ViewsHandlerDiscovery and Core AnnotatedClassDiscovery now no longer need $this->pluginNamespaces inherited from Component AnnotatedClassDiscovery. Instead, plugin lazily are dynamically determined in getPluginNamespaces();

This is since this patch went in:
http://drupalcode.org/project/drupal.git/blobdiff/8b0ea7503d7c61a19acb47...
http://drupalcode.org/project/drupal.git/blobdiff/8b0ea7503d7c61a19acb47...

However, ViewsHandlerDiscovery::__construct() still fills this variable for no reason.
This was introduced before the above mentioned change:
http://drupalcode.org/project/drupal.git/blobdiff/5fc86b002978fc160b98e7...

This should be removed.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

donquixote’s picture

Status: Active » Needs review
FileSize
1.76 KB
damiankloip’s picture

Status: Needs review » Reviewed & tested by the community

Looks fine to me.

dawehner’s picture

To be honest I would like to understand whether it would be possible to remove the custom discovery mechanism in general.

damiankloip’s picture

It's a good question.

dawehner’s picture

Issue tags: +VDC

We can remove the custom discovery in a follow up.

donquixote’s picture

dawehner: thanks!

This patch will remove yet another obstacle for PSR-4,
#2083547: PSR-4: Putting it all together

webchick’s picture

Status: Reviewed & tested by the community » Fixed

Yay, less code. :)

Committed and pushed to 8.x. Thanks!

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

Anonymous’s picture

Issue summary: View changes

give context to the other commit link.