Project

General

Profile

Actions

Bug #17152

closed

Error in function t3lib_div::array_merge_recursive_overrule

Added by Lukas Rüegg about 17 years ago. Updated almost 16 years ago.

Status:
Closed
Priority:
Should have
Assignee:
Category:
-
Target version:
-
Start date:
2007-03-24
Due date:
% Done:

0%

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

Description

The function t3lib_div::array_merge_recursive_overrule recursivly merges 2 arrays. on line 1657 The recursion however does not pass the optional 4th parameter so that using the function on a mutlidimensional array ignores the setting $includeEmtpyValues (i.e. $includeEmtpyValues is always true)

Line 1657 should read:
$arr0[$key] = t3lib_div::array_merge_recursive_overrule($arr0[$key],$arr1[$key],$notAddKeys,$includeEmtpyValues);
instead of:
$arr0[$key] = t3lib_div::array_merge_recursive_overrule($arr0[$key],$arr1[$key],$notAddKeys);

PS. I couldn't find an appropriate category for this issue. It's not backend, that's for sure. How about plain 'API' as a category?
(issue imported from #M5288)


Files

0005288.patch (1.5 KB) 0005288.patch Administrator Admin, 2007-03-24 16:23
Actions #1

Updated by Oliver Hader about 17 years ago

I can confirm this. The $includeEmtpyValues was only used on the first level but not in recursion. The attached patch solves this as suggested by Simon.

Actions #2

Updated by Oliver Hader about 17 years ago

Committed to SVN - Trunk & TYPO3_4-1

Actions

Also available in: Atom PDF