Bug #42775
closedSaving one tab, deleted entries from other tabs
100%
Description
When storing extension configuration in different tabs, saving one tab, will delete/reset all values in other tabs.
In order to provoke the error, just create an ext_conf_template with
- cat=Basic/; type=int+; label=Test key1
key1 =
- cat=WebService/; type=string; label=Test key2
key2 =
Then go to the extension manager, and setting value of key2, will reset the value of key1 and vice versa.
The error related to ConfigurationUtility::getCurrentConfiguration which looks for
$GLOBALS['TYPO3_LOADED_EXT'][$extensionKey]['key']
which is never set.
Updated by Jan-Erik Revsbech about 12 years ago
Debugging further, i see that the getCurrentConfiguration (which is only used by the extension manager backend) is calling $this->configurationItemRepository->createArrayFromConstants
which returns an array looking like this
array (
'key1' => array(
'cat' => ' Basic'
'subcat' => 'x/z',
...
'value' => ''
'default_value' => ''
)
)
Which makes a lot os sense. But it does not make sence to call array_merge_recursive_overrule with this array and the currenc extConf which will look simething like
array(
'key1' => 'value1'
)
Updated by Jan-Erik Revsbech about 12 years ago
Hmm, it gets even worse when having nested configuration with first.second = value
Updated by Jan-Erik Revsbech about 12 years ago
The error occurs when merging the ConfigurationUtility internal "valued" configuration with the serilized "nested" configuration, which doesnt make to much sense. I submitted a patch set which convertes to "valued" before merging default with existing. This seems to fix the problems.
Updated by Gerrit Code Review about 12 years ago
- Status changed from New to Under Review
Patch set 3 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/16280
Updated by Gerrit Code Review almost 12 years ago
Patch set 1 for branch TYPO3_6-0 has been pushed to the review server.
It is available at https://review.typo3.org/17720
Updated by Christian Kuhn almost 12 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Updated by Oliver Hader over 11 years ago
- Project changed from 2269 to TYPO3 Core