Clean up services, add autowiring, make use of new features, and fix deprecations

Created on 16 May 2024, about 1 month ago

Updated 21 May 2024, about 1 month ago

Problem/Motivation

This issue resolves to make use of new features, and fix deprecations.

Add Autowiring aliases so we can autowire Purge services into our own project services — https://www.drupal.org/node/3323122 →
Switch to Logger injection via autoconfiguration — https://www.drupal.org/node/3395436 →
Switch to event subscriber autoconfiguration so we dont need to add tags. — https://www.drupal.org/node/3357408 →
DiagnosticsService and TagsHeadersService make use of the ContainerAwareTrait/Interface which is deprecated and being removed. The usage of this is exclusively for "lazy-loading" the services. We can switch to service closures in order to keep this lazy behaviour. — https://www.drupal.org/node/3360604 →

Proposed resolution

The proposed changes require Drupal 10.2, which in a few weeks will be the lowest supported version of Drupal. So we can up the Purge project minimums.

Update services.yml and add appropriate infrastructure to make use of new features

The changes here introduce no new behaviours.

Remaining tasks

Review, test, commit

User interface changes

None

API changes

Changes to some service constructors
Changes to internals of our services that wouldnt be expected to be modified downstream or have an API contract.

Data model changes

Nil.

📌 Task

Status

Active

Version

3.5

Component

Code

Created by

🇦🇺Australia dpi Perth, Australia

Live updates comments and jobs are added and updated live.

Drupal11

Merge Requests

!41Clean up services, add autowiring, make use of new features, and fix deprecations
Open
🇦🇺Australia dpi
updated 2 days ago

Comments & Activities

Issue created by @dpi
Merge request !41Clean up services, add autowiring, make use of new features, and fix deprecations → (Open) created by dpi
Open in Jenkins → Open on Drupal.org →
Core: 9.5.5 + Environment: PHP 7.4 & MySQL 5.7
last update about 1 month ago
121 pass, 87 fail
Open in Jenkins → Open on Drupal.org →
Core: 9.5.5 + Environment: PHP 7.4 & MySQL 5.7
last update about 1 month ago
121 pass, 87 fail
Open in Jenkins → Open on Drupal.org →
Core: 10.2.x + Environment: php8.3_mysql8_ignore
last update about 1 month ago
215 pass, 78 fail
Comment about 1 month ago →
🇺🇸United States japerry KVUO
Thanks for looking at this, as it will be required for D11 compatibility. I've added this issue to our internal D11 tracker as well.
Open in Jenkins → Open on Drupal.org →
Core: 9.5.5 + Environment: PHP 7.4 & MySQL 5.7
last update 2 days ago
121 pass, 87 fail

contrib.social Blog FAQ Discussions

Production build 0.69.0 2024