"Delete this and later bids" doesn't work with double digit BID's

jesse4216 - March 4, 2009 - 15:03
Project:Ubercart Auction
Version:6.x-1.x-dev
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:active
Issue tags:delete this and later bids, uc_auction, uc_auction_bids
Description

HI. This is a great module. Great job.

In testing, I noticed that the "Delete this and later bids" doesn't work when you have lots of auctions. I am developing a auction for a non-profit that has over 35 auctions. When the BID's in "uc_auction_bids" get larger than 2 digits, when clicking on "Delete this and later bids", it doesn't update "uc_auction" "high_bid" in the database with the correct number.

I tested by manually modifying the database with various BID's to see what worked. It is probably something in uc_auction.module around line 971 which has to be changed but unsure what that might be.

I also thought it may be another module I installed which is conflicting in some way. Upon doing a clean install with just ubercart, ubercart auction, and required modules, it did the same thing with double digit BID's in "uc_auction_bids".

Thanks so much for the help. Any ideas?

#1

Garrett Albright - March 5, 2009 - 01:01

I don't think double digit bid ID values are the issue. For one, because I can't see why it would be, and for two, because I just tested deleting some bids with bid IDs well above 10 on our client's site, and everything went fine.

This could be a bug in the older code; something which will be fixed once the Drupal.org robot finally updates the dev release. I just did another commit. If I don't see a new package when I get off of work tomorrow, I'll go annoy chx in #drupal - he's been unreasonably helpful with me and my CVS/packaging robot problems in the past.

#2

jesse4216 - March 5, 2009 - 05:38

Thanks Garrett.

#3

Garrett Albright - March 10, 2009 - 23:00

Issue #396138: Deleting bids resets bid price to $45. outlines a similar problem, but it seems to have been resolved by the most recent (and finally correct) dev release. jesse, could you give it a try and let me know if it resolves the issue?

 
 

Drupal is a registered trademark of Dries Buytaert.