search

dmitryd / typo3-simplemvc

[OBSOLETE] A TYPO3 "simplemvc" extension. This extension adds a simple high performance MVC framework to TYPO3.
7 stars 0 forks source link

array_merge_recursive_overrule requires arrays #5

Open franzholz opened 11 years ago

franzholz commented 11 years ago

The function array_merge_recursive_overrule must be checked for array arguments before it is called.

GeneralUtility::readLLfile could return a boolean, because it returns the result of

 * @return array|boolean
 */
public function getParsedData

...

protected function addLanguageLabelsFromFile($fileRef) { $labels = GeneralUtility::readLLfile($fileRef, $this->tsfeLanguage); if (isset($labels[$this->tsfeLanguage])) { $labels['default'] = GeneralUtility::array_merge_recursive_overrule($labels['default'], $labels[$this->tsfeLanguage]);

}

You must check both parameters for an array if you want to avoid an exception.