Use dependency Injection instead of Drupal call directly

Created on 17 January 2023, about 2 years ago

Updated 23 March 2023, almost 2 years ago

This is a recommendation from best practice to use DI instead of Drupal:: directly for Request

\Drupal calls should be avoided in classes, use dependency injection instead

Create the constructor and receive this class and apply dependency injection to this

🐛 Bug report

Status

Needs review

Version

1.0

Component

Code

Created by

🇧🇷Brazil renatog Campinas

Live updates comments and jobs are added and updated live.

Incomplete comments

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

Status changed to Needs review about 2 years ago3:11am 17 January 2023
Status changed to Needs work almost 2 years ago7:08pm 22 March 2023
Comment almost 2 years ago →
🇺🇸United States Greg Boggs Portland Oregon
This needs a rebase because I merged 4.
Status changed to Needs review almost 2 years ago3:16am 23 March 2023
Comment almost 2 years ago →
🇧🇷Brazil renatog Campinas
MR rebased.

Now is mergeable: https://git.drupalcode.org/project/current_page_crumb/-/merge_requests/5...

Production build 0.71.5 2024