[Digikam-devel] [Bug 147439] It is too easy to delete a search

[Arnd Baecker]

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
         
http://bugs.kde.org/show_bug.cgi?id=147439         
arnd.baecker web de changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED



------- Additional Comments From arnd.baecker web de  2007-08-30 15:50 -------
SVN commit 706464 by abaecker:

Add Yes/No Warning when deleting a search to avoid accidental removal of complex searches.
BUG: 147439


 M  +13 -0     searchfolderview.cpp  


--- branches/extragear/kde3/graphics/digikam/digikam/searchfolderview.cpp #706463:706464
 @ -35,6 +35,7  @
 #include <kglobal.h>
 #include <kiconloader.h>
 #include <kdeversion.h>
+#include <kmessagebox.h>
 
 #if KDE_IS_VERSION(3,2,0)
 #include <kinputdialog.h>
 @ -270,6 +271,18  @
     if (!album)
         return;
 
+    // Make sure that a complicated search is not deleted accidentally
+    int result =KMessageBox::warningYesNo( this, i18n("Are you sure you want to "
+                                                      "delete the selected search "
+                                                      "\"%1\"?")
+                                           .arg(album->title()),
+                                       i18n("Delete Search?"),
+                                       i18n("Delete"),
+                                       KStdGuiItem::cancel() );
+
+    if (result != KMessageBox::Yes)
+        return;
+
     AlbumManager::instance()->deleteSAlbum(album);
 }