Switch to OOP hook implementations

Working on it.

sayan_k_dutta → closed merge request !217

sayan_k_dutta → changed the visibility of the branch 3489482-switch-to-oop to hidden.

sayan_k_dutta → changed the visibility of the branch 3489482-switch-to-oop to active.

Working further to convert the views_hooks in oop implementation.

Made the changes. Please review MR !218

Thanks a lot for your work on this, great job so far! However, there are still a couple of problems, as evidenced by the failing tests:

• I think SearchApiViewHooks is missing a services.yml entry.
• (I’d also call that class SearchApiViewsHooks, as the module name is “Views”.)
• I don’t think we can use autowire yet, but I just temporarily enabled testing against other Drupal versions in the fork to validate that assumption.
• Services in the two Hooks classes should be accessed via dependency injection (which is also why autowire would even be needed, I think), not via the global \Drupal class. As a rule, classes that are able to use dependency injection should not contain the string '\Drupal::'.

I already implemented the first two items, but please address the last one as well as the failing MR pipelines.