Merge remote-tracking branch 'origin/develop' into feature/elastic-upgrade

Conflicts:
    Classes/Configuration/ConfigurationContainer.php
This commit is contained in:
Daniel Siepmann 2018-12-27 10:12:18 +01:00
commit f9ef8cb3bb
Signed by: Daniel Siepmann
GPG key ID: 33D6629915560EF4
3 changed files with 19 additions and 2 deletions

View file

@ -44,8 +44,7 @@ class ConfigurationContainer implements ConfigurationContainerInterface
{
$this->settings = $configurationManager->getConfiguration(
ConfigurationManagerInterface::CONFIGURATION_TYPE_SETTINGS,
'SearchCore',
'Search'
'SearchCore'
);
if ($this->settings === null) {
throw new NoConfigurationException('Could not fetch configuration.', 1484226842);

View file

@ -17,3 +17,4 @@ Changelog
changelog/20180308-introduce-php70-type-hints
changelog/20180306-120-facet-configuration
changelog/20180926-163-allow-zero-as-typoscript-filter-value
changelog/20181106-170-do-not-specify-the-pluginname-in-configurationcontainer

View file

@ -0,0 +1,17 @@
Feature 170 "Do not specify the pluginName in ConfigurationContainer"
===============================================================================
Prior to the change it was not possible to create a second plugin in a
separate extension. The pluginName for the configuration was set to `search`.
The problem was that the plugin specific settings could not be fetched.
The configuration in `plugin.tx_exampleextension_pluginkey.settings {..}` and
from flexform were not fetched.
Now the pluginName is not set and the ConfigurationManager checks which plugin
is used in the current context.
It is now possible to create a second plugin. For example if you want to cache
the output of your query or the filters you specified.
See :issue:`170`.