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

db_verify::getIndex(): Support index_col_name optional parts

Browse Source 
In the MariaDB `CREATE TABLE` [`index_definition`](https://mariadb.com/kb/en/create-table/#index-definitions),
 the `index_col_name` could have an optional length and a sort order:

```
index_definition:
    {INDEX|KEY} [index_name] [index_type] (index_col_name,...) [index_option] ...
  {{{|}}} {FULLTEXT|SPATIAL} [INDEX|KEY] [index_name] (index_col_name,...) [index_option] ...
  {{{|}}} [CONSTRAINT [symbol]] PRIMARY KEY [index_type] (index_col_name,...) [index_option] ...
  {{{|}}} [CONSTRAINT [symbol]] UNIQUE [INDEX|KEY] [index_name] [index_type] (index_col_name,...) [index_option] ...
  {{{|}}} [CONSTRAINT [symbol]] FOREIGN KEY [index_name] (index_col_name,...) reference_definition

index_col_name:
    col_name [(length)] [ASC | DESC]

index_type:
    USING {BTREE | HASH | RTREE}

index_option:
    [ KEY_BLOCK_SIZE [=] value
  {{{|}}} index_type
  {{{|}}} WITH PARSER parser_name
  {{{|}}} COMMENT 'string'
  {{{|}}} CLUSTERING={YES| NO} ]
  [ IGNORED | NOT IGNORED ]

reference_definition:
    REFERENCES tbl_name (index_col_name,...)
      [MATCH FULL | MATCH PARTIAL | MATCH SIMPLE]
      [ON DELETE reference_option]
      [ON UPDATE reference_option]

reference_option:
    RESTRICT | CASCADE | SET NULL | NO ACTION
```

`db_verify::getIndex()` didn't handle this possibility, leading to a
database validity check failure despite the index actually existing.

Fixes: https://github.com/e107inc/e107/issues/5054
This commit is contained in:
Nick Liu
2023-08-02 15:03:23 +02:00
parent 5ff319cd5c
commit b623868756
2 changed files with 49 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

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!
*/