MDL-21432 backup - include both user and group "icon" areas (MDL-14589, MDL-23202)
authorEloy Lafuente <stronk7@moodle.org>
Sun, 11 Jul 2010 14:46:31 +0000 (14:46 +0000)
committerEloy Lafuente <stronk7@moodle.org>
Sun, 11 Jul 2010 14:46:31 +0000 (14:46 +0000)
backup/moodle2/backup_final_task.class.php
backup/moodle2/backup_stepslib.php

index d0461e3..fce06a4 100644 (file)
@@ -46,7 +46,7 @@ class backup_final_task extends backup_task {
         // including membership based on setting
         $this->add_step(new backup_groups_structure_step('groups', 'groups.xml'));
 
-        // Annotate all the user files (conditionally) (private files, and profile)
+        // Annotate all the user files (conditionally) (private profile and icon files)
         // Because each user has its own context, we need a separate/specialised step here
         // This step also ensures that the contexts for all the users exist, so next
         // step can be safely executed (join between users and contexts)
index c353699..346c5d5 100644 (file)
@@ -722,9 +722,8 @@ class backup_groups_structure_step extends backup_structure_step {
 
         // Define file annotations
 
-        //TODO: not implemented yet
         $group->annotate_files('group', 'description', 'id');
-        $group->annotate_files('group', 'image', 'id');
+        $group->annotate_files('group', 'icon', 'id');
 
         // Return the root element (groups)
         return $groups;
@@ -1306,8 +1305,7 @@ class backup_annotate_all_user_files extends backup_execution_step {
         global $DB;
 
         // List of fileareas we are going to annotate
-        // TODO: user image not implemented yet
-        $fileareas = array('private', 'profile', 'image');
+        $fileareas = array('private', 'profile', 'icon');
 
         // Fetch all annotated (final) users
         $rs = $DB->get_recordset('backup_ids_temp', array(
@@ -1317,7 +1315,7 @@ class backup_annotate_all_user_files extends backup_execution_step {
             $userctxid = get_context_instance(CONTEXT_USER, $userid)->id;
             // Proceed with every user filearea
             foreach ($fileareas as $filearea) {
-                // We don't need to specify itemid ($userid - 4th param) as far as by
+                // We don't need to specify itemid ($userid - 5th param) as far as by
                 // context we can get all the associated files. See MDL-22092
                 backup_structure_dbops::annotate_files($this->get_backupid(), $userctxid, 'user', $filearea, null);
             }