Problem/Motivation

On the site I'm working on we'd like to send some parameters that aren't currently catered to by the module (CATALOGURL, and HOMEURL). As it stands unless I'm mistaken there's not any particularly simple way to do so.

Proposed resolution

Adding an alter hook after this module has done everything it's going to do (but before calculating the shasign) would allow people do mess with the parameters as they see fit.

Remaining tasks

Upload a patch!

User interface changes

None.

API changes

Addition of an alter hook, but nothing else.

Data model changes

None.

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:

Comments

stevetweeddale created an issue. See original summary.

matthijs’s picture

Status: Active » Needs work

This might be a good idea, but I would prefer to use an event instead of a hook. Could you update your MR accordingly?
Also prevent the use of \Drupal, we should be able to use dependency injection.

ps: sorry for the delayed response, apparently I wasn't following the issues of this project.

james.williams’s picture

Assigned: stevetweeddale » Unassigned

As you might have guessed, Steve is no longer actively working on this. But I might pick it up for him :)

james.williams changed the visibility of the branch 3264483-allow-other-modules to hidden.

james.williams’s picture

Status: Needs work » Needs review

OK; I've opened MR !2 which uses a modern event instead of a hook, which I believe does the trick. That said, I can't easily test it as my client doesn't have a easily-usable test environment. I can certainly get as far as the initial KBC offsite payment page, and parameters altered via this method do indeed get passed along as intended, I just can't then actually proceed from my local test site.

matthijs’s picture

Status: Needs review » Fixed

Thanks for the MR, it looks okay (I can't easily test it either), so I merged it.

Status: Fixed » Closed (fixed)

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