Project

General

Profile

Actions

Bug #23823

closed

listNum = 0 not working anymore due to stdWrap changes

Added by Susanne Moog about 14 years ago. Updated over 13 years ago.

Status:
Closed
Priority:
Should have
Assignee:
Category:
-
Target version:
-
Start date:
2010-10-22
Due date:
% Done:

0%

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

Description

Since the stdWrap refactoring import.listNum = 0 does not work anymore and all other functions where = 0 is not meant to be false will fail, too. This is due to a wrong check in function stdWrap, line 4461.

If changed from
if ($conf[$functionName] || $conf[$functionProperties]) {
to
if (isset($conf[$functionName]) || $conf[$functionProperties]) {

it works again. I did not test possible other effects of that change at the moment.

(issue imported from #M16107)


Files

tslib_content_v2.diff (12 KB) tslib_content_v2.diff Administrator Admin, 2010-10-22 23:44
Actions #1

Updated by Jo Hasenau about 14 years ago

looks like a nobrainer to me (who was the nobrainer overlooking it) ;-)

Actions #2

Updated by Jo Hasenau about 14 years ago

but we have to make sure that "boolean" stdWrap functions will behave correctly when they get a 0 value

Actions #3

Updated by Jo Hasenau about 14 years ago

Attached patch introduces function types for the $stdWrapOrder array to make sure that boolean functions will not be executed when their value is 0, while the rest of the functions will work as before.

Additionally it gives us more options to do a kind of "syntax check"

Actions #4

Updated by Ingo Renner about 14 years ago

Fixed target version, always set target to final release, use "Fixed in Version" for specific preview releases...

Actions #5

Updated by Susanne Moog about 14 years ago

trunk rev 9430

Actions #6

Updated by Susanne Moog over 13 years ago

  • Target version deleted (4.5.0)
Actions

Also available in: Atom PDF