Closed (fixed)
Project:
UC Recurring Payments and Subscriptions
Version:
6.x-2.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
5 Jan 2010 at 19:49 UTC
Updated:
15 Mar 2010 at 12:10 UTC
Jump to comment: Most recent file
Comments
Comment #1
trevorwh commentedI'm not sure if this is currently supported by either recurring gateway, but I agree it is a valuable/necessary feature.
Does anyone have any input on how this should be accomplished? Should we just pull the product qty from the order?
This should also probably be independent of the gateway, which means it should be set in the fee amount upstream of the gateways.
Open to suggestions and I'll create the patch.
Comment #2
wftl commentedHello trevorwh,
Interesting that the gateways (e.g. PayPal) don't support this. Seems like a massive oversight for recurring fees. Nevertheless, modifying the recurring fee upstream -- telling PayPal that the recurring fee is $75 for the 3 quantity rather than $25 three times -- seems like the correct approach to me.
-- Marcel
Comment #3
virtualdrupal commentedSubscribing..
Does Authorize.net CIM have this same issue?
Comment #4
univate commentedWe just need a small patch to the uc_recurring order process function to include the quantity in calculating the recurring fee amount.
Willing to accept a patch.
Comment #5
jday commentedwould changing line 492 work?
$fee->fee_amount = $product_fee->fee_amount == 0 ? ($product['product']->price * $product['product']->qty) : $product_fee->fee_amount;
still missing the shipping cost though
Comment #6
univate commentedSomething like that is start, although I would also suggest either changing the fee title to show a quantity or store this information in the data field so that on renewal it can be extracted and used in the renewal order and have the invoice actually show quantity properly.
The shipping seems to be related to this issue: #648098: Renewed orders have no line items (e.g. taxes, shipping)
Comment #7
univate commentedsomething like this?
Comment #8
univate commentedOk committed this.