Needs review
Project:
Ubercart Payment Method Adjustments
Version:
6.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
27 May 2011 at 09:44 UTC
Updated:
23 Mar 2012 at 10:22 UTC
Jump to comment: Most recent file
Comments
Comment #1
omnius commentedWould be wonderful to know if there are any plans for D7 and Ubercart3
Comment #2
longwaveNo immediate plans, but if anyone posts a patch I will review it, or if you would like to sponsor the upgrade please contact me directly.
Comment #3
JoshOrndorff commentedThis looks like a wonderful module. I'll take a look at the source to consider a port.
Comment #4
JoshOrndorff commentedI'm not quite familiar enough with ubercart to work this out yet. It's still on my radar because I would love to see it, but I probably won't be able to do anything immediately.
I'll be happy to help or test if anyone else has a start.
-Josh
Comment #5
xdelfino commentedI need uc_pma too! If a 7 dev release available, i can try to help you!
Comment #6
pityu73 commentedVery good module, fill a large gap in the form of payment.
We have also had cash on delivery cost.
Please update to drupal7.x and ubercart3.x to.
thank you,
Stephen
Comment #7
tomasbarej commentedI ported this project to the Drupal 7 version. I tested it on Drupal 7.10 and Ubercart 7.x-3.0-rc2.
For me it worked well, as it is on 6.x branch. The javascript ajax function from 6.x is no longer necessary because ajax line_item refreshing is now in the Ubercart core.
So please test it and enjoy!
Comment #8
gynekolog commented#7
The price for delivery is not included in the total price of the order.
Comment #9
pityu73 commentedUnfortunately does not include the amount of the payment at the board and has not been included in the total.
I attached a picture of the problem. (uc_pma7x-bug.png)
Tested environment: drupal 7.10 and ubercart 7.x-3.0-rc4
Comment #10
tomasbarej commentedYeah you right. I have an error in hook_uc_line_item. Parameter stored should have value FALSE.
I attached a fixed version. I tested it on Drupal 7.12 and Ubercart 7.x-3.0
Comment #11
gynekolog commentedWorks, thank you!
Comment #12
pityu73 commentedGreat job, thank you.

Comment #13
xdelfino commentedGood job! Thank you!
Comment #14
xurizaemonPatch attached, with minimal changes from @tomasbarej's version.
Sandbox at http://drupalcode.org/sandbox/grobot/1471034.git/shortlog/refs/heads/7.x... has updated tarball ("snapshot" link).
Maintainer, can you please create a Git branch for 7.x-1.x so we can work from that?
See also patch in http://drupal.org/node/1471076#comment-5698070 for PHP5.3 fix.
Comment #15
cyu commentedgrobot, I've added you as a maintainer as requested. Feel free to start the D7 branch.
Comment #16
christian death commentedHi,
good to show the port for D7 is coming :-).
I have short testing and I must say it give problem with tax. The pma is not including in tax calculation.
Comment #17
xurizaemonThanks cYu for giving me access to work on this. 7.x-1.x-dev release is now available on the module homepage.
@christian death, please see #841348: pma + tax module re tax support. By default uc_pma only operates on products and ignores shipping, tax etc. See also #864726: Optionally apply uc_pma adjustments also to non-product line items (eg shipping) which has a patch (fix in 7.x-1.x-dev already).
Comment #18
christian death commented@grobot: thanxs,i have tax enabled for this and it works :-)
But can you including a solution, when i insert a positiv currency value so show this with inkl. tax on checkout?