[BUGFIX] Fix JS concat if first file is forced on top 18/27818/2
authorBenjamin Kott <info@bk2k.info>
Sun, 23 Feb 2014 12:00:32 +0000 (13:00 +0100)
committerBenjamin Mack <benni@typo3.org>
Mon, 24 Feb 2014 20:15:01 +0000 (21:15 +0100)
If a javascript file is forceOnTop the script uses array_unshift()
to put the file at the beginning of the section array. If this is
the first file that got processed, the array does not exist yet,
so the array_unshift function returns NULL and the file
will not added to the array.

The fix properly initalizes the section sub-array.

Resolves: #56242
Releases: 6.2, 6.1
Change-Id: Iadf2b6f2f512929ff5e8598af9972c5337949341
Reviewed-on: https://review.typo3.org/27818
Reviewed-by: Benjamin Mack
Tested-by: Benjamin Mack
typo3/sysext/core/Classes/Resource/ResourceCompressor.php

index 236e7aa..f537ba1 100644 (file)
@@ -227,6 +227,9 @@ class ResourceCompressor {
                        if (empty($fileOptions['section']) || !empty($fileOptions['excludeFromConcatenation'])) {
                                continue;
                        }
+                       if (!isset($filesToInclude[$fileOptions['section']])) {
+                               $filesToInclude[$fileOptions['section']] = array();
+                       }
                        // we remove BACK_PATH from $filename, so make it relative to root path
                        $filenameFromMainDir = $this->getFilenameFromMainDir($fileOptions['file']);
                        if ($fileOptions['forceOnTop']) {