[MDL-34085] Fix the backup options names in core_course_duplicate_course - Moodle Tracker

Details

Type: Bug
Status: Closed
Priority: Minor
Resolution: Fixed
Affects Version/s: 2.3, 2.3.7, 2.4.4, 2.5
Fix Version/s: 2.3.8, 2.4.5, 2.5.1
Component/s: Backup, Web Services
Labels:
- patch
- triaged

Testing Instructions:
Hide

Create a course with 2 students enrolled

enable actvity completion in course and site settings

Create an assignment and a page resource

Enable manual (student smarks as complete) activity completion on the page resource

For the assignment, set the activity as completed when the students submits the assignment and the teacher grades it.

As a student manually mark the page as completed

As a student submit an assignment

As a teaher grade the assignment

Enable webservices, providing access to the core_course_duplicate_course webservice.
#
Use this client:
https://github.com/moodlehq/sample-ws-clients/blob/master/PHP-REST/client.php
functionname
core_course_duplicate_course
/// PARAMETERS
$params = array(
'courseid' => 4, // CHANGE TO THE COURSE you created above
'fullname' => 'core_course_duplicate_course test', // New course full name
'shortname' => 'core_course_duplicate_course', // New course shortname
'categoryid' => 1, // New course category id
'visible' => 1, // Make the course visible after duplicating
'options' => array(array('name'=>'blocks', 'value'=>1), array('name'=>'activities', 'value'=>1), array('name'=>'users', 'value'=>1), array('name'=>'userscompletion', 'value'=>1), array('name'=>'grade_histories', 'value'=>1), ) // Backup options
);

VERIFY: that the course has been duplicated correctly

VERIFY: that the student has been enrolled correctly into the new course

VERIFY: that you can login as the student and see the completion information is restored in the new coruse

VERFIY: that the grade history has been retained in the new course
Show
Create a course with 2 students enrolled enable actvity completion in course and site settings Create an assignment and a page resource Enable manual (student smarks as complete) activity completion on the page resource For the assignment, set the activity as completed when the students submits the assignment and the teacher grades it. As a student manually mark the page as completed As a student submit an assignment As a teaher grade the assignment Enable webservices, providing access to the core_course_duplicate_course webservice. # Use this client: https://github.com/moodlehq/sample-ws-clients/blob/master/PHP-REST/client.php functionname core_course_duplicate_course /// PARAMETERS $params = array( 'courseid' => 4, // CHANGE TO THE COURSE you created above 'fullname' => 'core_course_duplicate_course test', // New course full name 'shortname' => 'core_course_duplicate_course', // New course shortname 'categoryid' => 1, // New course category id 'visible' => 1, // Make the course visible after duplicating 'options' => array(array('name'=>'blocks', 'value'=>1), array('name'=>'activities', 'value'=>1), array('name'=>'users', 'value'=>1), array('name'=>'userscompletion', 'value'=>1), array('name'=>'grade_histories', 'value'=>1), ) // Backup options ); VERIFY: that the course has been duplicated correctly VERIFY: that the student has been enrolled correctly into the new course VERIFY: that you can login as the student and see the completion information is restored in the new coruse VERFIY: that the grade history has been retained in the new course
Difficulty:
Easy
Affected Branches:
MOODLE_23_STABLE, MOODLE_24_STABLE, MOODLE_25_STABLE
Fixed Branches:
MOODLE_23_STABLE, MOODLE_24_STABLE, MOODLE_25_STABLE
Pull from Repository:
https://github.com/iarenaza/moodle.git
Pull 2.4 Branch:
wip_24_mdl-34085
Pull 2.4 Diff URL:
https://github.com/iarenaza/moodle/compare/MOODLE_24_STABLE...wip_24_mdl-34085
Pull 2.5 Branch:
wip_25_mdl-34085
Pull 2.5 Diff URL:
https://github.com/iarenaza/moodle/compare/MOODLE_25_STABLE...wip_25_mdl-34085
Pull Master Branch:
wip_master_mdl-34085
Pull Master Diff URL:
https://github.com/iarenaza/moodle/compare/master...wip_master_mdl-34085

Rank:
42392

Description

Some backup options name are incorrect:

"completion_information" (int) Include user course completion information (default to 0 that is equal to
"histories" (int) Include histories (default to 0 that is equal to no)'

Correct ones are:

anonymize is missing

"anonymize" (int) Anonymize backup (default to 0 that is equal to no),
"userscompletion" (int) Include user course completion information (default to 0 that is equal to no),
"grade_histories" (int) Include grade histories (default to 0 that is equal to no)'

Issue Links

testing discovered

MDL-40015 core_course_duplicate_course description seems incorrect

Open

MDL-40016 core_course_duplicate_course does not restore rubric and marking data

Open

MDL-40007 core_course_duplicate_course is missing anonymize option

Closed

will be (partly) resolved by

MDL-34083 Deprecated (and invalid) backup options in WS core_course_duplicate_course

Closed

Activity

Ascending order - Click to sort in descending order

Hide

Permalink

Brian King added a comment - 13/Jul/12 3:31 PM

Please also note that in 2.3.1, the external_value for 'options' is wrong; it should be PARAM_ALPHAEXT, not PARAM_ALPHA (otherwise things like 'role_assignments' fail validation).

diff --git a/course/externallib.php b/course/externallib.php
index 41c5ab4..41f3eaa 100644
--- a/course/externallib.php
+++ b/course/externallib.php
@@ -668,7 +668,7 @@ class core_course_external extends external_api {
                 'options' => new external_multiple_structure(
                     new external_single_structure(
                         array(
-                                'name' => new external_value(PARAM_ALPHA, 'The backup option name:
+                                'name' => new external_value(PARAM_ALPHAEXT, 'The backup option name:
                                             "activities" (int) Include course activites (default to 1 that is equal to yes),
                                             "blocks" (int) Include course blocks (default to 1 that is equal to yes),
                                             "filters" (int) Include course filters  (default to 1 that is equal to yes),

Show

Brian King added a comment - 13/Jul/12 3:31 PM Please also note that in 2.3.1, the external_value for 'options' is wrong; it should be PARAM_ALPHAEXT, not PARAM_ALPHA (otherwise things like 'role_assignments' fail validation). diff --git a/course/externallib.php b/course/externallib.php index 41c5ab4..41f3eaa 100644 --- a/course/externallib.php +++ b/course/externallib.php @@ -668,7 +668,7 @@ class core_course_external extends external_api { 'options' => new external_multiple_structure( new external_single_structure( array( - 'name' => new external_value(PARAM_ALPHA, 'The backup option name: + 'name' => new external_value(PARAM_ALPHAEXT, 'The backup option name: "activities" (int) Include course activites (default to 1 that is equal to yes), "blocks" (int) Include course blocks (default to 1 that is equal to yes), "filters" (int) Include course filters (default to 1 that is equal to yes),

Hide

Permalink

Dan Poltawski added a comment - 04/Jun/13 2:26 AM

Looks like this is your issue Iñaki

Show

Dan Poltawski added a comment - 04/Jun/13 2:26 AM Looks like this is your issue Iñaki

Hide

Permalink

Dan Poltawski added a comment - 04/Jun/13 2:30 AM

Hi Brian King,

Looks like the issue you mentioned was fixed in ~~MDL-34083~~,

thanks!

Show

Dan Poltawski added a comment - 04/Jun/13 2:30 AM Hi Brian King , Looks like the issue you mentioned was fixed in MDL-34083 , thanks!

Hide

Permalink

Dan Poltawski added a comment - 04/Jun/13 2:41 AM

Hi Iñaki,

The issue was missing testing instructions, so i've created some to save sending this back.

If you have some time, please could you verify that the instructions are correct and/or improve them to be more accurate. Thanks

Show

Dan Poltawski added a comment - 04/Jun/13 2:41 AM Hi Iñaki, The issue was missing testing instructions, so i've created some to save sending this back. If you have some time, please could you verify that the instructions are correct and/or improve them to be more accurate. Thanks

Hide

Permalink

Dan Poltawski added a comment - 04/Jun/13 2:45 AM

Oh, I just noticed that "anonymize" (int) Anonymize backup (default to 0 that is equal to no), was not addressed from issue description, will create an issue for that.

Show

Dan Poltawski added a comment - 04/Jun/13 2:45 AM Oh, I just noticed that "anonymize" (int) Anonymize backup (default to 0 that is equal to no), was not addressed from issue description, will create an issue for that.

Hide

Permalink

Dan Poltawski added a comment - 04/Jun/13 2:50 AM

Thanks, i've integrated this to master, 25, 24 and 23.

I noticed the anonymize issue to late and so have created ~~MDL-40007~~

Show

Dan Poltawski added a comment - 04/Jun/13 2:50 AM Thanks, i've integrated this to master, 25, 24 and 23. I noticed the anonymize issue to late and so have created MDL-40007

Hide

Permalink

Iñaki Arenaza added a comment - 04/Jun/13 12:15 PM

Hi Dan,

I left anonymize out intentionally. I added the option in the first place, but when testing the patch, it threw an exception inside the backup/restore code (saying the option name was unkown). So I left it out for this patch.

Saludos.
Iñaki.

Show

Iñaki Arenaza added a comment - 04/Jun/13 12:15 PM Hi Dan, I left anonymize out intentionally. I added the option in the first place, but when testing the patch, it threw an exception inside the backup/restore code (saying the option name was unkown). So I left it out for this patch. Saludos. Iñaki.

Hide

Permalink

Petr Škoda added a comment - 04/Jun/13 12:29 PM

Hmm, is restore of grade histories already implemented? I searched the codebase and I could not find anything apart from the settings.

Show

Petr Škoda added a comment - 04/Jun/13 12:29 PM Hmm, is restore of grade histories already implemented? I searched the codebase and I could not find anything apart from the settings.

Hide

Permalink

Petr Škoda added a comment - 04/Jun/13 6:30 PM - edited

This kind of works, sorry for the noise, I am going to create new issues for:
1/ weird function name - core_course_duplicate_course Duplicate an existing course (creating a new one) without user data - it restores user data now it seems
2/ the marking guide and rubric data is not restored.

The grade_grades_history restore seems not implemented...

Thanks

Show

Petr Škoda added a comment - 04/Jun/13 6:30 PM - edited This kind of works, sorry for the noise, I am going to create new issues for: 1/ weird function name - core_course_duplicate_course Duplicate an existing course (creating a new one) without user data - it restores user data now it seems 2/ the marking guide and rubric data is not restored. The grade_grades_history restore seems not implemented... Thanks

Hide

Permalink

Dan Poltawski added a comment - 06/Jun/13 5:22 AM

Feature: Thanks to our superb contributors
  In order to make Moodle better
  As an integrator
  I need to thank all our contributors

  Scenario: Dan thanks you all
    Given I log in as "dan"
    And I see "lots of fixed issues"
    When I follow "Close integrated issues"
    Then I should see "Lots of thanks to all our contributors"

Your changes are upstream

Show

Dan Poltawski added a comment - 06/Jun/13 5:22 AM Feature: Thanks to our superb contributors In order to make Moodle better As an integrator I need to thank all our contributors Scenario: Dan thanks you all Given I log in as "dan" And I see "lots of fixed issues" When I follow "Close integrated issues" Then I should see "Lots of thanks to all our contributors" Your changes are upstream

People

Assignee:

Iñaki Arenaza

Reporter:

Juan Leyva

Integrator:

Dan Poltawski

Tester:

Petr Škoda

Participants:

Brian King, Dan Poltawski, Iñaki Arenaza, Juan Leyva, Petr Škoda

Votes:

1 Vote for this issue

Watchers:

4 Start watching this issue

Dates

Created:

28/Jun/12 5:33 PM

Updated:

06/Jun/13 5:22 AM

Resolved:

06/Jun/13 5:22 AM

Agile

View on Board