Home » Database » SQL_Parser » Bug #4081
[PATCH] Added tinyint to synonyms
Details
| Submitted | 2005-04-06 15:06 UTC |
|---|---|
| From | epte at ruffdogs dot com |
| Assigned | epte |
| Status | Closed |
| Package | SQL_Parser |
| PHP Version | 4.3.8 |
| Roadmaps | (Not assigned) |
Comments
[2005-04-06 15:06 UTC] epte at ruffdogs dot com
Description:
------------
I needed to add tinyint to the dialect's synonyms list as well. Here's the patch:
Index: Dialect_MySQL.php
===================================================================
--- Dialect_MySQL.php (revision 2687)
+++ Dialect_MySQL.php (working copy)
@@ -115,10 +115,11 @@
'decimal'=>'numeric', 'dec'=>'numeric', 'numeric'=>'numeric',
'float'=>'float', 'real'=>'real', 'double'=>'real', 'int'=>'int',
'integer'=>'int', 'interval'=>'interval', 'smallint'=>'smallint',
- 'timestamp'=>'timestamp', 'bool'=>'bool', 'boolean'=>'bool', 'set'=>'set',
- 'enum'=>'enum', 'text'=>'text', 'char'=>'char', 'character'=>'char',
- 'varchar'=>'varchar', 'ascending'=>'asc', 'asc'=>'asc',
- 'descending'=>'desc', 'desc'=>'desc', 'date'=>'date', 'time'=>'time'),
+ 'tinyint'=>'tinyint', 'timestamp'=>'timestamp', 'bool'=>'bool',
+ 'boolean'=>'bool', 'set'=>'set', 'enum'=>'enum', 'text'=>'text',
+ 'char'=>'char', 'character'=>'char', 'varchar'=>'varchar',
+ 'ascending'=>'asc', 'asc'=>'asc', 'descending'=>'desc', 'desc'=>'desc',
+ 'date'=>'date', 'time'=>'time'),
'lexeropts'=>array('allowIdentFirstDigit'=>true),
[2005-04-06 15:11 UTC] epte at ruffdogs dot com
AND tinyint needed to be in the case statement. Here's that patch:
Index: Parser.php
===================================================================
--- Parser.php (revision 2688)
+++ Parser.php (working copy)
@@ -555,7 +555,7 @@
$fields[$name]['length'] = $values[0];
break;
case 'char': case 'varchar':
- case 'integer': case 'int':
+ case 'integer': case 'int': case 'tinyint':
if (sizeof($values) != 1) {
return $this->raiseError('Expected 1 parameter');