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

Issue created by @chaitanyadessai
Open in Jenkins → Open on Drupal.org →
Core: 10.2.1 + Environment: PHP 8.2 & pgsql-14.1
last update about 1 year ago
6 pass, 2 fail
Comment about 1 year ago →
🇮🇳India chaitanyadessai Goa
Please review.
Status changed to Needs review about 1 year ago7:40am 6 March 2024
First commit to issue fork.
Merge request !11#3425956 - Remove Drupal direct global functions calls. → (Open) created by rishabjasrotia
Open in Jenkins → Open on Drupal.org →
Core: 10.2.x + Environment: PHP 8.2 & MySQL 8
last update about 1 year ago
6 pass, 2 fail
Pipeline finished with Failed
about 1 year ago
Total: 149s
#112541
Comment about 1 year ago →
System Message
The last submitted patch, 2: 3425956-2.patch, failed testing. View results →
Open in Jenkins → Open on Drupal.org →
Core: 10.2.x + Environment: PHP 8.2 & MySQL 8
last update about 1 year ago
7 pass
Status changed to Needs review about 1 year ago9:20am 6 March 2024
Comment about 1 year ago →
rishabjasrotia
Hello @chaitanyadessai,
Thank you for the patch.

We should use the Drupal RouteObjectInterface as per Drupal 10 instead of \Symfony\Cmf\Component\Routing\RouteObjectInterface.
I have updated the code.
Pipeline finished with Success
about 1 year ago
Total: 211s
#112619
Open in Jenkins → Open on Drupal.org →
Core: 10.2.1 + Environment: PHP 8.2 & pgsql-14.1
last update about 1 year ago
Patch Failed to Apply
Comment about 1 year ago →
rishabjasrotia
Comment about 1 year ago →
🇮🇳India Jasjeet Kaur Brar
Jasjeet Kaur Brar → made their first commit to this issue’s fork.
Open in Jenkins → Open on Drupal.org →
Core: 10.2.x + Environment: PHP 8.2 & MySQL 8
last update about 1 year ago
7 pass
Open in Jenkins → Open on Drupal.org →
Core: 10.2.x + Environment: PHP 8.2 & MySQL 8
last update about 1 year ago
7 pass
Comment about 1 year ago →
🇮🇳India Jasjeet Kaur Brar
Green Tick for phpcs across board
Comment about 1 year ago →
rishabjasrotia
Thank you @jasjeet-kaur-brar

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

Problem/Motivation

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Merge Requests

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

Comments & Activities

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

Problem/Motivation

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Merge Requests

!11\Drupal calls should be avoided in classes, use dependency injection insteadOpen

Comments & Activities

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