Fixed issue: Database error when username or user full name too long

LimeSurvey · Jan 5, 2021 · dcbcae3 · dcbcae3
1 parent c8ebbca
commit dcbcae3
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 4 deletions.
diff --git a/application/models/User.php b/application/models/User.php
@@ -81,6 +81,8 @@ public function rules()
         return array(
             array('users_name, password, email', 'required'),
             array('users_name', 'unique'),
+            array('users_name', 'length','max'=>64),
+            array('full_name', 'length','max'=>50),
             array('email', 'email'),
             array('full_name', 'LSYii_Validators'), // XSS if non super-admin
             array('parent_id', 'default', 'value'=>0),

diff --git a/application/views/admin/user/editusers.php b/application/views/admin/user/editusers.php
@@ -107,7 +107,7 @@
                     <div class="form-group">
                         <label for="new_user" class="control-label"><?php eT("Username:");?></label>
                         <div class="">
-                            <input type='text' class="text input-sm form-control" id='new_user' name='new_user' required />
+                            <input type='text' class="text input-sm form-control" maxlength='64' id='new_user' name='new_user' required />
                         </div>
                     </div>
                     <div class="form-group">
@@ -119,7 +119,7 @@
                     <div class="form-group">
                         <label for="new_full_name" class="control-label "><?php eT("Full name:");?></label>
                         <div class="">
-                            <input type='text' class="text input-sm form-control" id='new_full_name' name='new_full_name' required />
+                            <input type='text' class="text input-sm form-control" maxlength='50' id='new_full_name' name='new_full_name' required />
                         </div>
                     </div>
                     <div class="form-group">

diff --git a/application/views/admin/user/modifyuser.php b/application/views/admin/user/modifyuser.php
@@ -43,8 +43,8 @@
                         <?php eT("Full name");?>
                     </label>
                     <div class="">
-                        <?php echo $form->textField($oUser, 'full_name');?>
-                    </div>
+                        <?php echo $form->textField($oUser, 'full_name', array('maxlength'=>50));?>
+                    </div> 
                 </div>
 
                 <?php if( !Permission::model()->hasGlobalPermission('superadmin','read', $oUser->uid) || (Permission::isForcedSuperAdmin(Permission::getUserId())) ): ?>