Task #86586

Use different placeholder than ### because of preg_quote since PHP 7.3.0

Added by Stefan Neufeind 9 months ago. Updated about 2 months ago.

Status:
Closed
Priority:
Should have
Category:
Code Cleanup
Target version:
Start date:
2018-10-06
Due date:
% Done:

100%

TYPO3 Version:
9
PHP Version:
Tags:
Complexity:
Sprint Focus:

Description

Since 7.3.0 preg_quote also quotes #. Simply use a different placeholder.

Associated revisions

Revision 3d0c216e (diff)
Added by Stefan Neufeind 8 months ago

[TASK] Use different placeholder than ###

preg_quote since PHP 7.3.0 also quotes #.
Simply use a different placeholder.

Resolves: #86586
Releases: master, 8.7, 7.6
Change-Id: I8ed9bd39605341a09347e21dd38c9a1824a01ee5
Reviewed-on: https://review.typo3.org/58586
Tested-by: TYPO3com <>
Reviewed-by: Jan Stockfisch <>
Reviewed-by: Tobi Kretschmann <>
Tested-by: Tobi Kretschmann <>
Reviewed-by: Tomas Norre Mikkelsen <>
Reviewed-by: Daniel Goerz <>
Reviewed-by: Jürgen Heym <>
Tested-by: Jürgen Heym <>
Reviewed-by: Benni Mack <>
Tested-by: Benni Mack <>

Revision 64664892 (diff)
Added by Benni Mack 8 months ago

[TASK] Use different placeholder than ###

preg_quote since PHP 7.3.0 also quotes #.
Simply use a different placeholder.

Resolves: #86586
Releases: master, 8.7, 7.6
Change-Id: I8ed9bd39605341a09347e21dd38c9a1824a01ee5
Reviewed-on: https://review.typo3.org/58763
Tested-by: TYPO3com <>
Reviewed-by: Benni Mack <>
Tested-by: Benni Mack <>

Revision 03d9625f (diff)
Added by Benni Mack 8 months ago

[TASK] Use different placeholder than ###

preg_quote since PHP 7.3.0 also quotes #.
Simply use a different placeholder.

Resolves: #86586
Releases: master, 8.7, 7.6
Change-Id: I8ed9bd39605341a09347e21dd38c9a1824a01ee5
Reviewed-on: https://review.typo3.org/58766
Tested-by: TYPO3com <>
Reviewed-by: Benni Mack <>
Tested-by: Benni Mack <>

History

#1 Updated by Gerrit Code Review 9 months ago

  • Status changed from New 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 https://review.typo3.org/58586

#2 Updated by Gerrit Code Review 9 months ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/58586

#3 Updated by Gerrit Code Review 8 months ago

Patch set 1 for branch TYPO3_8-7 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/58763

#4 Updated by Stefan Neufeind 8 months ago

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

#5 Updated by Gerrit Code Review 8 months ago

  • Status changed from Resolved to Under Review

Patch set 1 for branch TYPO3_7-6 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/58766

#6 Updated by Benni Mack 8 months ago

  • Status changed from Under Review to Resolved

#7 Updated by Benni Mack about 2 months ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF