Project

General

Profile

Actions

Bug #18922

closed

t3lib_parsehtml::substituteMarkerArray() deletes too much if wrap is not set

Added by Francois Suter over 16 years ago. Updated about 16 years ago.

Status:
Closed
Priority:
Should have
Category:
-
Target version:
-
Start date:
2008-06-07
Due date:
% Done:

0%

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

Description

In RFC #18115 a 5th parameter was added to t3lib_parsehtml::substituteMarkerArray() to remove unused markers. It does so with the following line:

$content = preg_replace('/'.preg_quote($wrapArr0).'([A-Z0-9_-|]*)'.preg_quote($wrapArr1).'/is', '', $content);

This will fail if the marker wrap is not defined, because the default wrap is ### where the $wrapArr is empty when no wrap is passed to this method.

The solution is to initialise $wrapArr when no marker wrap argument was passed.

(issue imported from #M8637)

Actions #1

Updated by Ernesto Baschny over 16 years ago

Committed to TYPO3_4-2 (rev.3770) and trunk (rev.3771).

Actions

Also available in: Atom PDF