[Change notice[ Move views_get_handler into the handler plugin manager

+++ b/core/modules/views/lib/Drupal/views/Plugin/ViewsHandlerManager.phpundefined
@@ -18,6 +20,22 @@
    * @param string $type

@@ -25,12 +43,80 @@ class ViewsHandlerManager extends PluginManagerBase {
+  public function __construct($type, \Traversable $namespaces, ViewsData $views_data) {

I think we should call this $handler_type

+++ b/core/modules/views/lib/Drupal/views/Plugin/ViewsHandlerManager.phpundefined
@@ -25,12 +43,80 @@ class ViewsHandlerManager extends PluginManagerBase {
+   * Fetch a handler from the data cache.

Fetches/Gets

+++ b/core/modules/views/lib/Drupal/views/Plugin/ViewsHandlerManager.phpundefined
@@ -25,12 +43,80 @@ class ViewsHandlerManager extends PluginManagerBase {
+          // First check the field level
...
+          // Then if that doesn't work, check the table level

nitpcik alert: full stops...

+++ b/core/modules/views/lib/Drupal/views/Plugin/ViewsHandlerManager.phpundefined
@@ -25,12 +43,80 @@ class ViewsHandlerManager extends PluginManagerBase {
+      // @todo This is crazy. Find a way to remove the override functionality.
+      $plugin_id = $override ?: $definition['id'];

Are we actually using this anywhere atm?

+++ b/core/modules/views/lib/Drupal/views/Tests/ModuleTest.phpundefined
@@ -78,7 +78,7 @@ public function testViewsGetHandler() {
+    $handler = Views::handlerManager('filter')->getHandler($item, 'standard');

Much nicer :)

+++ b/core/modules/views/views.moduleundefined
@@ -828,55 +828,11 @@ function views_library_info() {
 function views_get_handler($item, $type, $override = NULL) {

Didn't you replace all the calls to this in the patch anyway?

Subject to passing, this looks good. Just rerolled as reviewing (straight forward reroll).

Committed aa56ce3 and pushed to 8.x. Thanks!

+++ b/core/modules/views/lib/Drupal/views/Tests/Handler/HandlerTest.phpundefined
@@ -10,6 +10,7 @@
 use Drupal\views\ViewExecutable;
 use Drupal\views\Tests\ViewTestBase;
 use Drupal\views\Plugin\views\HandlerBase;
+use Drupal\views\Views;

Removed unnecessarily added use statement during commit.

Committed to 8.x. Thanks!

For more information as to why this issue was moved to the Drupal core project, please see issue #3030347: Plan to clean process issue queue

Moving back to the contributed Views issue queue and closing as outdated per https://www.drupal.org/project/views/issues/3030347#comment-13023447