diff --git a/core/lib/Drupal/Core/Plugin/DefaultLazyPluginCollection.php b/core/lib/Drupal/Core/Plugin/DefaultLazyPluginCollection.php index e420dfc150521c4ee4139335ab362e1d739620ae..9371de650a9224a2b62a0a299c7be0df214980ab 100644 --- a/core/lib/Drupal/Core/Plugin/DefaultLazyPluginCollection.php +++ b/core/lib/Drupal/Core/Plugin/DefaultLazyPluginCollection.php @@ -156,6 +156,17 @@ public function setConfiguration(array $configuration) { * The plugin configuration to set. */ public function setInstanceConfiguration($instance_id, array $configuration) { + if ( + isset($this->pluginInstances[$instance_id]) && + isset($configuration[$this->pluginKey]) && + isset($this->configurations[$instance_id][$this->pluginKey]) && + $configuration[$this->pluginKey] !== $this->configurations[$instance_id][$this->pluginKey] + ) { + // If the plugin has already been instantiated by the configuration was + // for a different plugin then we need to unset the instantiated plugin. + unset($this->pluginInstances[$instance_id]); + } + $this->configurations[$instance_id] = $configuration; $instance = $this->get($instance_id); if ($instance instanceof ConfigurableInterface) { diff --git a/core/tests/Drupal/Tests/Core/Plugin/DefaultLazyPluginCollectionTest.php b/core/tests/Drupal/Tests/Core/Plugin/DefaultLazyPluginCollectionTest.php index 71b67e43171d12f6b897c48fa809180973a9b968..a4b9e379caa9fafd13291e8054f1971a52a4b7ae 100644 --- a/core/tests/Drupal/Tests/Core/Plugin/DefaultLazyPluginCollectionTest.php +++ b/core/tests/Drupal/Tests/Core/Plugin/DefaultLazyPluginCollectionTest.php @@ -168,6 +168,37 @@ public function testSetInstanceConfiguration() { $this->assertSame($expected, $config['cherry']); } + /** + * Tests plugin instances are changed if the configuration plugin key changes. + * + * @covers ::setInstanceConfiguration + */ + public function testSetInstanceConfigurationPluginChange() { + $configurable_plugin = $this->prophesize(ConfigurableInterface::class); + $configurable_config = ['id' => 'configurable', 'foo' => 'bar']; + $configurable_plugin->getConfiguration()->willReturn($configurable_config); + + $nonconfigurable_plugin = $this->prophesize(PluginInspectionInterface::class); + $nonconfigurable_config = ['id' => 'non-configurable', 'baz' => 'qux']; + $nonconfigurable_plugin->configuration = $nonconfigurable_config; + + $configurations = [ + 'instance' => $configurable_config, + ]; + + $plugin_manager = $this->prophesize(PluginManagerInterface::class); + $plugin_manager->createInstance('configurable', $configurable_config)->willReturn($configurable_plugin->reveal()); + $plugin_manager->createInstance('non-configurable', $nonconfigurable_config)->willReturn($nonconfigurable_plugin->reveal()); + + $collection = new DefaultLazyPluginCollection($plugin_manager->reveal(), $configurations); + $this->assertInstanceOf(ConfigurableInterface::class, $collection->get('instance')); + + // Ensure changing the instance to a different plugin via + // setInstanceConfiguration() results in a different plugin instance. + $collection->setInstanceConfiguration('instance', $nonconfigurable_config); + $this->assertNotInstanceOf(ConfigurableInterface::class, $collection->get('instance')); + } + /** * @covers ::count */