We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Current return type hint of the Phalcon\Db\Column::getSize implies that method must return a integer value:
Phalcon\Db\Column::getSize
cphalcon/phalcon/Db/Column.zep
Lines 272 to 277 in 5c34a6c
But this method MAY return a string for enum columns:
use Phalcon\Db\Column; $column = new Column('field_enum', [ 'type' => Column::TYPE_ENUM, 'isNumeric' => false, 'size' => "'xs','s','m','l','xl','internal'", 'default' => null, 'unsigned' => false, 'notNull' => false, 'autoIncrement' => false, 'primary' => false, 'first' => false, 'after' => 'field_decimal_default', 'bindType' => Column::BIND_PARAM_STR, 'typeValues' => 'c', 'typeReference' => Column::TYPE_ENUM, ]); var_dump($column->getSize()); // string
This leads to a fatal error:
PHP Fatal error: Return value of Phalcon\Db\Column::getSize() must be of the type int, string returned in Unknown on line 0
The text was updated successfully, but these errors were encountered:
Fixed return type hint for Column::getSize
c633aa9
See: #15019
Fixed in 4.0.x
Sorry, something went wrong.
08aed34
sergeyklay
Successfully merging a pull request may close this issue.
Current return type hint of the
Phalcon\Db\Column::getSize
implies that method must return a integer value:cphalcon/phalcon/Db/Column.zep
Lines 272 to 277 in 5c34a6c
But this method MAY return a string for enum columns:
This leads to a fatal error:
The text was updated successfully, but these errors were encountered: