[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-bugs-dist
Subject:    [Bug 64388] save body part attachment problem
From:       Till Adam <adam () kde ! org>
Date:       2003-10-03 15:07:52
[Download RAW message or body]

------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.
     
http://bugs.kde.org/show_bug.cgi?id=64388     
adam@kde.org changed:

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



------- Additional Comments From adam@kde.org  2003-10-03 17:07 -------
CVS commit by tilladam:  
 
Don't just silently bail out when the user tries to save a part of the mime 
tree that is not an attachment, such as an html part. Ask whether that was 
a mistake or not instead. 
 
Carsten, ok with you? 
 
CCMAIL: 64388-done@bugs.kde.org 
 
 
  M +11 -2     kmcommands.cpp   1.69 
 
 
--- kdepim/kmail/kmcommands.cpp  #1.68:1.69 
@@ -1690,7 +1690,16 @@ void KMSaveAttachmentsCommand::slotSaveA 
     // we can't do the check earlier as we first need to load the mimeheader 
     // for imap attachments to do this check 
+     
+    /* Ask the user how to proceed, she could be wanting to save an html part, 
+     * or somesuch. -till */ 
     if ( s.isEmpty() ) { 
+      if ( KMessageBox::warningYesNo( mParent, 
+            i18n( "One of the parts of the message that you want to save does " 
+                  "not appear to be an attachment. Are you sure you want " 
+                  "to save it anyway? (If it is an html part, for example.)" )  
+            ) == KMessageBox::No ) { 
       ++itr; 
       continue; 
+      } 
     }
[prev in list] [next in list] [prev in thread] [next in thread]