Closed (fixed)
Project:
Ubercart
Version:
6.x-2.0-rc3
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
18 May 2009 at 16:00 UTC
Updated:
5 Jun 2009 at 20:20 UTC
Jump to comment: Most recent file
As mentioned here: http://www.ubercart.org/forum/development/5111/cart_id_varchar
Session ids can be higher than 32 characters so cart_id should at minimum match the sessions table's sid of varchar(64) and possibly use varchar(255) to add the ability of using a prefix or suffix along with the session id to create multiple carts for users, as is done with http://drupal.org/project/uc_save_for_later
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | cart_id_255.patch | 887 bytes | cyu |
Comments
Comment #1
cyu commentedHere is a patch to bump the cart_id field to varchar(255) with an update function.
Comment #2
Island Usurper commentedSounds like a plan.
Committed.