Replace non-test usages of \Drupal::request() with IoC injection

This issue is being reviewed by the kind folks in Slack, #needs-review-queue-initiative. We are working to keep the size of Needs Review queue [2700+ issues] to around 400 (1 month or less), following Review a patch or merge request → as a guide.

At this time we will need a D10 version of the patch.

Also D10 should be checked to see if there are additional instances of \Drupal::request() that need to be addressed.

Thanks

reroll for drupal 10

Still seem to be instances

ViewExecutable
DbUpdateController

Not sure if entities or traits count.

Updating deprecation message for drupal:11.0.0.
The invocation of \Drupal::request() is located within a static function in DbUpdateController, making it uncertain whether it should be modified.

samit.310@gmail.com → made their first commit to this issue’s fork.

samit.310@gmail.com → changed the visibility of the branch 3123224-replace-non-test-usages to hidden.

Should use constructor promotion now.

Hi @smustgrave,

Please review.

Thanks
Samit K.

Left a comment but not sure we can fully remove parameters and add new ones without doing a BC step.