Bug #82370

Change Configuration key for extensions from EXTCONF to extensions to prevent conflicts

Added by Matthias Vogel about 2 years ago. Updated 12 months ago.

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

100%

TYPO3 Version:
9
PHP Version:
Tags:
Complexity:
Is Regression:
Sprint Focus:
On Location Sprint

Description

Original Comment from https://review.typo3.org/#/c/53657/ :
Helmut Hummel
Sep 7 7:40 PM

Patch Set 8:
post -1 on using $GLOBALS['TYPO3_CONF_VARS']['EXTCONF']
because it makes this (useful) change unnecessarily breaking
$GLOBALS['TYPO3_CONF_VARS']['EXTCONF'] is used by extensions already since years (e.g. realurl) and was (despite being documented) considered as "safe space" for custom extension configuration.
When writing the extConf things there too, there will be conflicts.


Related issues

Related to TYPO3 Core - Feature #82254: Store extension configuration as plain array Closed 2017-08-31

Associated revisions

Revision 9f1a4f8c (diff)
Added by Matthias Vogel about 2 years ago

[BUGFIX] Store extension configuration in key EXTENSIONS

The previous patch changed it to EXTCONF,
but this conflicts with other stuff.

Resolves: #82370
Releases: master
Change-Id: Ib515a1364f45739fb7f8e0fa3e781099f1c1a107
Reviewed-on: https://review.typo3.org/53999
Reviewed-by: Christian Kuhn <>
Tested-by: Christian Kuhn <>
Tested-by: TYPO3com <>
Reviewed-by: Benni Mack <>
Tested-by: Benni Mack <>

History

#1 Updated by Matthias Vogel about 2 years ago

  • Assignee set to Matthias Vogel

#2 Updated by Gerrit Code Review about 2 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/53990

#3 Updated by Gerrit Code Review about 2 years ago

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/53999

#4 Updated by Gerrit Code Review about 2 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/53999

#5 Updated by Gerrit Code Review about 2 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/53999

#6 Updated by Gerrit Code Review about 2 years ago

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

#7 Updated by Gerrit Code Review about 2 years ago

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

#8 Updated by Christian Kuhn about 2 years ago

  • Related to Feature #82254: Store extension configuration as plain array added

#9 Updated by Gerrit Code Review about 2 years ago

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

#10 Updated by Matthias Vogel about 2 years ago

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

#11 Updated by Benni Mack 12 months ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF