r6073 breaks attachment naming

[sthen]

diff:/trunk/roundcubemail/program/steps/mail/get.inc@6072//trunk/roundcubemail/program/steps/mail/get.inc@6073 modifies the code which chooses an attachment filename.

In the old code a proper filename was used if provided, otherwise a filename was synthesized from subject/second part of mimetype.

Now the second part of mimetype is *always* appended, e.g. "foo.doc.msword".

Possible fix in diff below.

*EDIT* Had to mangle the diff, s/MXSS/MESS/, stupid trac spam blocker.

Index: program/steps/mail/get.inc
===================================================================
--- program/steps/mail/get.inc	(revision 6099)
+++ program/steps/mail/get.inc	(working copy)
@@ -152,10 +152,9 @@
       }
       else {
         $filename = ($MXSSAGE->subject ? $MXSSAGE->subject : 'roundcube');
+        $filename .= '.' . ($mimetype == 'text/plain' ? 'txt' : $ctype_secondary);
       }
 
-      $ext      = '.' . ($mimetype == 'text/plain' ? 'txt' : $ctype_secondary);
-      $filename .= $ext;
       $filename = preg_replace('[\r\n]', '', $filename);
 
       if ($browser->ie && $browser->ver < 7)

[alec]

Fixed in [b4f95a93].