11 replies [Last post]
rak
rak's picture
Offline
Internationalizationizer
Joined: 11/07/2007
Juice: 310
Was this information Helpful?

After checkout, at order review, if somebody for some reason clicks 'Back', the lines containing tax are duplicated. It happens on ubercart 2.0-beta5, another user (Bentley) also confirmed the bug.

Issue at: http://drupal.org/node/406470

--
RAk

ahizzle's picture
Offline
Joined: 02/23/2009
Juice: 8
Re: tax recalculation at order review

Have the exact same issue. Rather disappointed that the reported issue doesn't include a patch or rev number. Sad

ahizzle's picture
Offline
Joined: 02/23/2009
Juice: 8
found it...

revno: 1628
committer: Lyle
branch nick: ubercart-PHOSPHOR-2
timestamp: Wed 2009-03-11 09:18:12 -0400
message:
#377064 by rszrama: Fix duplicate tax issue when going back to the checkout page.

Is there anywhere this can be flagged? IMO this should be a show-stopper for 2.0-beta5.

Ryan's picture
Offline
Joined: 08/07/2007
Juice: 15438
Re: found it...

That commit message indicates this was fixed on March 11. Are you using the latest dev release? If not, can you test it to make sure it solves your problem?

trk247's picture
Offline
Joined: 03/24/2009
Juice: 2
Re: Re: found it...

Same issue here - we have 3 different sites doing this on the latest releases.

It's also an issue with the order history - if you edit and submit an existing order with tax, it adds another tax line item which then gets added to the order total.

gooddesignusa's picture
Offline
Joined: 04/13/2009
Juice: 41
Same Issue

Drupal 6.10 with ubercart 6.x-2.0-beta5. Noticed when entering wrong credit card and having to hit back. After clicking review order again. tax was being applied multiple times. Kinda of a show stopper Sad
About to go live with a store in the next week or 2.

ahizzle's picture
Offline
Joined: 02/23/2009
Juice: 8
Re: Re: found it...

Ryan, I've tested it briefly with the fix indicated in the march 11 check-in. Seems to work OK... But I'm more concerned that this issue hasn't been flagged as a showstopper for 2.x-beta5.

gooddesignusa's picture
Offline
Joined: 04/13/2009
Juice: 41
Sorry Still A Noob

Sorry I'm still a noob to drupal. When you say "fix indicated in the march 11 check-in" what are you referring to? The latest release was on 2009-Mar-02. Are you talking about some type of patch I'm not aware of? Thanks in advance.

edit: after doing some research I found this patch. Is this the one you are talking about? http://drupal.org/node/377064#comment-1330056

Lyle's picture
Offline
AdministratoreLiTe!
Joined: 08/07/2007
Juice: 6846
Re: Sorry Still A Noob

The dev version had the fix in it because it was updated on Apr 3. However, I've just released Ubercart 6.x-2.0-beta6, so you can upgrade to that and have this fix as well as several others.

gooddesignusa's picture
Offline
Joined: 04/13/2009
Juice: 41
thanks

wow thanks a lot. I will install and do some tests today.

gooddesignusa's picture
Offline
Joined: 04/13/2009
Juice: 41
Seems Ok

I've had a little time to test it. Seems to be working. I will do some more testing tonight / tomorrow. Thanks again

Ryan's picture
Offline
Joined: 08/07/2007
Juice: 15438
ahizzle wrote:Ryan, I've
ahizzle wrote:

Ryan, I've tested it briefly with the fix indicated in the march 11 check-in. Seems to work OK... But I'm more concerned that this issue hasn't been flagged as a showstopper for 2.x-beta5.

If this question still needs to be answered, it's because it wasn't a showstopper any more. It was already fixed. Sticking out tongue

Now it's a moot issue, so it won't come up any more in the forums.