Welcome, Guest
Username: Password: Remember me
  • Page:
  • 1

TOPIC: Sorry for the checkout bug at td.events today

Sorry for the checkout bug at td.events today 5 years 11 months ago #1

Hey folks. I'm Doug Martin (Jeff's brother) and I built the td.events site.

I'd like to apologize for the checkout bug some patrons encountered at td.events today -- some patrons had their checkout automatically abort with an error message that read the "slot quantities do not match up".

This not very helpful message was from a broken tripwire test in checkout that was added late in the beta testing. The test was there to ensure that the number of tickets the user wanted to purchase was not more than the total number of tickets reserved for the slot (a separate denormalized count of reserved tickets per slot is maintained to speed the slot search). The broken test looked like
if (a < b) or (a > c) then abort with "slot quantities do not match up"
when if should have been
if (a > b) or (a > c) then abort with "slot quantities do not match up"

There is also code in the add to cart that also does an extensive set of tests to ensure we maintain consistency in the database but I wanted a backup tripwire to ensure we didn't oversell slots. The bug crept in due to a combination of it being added late in the beta testing when we didn't have multiple users trying to get the same slot and no unit test coverage since the code was (in my mind) double checking other carefully constructed and tested code in the add to cart system. Its kind of ironic that I introduced a bug with code meant to check to ensure there wasn't a bug.

Again, sorry for any inconvenience or aggravation I might have caused. I hope you have a great time at Origins 2018!

Please Log in or Create an account to join the conversation.

Sorry for the checkout bug at td.events today 5 years 11 months ago #2

Thanks for getting it fixed so quickly!
First ever death in True Horde
"Well, with you guarding 2 players, that means you take 90. Are you dead?"
-Incognito

My token shop/trade thread: Wade's Wide World of Wonder 

My Current Paladin Build 

Please Log in or Create an account to join the conversation.

Sorry for the checkout bug at td.events today 5 years 11 months ago #3

Wade Schwendemann wrote: Thanks for getting it fixed so quickly!


Agreed. The unexpected happens from time to time, but a quick response is very much appreciated.

Please Log in or Create an account to join the conversation.

Sorry for the checkout bug at td.events today 5 years 11 months ago #4

Thanks Doug for the quick resolution.

1. Can you look into why this event shows the following td.events/conventions/origins-2018/slots/482
Total tickets: 10
Available tickets: 0
Tickets in carts: 0
Purchased tickets: 5

2. Minor suggestion to increase the cart timer from 30 minutes. For comparison, Gen Con uses 2 hours. That is a little more forgiving in case issues do arise. Even issues that aren't your fault such as purchaser's credit card being declined.
Classes Played: Barbarian (65 times), Monk (56), Ranger (33), Rogue (25), Cleric (21), Fighter (13), Druid (12), Paladin (11), Dwarf Fighter (10), Bard (7), Elf Wizard (2), Wizard (2)

Please Log in or Create an account to join the conversation.

Sorry for the checkout bug at td.events today 5 years 11 months ago #5

@Jeff312: thanks for the heads up - I've fixed the available ticket count for that slot.

Please Log in or Create an account to join the conversation.

Sorry for the checkout bug at td.events today 5 years 11 months ago #6

  • TJRat
  • TJRat's Avatar
  • Offline
  • 10th Level
  • Supporter
  • Yeti Cleric; Mercenary Druid
  • Posts: 678
...I'm sending you the bill for my next month's blood pressure pills... :whistle:
You still got hit points! Get back in there and fight!!

Please Log in or Create an account to join the conversation.

  • Page:
  • 1
Time to create page: 0.080 seconds