Bug #99503: Inline script concatenation requires scripts to have trailing ";" - TYPO3 Core - TYPO3 Forge

Actions

Copy link

Bug #99503

closed

Inline script concatenation requires scripts to have trailing ";"

Added by Jonas Eberle almost 2 years ago. Updated 4 months ago.

Status:

Closed

Priority:

Must have

Assignee:

Category:

Frontend

Target version:

Start date:

2023-01-10

Due date:

% Done:

100%

Estimated time:

TYPO3 Version:

PHP Version:

Tags:

javascript

Complexity:

easy

Is Regression:

Sprint Focus:

Description

concatenateJs produces invalid JavaScript if scripts do not end with ';'.

In JavaScript ';' line endings are optional, but when concatenating scripts a ';' (or a line feed) needs to be added.

Minimum reproducible example:

(TypoScript settings)

config.concatenateJs = 1  
page.jsInline {
  script1 = TEXT
  script1.value = alert(1)
  script2 = TEXT
  script2.value = alert(2)
}

This results in

alert(1)alert(2)

(This applies to concatenated files in any of the PAGE.includeJS functions and friends, too)

Related issues 1 (0 open — 1 closed)

Actions

Copy link

Updated by Jonas Eberle almost 2 years ago

Correction: it seems string keys are not possible there.

So the MRE is:

config.concatenateJs = 1  
page.jsInline {
  10 = TEXT
  10.value = alert(1)
  20 = TEXT
  20.value = alert(2)
}

Actions

Copy link

Updated by Jonas Eberle almost 2 years ago

My debugging went as far as to understand that page.jsInline is quite different from the others like page.includeJS.

page.jsInline renders no separators because it gets handed to \TYPO3\CMS\Frontend\ContentObject\ContentObjectRenderer::cObjGet directly.

Actions

Copy link

Updated by Gerrit Code Review almost 2 years ago

Status changed from New to Under Review

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

Actions

Copy link

Updated by Jonas Eberle almost 2 years ago

Subject changed from config.concatenateJs requires scripts to finish with ";" to Inline script concatenation requires scripts to have trailing ";"

Actions

Copy link

Updated by Gerrit Code Review over 1 year ago

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

Actions

Copy link

Updated by Gerrit Code Review over 1 year ago

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

Actions

Copy link

Updated by Gerrit Code Review over 1 year ago

Patch set 4 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/77320

Actions

Copy link

Updated by Gerrit Code Review over 1 year ago

Patch set 5 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/77320

Actions

Copy link

Updated by Gerrit Code Review about 1 year ago

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

Actions

Copy link

#10

Updated by Gerrit Code Review about 1 year ago

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

Actions

Copy link

#11

Updated by Gerrit Code Review about 1 year ago

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

Actions

Copy link

#12

Updated by Anonymous about 1 year ago

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

Applied in changeset 52795f16b592f3c1e8d76c4ca99dd12f4ea21609.

Actions

Copy link

#13

Updated by Rafael Kähm about 1 year ago

Please reopen, because applied patch makes unnecessary BC at least on 11.5.
See:

    public function cObjGet($setup, $addKey = '')
    {
        return implode('', $this->cObjGetSeparated($setup, $addKey));
    }

    public function cObjGetSeparated(?array $setup, string $addKey = ''): array
    {
        if (!is_array($setup) || $setup === []) {
            return [];
        }

Proposal:

    public function cObjGet($setup, $addKey = '')
    {
        if (!is_array($setup)) {
            return '';
        }
        return implode('', $this->cObjGetSeparated($setup, $addKey));
    }

    public function cObjGetSeparated(?array $setup = [], string $addKey = ''): array
    {
        if (empty($setup)) {
            return [];
        }