Bug #53116

Epic #55575: Streamline JavaScript Code in the TYPO3 Backend

concatenateJs in conjunction with forceOnTop

Added by Matthias Kappenberg over 6 years ago. Updated over 1 year ago.

Status:
Closed
Priority:
Must have
Assignee:
-
Category:
Frontend
Target version:
-
Start date:
2013-10-25
Due date:
% Done:

100%

TYPO3 Version:
6.1
PHP Version:
Tags:
Complexity:
Is Regression:
No
Sprint Focus:

Description

Hi All,

if I'm right the Typoscript setting "forceOnTop" has no effect in conjuntion with "concatenateJs".

Please try:

config {
  concatenateJs = 1
}

page {
  includeJSFooter {
    somefile = path/to/functions.js
    someotherfile = path/to/functions.js
    someotherfile.forceOnTop = 1
  }
}

If concatenateJs is disabled, it's working.

Can you confirm this behavior?


Related issues

Related to TYPO3 Core - Bug #56019: ResourceCompressor: concatenateCss with forceOnTop gives wrong order Closed 2014-02-16
Related to TYPO3 Core - Bug #56242: Fix JS concatenation if first file is forceOnTop = 1 Closed 2014-02-24
Duplicated by TYPO3 Core - Bug #56233: Load order of js files differs Closed 2014-02-24

Associated revisions

Revision 599cdb66 (diff)
Added by Markus Klein almost 6 years ago

[BUGFIX] concatenateJs/Css does not consider forceOnTop

Setting the forceOnTop property for a JS or CSS file is not
observed if concatenateJs/Css is set as well.

Resolves: #53116
Resolves: #56019
Releases: 6.2, 6.1
Change-Id: Ica31656bd72f4a0249513206bac45aa16b929761
Reviewed-on: https://review.typo3.org/27640
Reviewed-by: Stefan Neufeind
Tested-by: Stefan Neufeind
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters

Revision c9ae284b (diff)
Added by Markus Klein almost 6 years ago

[BUGFIX] concatenateJs/Css does not consider forceOnTop

Setting the forceOnTop property for a JS or CSS file is not
observed if concatenateJs/Css is set as well.

Resolves: #53116
Resolves: #56019
Releases: 6.2, 6.1
Change-Id: Ica31656bd72f4a0249513206bac45aa16b929761
Reviewed-on: https://review.typo3.org/27672
Reviewed-by: Stefan Neufeind
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters

History

#1 Updated by Markus Klein almost 6 years ago

  • Status changed from New to Accepted
  • Priority changed from Should have to Must have

#2 Updated by Gerrit Code Review almost 6 years ago

  • Status changed from Accepted to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/27640

#3 Updated by Gerrit Code Review almost 6 years ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/27640

#4 Updated by Gerrit Code Review almost 6 years ago

Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/27640

#5 Updated by Gerrit Code Review almost 6 years ago

Patch set 1 for branch TYPO3_6-1 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/27672

#6 Updated by Markus Klein almost 6 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100

#7 Updated by Daniel Siepmann almost 6 years ago

  • Parent task set to #55575

#8 Updated by Benni Mack over 1 year ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF