Move check for mono icons into separate function

owncloud · Apr 26, 2017 · 72a3e08 · 72a3e08
1 parent 4202387
commit 72a3e08
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 3 deletions.
diff --git a/src/gui/generalsettings.cpp b/src/gui/generalsettings.cpp
@@ -83,9 +83,7 @@ GeneralSettings::GeneralSettings(QWidget *parent) :
 
     // OEM themes are not obliged to ship mono icons, so there
     // is no point in offering an option
-    QString themeDir = QString::fromLatin1(":/client/theme/%1/")
-            .arg(Theme::instance()->systrayIconFlavor(true));
-    _ui->monoIconsCheckBox->setVisible(QDir(themeDir).exists());
+    _ui->monoIconsCheckBox->setVisible(Theme::instance()->monoIconsAvailable());
 
     connect(_ui->ignoredFilesButton, SIGNAL(clicked()), SLOT(slotIgnoreFilesEditor()));
 

diff --git a/src/libsync/theme.cpp b/src/libsync/theme.cpp
@@ -265,6 +265,12 @@ bool Theme::systrayUseMonoIcons() const
     return _mono;
 }
 
+bool Theme::monoIconsAvailable() const
+{
+    QString themeDir = QString::fromLatin1(":/client/theme/%1/").arg(Theme::instance()->systrayIconFlavor(true));
+    return QDir(themeDir).exists();
+}
+
 QString Theme::updateCheckUrl() const
 {
     return QLatin1String("https://updates.owncloud.com/client/");

diff --git a/src/libsync/theme.h b/src/libsync/theme.h
@@ -201,6 +201,11 @@ class OWNCLOUDSYNC_EXPORT Theme : public QObject
      */
     bool systrayUseMonoIcons() const;
 
+    /**
+     * Check if mono icons are available
+     */
+    bool monoIconsAvailable() const;
+
     /**
      * @brief Where to check for new Updates.
      */