From 66510e20d36cb8da4f3012ef063de7bfce9b51aa Mon Sep 17 00:00:00 2001 From: Aleksander Machniak <alec@alec.pl> Date: Wed, 09 May 2012 07:52:28 -0400 Subject: [PATCH] Merge pull request #1 from Brianetta/de-svn --- program/include/rcube_vcard.php | 65 +++++++++++++++++++++++--------- 1 files changed, 47 insertions(+), 18 deletions(-) diff --git a/program/include/rcube_vcard.php b/program/include/rcube_vcard.php index 283da1d..e453af7 100644 --- a/program/include/rcube_vcard.php +++ b/program/include/rcube_vcard.php @@ -6,7 +6,10 @@ | | | This file is part of the Roundcube Webmail client | | Copyright (C) 2008-2011, The Roundcube Dev Team | - | Licensed under the GNU GPL | + | | + | Licensed under the GNU General Public License version 3 or | + | any later version with exceptions for skins & plugins. | + | See the README file for a full license statement. | | | | PURPOSE: | | Logical representation of a vcard address record | @@ -292,11 +295,10 @@ case 'photo': if (strpos($value, 'http:') === 0) { // TODO: fetch file from URL and save it locally? - $this->raw['PHOTO'][0] = array(0 => $value, 'URL' => true); + $this->raw['PHOTO'][0] = array(0 => $value, 'url' => true); } else { - $encoded = !preg_match('![^a-z0-9/=+-]!i', $value); - $this->raw['PHOTO'][0] = array(0 => $encoded ? $value : base64_encode($value), 'BASE64' => true); + $this->raw['PHOTO'][0] = array(0 => $value, 'base64' => (bool) preg_match('![^a-z0-9/=+-]!i', $value)); } break; @@ -386,7 +388,7 @@ if (is_array($subnode) && (($charset = $force_charset) || ($subnode['charset'] && ($charset = $subnode['charset'][0])))) { foreach ($subnode as $j => $value) { if (is_numeric($j) && is_string($value)) - $card[$key][$i][$j] = rcube_charset_convert($value, $charset); + $card[$key][$i][$j] = rcube_charset::convert($value, $charset); } unset($card[$key][$i]['charset']); } @@ -422,7 +424,7 @@ $charset = null; // detect charset and convert to utf-8 else if (($charset = self::detect_encoding($data)) && $charset != RCMAIL_CHARSET) { - $data = rcube_charset_convert($data, $charset); + $data = rcube_charset::convert($data, $charset); $data = preg_replace(array('/^[\xFE\xFF]{2}/', '/^\xEF\xBB\xBF/', '/^\x00+/'), '', $data); // also remove BOM $charset = RCMAIL_CHARSET; } @@ -549,26 +551,45 @@ if (preg_match_all('/([^\\;]+);?/', $line[1], $regs2)) { $entry = array(); $field = strtoupper($regs2[1][0]); + $enc = null; foreach($regs2[1] as $attrid => $attr) { if ((list($key, $value) = explode('=', $attr)) && $value) { $value = trim($value); if ($key == 'ENCODING') { // add next line(s) to value string if QP line end detected - while ($value == 'QUOTED-PRINTABLE' && preg_match('/=$/', $lines[$i])) + if ($value == 'QUOTED-PRINTABLE') { + while (preg_match('/=$/', $lines[$i])) $line[2] .= "\n" . $lines[++$i]; - - $line[2] = self::decode_value($line[2], $value); + } + $enc = $value; } - else - $entry[strtolower($key)] = array_merge((array)$entry[strtolower($key)], (array)self::vcard_unquote($value, ',')); + else { + $lc_key = strtolower($key); + $entry[$lc_key] = array_merge((array)$entry[$lc_key], (array)self::vcard_unquote($value, ',')); + } } else if ($attrid > 0) { - $entry[$key] = true; // true means attr without =value + $entry[strtolower($key)] = true; // true means attr without =value } } - $entry = array_merge($entry, (array)self::vcard_unquote($line[2])); + // decode value + if ($enc || !empty($entry['base64'])) { + // save encoding type (#1488432) + if ($enc == 'B') { + $entry['encoding'] = 'B'; + // should we use vCard 3.0 instead? + // $entry['base64'] = true; + } + $line[2] = self::decode_value($line[2], $enc ? $enc : 'base64'); + } + + if ($enc != 'B' && empty($entry['base64'])) { + $line[2] = self::vcard_unquote($line[2]); + } + + $entry = array_merge($entry, (array) $line[2]); $data[$field][] = $entry; } } @@ -593,6 +614,7 @@ return quoted_printable_decode($value); case 'base64': + case 'b': self::$values_decoded = true; return base64_decode($value); @@ -624,13 +646,20 @@ if (is_array($entry)) { $value = array(); foreach($entry as $attrname => $attrvalues) { - if (is_int($attrname)) + if (is_int($attrname)) { + if (!empty($entry['base64']) || $entry['encoding'] == 'B') { + $attrvalues = base64_encode($attrvalues); + } $value[] = $attrvalues; - elseif ($attrvalues === true) - $attr .= ";$attrname"; // true means just tag, not tag=value, as in PHOTO;BASE64:... + } + else if (is_bool($attrvalues)) { + if ($attrvalues) { + $attr .= strtoupper(";$attrname"); // true means just tag, not tag=value, as in PHOTO;BASE64:... + } + } else { foreach((array)$attrvalues as $attrvalue) - $attr .= ";$attrname=" . self::vcard_quote($attrvalue, ','); + $attr .= strtoupper(";$attrname=") . self::vcard_quote($attrvalue, ','); } } } @@ -777,7 +806,7 @@ )*\z/xs', substr($string, 0, 2048))) return 'UTF-8'; - return rcmail::get_instance()->config->get('default_charset', 'ISO-8859-1'); # fallback to Latin-1 + return rcube::get_instance()->config->get('default_charset', 'ISO-8859-1'); # fallback to Latin-1 } } -- Gitblit v1.9.1