When you go to the checkout page with a uc_node_checkout node in the cart, or when you remove any item from the cart when a uc_node_checkout node is present in the cart, the uc_node_checkout node gets erroneously deleted, which also removes it from the cart.

The attached patch fixed the problem by implementing the "remove" operation in hook_cart_item(). Note that I'm using the dev versions of both uc_node_checkout and the standard ubercart modules.

CommentFileSizeAuthor
uc_node_checkout_remove_item.patch1.75 KBdwightaspinwall

Comments

kiero63’s picture

kiero63 commented

Thanks for this patch, I had exactly the problem you describe and your patch fix it.

dwightaspinwall’s picture

dwightaspinwall commented
Status: Needs review » Reviewed & tested by the community

Super. Now hopefully we can get it committed.

aidanlis’s picture

aidanlis commented

This is fixed in http://drupal.org/project/1114812

dwightaspinwall’s picture

dwightaspinwall commented

That page 404s.

aidanlis’s picture

aidanlis commented
Status: Reviewed & tested by the community » Fixed

Fixed: Commit 34214ba on 6.x-2.x

Status: Fixed » Closed (fixed)

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

aidanlis’s picture

aidanlis commented
Status: Closed (fixed) » Active

The patch is totally bogus and causes #1131406: Nodes being deleted on successful checkout

aidanlis’s picture

aidanlis commented
Status: Active » Closed (fixed)

Maybe ... I don't know. I'll investigate in the other report.