Project

General

Profile

Actions

Bug #82543

closed

INCLUDE_TYPOSCRIPT extensions can not handle double extension

Added by Jan Kiesewetter about 7 years ago. Updated about 6 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
TypoScript
Target version:
-
Start date:
2017-09-22
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
7
PHP Version:
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

To include TypoScript setup and constants I would like to do something like this

on the setup side:

<INCLUDE_TYPOSCRIPT: source="DIR:./plugin" extensions="setup.typoscript">

and this on the constants side:

<INCLUDE_TYPOSCRIPT: source="DIR:./plugin" extensions="constants.typoscript">

Files in the plugin folder, e.g.:

tx_news.setup.typoscript
tx_news.constants.typoscript

Expected behavior:
tx_news.setup.typoscript is included at the setup, tx_news.constants.typoscript is included at the constants.

Current behavior:
Every file with *.typoscript as extension is included in setup and constants.

I haven't checked this in 8.7 yet, but the code of TypoScriptParser::includeDirectory didn't changed, as far as I can overlook.

Actions #1

Updated by Jan Kiesewetter about 7 years ago

Jan Kiesewetter wrote:

Current behavior:
Every file with *.typoscript as extension is included in setup and constants.

Sorry, the current behavior is that NO file is included.

Actions #2

Updated by Susanne Moog about 7 years ago

  • Category set to TypoScript
Actions #3

Updated by Gerrit Code Review about 7 years ago

  • Status changed from New 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/54443

Actions #4

Updated by Gerrit Code Review about 7 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/54443

Actions #5

Updated by Gerrit Code Review about 7 years ago

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

Actions #6

Updated by Anonymous about 7 years ago

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

Updated by Gerrit Code Review about 7 years ago

  • Status changed from Resolved 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/54460

Actions #8

Updated by Gerrit Code Review about 7 years ago

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

Actions #9

Updated by Benni Mack about 7 years ago

  • Status changed from Under Review to Resolved
Actions #10

Updated by Benni Mack about 6 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF