Be less restrictive on vCard import, do not require FN when N exists

Aleksander Machniak

2013-03-24 ae0821f1dd15baacd436d3337eb26a41d72204d0

 program/lib/Roundcube/rcube_vcard.php

@@ -491,7 +491,9 @@
            if (preg_match('/^END:VCARD$/i', $line)) {
                // parse vcard
                $obj = new rcube_vcard(self::cleanup($vcard_block), $charset, true, self::$fieldmap);
                if (!empty($obj->displayname) || !empty($obj->email)) {
                // FN and N is required by vCard format (RFC 2426)
                // on import we can be less restrictive, let's addressbook decide
                if (!empty($obj->displayname) || !empty($obj->surname) || !empty($obj->firstname) || !empty($obj->email)) {
                    $out[] = $obj;
                }

@@ -513,7 +515,7 @@
     *
     * @return string Cleaned vcard block
     */
    private static function cleanup($vcard)
    public static function cleanup($vcard)
    {
        // Convert special types (like Skype) to normal type='skype' classes with this simple regex ;)
        $vcard = preg_replace(
@@ -770,15 +772,21 @@
     */
    private static function vcard_unquote($s, $sep = ';')
    {
        // break string into parts separated by $sep, but leave escaped $sep alone
        if (count($parts = explode($sep, strtr($s, array("\\$sep" => "\007")))) > 1) {
            foreach($parts as $s) {
                $result[] = self::vcard_unquote(strtr($s, array("\007" => "\\$sep")), $sep);
        // break string into parts separated by $sep
        if (!empty($sep)) {
            // Handle properly backslash escaping (#1488896)
            $rep1 = array("\\\\" => "\010", "\\$sep" => "\007");
            $rep2 = array("\007" => "\\$sep", "\010" => "\\\\");

            if (count($parts = explode($sep, strtr($s, $rep1))) > 1) {
                foreach ($parts as $s) {
                    $result[] = self::vcard_unquote(strtr($s, $rep2));
                }
                return $result;
            }
            return $result;
        }

        return strtr($s, array("\r" => '', '\\\\' => '\\', '\n' => "\n", '\N' => "\n", '\,' => ',', '\;' => ';', '\:' => ':'));
        return strtr($s, array("\r" => '', '\\\\' => '\\', '\n' => "\n", '\N' => "\n", '\,' => ',', '\;' => ';'));
    }

    /**