Closed (fixed)
Project:
Ubercart Payment Method Adjustments
Version:
5.x-1.0
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Anonymous (not verified)
Created:
6 Jun 2008 at 08:59 UTC
Updated:
7 May 2015 at 14:53 UTC
Comments
Comment #1
cyu commentedI tested this out and was able to duplicate the problem. I also tried applying a tax to shipping and saw this same behavior. I think there is a fundamental problem with the line item system in this case, but I'll try and dig a little deeper.
Comment #2
cyu commentedI took a look at this and the problem exists because uc_pma does not call getTax(). Adding this in what would seem like the appropriate places does not work, though. There are timing issues somewhere with set_line_item(), render_line_items(), remove_line_item(), and getTax().
I'm just going to leave this as it stands and say that this module was not intended to create payment method fees which are taxable.
Comment #3
cyu commentedComment #4
cyu commentedOk, looks like the problem was with the remove_line_item() function and a fix for uc_payment.js is here: http://drupal.org/node/274856
uc_pma.js has been updated as well for tax support here http://drupal.org/cvs?commit=123506 and will show up in the next release.
Comment #5
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.