Use class "error" when appropriate

pombredanne · Mar 1, 2021 · b27a7a3 · b27a7a3
1 parent 8b33c72
commit b27a7a3
Showing 1 changed file with 9 additions and 1 deletion.
diff --git a/lib/plugin/UserPreferences.php b/lib/plugin/UserPreferences.php
@@ -78,6 +78,7 @@ function run($dbi, $argstr, &$request, $basepage)
         $args = $this->getArgs($argstr, $request);
         $user =& $request->_user;
         $user->_request = $request;
+        $iserror = false;
         if (defined('FUSIONFORGE') && FUSIONFORGE) {
             if (!($user->isAuthenticated())) {
                 return HTML::p(array('class' => 'error'),
@@ -129,6 +130,7 @@ function run($dbi, $argstr, &$request, $basepage)
                 } elseif ($rp = $request->getArg('pref')) {
                     // replace only changed prefs in $pref with those from request
                     if (!empty($rp['passwd']) and ($rp['passwd2'] != $rp['passwd'])) {
+                        $iserror = true;
                         $errmsg = _("Wrong password. Try again.");
                     } else {
                         if (empty($rp['passwd'])) unset($rp['passwd']);
@@ -148,9 +150,11 @@ function run($dbi, $argstr, &$request, $basepage)
                                 if ($passchanged) {
                                     $errmsg = _("Password updated.") . " ";
                                 } else {
+                                    $iserror = true;
                                     $errmsg = _("Password was not changed.") . " ";
                                 }
                             } else {
+                                $iserror = true;
                                 $errmsg = _("Password cannot be changed.");
                             }
                         }
@@ -167,7 +171,11 @@ function run($dbi, $argstr, &$request, $basepage)
                             }
                         }
                     }
-                    $args['errmsg'] = HTML::div(array('class' => 'feedback'), HTML::p($errmsg));
+                    if ($iserror) {
+                        $args['errmsg'] = HTML::div(array('class' => 'error'), HTML::p($errmsg));
+                    } else {
+                        $args['errmsg'] = HTML::div(array('class' => 'feedback'), HTML::p($errmsg));
+                    }
 
                 }
             }