hello, on a cart page (/cart), if a qty is updated, and the button update is clicked, it does not update the cart. I tracked this down to line 967/968 in uc_cart. The following needs to be added:

'#submit' => array('uc_cart_view_form_submit'),

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

TR’s picture

Status: Active » Postponed (maintainer needs more info)

Works fine for me in 6.x-2.x-dev. You should upgrade and try again, but there haven't been any changes that would affect this so I suspect you may have a contributed module installed which affects the cart page.

TR’s picture

Oh, and there are SimpleTest test cases which check to see if the "Update" button works properly when quantities are changed. You can check the automated testing tab on the Ubercart project page to see that these tests produce the correct results, which again implies that this is something specific to your site.

longwave’s picture

Notably if you are using uc_attributes_in_cart, see #1540028: Update cart not working

longwave’s picture

Version: 6.x-2.9 » 6.x-2.x-dev
Status: Postponed (maintainer needs more info) » Needs review
FileSize
461 bytes

I guess adding #submit consistently to all buttons on the cart page can't hurt.

longwave’s picture

We can even remove the reliance on testing $form_state['clicked_button'] by expanding this a bit further.

Status: Needs review » Needs work

The last submitted patch, 1837876-update-cart-submit-5.patch, failed testing.

longwave’s picture

Status: Needs work » Needs review

#5: 1837876-update-cart-submit-5.patch queued for re-testing.

longwave’s picture

Version: 6.x-2.x-dev » 7.x-3.x-dev
FileSize
1.02 KB

Committed to 6.x, patch for 7.x attached.

longwave’s picture

Status: Needs review » Fixed

Committed to 7.x.

Status: Fixed » Closed (fixed)

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