Do not encode contacts menu mailto links

Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>

diff --git a/lib/private/Contacts/ContactsMenu/ActionFactory.php b/lib/private/Contacts/ContactsMenu/ActionFactory.php
index 6bc724bd4392e094b3e61f31fef3368d05667045..2841c5ad206d355338d9f76d9d63148530e5ada6 100644 (file)
--- a/lib/private/Contacts/ContactsMenu/ActionFactory.php
+++ b/lib/private/Contacts/ContactsMenu/ActionFactory.php
@@ -50,7 +50,7 @@ class ActionFactory implements IActionFactory {
         * @return ILinkAction
         */
        public function newEMailAction($icon, $name, $email) {
-               return $this->newLinkAction($icon, $name, 'mailto:' . urlencode($email));
+               return $this->newLinkAction($icon, $name, 'mailto:' . $email);
        }
 
 }

diff --git a/tests/lib/Contacts/ContactsMenu/ActionFactoryTest.php b/tests/lib/Contacts/ContactsMenu/ActionFactoryTest.php
index d1273c2b9adc3796512f9e384579c8b8137cb0e9..299946ed21c3e2ae9a2d884ba48e2a929596c9fc 100644 (file)
--- a/tests/lib/Contacts/ContactsMenu/ActionFactoryTest.php
+++ b/tests/lib/Contacts/ContactsMenu/ActionFactoryTest.php
@@ -61,7 +61,7 @@ class ActionFactoryTest extends TestCase {
                $this->assertInstanceOf(IAction::class, $action);
                $this->assertEquals($name, $action->getName());
                $this->assertEquals(10, $action->getPriority());
-               $this->assertEquals('mailto:user%40example.com', $action->getHref());
+               $this->assertEquals('mailto:user@example.com', $action->getHref());
        }
 
 }