MDL-21666 backup & restore caps - upload/download renamed

robjoyce · Feb 19, 2010 · 2fdde5d · 2fdde5d
1 parent d294000
commit 2fdde5d
Show file tree

Hide file tree

Showing 3 changed files with 18 additions and 18 deletions.
diff --git a/lib/db/access.php b/lib/db/access.php
@@ -167,7 +167,7 @@
         'clonepermissionsfrom' =>  'moodle/site:backup'
     ),
 
-    'moodle/site:backupdownload' => array(
+    'moodle/backup:downloadfile' => array(
 
         'riskbitmask' => RISK_SPAM | RISK_PERSONAL | RISK_XSS,
 
@@ -178,35 +178,35 @@
             'admin' => CAP_ALLOW
         ),
 
-        'clonepermissionsfrom' =>  'moodle/site:backup'
+        'clonepermissionsfrom' =>  'moodle/site:backupdownload'
     ),
 
-    'moodle/site:backupupload' => array(
+    'moodle/backup:userinfo' => array(
 
-        'riskbitmask' => RISK_SPAM | RISK_PERSONAL | RISK_XSS,
+        'riskbitmask' => RISK_PERSONAL,
 
-        'captype' => 'write',
+        'captype' => 'read',
         'contextlevel' => CONTEXT_COURSE,
         'legacy' => array(
-            'editingteacher' => CAP_ALLOW,
             'admin' => CAP_ALLOW
-        ),
-
-        'clonepermissionsfrom' =>  'moodle/site:restore'
+        )
     ),
 
-    'moodle/backup:userinfo' => array(
+    'moodle/restore:restorecourse' => array(
 
-        'riskbitmask' => RISK_PERSONAL,
+        'riskbitmask' => RISK_SPAM | RISK_PERSONAL | RISK_XSS,
 
-        'captype' => 'read',
+        'captype' => 'write',
         'contextlevel' => CONTEXT_COURSE,
         'legacy' => array(
+            'editingteacher' => CAP_ALLOW,
             'admin' => CAP_ALLOW
-        )
+        ),
+
+        'clonepermissionsfrom' =>  'moodle/site:restore'
     ),
 
-    'moodle/restore:restorecourse' => array(
+    'moodle/restore:uploadfile' => array(
 
         'riskbitmask' => RISK_SPAM | RISK_PERSONAL | RISK_XSS,
 
@@ -217,7 +217,7 @@
             'admin' => CAP_ALLOW
         ),
 
-        'clonepermissionsfrom' =>  'moodle/site:restore'
+        'clonepermissionsfrom' =>  'moodle/site:backupupload'
     ),
 
     'moodle/restore:createuser' => array(

diff --git a/lib/file/file_browser.php b/lib/file/file_browser.php
@@ -387,8 +387,8 @@ private function get_file_info_course_backup($course, $context, $filearea=null,
             }
         }
 
-        $downloadable = has_capability('moodle/site:backupdownload', $context);
-        $uploadable   = has_capability('moodle/site:backupupload', $context);
+        $downloadable = has_capability('moodle/backup:downloadfile', $context);
+        $uploadable   = has_capability('moodle/backup:uploadfile', $context);
         return new file_info_stored($this, $context, $storedfile, $urlbase, get_string('areabackup', 'repository'), false, $downloadable, $uploadable, false);
 
     }

diff --git a/pluginfile.php b/pluginfile.php
@@ -281,7 +281,7 @@
 
     if ($filearea === 'course_backup') {
         require_login($course);
-        require_capability('moodle/site:backupdownload', $context);
+        require_capability('moodle/backup:downloadfile', $context);
 
         $relativepath = '/'.implode('/', $args);
         $fullpath = $context->id.'course_backup0'.$relativepath;