You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
if (false === strpos($this->parseFormat, '|')) {
$this->parseFormat .= '|';
}
According to notes in the php manual, the trailing | does not work correctly on all php versions:
"It seems that a pipe ('|') option in formating string works only with PHP version 5.3.8 and newer.
We had an issue with it on versions 5.3.2, 5.3.3, 5.3.6. Yet it was fine with 5.3.8 and 5.3.10."
This fix broke my applications running on 5.3.3 and 5.3.5 applications. Commenting out the above line fixed the problem.
Since 5.5.3 is officially supported by S2 we might consider an adjustment somewhere.
The text was updated successfully, but these errors were encountered:
This PR was merged into the 2.1 branch.
Commits
-------
ad29df5 [Form] Fixed DateTimeToStringTransformer parsing on PHP < 5.3.8
Discussion
----------
[Form] Fixed DateTimeToStringTransformer parsing on PHP < 5.3.8
Bug fix: yes
Feature addition: no
Backwards compatibility break: no
Symfony2 tests pass: yes
Fixes the following tickets: #6351, see comments in b20c5ca
Todo: -
License of the code: MIT
Documentation PR: -
The subject commit added (among other things):
According to notes in the php manual, the trailing | does not work correctly on all php versions:
This fix broke my applications running on 5.3.3 and 5.3.5 applications. Commenting out the above line fixed the problem.
Since 5.5.3 is officially supported by S2 we might consider an adjustment somewhere.
The text was updated successfully, but these errors were encountered: