-
Notifications
You must be signed in to change notification settings - Fork 2.5k
New issue
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
XmlDriver incorrectly parses boolean option values #6129
Comments
@kalimatas can this be converted into a test case? |
I'm on it right now. |
Tests in #6130 |
I would like also to fix it, though I'm not sure, if maintaining an array of option names, that are supposed to be boolean in the |
@kalimatas give it a shot - it can be a quick fix with a later refactoring, but if there's a possible BC break, we need to expose that as well. |
I created a generic test for all drivers. |
Closing as per work done in #6130 |
#6129 Added unit test for boolean option values.
Note: this is only merged into |
I have a mapping for a column like that:
Despite that I specified
false
as a value offixed
option, it will betrue
, because inXmlDriver::_parseOptions
method the stringfalse
will be casted totrue
with(bool)
:It probably should be:
The text was updated successfully, but these errors were encountered: