MDL-66072 core: Remove course sharing and related capabilities

rboyatt · Sep 3, 2019 · 020bad7 · 020bad7
1 parent 1d6f041
commit 020bad7
Show file tree

Hide file tree

Showing 18 changed files with 8 additions and 564 deletions.
diff --git a/admin/settings/subsystems.php b/admin/settings/subsystems.php
@@ -48,7 +48,4 @@
 
     $optionalsubsystems->add(new admin_setting_configcheckbox('allowstealth', new lang_string('allowstealthmodules'),
         new lang_string('allowstealthmodules_help'), 0, 1, 0));
-
-    $optionalsubsystems->add(new admin_setting_configcheckbox('enablecoursepublishing',
-        new lang_string('enablecoursepublishing', 'hub'), new lang_string('enablecoursepublishing_help', 'hub'), 0));
 }
diff --git a/backup/util/checks/backup_check.class.php b/backup/util/checks/backup_check.class.php
@@ -128,15 +128,6 @@ public static function check_security($backup_controller, $apply) {
         // Now, if backup mode is hub or import, check userid has permissions for those modes
         // other modes will perform common checks only (backupxxxx capabilities in $typecapstocheck)
         switch ($mode) {
-            case backup::MODE_HUB:
-                if (!has_capability('moodle/backup:backuptargethub', $coursectx, $userid)) {
-                    $a = new stdclass();
-                    $a->userid = $userid;
-                    $a->courseid = $courseid;
-                    $a->capability = 'moodle/backup:backuptargethub';
-                    throw new backup_controller_exception('backup_user_missing_capability', $a);
-                }
-                break;
             case backup::MODE_IMPORT:
                 if (!has_capability('moodle/backup:backuptargetimport', $coursectx, $userid)) {
                     $a = new stdclass();

diff --git a/backup/util/checks/restore_check.class.php b/backup/util/checks/restore_check.class.php
@@ -90,15 +90,6 @@ public static function check_security($restore_controller, $apply) {
         // Now, if restore mode is hub or import, check userid has permissions for those modes
         // other modes will perform common checks only (restorexxxx capabilities in $typecapstocheck)
         switch ($mode) {
-            case backup::MODE_HUB:
-                if (!has_capability('moodle/restore:restoretargethub', $coursectx, $userid)) {
-                    $a = new stdclass();
-                    $a->userid = $userid;
-                    $a->courseid = $courseid;
-                    $a->capability = 'moodle/restore:restoretargethub';
-                    throw new restore_controller_exception('restore_user_missing_capability', $a);
-                }
-                break;
             case backup::MODE_IMPORT:
                 if (!has_capability('moodle/restore:restoretargetimport', $coursectx, $userid)) {
                     $a = new stdclass();

diff --git a/course/lib.php b/course/lib.php
@@ -4045,7 +4045,6 @@ function course_get_user_administration_options($course, $context) {
         $options->outcomes = !empty($CFG->enableoutcomes) && has_capability('moodle/course:update', $context);
         $options->badges = !empty($CFG->enablebadges);
         $options->import = has_capability('moodle/restore:restoretargetimport', $context);
-        $options->publish = !empty($CFG->enablecoursepublishing) && has_capability('moodle/course:publish', $context);
         $options->reset = has_capability('moodle/course:reset', $context);
         $options->roles = has_capability('moodle/role:switchroles', $context);
     } else {

diff --git a/course/publish/backup.php b/course/publish/backup.php
diff --git a/course/publish/forms.php b/course/publish/forms.php
diff --git a/course/publish/hubselector.php b/course/publish/hubselector.php
diff --git a/course/publish/index.php b/course/publish/index.php
diff --git a/course/publish/lib.php b/course/publish/lib.php