Fix so mobile phone has TYPE=CELL in exported vCard (#1488812)

CHANGELOG

@@ -1,6 +1,7 @@
 CHANGELOG Roundcube Webmail
 ===========================
 
+- Fix so mobile phone has TYPE=CELL in exported vCard (#1488812)
 - Support contacts import from CSV file (#1486399)
 - Improved keep-alive action. Now the interval is based on session_lifetime (#1488507)
 - Added cross-task 'refresh' request for system state updates (#1488507)

program/include/rcube_vcard.php

@@ -52,7 +52,7 @@ class rcube_vcard
     'edit'        => 'X-AB-EDIT',
   );
   private $typemap = array('IPHONE' => 'mobile', 'CELL' => 'mobile', 'WORK,FAX' => 'workfax');
-  private $phonetypemap = array('HOME1' => 'HOME', 'BUSINESS1' => 'WORK', 'BUSINESS2' => 'WORK2', 'BUSINESSFAX' => 'WORK,FAX');
+  private $phonetypemap = array('HOME1' => 'HOME', 'BUSINESS1' => 'WORK', 'BUSINESS2' => 'WORK2', 'BUSINESSFAX' => 'WORK,FAX', 'MOBILE' => 'CELL');
   private $addresstypemap = array('BUSINESS' => 'WORK');
   private $immap = array('X-JABBER' => 'jabber', 'X-ICQ' => 'icq', 'X-MSN' => 'msn', 'X-AIM' => 'aim', 'X-YAHOO' => 'yahoo', 'X-SKYPE' => 'skype', 'X-SKYPE-USERNAME' => 'skype');
 

tests/Framework/VCard.php

@@ -33,6 +33,23 @@ class Framework_VCard extends PHPUnit_Framework_TestCase
         $this->assertEquals("roundcube@gmail.com", $vcard->email[0], "Use PREF e-mail as primary");
     }
 
+    /**
+     * Make sure MOBILE phone is returned as CELL (as specified in standard)
+     */
+    function test_parse_three()
+    {
+        $vcard = new rcube_vcard(file_get_contents($this->_srcpath('johndoe.vcf')), null);
+
+        $vcf = $vcard->export();
+        $this->assertRegExp('/TEL;CELL:\+987654321/', $vcf, "Return CELL instead of MOBILE (import)");
+
+        $vcard = new rcube_vcard();
+        $vcard->set('phone', '+987654321', 'MOBILE');
+
+        $vcf = $vcard->export();
+        $this->assertRegExp('/TEL;TYPE=CELL:\+987654321/', $vcf, "Return CELL instead of MOBILE (set)");
+    }
+
     function test_import()
     {
         $input = file_get_contents($this->_srcpath('apple.vcf'));

tests/src/johndoe.vcf

@@ -6,6 +6,7 @@ ORG:roundcube.net;
 EMAIL;INTERNET;WORK:inbox@roundcube.net
 EMAIL;INTERNET;HOME;TYPE=pref:roundcube@gmail.com
 TEL;WORK:+123456789
+TEL;CELL:+987654321
 ADR;WORK:;;The street;Hometown;;5555;Cayman Islands
 NOTE:The notes...
 END:VCARD