AzureMailer should implement ContainerFactoryPluginInterface

Created on 25 November 2023, about 1 year ago
Updated 13 February 2024, 10 months ago

Problem/Motivation

The Drupal\azure_mailer\Plugin\Mail\AzureMailer class uses the container directly instead of having its dependencies injected into the class. This makes it difficult to write a test for this class.

Steps to reproduce

Proposed resolution

The class could instead impalement Drupal\Core\Plugin\ContainerFactoryPluginInterface which would allow the dependencies to be injected. This would make it significantly easier to write a test for this class.

Remaining tasks

User interface changes

API changes

Data model changes

πŸ“Œ Task
Status

Closed: duplicate

Version

1.0

Component

Code

Created by

πŸ‡ΊπŸ‡ΈUnited States davidwbarratt

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

Comments & Activities

Production build 0.71.5 2024