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.

Files: 
CommentFileSizeAuthor
#1 D8-ViewsHandlerDiscovery-2087551-1.patch1.76 KBdonquixote
PASSED: [[SimpleTest]]: [MySQL] 59,418 pass(es).
[ View ]

Comments

Status:Active» Needs review
StatusFileSize
new1.76 KB
PASSED: [[SimpleTest]]: [MySQL] 59,418 pass(es).
[ View ]

Status:Needs review» Reviewed & tested by the community

Looks fine to me.

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

It's a good question.

Issue tags:+VDC

We can remove the custom discovery in a follow up.

dawehner: thanks!

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

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.

Issue summary:View changes

give context to the other commit link.