Decode header value in rcube_mime::get() by default (#1488511)

Conflicts:

	program/include/rcube_message_header.php
	program/include/rcube_mime.php

1 files changed, 21 insertions, 7 deletions

diff --git a/program/include/rcube_mime.php b/program/include/rcube_mime.php
index 538c8ea45..287bde851 100644
--- a/program/include/rcube_mime.php
+++ b/program/include/rcube_mime.php
@@ -34,7 +34,7 @@
  */
 class rcube_mime
 {
-    private static $default_charset = RCMAIL_CHARSET;
+    private static $default_charset;
 
 
     /**
@@ -42,12 +42,26 @@ class rcube_mime
      */
     function __construct($default_charset = null)
     {
-        if ($default_charset) {
-            self::$default_charset = $default_charset;
+        self::$default_charset = $default_charset;
+    }
+
+
+    /**
+     * Returns message/object character set name
+     *
+     * @return string Characted set name
+     */
+    public static function get_charset()
+    {
+        if (self::$default_charset) {
+            return self::$default_charset;
         }
-        else {
-            self::$default_charset = rcmail::get_instance()->config->get('default_charset', RCMAIL_CHARSET);
+
+        if ($charset = rcmail::get_instance()->config->get('default_charset')) {
+            return $charset;
         }
+
+        return RCMAIL_CHARSET;
     }
 
 
@@ -92,7 +106,7 @@ class rcube_mime
         if ($part->ctype_parameters['charset'])
             $struct->charset = $part->ctype_parameters['charset'];
 
-        $part_charset = $struct->charset ? $struct->charset : self::$default_charset;
+        $part_charset = $struct->charset ? $struct->charset : self::get_charset();
 
         // determine filename
         if (($filename = $part->d_parameters['filename']) || ($filename = $part->ctype_parameters['name'])) {
@@ -186,7 +200,7 @@ class rcube_mime
      */
     public static function decode_mime_string($input, $fallback = null)
     {
-        $default_charset = !empty($fallback) ? $fallback : self::$default_charset;
+        $default_charset = !empty($fallback) ? $fallback : self::get_charset();
 
         // rfc: all line breaks or other characters not found
         // in the Base64 Alphabet must be ignored by decoding software