Bug #67013
closed
felogin setting plugin.tx_felogin_pi1.redirectMode = refererDomains causes an exception
Added by Mike Wanush over 9 years ago.
Updated about 7 years ago.
Description
Line 692 (of the current git version) uses the following regular expression:
if (preg_match('/(^|\\.)/' . $d . '$', $redirect_domain)) {
As $d
in this case is a domain string, it causes an exception:
#1: PHP Warning: preg_match(): Unknown modifier 'w' in ...
The line should be changed to
if (preg_match('/(^|\\.)' . $d . '$/', $redirect_domain)) {
See attached felogin.patch.
Files
Can you please describe the full usecase?
- Status changed from New to Accepted
- Target version set to 6.2.13
Had a quick look, you're right of course.
Can you push the patch to our review system?
Funny, this traces back to 2009: #19728
- Status changed from Accepted to Under Review
Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/39600
Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/39600
Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/39600
Patch set 1 for branch TYPO3_6-2 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/39601
Patch set 2 for branch TYPO3_6-2 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/39601
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
- Status changed from Resolved to Closed
Also available in: Atom
PDF