Changeset 435c315 in github

Timestamp:

Oct 20, 2010 4:58:21 AM (3 years ago)

Author:

alecpl <alec@…>

Branches:

master, HEAD, courier-fix, dev-browser-capabilities, pdo, release-0.6, release-0.7, release-0.8

Children:

8bac7e9

Parents:

13326b7

Message:

• Fix decoding of e-mail address strings in message headers (#1487068)

Files:

• CHANGELOG (modified) (1 diff)
• program/include/rcube_imap.php (modified) (2 diffs)

CHANGELOG

@@ -36 +36 @@
 - Add ANNOTATEMORE extension support into IMAP classes (draft-daboo-imap-annotatemore)
 - Add METADATA extension support into IMAP classes (RFC5464)
+- Fix decoding of e-mail address strings in message headers (#1487068)
 
 RELEASE 0.4.2

program/include/rcube_imap.php

@@ -4001 +4001 @@
             $j++;
             $address = trim($val['address']);
-            $name = trim($val['name']);
-
-            if ($name && preg_match('/^[\'"]/', $name) && preg_match('/[\'"]$/', $name))
-                $name = trim($name, '\'"');
+            $name    = trim($val['name']);
 
             if ($name && $address && $name != $address)
@@ -4420 +4417 @@
 
         foreach ($a as $key => $val) {
-            $val = preg_replace("/([\"\w])</", "$1 <", $val);
-            $sub_a = rcube_explode_quoted_string(' ', $decode ? $this->decode_header($val) : $val);
-            $result[$key]['name'] = '';
-
-            foreach ($sub_a as $k => $v) {
-                    // use angle brackets in regexp to not handle names with @ sign
-                if (preg_match('/^<\S+@\S+>$/', $v))
-                    $result[$key]['address'] = trim($v, '<>');
-                else
-                    $result[$key]['name'] .= (empty($result[$key]['name'])?'':' ').str_replace("\"",'',stripslashes($v));
-            }
-
-//          if (empty($result[$key]['name']))
-//              $result[$key]['name'] = $result[$key]['address'];
-            if (empty($result[$key]['address']))
-                $result[$key]['address'] = $result[$key]['name'];
+            $name    = '';
+            $address = '';
+            $val     = trim($val);
+
+            if (preg_match('/(.*)<(\S+@\S+)>$/', $val, $m)) {
+                $address = $m[2];
+                $name    = trim($m[1]);
+            }
+            else if (preg_match('/^(\S+@\S+)$/', $val, $m)) {
+                $address = $m[1];
+                $name    = '';
+            }
+            else {
+                $name = $val;
+            }
+
+            // dequote and/or decode name
+            if ($name) {
+                if ($name[0] == '"') {
+                    $name = substr($name, 1, -1);
+                    $name = stripslashes($name);
+                }
+                else if ($decode) {
+                    $name = $this->decode_header($name);
+                }
+            }
+
+            if (!$address && $name) {
+                $address = $name;
+            }
+
+            if ($address) {
+                $result[$key] = array('name' => $name, 'address' => $address);
+            }
         }