From a104debf4fe50fe8ef9d50ba50988e997ab4ed81 Mon Sep 17 00:00:00 2001
From: skodak <skodak>
Date: Fri, 17 Aug 2007 12:29:18 +0000
Subject: [PATCH] MDL-10892 modules explicitly declare support for
 groupmembersonly; the grouping and groupmembersonly elements are now marked
 as advanced; only Resource supports groupmembers only now

---
 course/moodleform_mod.php | 17 +++++++++++++++--
 mod/resource/mod_form.php |  2 +-
 2 files changed, 16 insertions(+), 3 deletions(-)

diff --git a/course/moodleform_mod.php b/course/moodleform_mod.php
index 261c789ce6..adc7f69e02 100644
--- a/course/moodleform_mod.php
+++ b/course/moodleform_mod.php
@@ -70,6 +70,13 @@ class moodleform_mod extends moodleform {
                 $mform->hardFreeze('groupmode'); // groupmode can not be changed if forced from course settings
             }
         }
+
+        // groupings have no use without groupmode or groupmembersonly
+        if (!$mform->elementExists('groupmode') and !$mform->elementExists('groupmembersonly')) {
+            if ($mform->elementExists('groupingid')) {
+                $mform->removeElement('groupingid');
+            }
+        }
     }
 
     // form verification
@@ -117,8 +124,9 @@ class moodleform_mod extends moodleform {
      * Adds all the standard elements to a form to edit the settings for an activity module.
      *
      * @param bool $supportsgroups does this module support groups?
+     * @param bool $supportgroupmembersonly does this module support groupmembersonly access?
      */
-    function standard_coursemodule_elements($supportsgroups=true){
+    function standard_coursemodule_elements($supportsgroups=true, $supportgroupmembersonly=false){
         global $COURSE, $CFG;
         $mform =& $this->_form;
 
@@ -146,7 +154,12 @@ class moodleform_mod extends moodleform {
                 }
             }
             $mform->addElement('select', 'groupingid', get_string('grouping', 'group'), $options);
-            $mform->addElement('advcheckbox', 'groupmembersonly', get_string('groupmembersonly', 'group'));
+            $mform->setAdvanced('groupingid');
+            
+            if ($supportgroupmembersonly) {
+                $mform->addElement('advcheckbox', 'groupmembersonly', get_string('groupmembersonly', 'group'));
+                $mform->setAdvanced('groupmembersonly');
+            }
         }
 
         $mform->addElement('modvisible', 'visible', get_string('visible'));
diff --git a/mod/resource/mod_form.php b/mod/resource/mod_form.php
index 1948c22130..3d9b4bb076 100644
--- a/mod/resource/mod_form.php
+++ b/mod/resource/mod_form.php
@@ -43,7 +43,7 @@ class mod_resource_mod_form extends moodleform_mod {
         $mform->addElement('header', 'typedesc', get_string('resourcetype'.$type,'resource'));
         $this->_resinstance->setup_elements($mform);
 
-        $this->standard_coursemodule_elements(false);
+        $this->standard_coursemodule_elements(false, true);
 
         $this->add_action_buttons();
     }
-- 
2.39.5