From: nfreear <nfreear>
Date: Thu, 8 Mar 2007 10:35:28 +0000 (+0000)
Subject: Follow-up fixes for MDL-8531 "Groups icons..." -PARAM_ALPHANUM bug.
X-Git-Url: http://git.mjollnir.org/gw?a=commitdiff_plain;h=8834b12ad2c841852f899b19137bdfe132c2bbf3;p=moodle.git

Follow-up fixes for MDL-8531 "Groups icons..." -PARAM_ALPHANUM bug.
---

diff --git a/group/group.php b/group/group.php
index 6e028b7edd..060f70a031 100644
--- a/group/group.php
+++ b/group/group.php
@@ -21,9 +21,9 @@ $groupingid = optional_param('grouping', false, PARAM_INT);
 $newgrouping= optional_param('newgrouping', false, PARAM_INT);
 $groupid    = optional_param('group', false, PARAM_INT);
 
-$groupsettings->name       = optional_param('name', false, PARAM_ALPHANUM);
-$groupsettings->description= optional_param('description', '', PARAM_ALPHANUM);
-$groupsettings->enrolmentkey= optional_param('enrolmentkey', '', PARAM_ALPHANUM);
+$groupsettings->name       = optional_param('name', false, PARAM_TEXT);
+$groupsettings->description= optional_param('description', '', PARAM_TEXT);
+$groupsettings->enrolmentkey= optional_param('enrolmentkey', '', PARAM_TEXT);
 $groupsettings->hidepicture= optional_param('hidepicture', true, PARAM_BOOL);
 
 $delete = optional_param('delete', false, PARAM_BOOL);
diff --git a/group/grouping.php b/group/grouping.php
index 0ea021bb81..d6af0eadd0 100644
--- a/group/grouping.php
+++ b/group/grouping.php
@@ -17,8 +17,8 @@ $success = true;
 $courseid   = required_param('courseid', PARAM_INT);         
 $groupingid = optional_param('grouping', false, PARAM_INT);
 
-$groupingsettings->name       = optional_param('name', PARAM_ALPHANUM);
-$groupingsettings->description= optional_param('description', PARAM_ALPHANUM);
+$groupingsettings->name       = optional_param('name', false, PARAM_TEXT);
+$groupingsettings->description= optional_param('description', '', PARAM_TEXT);
 
 $delete = optional_param('delete', false, PARAM_BOOL);
 
@@ -64,7 +64,7 @@ if ($success) {
             if ($groupingid) {
                 $success = (bool)groups_set_grouping_settings($groupingid, $groupingsettings);
             }
-            else {
+            else { //OK, new group.
                 $success = (bool)$groupingid = groups_create_grouping($courseid, $groupingsettings);
             }
             if ($success) {
@@ -135,7 +135,12 @@ if ($success) {
 ?>
 
 <div class="f-item">
-<p><label for="groupingname"><?php print_string('groupingname', 'group'); ?>&nbsp;</label></p>
+<p><label for="groupingname"><?php
+   print_string('groupingname', 'group'); 
+   if (isset($err['name'])) {
+       echo' ';
+       formerr($err['name']);
+   } ?>&nbsp; </label></p>
 <p><input id="groupingname" name="name" type="text" size="40" value="<?php echo $strname; ?>" /></p>
 </div>