Merge pull request #20 from crazyfactory/prevent-emoji-validator

feat(latin validator): exclude c0 latin chars
crazyfactory · Dec 1, 2021 · 3619681 · 3619681
2 parents f33acc7 + bae722c
commit 3619681
Show file tree

Hide file tree

Showing 2 changed files with 41 additions and 1 deletion.
diff --git a/src/LatinCharValidator.php b/src/LatinCharValidator.php
@@ -4,8 +4,16 @@
 
 class LatinCharValidator
 {
+    /**
+     * exclude C0 controls (basic latin): https://unicode-table.com/en/blocks/basic-latin/
+     *
+     * @param string $txt
+     *
+     * @return bool
+     */
     public static function isValid(string $txt): bool
     {
-        return empty($txt) || preg_match('/^[\p{Latin}\p{Common}]+$/u', $txt) > 0;
+        return empty($txt) || (preg_match('/^[\p{Latin}\p{Common}]+$/u', $txt) > 0
+                && preg_match('/[\x{0000}-\x{001F}]/u', $txt) == 0);
     }
 }
diff --git a/tests/unit/LatinCharValidatorTest.php b/tests/unit/LatinCharValidatorTest.php
@@ -54,5 +54,37 @@ public function testIsValid()
         $this->assertFalse(LatinCharValidator::isValid('الْأَبْجَدِيَّة الْعَرَبِيَّة‎'));
         // Russian chars
         $this->assertFalse(LatinCharValidator::isValid('Меня зовут Мандли'));
+        // Basic Latin C0 controls
+        $c0 = [
+            '',
+            '',
+            '',
+            '',
+            '',
+            '',
+            '	',
+            '',
+            '',
+            '',
+            '',
+            '',
+            '',
+            '',
+            '',
+            '',
+            '',
+            '',
+            '',
+            '',
+            '',
+            '',
+            '',
+            '',
+            '',
+            ''
+        ];
+        foreach ($c0 as $item) {
+            $this->assertFalse(LatinCharValidator::isValid('Hello World ' . $item));
+        }
     }
 }