From 383453148806fd29552eb9ef9fd5a16424c91ab4 Mon Sep 17 00:00:00 2001
From: Felix Geyer <debfx@fobos.de>
Date: Fri, 25 May 2012 13:32:37 +0200
Subject: [PATCH] Simplify MainWindow::setMenuActionState().

---
 src/gui/MainWindow.cpp | 56 +++++++++---------------------------------
 1 file changed, 12 insertions(+), 44 deletions(-)

diff --git a/src/gui/MainWindow.cpp b/src/gui/MainWindow.cpp
index 0e65da6ac..2e67e4fdc 100644
--- a/src/gui/MainWindow.cpp
+++ b/src/gui/MainWindow.cpp
@@ -113,57 +113,25 @@ void MainWindow::setMenuActionState(DatabaseWidget::Mode mode)
         }
 
         switch (mode) {
-        case DatabaseWidget::ViewMode:
-            if (dbWidget->entryView()->inSearch()) {
-                m_ui->actionEntryNew->setEnabled(false);
-            }
-            else {
-                m_ui->actionEntryNew->setEnabled(true);
-            }
-            if (dbWidget->entryView()->inSearch() ||
-                    !dbWidget->entryView()->isSingleEntrySelected()) {
-                m_ui->actionEntryClone->setEnabled(false);
-            }
-            else {
-                m_ui->actionEntryClone->setEnabled(true);
-            }
+        case DatabaseWidget::ViewMode: {
+            bool inSearch = dbWidget->entryView()->inSearch();
+            bool singleEntry = dbWidget->entryView()->isSingleEntrySelected();
 
-            if (dbWidget->entryView()->isSingleEntrySelected()) {
-               m_ui->actionEntryEdit->setEnabled(true);
-               m_ui->actionEntryDelete->setEnabled(true);
-            }
-            else {
-                m_ui->actionEntryEdit->setEnabled(false);
-                m_ui->actionEntryDelete->setEnabled(false);
-            }
-
-            if (dbWidget->entryView()->inSearch()) {
-                m_ui->actionGroupNew->setEnabled(false);
-                m_ui->actionGroupEdit->setEnabled(false);
-                m_ui->actionGroupDelete->setEnabled(false);
-            }
-            else {
-                m_ui->actionGroupNew->setEnabled(true);
-                m_ui->actionGroupEdit->setEnabled(true);
-                if (dbWidget->canDeleteCurrentGoup()) {
-                    m_ui->actionGroupDelete->setEnabled(true);
-                }
-                else {
-                    m_ui->actionGroupDelete->setEnabled(false);
-                }
-            }
+            m_ui->actionEntryNew->setEnabled(!inSearch);
+            m_ui->actionEntryClone->setEnabled(!inSearch && singleEntry);
+            m_ui->actionEntryEdit->setEnabled(singleEntry);
+            m_ui->actionEntryDelete->setEnabled(singleEntry);
+            m_ui->actionGroupNew->setEnabled(!inSearch);
+            m_ui->actionGroupEdit->setEnabled(!inSearch);
+            m_ui->actionGroupDelete->setEnabled(!inSearch && dbWidget->canDeleteCurrentGoup());
             m_ui->actionSearch->setEnabled(true);
-            if (dbWidget->entryView()->inSearch()) {
-                m_ui->actionSearch->setChecked(true);
-            }
-            else {
-                m_ui->actionSearch->setChecked(false);
-            }
+            m_ui->actionSearch->setChecked(inSearch);
             m_ui->actionChangeMasterKey->setEnabled(true);
             m_ui->actionChangeDatabaseSettings->setEnabled(true);
             m_ui->actionDatabaseSave->setEnabled(true);
             m_ui->actionDatabaseSaveAs->setEnabled(true);
             break;
+        }
         case DatabaseWidget::EditMode:
             m_ui->actionEntryNew->setEnabled(false);
             m_ui->actionGroupNew->setEnabled(false);