Browse Source

feat(Settings): Add setting for hiding group join and leave system messages

Messages can become spammy is long lasting quiet groups, drowning out real user
messages
reviewable/pr6459/r4
Anthony Bilinski 3 years ago
parent
commit
1be5b99d17
No known key found for this signature in database
GPG Key ID: 2AA8E0DA1B31FB3C
  1. 4
      src/persistence/igroupsettings.h
  2. 15
      src/persistence/settings.cpp
  3. 6
      src/persistence/settings.h
  4. 5
      test/model/groupmessagedispatcher_test.cpp

4
src/persistence/igroupsettings.h

@ -36,5 +36,9 @@ public: @@ -36,5 +36,9 @@ public:
virtual QStringList getBlackList() const = 0;
virtual void setBlackList(const QStringList& blist) = 0;
virtual bool getShowGroupJoinLeaveMessages() const = 0;
virtual void setShowGroupJoinLeaveMessages(bool newValue) = 0;
DECLARE_SIGNAL(blackListChanged, QStringList const& blist);
DECLARE_SIGNAL(showGroupJoinLeaveMessagesChanged, bool show);
};

15
src/persistence/settings.cpp

@ -200,6 +200,7 @@ void Settings::loadGlobal() @@ -200,6 +200,7 @@ void Settings::loadGlobal()
lightTrayIcon = s.value("lightTrayIcon", false).toBool();
useEmoticons = s.value("useEmoticons", true).toBool();
statusChangeNotificationEnabled = s.value("statusChangeNotificationEnabled", false).toBool();
showGroupJoinLeaveMessages = s.value("showGroupJoinLeaveMessages", true).toBool();
spellCheckingEnabled = s.value("spellCheckingEnabled", true).toBool();
themeColor = s.value("themeColor", 0).toInt();
style = s.value("style", "").toString();
@ -663,6 +664,7 @@ void Settings::saveGlobal() @@ -663,6 +664,7 @@ void Settings::saveGlobal()
s.setValue("style", style);
s.setValue("nameColors", nameColors);
s.setValue("statusChangeNotificationEnabled", statusChangeNotificationEnabled);
s.setValue("showGroupJoinLeaveMessages", showGroupJoinLeaveMessages);
s.setValue("spellCheckingEnabled", spellCheckingEnabled);
}
s.endGroup();
@ -1023,6 +1025,19 @@ void Settings::setStatusChangeNotificationEnabled(bool newValue) @@ -1023,6 +1025,19 @@ void Settings::setStatusChangeNotificationEnabled(bool newValue)
}
}
bool Settings::getShowGroupJoinLeaveMessages() const
{
QMutexLocker locker{&bigLock};
return showGroupJoinLeaveMessages;
}
void Settings::setShowGroupJoinLeaveMessages(bool newValue)
{
if (setVal(showGroupJoinLeaveMessages, newValue)) {
emit showGroupJoinLeaveMessagesChanged(newValue);
}
}
bool Settings::getSpellCheckingEnabled() const
{
const QMutexLocker locker{&bigLock};

6
src/persistence/settings.h

@ -463,9 +463,12 @@ public: @@ -463,9 +463,12 @@ public:
QStringList getBlackList() const override;
void setBlackList(const QStringList& blist) override;
SIGNAL_IMPL(Settings, blackListChanged, QStringList const& blist)
bool getShowGroupJoinLeaveMessages() const override;
void setShowGroupJoinLeaveMessages(bool newValue) override;
SIGNAL_IMPL(Settings, showGroupJoinLeaveMessagesChanged, bool show)
// State
QByteArray getWindowGeometry() const;
void setWindowGeometry(const QByteArray& value);
@ -655,6 +658,7 @@ private: @@ -655,6 +658,7 @@ private:
QString timestampFormat;
QString dateFormat;
bool statusChangeNotificationEnabled;
bool showGroupJoinLeaveMessages;
bool spellCheckingEnabled;
// Privacy

5
test/model/groupmessagedispatcher_test.cpp

@ -63,9 +63,12 @@ class MockGroupSettings : public QObject, public IGroupSettings @@ -63,9 +63,12 @@ class MockGroupSettings : public QObject, public IGroupSettings
public:
QStringList getBlackList() const override;
void setBlackList(const QStringList& blist) override;
SIGNAL_IMPL(MockGroupSettings, blackListChanged, QStringList const& blist)
bool getShowGroupJoinLeaveMessages() const override { return true; };
void setShowGroupJoinLeaveMessages(bool newValue) override {};
SIGNAL_IMPL(MockGroupSettings, showGroupJoinLeaveMessagesChanged, bool show)
private:
QStringList blacklist;
};

Loading…
Cancel
Save