[BUGFIX] Follow up: Use strrpos to find last occurence 39/34639/2
authorAndreas Fernandez <andreas.fernandez@aspedia.de>
Wed, 26 Nov 2014 15:07:19 +0000 (16:07 +0100)
committerOliver Hader <oliver.hader@typo3.org>
Thu, 27 Nov 2014 10:31:40 +0000 (11:31 +0100)
commitb9756f6733f17a894248c7b6d9735152ca6ae544
tree532fc931d856c122683d40d8666fb9083b4994c3
parentabaab88f48379fd7d7fd099c00e4eb867788caef
[BUGFIX] Follow up: Use strrpos to find last occurence

The patch #61654 introduced a bug, where strpos is mistakenly
used to get the last position of the currently processed field.
It must be changed to strrpos, otherwise the opening brace
breaks the SQL.

Also, a unit test for this case is added.

Resolves: #63345
Related: #61654
Releases: master, 6.2
Change-Id: Ibdbc7975b8f750060980f811cd0089b1a7b7de2b
Reviewed-on: http://review.typo3.org/34639
Reviewed-by: Mathias Schreiber <mathias.schreiber@wmdb.de>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Markus Klein <klein.t3@reelworx.at>
Tested-by: Markus Klein <klein.t3@reelworx.at>
Reviewed-by: Jigal van Hemert <jigal.van.hemert@typo3.org>
Reviewed-by: Oliver Hader <oliver.hader@typo3.org>
Tested-by: Oliver Hader <oliver.hader@typo3.org>
typo3/sysext/dbal/Classes/Database/SqlParser.php
typo3/sysext/dbal/Tests/Unit/Database/DatabaseConnectionOracleTest.php