Support from Acquia helps fund testing for Drupal Acquia logo

Comments

damiankloip’s picture

Status: Active » Postponed

Let's wait for that issue, see how it unfolds. This should be pretty quick when we need to do it.

damiankloip’s picture

Status: Postponed » Needs review
FileSize
2.76 KB

That patch is in now, so we can do this.

dawehner’s picture

Status: Needs review » Reviewed & tested by the community

Awesome!

tim.plunkett’s picture

Title: Inject display plugin manager into DisplayArray class » Inject display plugin manager into DisplayBag class

RTBC if green, thanks @damiankloip!

catch’s picture

Status: Reviewed & tested by the community » Fixed
-    $this->pluginInstances[$display_id] = drupal_container()->get("plugin.manager.views.display")->createInstance($display['display_plugin']);
+    $this->pluginInstances[$display_id] = $this->manager->createInstance($display['display_plugin']);

Very nice. Committed/pushed to 8.x.

Automatically closed -- issue fixed for 2 weeks with no activity.