MAGETWO-31904: There is no validation for 'Table prefix' on Step 2: Add a Database during instalation...

Ivan Gavryshko · Ivan Gavryshko · commit c1b30b7db108 · 2015-02-18T11:22:39.000-06:00
- removed unnesessary statement.
diff --git a/setup/src/Magento/Setup/Model/Installer.php b/setup/src/Magento/Setup/Model/Installer.php
@@ -768,7 +768,7 @@ public function checkDatabaseTablePrefix($prefix)
     {
         //The table prefix should contain only letters (a-z), numbers (0-9) or underscores (_);
         // the first character should be a letter.
-        if ($prefix !== '' && !preg_match('/^([a-zA-Z])([[:alnum:]]+)([[:alnum:]_]+)$/', $prefix)) {
+        if ($prefix !== '' && !preg_match('/^([a-zA-Z])([[:alnum:]_]+)$/', $prefix)) {
             throw new \InvalidArgumentException('Please correct the table prefix format.');
         }
 

Original file line number	Diff line number	Diff line change
`@@ -768,7 +768,7 @@ public function checkDatabaseTablePrefix($prefix)`
`768`	`768`	`{`
`769`	`769`	`//The table prefix should contain only letters (a-z), numbers (0-9) or underscores (_);`
`770`	`770`	`// the first character should be a letter.`
`771`		`- if ($prefix !== '' && !preg_match('/^([a-zA-Z])([[:alnum:]]+)([[:alnum:]_]+)$/', $prefix)) {`
	`771`	`+ if ($prefix !== '' && !preg_match('/^([a-zA-Z])([[:alnum:]_]+)$/', $prefix)) {`
`772`	`772`	`throw new \InvalidArgumentException('Please correct the table prefix format.');`
`773`	`773`	`}`
`774`	`774`