Task #36793
Add/drop usage of preg_quote() where needed
| 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
[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
[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
[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
[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
Applied in changeset 292b6d2e14fe48b03eb8d33043e7d96fc26dc115.
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
Applied in changeset c40805d9de25193d37a90dcac20c802859a49e6a.
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
Applied in changeset ecadba263992933d8fc6450788be0482b7ff8d28.