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

CommentFileSizeAuthor
#1 cart_id_255.patch887 bytescyu

Comments

cyu’s picture

Version: 6.x-2.x-dev » 6.x-2.0-rc3
Status: Active » Needs review
StatusFileSize
new887 bytes

Here is a patch to bump the cart_id field to varchar(255) with an update function.

Island Usurper’s picture

Status: Needs review » Fixed

Sounds like a plan.

Committed.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.