mirror/php-e107
1
0
Fork 0
mirror of https://github.com/e107inc/e107.git synced 2025-04-21 21:21:54 +02:00
Code Issues Releases Wiki Activity

Merge pull request #5055 from Deltik/fix/5054

Browse Source 
Fixes #5054 db_verify::getIndex(): Support index_col_name optional parts
This commit is contained in:
Cameron 2023-08-17 11:52:47 -07:00 committed by GitHub
commit 068bf05e92
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 49 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
e107_handlers/db_verify_class.php
View File

@ -1065,9 +1065,8 @@ class db_verify
*/
function getIndex($data, $print = false)
{
// $regex = "/(?:(PRIMARY|UNIQUE|FULLTEXT))?[\s]*?KEY (?: ?`?([\w]*)`?)[\s]* ?(?:\([\s]?`?([\w,]*[\s]?)`?\))?,?/i";
// $regex = "/(?:(PRIMARY|UNIQUE|FULLTEXT|FOREIGN))?[\s]*?KEY (?: ?`?([\w]*)`?)[\s]* ?(?:\([\s]?([\w\s,`]*[\s]?)`?\))?,?/i";
$regex = "/(?:(PRIMARY|UNIQUE|FULLTEXT|FOREIGN))?[\s]*?(INDEX|KEY) (?: ?`?([\w]*)`?)[\s]* ?(?:\([\s]?([\w\s,`]*[\s]?)`?\))?,?/i";
$regex = "/(PRIMARY|UNIQUE|FULLTEXT|FOREIGN)?[\s]*?(INDEX|KEY)[\s]*(?:`?([\w]*)`?)?[\s]*?\(`?([\w]+)`?(?:\s*\(\d+\))?(?:\s*(ASC|DESC))?\)[\s]*,?/i";
preg_match_all($regex,$data,$m);
if (count($m) > 0)

47
e107_tests/tests/unit/db_verifyTest.php
View File

@ -263,6 +263,53 @@
$this->assertEquals($expected,$result);
}
/**
* @see https://github.com/e107inc/e107/issues/5054
*/
public function testGetIndexOptionalLengthAndSortOrder()
{
$data = <<<EOF
`field1` int(10) unsigned NOT NULL AUTO_INCREMENT,
`field2` varchar(100) NOT NULL DEFAULT '',
`field3` varchar(100) NOT NULL DEFAULT '',
`field4` varchar(100) NOT NULL DEFAULT '',
KEY (`field1`),
INDEX `field2` (`field2` DESC),
KEY `field3` (`field3` (100) ASC),
INDEX `field4` (`field4` (100)),
EOF;
$expected = array(
'field1' =>
array(
'type' => '',
'keyname' => 'field1',
'field' => 'field1',
),
'field2' =>
array(
'type' => '',
'keyname' => 'field2',
'field' => 'field2',
),
'field3' =>
array(
'type' => '',
'keyname' => 'field3',
'field' => 'field3',
),
'field4' =>
array(
'type' => '',
'keyname' => 'field4',
'field' => 'field4',
),
);
$result = $this->dbv->getIndex($data);
$this->assertEquals($expected,$result);
}
/**
* FIXME: This test has no assertions!
*/