Download & Extend

Allow other modules to change payment amounts

Project:Signup Pay for a node
Version:5.x-1.x-dev
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed (fixed)

Issue Summary

We need this for early bird discounts.

AttachmentSize
signup_pay5.diff1.84 KB

Comments

#1

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) {

#2

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.

AttachmentSize
signup_pay_263575.diff 1.91 KB

#3

Status:postponed (maintainer needs more info)» fixed

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

Committed.

Thank you.

#4

Cool, yeah, thats nicer.

#5

Status:fixed» closed (fixed)

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