Reverting PR #51.

We should try to switch to filter_var in 2.x
matomo-org · Apr 22, 2013 · b84e9c8 · b84e9c8
1 parent 45fa8a5
commit b84e9c8
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/core/Piwik.php b/core/Piwik.php
@@ -2037,7 +2037,7 @@ static public function createAccessObject()
      */
     static public function isValidEmailString($email)
     {
-        return (preg_match("/^[a-zA-Z0-9_.+'-]+@[a-zA-Z0-9_.-]+\.[a-zA-Z]{2,7}$/D", $email) > 0);
+        return (preg_match('/^[a-zA-Z0-9_.+-]+@[a-zA-Z0-9_.-]+\.[a-zA-Z]{2,7}$/D', $email) > 0);
     }
 
     /**

diff --git a/tests/PHPUnit/Plugins/UsersManagerTest.php b/tests/PHPUnit/Plugins/UsersManagerTest.php
@@ -226,6 +226,7 @@ public function testAddUserWrongPassword($userLogin, $password, $email, $alias)
     public function getWrongEmailTestData()
     {
         return array(
+            array("geggeqgeqag", "geqgeagae", "ema'[email protected]", "alias"),
             array("geggeqgeqag", "geqgeagae", "@email.com", "alias"),
             array("geggeqgeqag", "geqgeagae", "[email protected]", "alias"),
             array("geggeqgeqag", "geqgeagae", "email@4.", "alias"),
-Original file line number
+Diff line change
@@ Expand Up / @@ -2037,7 +2037,7 @@ static public function createAccessObject() @@
          */
         static public function isValidEmailString($email)
         {
-            return (preg_match("/^[a-zA-Z0-9_.+'-]+@[a-zA-Z0-9_.-]+\.[a-zA-Z]{2,7}$/D", $email) > 0);
+            return (preg_match('/^[a-zA-Z0-9_.+-]+@[a-zA-Z0-9_.-]+\.[a-zA-Z]{2,7}$/D', $email) > 0);
         }
         /**
@@ Expand Down @@