ciphers: Fix the default cipher option

 * Also add a 'modified' signal to make it easier to the clients
   to track the property change
 * Add a missing comment (for conformance with the coding guidelines)

Refs #72494

(cherry picked from commit 721a1548)

src/ciphermodel.cpp

@@ -60,8 +60,10 @@ CipherModelPrivate::CipherModelPrivate(Account* parent) : m_pAccount(parent),m_U
    m_lChecked = new bool[m_slSupportedCiphers.size()]{};
 
    foreach(const QString& cipher, parent->d_ptr->accountDetail(DRing::Account::ConfProperties::TLS::CIPHERS).split(' ')) {
-      m_lChecked[m_shMapping[cipher]] = true;
-      m_UseDefault = false;
+      if (!cipher.trimmed().isEmpty()) {
+         m_lChecked[m_shMapping[cipher]] = true;
+         m_UseDefault = false;
+      }
    }
 }
 
@@ -131,6 +133,9 @@ bool CipherModel::setData( const QModelIndex& index, const QVariant &value, int
             ciphers << d_ptr->m_slSupportedCiphers[i];
       }
       d_ptr->m_pAccount->d_ptr->setAccountProperty(DRing::Account::ConfProperties::TLS::CIPHERS,ciphers.join(QString(' ')));
+
+      emit modified();
+
       return true;
    }
    return false;

src/ciphermodel.h

@@ -42,6 +42,7 @@ class LIB_EXPORT CipherModel : public QAbstractListModel {
 
 public:
 
+   //Property
    Q_PROPERTY(bool useDefault READ useDefault WRITE setUseDefault)
 
    //Model functions
@@ -66,6 +67,9 @@ private:
    CipherModelPrivate* d_ptr;
    Q_DECLARE_PRIVATE(CipherModel)
 
+Q_SIGNALS:
+   void modified();
+
 };
 Q_DECLARE_METATYPE(CipherModel*)
 #endif