Quantity issue when adding wish list product to cart [

i'm having the same issue.

The source of the problem is the WantQty field being disabled for non-owner users. This thread (http://drupal.org/node/227966) has a discussion on the topic.

There appears to have been a change in form processing behavior between Drupal 5 and Drupal 6. In D5 the '#disabled' option to a form element would cause the value of the element to still be available when the element was disabled. In D6 this no longer seems to be the case. As far as I can tell this is a bug in drupal core.

So, what's the solution? This post (http://drupaleasy.com/quicktips/module-development-disabled-vs-readonly-...) has some thoughts on disabled vs. read-only. The readonly solution does not work. The solution is to add the '#value' option to the form element.

Following is the quick fix to the existing code:
In the uc_wishlist.pages.inc file, change the uc_wishlist_view_form() function:

    $element['wantqty'] = array(
      '#type' => 'textfield',
      '#value' => $item->wantqty, // <<< add this line
      '#default_value' => $item->wantqty,
      '#size' => 5,
      '#maxlength' => 6,
      '#disabled' => $own ? FALSE : TRUE
    );
    $element['haveqty'] = array(
      '#type' => 'textfield',
      '#value' => $item->haveqty, // <<< add this line
      '#default_value' => $item->haveqty,
      '#size' => 5,
      '#maxlength' => 6,
      '#disabled' => TRUE
    );

In looking at the code I found a few other inconsistencies that made things more intuitive for the end-user. Instead of allowing the user to order more items than are requested (wantqty), why not just limit the form's input using a select box? This way there's no confusing error to the end-user, and the actual selection process is simpler.

I made the following change to accommodate this (also in the uc_wishlist.pages.inc file, uc_wishlist_view_form() function):

    $element['qty'] = array(
      '#type' => 'textfield',
      '#default_value' => 0,
      // '#default_value' => $item->qty - $item->haveqty > 0 ? $item->qty - $item->haveqty : 1,
      '#size' => 5,
      '#maxlength' => 6,
    );

Changed to use a select-list instead:

    $element['qty'] = array(
      '#type' => 'select',
      '#options' => range(0,$item->qty),
      '#default_value' => $item->qty - $item->haveqty > 0 ? $item->qty - $item->haveqty : 1,
    );

Now the '#default_value' above has some sketchyness associated with it. If the #default_value uses the $item->qty-$item->haveqty logic, you'll have to change uc_wishlist_view_form_submit() to ensure that clicking one 'Add To Cart' button does not automagically add all items to the cart. The #default_value => 0 in the first block is based on a fix in a different post addressing this issue.

I've attached a patch here which has a set of changes I made to allow both singular and batch addition of wishlist items to the cart. I found it to be useful since I can test out how people respond to different options then. The patch includes a bunch of different things, so if you're not interested in all the features, please diff and only take what you need. But it incorporates all of the above discussion.

(BTW if you don't want to patch, and just want the whole bundle, it's attached as uc_wishlist_sr.tgz. Apologies to the module maintainers if this is not cool - please let me know if the proper code of conduct is to only provide patches).

I screwed up my patch!!!

Please ignore the previous patch and attachment and use the one in this post instead. The problem is that I had a reference to $item->wantqty in uc_wishlist.pages.inc which does not exist (it should be $item->qty, in the form element default and value options).

Many apologies!

am also experiencing this issue. Any chance cybertoast's patch can make it into a core release of UC Wishlist? The wishlist is rendered pretty much useless if people can't buy stuff from it...

@ cybertoast - many thanks for the module bud - works a treat!!!!!

// Why not just limit the user to the range of possible options?
$element['qty'] = array(
'#type' => 'select',
'#options' => range(0,$item->qty),
'#default_value' => $item->qty - $item->haveqty > 0 ? $item->qty - $item->haveqty : 1,
);

this part, i think it's more correct as this

'#default_value' => $item->qty - $item->haveqty > 0 ? $item->qty - $item->haveqty : 0,

+ Subscribing

This works now for when someone adds to cart but when you try to change the quantity wanted when you are logged in it doesn't work.
Please help.

Problem solved. If you have this same experience this is how I fixed it.

Add the below code to the uc_wishlist_pages.inc page:

if($own){
$element['wantqty'] = array(
'#type' => 'textfield',
'#default_value' => $item->qty,
'#size' => 5,
'#maxlength' => 6,
);
}else{
$element['wantqty'] = array(
'#type' => 'textfield',
'#default_value' => $item->qty,
'#value' => $item->qty, // Necessary with Disabled
'#size' => 5,
'#maxlength' => 6,
'#disabled' => TRUE
);
}

gdblackx what line #/ where ??

Patch attached that fixes this issue.

Fixed in uc_wishlist-6.x-1.1.