Task #36793

Add/drop usage of preg_quote() where needed

Added by Stefan Neufeind about 1 year ago. Updated about 1 month ago.

Status:Resolved Start date:2012-05-03
Priority:Should have Due date:
Assignee:- % Done:

100%

Category:-
Target version:-
TYPO3 Version:4.5 Complexity:
PHP Version:
Votes: 0

Description

In some cases it might be unusual the delimiter-char might appear in the string, but it's done for consistency.

When using CR of LF calling preg_quote() does not change a thing. So it can be left out.


Related issues

duplicated by Core - Bug #36245: Escape variable in regular expression Closed 2012-04-17

Associated revisions

Revision 292b6d2e
Added by Stefan Neufeind about 1 month ago

[TASK] Add/drop usage of preg_quote() where needed

In some places preg_quote() is not used or is used incorrectly.
In other places there are preg_quote() calls which had no
real function.

Change-Id: If201394036d17a68cea41903f849c6c5ea11b069
Releases: 4.5, 4.6, 4.7, 6.0, 6.1
Resolves: #36793
Reviewed-on: https://review.typo3.org/10943
Reviewed-by: Dmitry Dulepov
Reviewed-by: Markus Klein
Tested-by: Markus Klein
Tested-by: Philipp Gampe
Reviewed-by: Philipp Gampe
Reviewed-by: Jigal van Hemert
Tested-by: Jigal van Hemert

Revision c40805d9
Added by Stefan Neufeind about 1 month ago

[TASK] Add/drop usage of preg_quote() where needed

In some places preg_quote() is not used or is used incorrectly.
In other places there are preg_quote() calls which had no
real function.

Change-Id: If201394036d17a68cea41903f849c6c5ea11b069
Releases: 4.5, 4.6, 4.7, 6.0, 6.1
Resolves: #36793
Reviewed-on: https://review.typo3.org/19925
Reviewed-by: Jigal van Hemert
Tested-by: Jigal van Hemert

Revision b415c23f
Added by Jigal van Hemert about 1 month ago

[TASK] Add/drop usage of preg_quote() where needed

In some places preg_quote() is not used or is used incorrectly.
In other places there are preg_quote() calls which had no
real function.

Change-Id: If201394036d17a68cea41903f849c6c5ea11b069
Releases: 4.5, 4.6, 4.7, 6.0, 6.1
Resolves: #36793
Reviewed-on: https://review.typo3.org/19929
Reviewed-by: Andreas Wolf
Reviewed-by: Stefan Neufeind
Reviewed-by: Jigal van Hemert
Tested-by: Jigal van Hemert

Revision ecadba26
Added by Jigal van Hemert about 1 month ago

[TASK] Add/drop usage of preg_quote() where needed

In some places preg_quote() is not used or is used incorrectly.
In other places there are preg_quote() calls which had no
real function.

Change-Id: If201394036d17a68cea41903f849c6c5ea11b069
Releases: 4.5, 4.6, 4.7, 6.0, 6.1
Resolves: #36793
Reviewed-on: https://review.typo3.org/19938
Reviewed-by: Jigal van Hemert
Tested-by: Jigal van Hemert

History

Updated by Gerrit Code Review about 1 year ago

  • Status changed from New to Under Review

Patch set 1 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/10943

Updated by Gerrit Code Review about 1 year ago

Patch set 2 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/10943

Updated by Gerrit Code Review 7 months ago

Patch set 3 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/10943

Updated by Gerrit Code Review 4 months ago

Patch set 4 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/10943

Updated by Gerrit Code Review 4 months ago

Patch set 5 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/10943

Updated by Gerrit Code Review 3 months ago

Patch set 6 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/10943

Updated by Stefan Neufeind about 1 month ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100

Updated by Gerrit Code Review about 1 month ago

  • Status changed from Resolved to Under Review

Patch set 1 for branch TYPO3_6-0 has been pushed to the review server.
It is available at https://review.typo3.org/19925

Updated by Gerrit Code Review about 1 month ago

Patch set 1 for branch TYPO3_4-7 has been pushed to the review server.
It is available at https://review.typo3.org/19929

Updated by Stefan Neufeind about 1 month ago

  • Status changed from Under Review to Resolved

Updated by Gerrit Code Review about 1 month ago

  • Status changed from Resolved to Under Review

Patch set 1 for branch TYPO3_4-5 has been pushed to the review server.
It is available at https://review.typo3.org/19938

Updated by Jigal van Hemert about 1 month ago

  • Status changed from Under Review to Resolved

Also available in: Atom PDF