Skip to content
Snippets Groups Projects
Unverified Commit 9ed106f6 authored by Roeland Jago Douma's avatar Roeland Jago Douma Committed by GitHub
Browse files

Merge pull request #19176 from nextcloud/fix/contacts-menu-mailt-encoding

Do not encode contacts menu mailto links
parents 792c32aa 87ec729f
No related branches found
No related tags found
No related merge requests found
...@@ -50,7 +50,7 @@ class ActionFactory implements IActionFactory { ...@@ -50,7 +50,7 @@ class ActionFactory implements IActionFactory {
* @return ILinkAction * @return ILinkAction
*/ */
public function newEMailAction($icon, $name, $email) { public function newEMailAction($icon, $name, $email) {
return $this->newLinkAction($icon, $name, 'mailto:' . urlencode($email)); return $this->newLinkAction($icon, $name, 'mailto:' . $email);
} }
} }
...@@ -61,7 +61,7 @@ class ActionFactoryTest extends TestCase { ...@@ -61,7 +61,7 @@ class ActionFactoryTest extends TestCase {
$this->assertInstanceOf(IAction::class, $action); $this->assertInstanceOf(IAction::class, $action);
$this->assertEquals($name, $action->getName()); $this->assertEquals($name, $action->getName());
$this->assertEquals(10, $action->getPriority()); $this->assertEquals(10, $action->getPriority());
$this->assertEquals('mailto:user%40example.com', $action->getHref()); $this->assertEquals('mailto:user@example.com', $action->getHref());
} }
} }
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment