Clean up code as discussed with David

7fa92224 · Laurent Montel · 6737caaf · 7fa92224
Commit 7fa92224 authored Dec 02, 2014 by Laurent Montel 😁
Hide whitespace changes
Inline Side-by-side

Showing with 6 additions and 8 deletions

kmail/manageshowcollectionproperties.cpp kmail/manageshowcollectionproperties.cpp +6 -8

No files found.
--- a/kmail/manageshowcollectionproperties.cpp
+++ b/kmail/manageshowcollectionproperties.cpp
@@ -80,14 +80,12 @@ void ManageShowCollectionProperties::showCollectionProperties( const QString &pa
 {
    if ( !mMainWidget->currentFolder() )
        return;
-    Akonadi::Collection::Id id = mMainWidget->currentFolder()->collection().id();
-    if (mHashDialogBox.contains(id)) {
-        QPointer<Akonadi::CollectionPropertiesDialog> dlg = mHashDialogBox.value(id);
-        if (dlg) {
-            dlg->activateWindow();
-            dlg->raise();
-            return;
-        }
+    const Akonadi::Collection::Id id = mMainWidget->currentFolder()->collection().id();
+    QPointer<Akonadi::CollectionPropertiesDialog> dlg = mHashDialogBox.value(id);
+    if (dlg) {
+        dlg->activateWindow();
+        dlg->raise();
+        return;
    }
    if ( Solid::Networking::status() == Solid::Networking::Unconnected ) {
        KMessageBox::information(