We need this for early bird discounts.

CommentFileSizeAuthor
#2 signup_pay_263575.diff1.91 KBsingularo
signup_pay5.diff1.84 KBsingularo

Comments

kbahey’s picture

kbahey commented
Status: Active » Postponed (maintainer needs more info)

What happens if more than one module try to change the amount? Which one "sticks"?

Also, does this code:

if ($adjusted_amount) {

Work with an array?

Shouldn't this be something like:

if (count($adjusted_amount) > 0) {

singularo’s picture

singularo
Location Adelaide, AUS
commented
StatusFileSize
new signup_pay_263575.diff1.91 KB

I've changed the patch slightly, as we have more than one module changing the code, exactly as you say. Basically it uses a copy of the code from module_invoke instead using module_invoke to do cumulative adjustments. Only a couple of lines extra. Revised patch attached.

kbahey’s picture

kbahey commented
Status: Postponed (maintainer needs more info) » Fixed

Modified it a bit to make it one place to call from.

Committed.

Thank you.

singularo’s picture

singularo
Location Adelaide, AUS
commented

Cool, yeah, thats nicer.

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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