Switch to new constructor/dependency-injection pattern

Created on 20 October 2024, 7 months ago

Problem/Motivation

Since working on this module last in #2708461: Port to Drupal 8/9 , I've started using a new pattern for constructing dependency-injected classes which avoids some of the constructor complexity and backwards-compatibility concerns and also avoids the "unsafe usage of new static" PHPStan warning that's sometimes present in old code. I didn't invent it, but I don't know if it has a name.

Proposed resolution

Start using the new pattern. Make sure tests pass.

Remaining tasks

  1. Write a merge request
  2. Commit

User interface changes

None.

API changes

None.

Data model changes

None.

Feature request
Status

Active

Version

3.0

Component

Code

Created by

🇨🇦Canada mparker17 UTC-4

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

Production build 0.71.5 2024