- Don't set attachment's charset in Content-type header (#1487122)

alecpl

2010-11-18 0f9d8ca40de826733f0507b9706d18dd1e8719f4

- Don't set attachment's charset in Content-type header (#1487122)

 CHANGELOG

@@ -73,6 +73,7 @@
- Fix focused elements aren't unfocused when clicking on the list (#1487123)
- Fix error in MSSQL DDL scripts (#1487112)
- Lock submit button in onsubmit event on login page (#1487036)
- Don't set attachment's charset in Content-type header (#1487122)

RELEASE 0.4.2
-------------

 program/include/rcube_shared.inc

@@ -477,9 +477,9 @@
        $mime_type = $failover;
    }
    else {
        // sometimes content-type contains charset definition,
        // remove useless "charset=binary", should we remove any charset def. here?
        $mime_type = preg_replace('/; charset=binary;*/i', '', $mime_type);
        // Sometimes (PHP-5.3?) content-type contains charset definition,
        // Remove it (#1487122) also "charset=binary" is useless
        $mime_type = array_shift(preg_split('/[; ]/', $mime_type));
    }

    return $mime_type;

 program/steps/mail/sendmail.inc

@@ -503,7 +503,7 @@
        ($attachment['data'] ? false : true),
        ($ctype == 'message/rfc822' ? '8bit' : 'base64'),
        ($ctype == 'message/rfc822' ? 'inline' : 'attachment'),
        $message_charset, '', '', 
        '', '', '',
        $CONFIG['mime_param_folding'] ? 'quoted-printable' : NULL,
        $CONFIG['mime_param_folding'] == 2 ? 'quoted-printable' : NULL
      );