MDL-8096 - some more fixes for custom profile fields

diff --git a/user/profile/definelib.php b/user/profile/definelib.php
index ec208263e4981209cc02e7bd2323b22058ed1491..1e1a46a9a3523d3d423a8d751b5841ad433cf261 100644 (file)
--- a/user/profile/definelib.php
+++ b/user/profile/definelib.php
@@ -23,11 +23,11 @@ class profile_define_base {
 
         $strrequired = get_string('required');
 
-        $form->addElement('text', 'shortname', get_string('profileshortname', 'admin'), 'maxlength="100" size="30"');
+        $form->addElement('text', 'shortname', get_string('profileshortname', 'admin'), 'maxlength="100" size="25"');
         $form->addRule('shortname', $strrequired, 'required', null, 'client');
         $form->setType('shortname', PARAM_ALPHANUM);
 
-        $form->addElement('text', 'name', get_string('profilename', 'admin'), 'size="30"');
+        $form->addElement('text', 'name', get_string('profilename', 'admin'), 'size="50"');
         $form->addRule('name', $strrequired, 'required', null, 'client');
         $form->setType('name', PARAM_MULTILANG);
 
@@ -92,9 +92,12 @@ class profile_define_base {
     function define_validate_common($data) {
         $err = array();
 
+        /// Check the shortname was not truncated by cleaning
+        if (empty($data->shortname)) {
+            $err['shortname'] = get_string('required');
+
         /// Check the shortname is unique
-        if (($field = get_record('user_info_field', 'shortname', $data->shortname)) and ($field->id <> $data->id)) {
-        //if (record_exists_select('user_info_field', 'shortname='.$data->shortname.' AND id<>'.$data->id)) {
+        } else if (($field = get_record('user_info_field', 'shortname', $data->shortname)) and ($field->id <> $data->id)) {
             $err['shortname'] = get_string('profileshortnamenotunique', 'admin');
         }
 

diff --git a/user/profile/field/menu/define.class.php b/user/profile/field/menu/define.class.php
index 126457a929ad3d1c0304f52eeab1087afbbe0f7e..f824f07684ddeadf9fba312512bdbf15dcc9f56f 100644 (file)
--- a/user/profile/field/menu/define.class.php
+++ b/user/profile/field/menu/define.class.php
@@ -8,7 +8,7 @@ class profile_define_menu extends profile_define_base {
         $form->setType('param1', PARAM_MULTILANG);
 
         /// Default data
-        $form->addElement('text', 'defaultdata', get_string('profiledefaultdata', 'admin'), 'size="30"');
+        $form->addElement('text', 'defaultdata', get_string('profiledefaultdata', 'admin'), 'size="50"');
         $form->setType('defaultdata', PARAM_MULTILANG);
     }
 

diff --git a/user/profile/field/text/define.class.php b/user/profile/field/text/define.class.php
index 4c8c758b354037efe2619a101a5a421e36283ae9..f0b6a2d6c8f72c16a8cc27a9523f15775115dc8e 100644 (file)
--- a/user/profile/field/text/define.class.php
+++ b/user/profile/field/text/define.class.php
@@ -4,15 +4,17 @@ class profile_define_text extends profile_define_base {
 
     function define_form_specific(&$form) {
         /// Default data
-        $form->addElement('text', 'defaultdata', get_string('profiledefaultdata', 'admin'), 'size="30"');
+        $form->addElement('text', 'defaultdata', get_string('profiledefaultdata', 'admin'), 'size="50"');
         $form->setType('defaultdata', PARAM_MULTILANG);
 
         /// Param 1 for text type is the size of the field
         $form->addElement('text', 'param1', get_string('profilefieldsize', 'admin'), 'size="6"');
+        $form->setDefault('param1', 50);
         $form->setType('param1', PARAM_INT);
 
         /// Param 2 for text type is the maxlength of the field
         $form->addElement('text', 'param2', get_string('profilefieldmaxlength', 'admin'), 'size="6"');
+        $form->setDefault('param2', 100);
         $form->setType('param2', PARAM_INT);
     }
 

diff --git a/user/profile/lib.php b/user/profile/lib.php
index 8f6036daa965c3649755b67b986a979c68ef318f..4c13ab4867d36a844841c2a9539d8e6e8a1492c2 100644 (file)
--- a/user/profile/lib.php
+++ b/user/profile/lib.php
@@ -72,7 +72,7 @@ class profile_field_base {
         $usernew = $this->save_data_preprocess($usernew);
 
         if (!isset($usernew->{$this->inputname})) {
-            // field not present in form, probably locked and incisible - skip it!
+            // field not present in form, probably locked and invisible - skip it
             return;
         }