Project

General

Profile

Actions

Bug #85015

closed

stdWrap replacement: useRegExp = 0 should not use preg_replace()

Added by Andreas Krämer over 6 years ago. Updated almost 5 years ago.

Status:
Closed
Priority:
Could have
Assignee:
-
Category:
TypoScript
Target version:
-
Start date:
2018-05-15
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
8
PHP Version:
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

Hi,

if I write "useRegExp = 0" within a replacement rule it leads to the following error:
PHP Warning: preg_replace(): Delimiter must not be alphanumeric or backslash

I know it is not necessary to write the line at all but I think it should work anyway and should not accept 0 as "true".

Example:

page.3 = TEXT
page.3 {
    value = There_are_a_cat,_a_dog_and_a_tiger_in_da_hood!_Yeah!
    stdWrap.replacement {
        10 {
            search = in da hood
            replace = around the block
            useRegExp = 0
        }
    }
}

Actions #1

Updated by Riccardo De Contardi over 6 years ago

  • Category set to TypoScript
Actions #2

Updated by Gerrit Code Review almost 6 years 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/58945

Actions #3

Updated by Gerrit Code Review almost 6 years 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/58945

Actions #4

Updated by Gerrit Code Review about 5 years ago

Patch set 1 for branch 9.5 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/62144

Actions #5

Updated by Wolfgang Klinger about 5 years ago

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

Updated by Benni Mack almost 5 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF