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

Issue created by @chaitanyadessai
Issue was unassigned.
Status changed to Needs review 8 months ago1:45pm 11 March 2024
Open in Jenkins → Open on Drupal.org →
Core: 10.2.1 + Environment: PHP 8.1 & MySQL 5.7
last update 8 months ago
7 pass, 3 fail
Comment 8 months ago →
🇮🇳India chaitanyadessai Goa
Please review.
Comment 8 months ago →
System Message
The last submitted patch, 2: 3427093-2.patch, failed testing. View results →
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.
Status changed to Needs review 8 months ago3:21pm 11 March 2024
Open in Jenkins → Open on Drupal.org →
Core: 10.2.1 + Environment: PHP 8.1 & MySQL 5.7
last update 8 months ago
7 pass, 3 fail
Comment 8 months ago →
🇮🇳India Chandreshgiri Gauswami
Comment 8 months ago →
System Message
The last submitted patch, 4: 3427093-4.patch, failed testing. View results →
First commit to issue fork.
Merge request !20#3427093 Replaced static calls in favor of DI, small code improvements, fixing tests → (Open) created by quadrexdev
Open in Jenkins → Open on Drupal.org →
Core: 10.2.1 + Environment: PHP 8.1 & MySQL 5.7
last update 7 months ago
18 pass
Pipeline finished with Success
7 months ago
Total: 179s
#156612
Status changed to Needs review 7 months ago3:58pm 25 April 2024
Comment 7 months ago →
🇺🇦Ukraine quadrexdev Lutsk
Applied patch from #4 + fixed tests + fixed some changes from the patch -> moved all these in the MR.

Please review.

Merge Requests