Add a ! to preg_match of english-name value check

Crizz0 · Crizz0 · commit a78a007caf6b · 2025-09-26T19:51:51.000+02:00
diff --git a/src/Phpbb/TranslationValidator/Validator/FileValidator.php b/src/Phpbb/TranslationValidator/Validator/FileValidator.php
@@ -641,7 +641,7 @@ public function validateJsonFile($originFile)
             $this->output->addMessage(Output::FATAL, 'Language iso is not valid', $originFile);
         }
         // Check for english name
-        if ($jsonContent['extra']['english-name'] == '' || preg_match('/^[a-zA-Z\s]+$/', $jsonContent['extra']['english-name']))
+        if ($jsonContent['extra']['english-name'] == '' || !preg_match('/^[a-zA-Z\s]+$/', $jsonContent['extra']['english-name']))
         {
             $this->output->addMessage(Output::ERROR, 'The english-name value should only contain letters aA-zZ and spaces.', $originFile);
         }

Original file line number	Diff line number	Diff line change
`@@ -641,7 +641,7 @@ public function validateJsonFile($originFile)`
`641`	`641`	`$this->output->addMessage(Output::FATAL, 'Language iso is not valid', $originFile);`
`642`	`642`	`}`
`643`	`643`	`// Check for english name`
`644`		`- if ($jsonContent['extra']['english-name'] == '' \|\| preg_match('/^[a-zA-Z\s]+$/', $jsonContent['extra']['english-name']))`
	`644`	`+ if ($jsonContent['extra']['english-name'] == '' \|\| !preg_match('/^[a-zA-Z\s]+$/', $jsonContent['extra']['english-name']))`
`645`	`645`	`{`
`646`	`646`	`$this->output->addMessage(Output::ERROR, 'The english-name value should only contain letters aA-zZ and spaces.', $originFile);`
`647`	`647`	`}`