Use dependency injection

Fixed a couple of other PHPCS things that popped up after applying that patch.
Here's the updated version that has everything passing.

The issue summary should always describe what the issue is trying to fix and, in the case, of coding standards issues, report which command has been used, which arguments have been used, and which report that command shown.

(The patches fail to apply too.)

Reviewed MR 2, Looks good, +1RTBC

Attaching screenshots for the reference.

This will need a re-roll because of 📌 Use constructor property promotion Fixed . Please use constructor property promotion for the new properties.

i try to resolve this issue

Hi, Working on this issue.

Hi, I tried to solve the issue. Here is the patch attached for the same.

On PHP 8.2, I expect there will be deprecation notices because of the use of dynamic properties. $this->destination and $this->renderer need to be declared. This can be done like it is for $entityTypeManager by adding a keyword like protected in the constructor.

Hi, I have added protected keyword in constructor as said. Please review my new patch.

Thanks @priyanka_chauhan31. After adding protected, you don't need to assign the values; that happens automatically. Please test the merge request.

MR as a patch.

MR as a patch.

Automatically closed - issue fixed for 2 weeks with no activity.