Describe your bug or feature request.

Let's convert payment hook implementations to OOP hooks. We still need the procedural hooks due to the D10 support.

Issue fork commerce-3560458

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

jsacksick created an issue. See original summary.

ritarshi_chakraborty’s picture

ritarshi_chakraborty commented

Hey @jsacksick, just wanted to inform you that there is some redundant code in this issue: #3560453: Convert promotion hooks to OOP hooks

In commerce_promotion.services.yml, you don't have to add autowire: true for each hook service as it is already present in the defaults.

jsacksick opened merge request !564

  • jsacksick committed cae57971 on 3.x 
    feat: #3560458 Convert payment hooks to OOP hooks
jsacksick’s picture

jsacksick commented
Status: Active » Fixed

Number pattern, store hooks and log hoors were also converted. Closing this.

Will probably convert all remaining modules in a single separate issue.

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

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