Project

General

Profile

Actions

Bug #50786

closed

Loading TypoScript files is more picky about doubletick encapsulation

Added by Christian Kuhn over 11 years ago. Updated over 7 years ago.

Status:
Rejected
Priority:
Should have
Category:
TypoScript
Target version:
-
Start date:
2013-08-05
Due date:
% Done:

0%

Estimated time:
TYPO3 Version:
6.2
PHP Version:
5.4
Tags:
Complexity:
medium
Is Regression:
No
Sprint Focus:

Description

With the change of #34621 , the TypoScript inclusion requires " around the file path. This worked before:

<INCLUDE_TYPOSCRIPT: source=FILE:EXT:fx_library/Configuration/TypoScript/setup.txt>

now, only this works:

<INCLUDE_TYPOSCRIPT: source="FILE:EXT:fx_library/Configuration/TypoScript/setup.txt">

Imho, this is a pretty nasty regression, it took us an hour to nail our code down to these missing "


Related issues 1 (0 open1 closed)

Related to TYPO3 Core - Feature #34621: Allow <INCLUDE_TYPOSCRIPT to include directories (recursive)ClosedLoek Hilgersom2012-03-07

Actions
Actions #1

Updated by Mathias Schreiber almost 10 years ago

  • Status changed from New to Needs Feedback
  • Assignee set to Mathias Schreiber
  • Is Regression set to No

would you still fix this?

Actions #2

Updated by Alexander Opitz over 9 years ago

Is this still open?

Actions #3

Updated by Riccardo De Contardi about 9 years ago

yes, it is still open: in both 6.2.15 and 7.5-dev (latest master) the

<INCLUDE_TYPOSCRIPT: source=FILE:fileadmin/setup.txt>

syntax does not work and it is simply ignored.

But on the manual "TypoScript Syntax and In-depth Study", the only syntax reported is the one with the quotes: see https://docs.typo3.org/typo3cms/TyposcriptSyntaxReference/Syntax/Includes/Index.html

maybe it could be added in the documentation that from now on this is the only accepted syntax.

Actions #4

Updated by Mathias Schreiber about 9 years ago

Testing setup:

https://regex101.com/r/qZ7zH7/1

These syntax perversions should be supported. I will take care every once in a while.

Actions #5

Updated by Mathias Schreiber about 9 years ago

  • Target version deleted (6.2.0)
Actions #6

Updated by Alexander Opitz over 8 years ago

  • Status changed from Needs Feedback to New

So, I would vote to update documentation.

Actions #7

Updated by Benni Mack over 7 years ago

  • Status changed from New to Rejected

Sorry, we won't support any nasty perversions on the syntax, it is bad enough already!!!

Actions

Also available in: Atom PDF