* finish task #4823.

module/common/view/customfield.html.php

@@ -25,7 +25,7 @@
         <?php echo html::checkbox('fields', $customFields, $showFields);?>
       </div>
       <div>
-        <?php echo html::submitButton($lang->save);?>
+        <button type="submit" class="btn btn-primary" data-loading="<?php echo $lang->submitting;?>"><?php echo $lang->save;?></button>
         <?php echo html::commonButton($lang->cancel, '', "btn close-dropdown");?>
         <?php echo html::a($customLink, $lang->restore, 'hiddenwin', "class='btn'");?>
       </div>

module/user/control.php

@@ -694,6 +694,7 @@ class user extends control
             if($this->post->password) $password = $this->post->password;
             if($this->get->password)  $password = $this->get->password;
 
+            $account = trim($account);
             if($this->user->checkLocked($account))
             {
                 $failReason = sprintf($this->lang->user->loginLocked, $this->config->user->lockMinutes);

module/user/js/common.js

@@ -10,7 +10,7 @@ $(document).ready(function()
 {
     $('#verifyPassword').closest('form').find('#submit').click(function()
     {
-        var password = $('input#verifyPassword').val();
+        var password = $('input#verifyPassword').val().trim();
         var rand = $('input#verifyRand').val();
         $('input#verifyPassword').val(md5(md5(password) + rand));
     });

module/user/js/login.js

@@ -24,7 +24,7 @@ $(document).ready(function()
 
     $('#loginPanel #submit').click(function()
     {
-        var password = $('input:password').val();
+        var password = $('input:password').val().trim();
         var rand = $('input#verifyRand').val();
         if(password.length != 32 && typeof(md5) == 'function') $('input:password').val(md5(md5(password) + rand));
     });

module/user/model.php

@@ -206,6 +206,7 @@ class userModel extends model
      */
     public function create()
     {
+        $_POST['account'] = trim($_POST['account']);
         if(!$this->checkPassword()) return;
         if(strtolower($_POST['account']) == 'guest') return false;
 
@@ -236,17 +237,18 @@ class userModel extends model
             ->check('account', 'account')
             ->checkIF($this->post->email != '', 'email', 'email')
             ->exec();
-        if($this->post->group)
-        {
-            $data = new stdClass();
-            $data->account = $this->post->account;
-            $data->group   = $this->post->group;
-            $this->dao->insert(TABLE_USERGROUP)->data($data)->exec();
-        }
-
         if(!dao::isError())
         {
-            $this->loadModel('action')->create('user', $user->id, 'create');
+            $userID = $this->dao->lastInsertID();
+            if($this->post->group)
+            {
+                $data = new stdClass();
+                $data->account = $this->post->account;
+                $data->group   = $this->post->group;
+                $this->dao->insert(TABLE_USERGROUP)->data($data)->exec();
+            }
+
+            $this->loadModel('action')->create('user', $userID, 'Created');
             $this->loadModel('mail');
             if($this->config->mail->mta == 'sendcloud' and !empty($user->email)) $this->mail->syncSendCloud('sync', $user->email, $user->realname);
         }
@@ -268,6 +270,7 @@ class userModel extends model
         $accounts = array();
         for($i = 0; $i < $this->config->user->batchCreate; $i++)
         {
+            $users->account[$i] = trim($users->account[$i]);
             if($users->account[$i] != '')
             {
                 if(strtolower($users->account[$i]) == 'guest') die(js::error(sprintf($this->lang->user->error->reserved, $i+1)));
@@ -289,7 +292,7 @@ class userModel extends model
                 $data[$i]->group    = $users->group[$i] == 'ditto' ? (isset($prev['group']) ? $prev['group'] : '') : $users->group[$i];
                 $data[$i]->email    = $users->email[$i];
                 $data[$i]->gender   = $users->gender[$i];
-                $data[$i]->password = md5($users->password[$i]);
+                $data[$i]->password = md5(trim($users->password[$i]));
                 $data[$i]->commiter = $users->commiter[$i];
                 $data[$i]->join     = empty($users->join[$i]) ? '0000-00-00' : ($users->join[$i]);
                 $data[$i]->skype    = $users->skype[$i];
@@ -356,6 +359,7 @@ class userModel extends model
      */
     public function update($userID)
     {
+        $_POST['account'] = trim($_POST['account']);
         if(!$this->checkPassword(true)) return;
 
         $oldUser = $this->getById($userID, 'id');
@@ -457,7 +461,7 @@ class userModel extends model
         $accounts = array();
         foreach($data->account as $id => $account)
         {
-            $users[$id]['account']  = $account;
+            $users[$id]['account']  = trim($account);
             $users[$id]['realname'] = $data->realname[$id];
             $users[$id]['commiter'] = $data->commiter[$id];
             $users[$id]['email']    = $data->email[$id];
@@ -570,6 +574,7 @@ class userModel extends model
      */
     public function resetPassword()
     {
+        $_POST['account'] = trim($_POST['account']);
         if(!$this->checkPassword()) return;
 
         $user = $this->getById($this->post->account);
@@ -594,6 +599,8 @@ class userModel extends model
      */
     public function checkPassword($canNoPassword = false)
     {
+        $_POST['password1'] = trim($_POST['password1']);
+        $_POST['password2'] = trim($_POST['password2']);
         if(!$canNoPassword and empty($_POST['password1'])) dao::$errors['password'][] = sprintf($this->lang->error->notempty, $this->lang->user->password);
         if($this->post->password1 != false)
         {