removing a line item des not work [#1186988]

confirm_line_item_delete js function does not find #edit-li-delete-id

function should become

function confirm_line_item_delete(message, img_id) {
  if (confirm(message)) {
    var li_id = img_id.substring(3);
    jQuery('[name=li_delete_id]').val(li_id);
    jQuery('#uc-order-edit-form #edit-submit-changes').get(0).click();
  }
}

Comment	File	Size	Author
#4	1186988-removelineitem.patch	813 bytes	bwynants
#4
#3	1186988-removelineitem.patch	2.72 KB	bwynants
#3
#1	removelineitem-1186988.patch	435 bytes	bwynants
#1

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

File	Size
removelineitem-1186988.patch	435 bytes

Comment #2

13 June 2011 at 18:42

Status:

Needs review

» Needs work

The last submitted patch, removelineitem-1186988.patch, failed testing.

Comment #3

bwynants CreditAttribution: bwynants commented 13 June 2011 at 19:33

Version:	7.x-3.0-beta3	» 7.x-3.x-dev
Status:	Needs work	» Needs review

File	Size
1186988-removelineitem.patch	2.72 KB

sorry new at git, i followed the http://drupal.org/node/1054616 instructions but how do I get rid of the other fix I did (Issue #1186914)? it is also in this path?
or should it be in here? some help/advice is appreciated...

Comment #4

bwynants CreditAttribution: bwynants commented 14 June 2011 at 17:36

File	Size
1186988-removelineitem.patch	813 bytes

correct patch

Comment #5

TR CreditAttribution: TR commented 15 June 2011 at 18:34

OK, but what's the error you're seeing and where do you see the error? I can't test your patch to see if it fixes the error if I don't know what the error is.

Comment #6

bwynants CreditAttribution: bwynants commented 16 June 2011 at 10:43

on the order edit you can not delete line items that you have added, the delete button is available for manually added line items but it does not work.

Prerequisite: #1186894: add a line item to be able to add line items when editing an order.