[users] Sort the password checks before applying them

src/modules/users/CheckPWQuality.h

@@ -60,6 +60,7 @@ public:
     QString filter( const QString& s ) const { return m_accept( s ) ? QString() : m_message(); }
 
     Weight weight() const { return m_weight; }
+    bool operator<( const PasswordCheck& other ) const { return weight() < other.weight(); }
 
 private:
     Weight m_weight;

src/modules/users/UsersPage.cpp

@@ -424,6 +424,12 @@ UsersPage::checkPasswordAcceptance( const QString& pw1, const QString& pw2, QLab
         bool failureIsFatal = ui->checkBoxValidatePassword->isChecked();
         bool failureFound = false;
 
+        if ( m_passwordChecksChanged )
+        {
+            std::sort( m_passwordChecks.begin(), m_passwordChecks.end() );
+            m_passwordChecksChanged = false;
+        }
+
         for ( auto pc : m_passwordChecks )
         {
             QString s = pc.filter( pw1 );
@@ -502,6 +508,8 @@ UsersPage::setReusePasswordDefault( bool checked )
 void
 UsersPage::addPasswordCheck( const QString& key, const QVariant& value )
 {
+    m_passwordChecksChanged = true;
+
     if ( key == "minLength" )
     {
         add_check_minLength( m_passwordChecks, value );

src/modules/users/UsersPage.h

@@ -90,6 +90,7 @@ private:
     Ui::Page_UserSetup* ui;
 
     PasswordCheckList m_passwordChecks;
+    bool m_passwordChecksChanged = false;
 
     bool m_readyFullName;
     bool m_readyUsername;