This may be difficult to implement, but it would be nice to provide a function for an owner of a wishlist to email it out to others, even if it is just a link to their wishlist.
This would allow those emailed to become aware of the wishlist.
Another sub feature would be to add a "commits" column in the wishlist to allow people to commit to buying from the wishlist.
These features would allow this module to be more similar to other wishlists out there.

CommentFileSizeAuthor
#8 uc_wishlist_2094159_8.patch16.39 KBnaveenvalecha
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jaykainthola’s picture

jaykainthola CreditAttribution: jaykainthola commented

Hi,

I will add this feature into module.

Thanks

tparc’s picture

tparc CreditAttribution: tparc commented

Thank you for committing to this feature. I will be happy to provide any feedback.
Was there any plans to port the following feature to D7: https://drupal.org/node/1234434 ?

jaykainthola’s picture

jaykainthola CreditAttribution: jaykainthola commented

Hi tparc,

I am working on this feature to port into Drupal 7.

I have not yet added this feature to Drupal 6 and I will discuss this with other maintainer of the project that will request him to implement this into Drupal 6.

Thanks

tparc’s picture

tparc CreditAttribution: tparc commented

Hi. Has any progress been made on implementing this feature yet? Thx

jaykainthola’s picture

jaykainthola CreditAttribution: jaykainthola commented

Hi Tparc,

Due to some personal problem, I could not work on this. So, I have assigned another maintainer(naveenvalecha) for this project. I hope, he has some update regarding the same.

naveenvalecha’s picture

naveenvalecha
He/Him
Primary language Hindi
CreditAttribution: naveenvalecha commented

Hi Tparc,
I am quite busy in this week due to my exams.I will see this issue after 10 December and may provide the patch for the same.Hope you understand the same.

naveenvalecha’s picture

naveenvalecha
He/Him
Primary language Hindi
CreditAttribution: naveenvalecha commented

Hi Tparc,
I am still working on this feature request and have done a small part of this.I will update you after getting done.

naveenvalecha’s picture

naveenvalecha
He/Him
Primary language Hindi
CreditAttribution: naveenvalecha commented
Component: Miscellaneous » Code
Assigned: Unassigned » naveenvalecha
Issue summary: View changes
FileSize
uc_wishlist_2094159_8.patch16.39 KB

Hi All,
Here is the attached patch that will provide the tab on the user profile page for emailing the wishlist.Also it provides us to set the from email address on the configuration page.Please provide your feedback on this.

tparc’s picture

tparc CreditAttribution: tparc commented

Thanks, naveenvalecha. I will test this as soon as I can find a moment.

tparc’s picture

tparc CreditAttribution: tparc commented

I am getting the error message "Undefined variable: wid in uc_wishlist_email_form_submit() (line 713 of /home3/tparc/test/sites/all/modules/uc_wishlist/uc_wishlist.pages.inc", which relates to this line in the code: $message = str_replace('["wid"]', $wid, $message);
when I send an email. I tried to change syntax of command and $message but with no luck.

tparc’s picture

tparc CreditAttribution: tparc commented

Any progress on this issue? Has error message been replicated? Thanks

jaykainthola’s picture

jaykainthola CreditAttribution: jaykainthola commented

@naveenvalecha , I have checked your patch and I found that you have added another tab for the email wishlist. As per my understanding, there should not require extra tab for only email functionality. We can create a separate page for email wishlist and link that page in wishlist.

@Tparc, please suggest on my point, so we can implement the same.

Thanks

jaykainthola’s picture

jaykainthola CreditAttribution: jaykainthola commented

Hi tparc,

I have applied patch and fixed the issue. Please review.

Thanks

naveenvalecha’s picture

naveenvalecha
He/Him
Primary language Hindi
CreditAttribution: naveenvalecha commented
Status: Active » Closed (fixed)

Hi Folks

#8 patch committed 3e4e55f with some little ambedments by Jaykainthola.

Thanks
Naveen Valecha

tparc’s picture

tparc CreditAttribution: tparc commented

Has this patch been tested? I would offer several improvements, if resources allow.
1) jaykainthola's #12 comment suggests that another tab should not be created to email wishlist. It appears the tab still exists in the user profile - Perhaps there could be an option in the Wishlist settings tab instead to email the wishlist, not a new tab. Not entirely important.
2) Could some instructions be provided to users when emailing a wishlist (such as 'Please fill out the form below to email your wishlist to others', or give the administrator option to provide custom instructions to users on emailing wishlist? Also a status message to the user when the wishlist has been sent successfully?
3) When wishlist is emailed, the link to the wishlist is incomplete. Instead of http://[site:name]/wishlist/1, only http://wishlist/1 is provided. This seems like a major bug.
4) Formatting: Email 'message content' is not separated from link to wishlist. Link to wishlist should be on a new line.
Thank you.