Tie in to Product Kits

I'm experiencing the same behavior. Is there an easy way to allow this module to affect Product Kits as well?

This patch should enable wish lists for product kits.

Whoops, that didn't actually fix anything, this should be the correct patch.

This gets the button on the page and will add the entire kit as a wishlist item, but does not seem to add the individual items of a kit set "as individual products", i.e. ones that explode when you add them to the cart. I am trying to figure out how to get that part to work, which will hopefully take into account any attributes for parts that are in the kit. Will chime back in if I figure it out. If anyone has any pointers, that would be appreciated.

Fix spelling

The patch in #3 seems to also still add the product kit to the cart. Perhaps it's not removing a submit handler that it ought to be.

Here's a patch that fixes the behavior which sees the product kit simultaneously added to the cart. This is because Ubercart Product Kit handles this in uc_product_kit_add_to_cart; it's not supposed to do that, but that breaks things because we invoke that hook. I made a custom module invocation function for that hook which doesn't invoke it for uc_product_kit.

Not fully sure if this is working properly completely through checkout. Investigating that.

Nah, it doesn't break checkout. It doesn't update the Have total properly but I don't think I'll be figuring that one out at this stage as quantity doesn't really apply to the site I'm working on.

So this should be good.

Could this be committed to D7? Anyone willing to pick it up? Wish I had more experience.

@tparc, Kindly find the latest dev version of Drupal 7 version of the module. Its already implemented into module.

Thanks,
Jayendra Kumar

Fixed this is in the drupal 7 dev version.Needs to backport this request to drupal 6 version.

Thanks
Naveen Valecha

Thank you. Appears to be functioning well in D7. Have not tested all the way through checkout to see if 'wanted' quantity is reduced when item is purchased, but will try to do so.

The needs work status is for the D6 version. That's unlikely to happen. And review has been done by tparc. I propose we change the version back to 7.x, and close the issue.

Surely sagar Ramgade would take this and backport this feature in d6 if he wants to.

I had ported the D7 patch to D6, will commit it.

Sagar,
Great! A minor nutpick.

+++ uc_wishlist.module	(revision )
@@ -781,4 +794,29 @@
+function _uc_wishlist_invoke_hook_add_to_cart() {

We should not use the prefix with this function name.

Thanks naveen yes understand it shouldn't be a private function, corrected in the next commit.

ok Remember it yourself. But do it before any release.Not a problem in dev branch.
Please change the status of the issue after fixing the issue.