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

Open on Drupal.org →

Created on 18 April 2024, 12 months ago

Updated 19 April 2024, 12 months ago

Problem/Motivation

\Drupal calls should be avoided in classes.

Proposed resolution

Use dependency injection instead

📌 Task

Status

Needs work

Version

2.0

Component

Code

Created by

🇮🇳India chaitanyadessai Goa

Live updates comments and jobs are added and updated live.

Sign in to follow issues

Merge Requests

!5\Drupal calls should be avoided in classes, use dependency injection instead
Open
🇮🇳India chaitanyadessai
updated 12 months ago

Comments & Activities

Issue created by @chaitanyadessai
Merge request !5Fixed Drupal calls. → (Open) created by chaitanyadessai
Issue was unassigned.
Status changed to Needs review 12 months ago5:57am 18 April 2024
Comment 12 months ago →
🇮🇳India chaitanyadessai Goa
Please review MR.
Status changed to Needs work 12 months ago6:54am 19 April 2024
Comment 12 months ago →
🇳🇴Norway hansfn
Thanks for the contribution.

I have reviewed the MR:

Many unrelated (array) changes. In general that should be avoided, and it seems you are breaking coding standards → .

I'm sure there are more \Drupal usages, but maybe not in classes. Have you checked?

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024