Inject the display plugin manager into the view and view factory

Issue summary should be updated to the standard issue template.

Left some feedback but new parameters can probably be typehinted and new parameters probably have to do the backwards compatibility thing.

smustgrave → changed the visibility of the branch 2002012-inject-the-display to hidden.

smustgrave → changed the visibility of the branch 11.x to hidden.

Great job! Think only thing missing is an a test for the deprecation.

Will keep an eye out for this to come back.

Feedback has been addressed.

Lovely to see an old issue nearly completed!

I'm triaging RTBC issues → . I read the IS and the comments. I didn't find any unanswered questions.

I read the MR and made suggestions in the MR so setting to NW for that.

I read the change record and made some changes to the text. However, the branch and version number are incorrect. I am tagging for CR update. The introduced in branch is not 11.x it will be 10.3.x.

Done and done

Feedback has been addressed.

Couple of points on the MR.

Appears feedback from @catch has been addressed.

Committed and pushed 23bb407de3 to 11.x and 7f5ab91043 to 10.3.x. Thanks!

I think we could open a follow-up to use the DependencySerializationTrait in ViewsExecutabl... it'll be a little complex because we'll need to do a little work to make things work...
Also I think we should open a follow-up to use $this->provider in \Drupal\views\ViewExecutable::hasUrl() instead of \Drupal::service().

Opened up follow ups.

Not sure if either of them should be tagged novice though?

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