summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorBharat Mediratta <bharat@menalto.com>2010-09-22 21:43:44 -0700
committerBharat Mediratta <bharat@menalto.com>2010-09-22 21:43:44 -0700
commit7116f820ca2b6338abdca233a4b1d42d29a41150 (patch)
tree7c7fcf559e91da34b46c7c9af8575dcd43ac83f0
parent790bf4e871b1a1555dc32287ab222b6694d402a2 (diff)
Remove the 4 character minimum for group name length. Fixes ticket #1396.
-rw-r--r--modules/user/controllers/admin_users.php8
-rw-r--r--modules/user/models/group.php2
2 files changed, 4 insertions, 6 deletions
diff --git a/modules/user/controllers/admin_users.php b/modules/user/controllers/admin_users.php
index c22fcc2e..23032ab3 100644
--- a/modules/user/controllers/admin_users.php
+++ b/modules/user/controllers/admin_users.php
@@ -381,8 +381,8 @@ class Admin_Users_Controller extends Admin_Controller {
$form_group->inputs["name"]->error_messages("conflict", t("There is already a group with that name"))
->error_messages("required", t("You must enter a group name"))
->error_messages("length",
- t("The group name must be between %min_length and %max_length characters",
- array("min_length" => 4, "max_length" => 255)));
+ t("The group name must be less than %max_length characters",
+ array("max_length" => 255)));
$form_group->submit("")->value(t("Save"));
return $form;
}
@@ -392,9 +392,7 @@ class Admin_Users_Controller extends Admin_Controller {
$form_group = $form->group("add_group")->label(t("Add group"));
$form_group->input("name")->label(t("Name"))->id("g-name");
$form_group->inputs["name"]->error_messages("conflict", t("There is already a group with that name"))
- ->error_messages("required", t("You must enter a group name"))
- ->error_messages("length", t("The group name must be at least %min_length characters",
- array("min_length" => 4)));
+ ->error_messages("required", t("You must enter a group name"));
$form_group->submit("")->value(t("Add group"));
return $form;
}
diff --git a/modules/user/models/group.php b/modules/user/models/group.php
index 57b12963..b27c7250 100644
--- a/modules/user/models/group.php
+++ b/modules/user/models/group.php
@@ -41,7 +41,7 @@ class Group_Model extends ORM implements Group_Definition {
// validate() is recursive, only modify the rules on the outermost call.
if (!$array) {
$this->rules = array(
- "name" => array("rules" => array("required", "length[4,255]"),
+ "name" => array("rules" => array("required", "length[1,255]"),
"callbacks" => array(array($this, "valid_name"))));
}