Index: uc_cart/uc_cart.module =================================================================== RCS file: /cvs/drupal-contrib/contributions/modules/ubercart/uc_cart/uc_cart.module,v retrieving revision 1.15.2.37 diff -u -p -r1.15.2.37 uc_cart.module --- uc_cart/uc_cart.module 28 Sep 2010 20:10:56 -0000 1.15.2.37 +++ uc_cart/uc_cart.module 5 Oct 2010 22:32:24 -0000 @@ -946,6 +946,12 @@ function uc_cart_view_form($form_state, $description = $display_item['title']['#value'] . $display_item['description']['#value']; $form['items'][$i]['desc']['#value'] = $description; + $form['items'][$i]['remove'] = array( + '#type' => 'submit', + '#value' => t('Remove'), + '#name' => 'remove-' . $i, + ); + $form['items'][$i]['title']['#type'] = 'value'; $form['items'][$i]['description']['#type'] = 'value'; @@ -980,7 +986,6 @@ function uc_cart_view_form($form_state, $form['continue_shopping'] = array( '#type' => 'submit', '#value' => $cs_text, - '#submit' => array('uc_cart_view_form_submit'), ); $form['continue_shopping_text'] = array( '#type' => 'hidden', @@ -993,13 +998,11 @@ function uc_cart_view_form($form_state, $form['update'] = array( '#type' => 'submit', '#value' => t('Update cart'), - '#submit' => array('uc_cart_view_form_submit'), ); if (variable_get('uc_checkout_enabled', TRUE)) { $form['checkout'] = array( '#type' => 'submit', '#value' => t('Checkout'), - '#submit' => array('uc_cart_view_form_submit'), ); } @@ -1015,11 +1018,18 @@ function uc_cart_view_form_submit($form, unset($_SESSION['cart_order']); } + // If a remove button was clicked, set the quantity for that item to 0. + if (substr($form_state['clicked_button']['#name'], 0, 7) == 'remove-') { + $item = substr($form_state['clicked_button']['#name'], 7); + $form_state['values']['items'][$item]['qty'] = 0; + drupal_set_message(t('!product-title removed from your shopping cart.', array('!product-title' => $form_state['values']['items'][$item]['title']))); + } + // Update the items in the shopping cart based on the form values. uc_cart_update_item_object((object)$form_state['values']); // Specify the appropriate redirect based on the button used to submit. - switch ($form_state['values']['op']) { + switch ($form_state['clicked_button']['#value']) { // Continue shopping button. case $form_state['values']['continue_shopping_text']: $form_state['redirect'] = uc_cart_continue_shopping_url(); @@ -1597,12 +1607,7 @@ function uc_cart_remove_item($nid, $cid function uc_cart_update_item_object($cart) { if (is_object($cart)) { foreach ($cart->items as $item) { - if ($item['remove']) { - module_invoke($item['module'], 'update_cart_item', $item['nid'], unserialize($item['data']), 0); - } - else { - module_invoke($item['module'], 'update_cart_item', $item['nid'], unserialize($item['data']), $item['qty']); - } + module_invoke($item['module'], 'update_cart_item', $item['nid'], unserialize($item['data']), $item['qty']); } // Rebuild the cached cart items. Index: uc_product/uc_product.module =================================================================== RCS file: /cvs/drupal-contrib/contributions/modules/ubercart/uc_product/uc_product.module,v retrieving revision 1.14.2.44 diff -u -p -r1.14.2.44 uc_product.module --- uc_product/uc_product.module 28 Sep 2010 20:08:54 -0000 1.14.2.44 +++ uc_product/uc_product.module 5 Oct 2010 22:32:26 -0000 @@ -1180,7 +1180,6 @@ function uc_product_cart_display($item) $element = array(); $element['nid'] = array('#type' => 'value', '#value' => $node->nid); $element['module'] = array('#type' => 'value', '#value' => 'uc_product'); - $element['remove'] = array('#type' => 'checkbox'); $element['title'] = array( '#value' => node_access('view', $node) ? l($item->title, 'node/'. $node->nid) : check_plain($item->title),