Jump to content
Mort-lemur

Wishlist for 2.3 - Problem with emails

Recommended Posts

Following my previous thread on database problem I have now installed this Mod: http://addons.oscommerce.com/info/8665 and everything works as it should on the customer and admin side, with one exception:

 

When a customer attempts to email their wishlist to a friend using the wishlist email function, he is presented with a blank screen, and no email is sent.

 

The error log entry is:

 

PHP Fatal error:  Call to undefined function tep_mail() in /home/username/public_html/wishlist.php on line 204

 

with Line 204 of wishlist.php being:

tep_mail($friend[$j], $email[$j], $subject, $friend[$j] . ",\n\n" . $body, $from_name, $from_email);

Any pointers where to look to solve this please?

 


Now running on a fully modded, Mobile Friendly 2.3.4 Store with the Excellent MTS installed - See my profile for the mods installed ..... So much thanks for all the help given along the way by forum members.

Share this post


Link to post
Share on other sites

Thanks Luca, but I already checked there and the function is defined


Now running on a fully modded, Mobile Friendly 2.3.4 Store with the Excellent MTS installed - See my profile for the mods installed ..... So much thanks for all the help given along the way by forum members.

Share this post


Link to post
Share on other sites

If tep_mail is defined in general.php and application_top.php is included in wishlist.php the function should work. Very strange. what version of osc you have?

Share this post


Link to post
Share on other sites

2.3.4


Now running on a fully modded, Mobile Friendly 2.3.4 Store with the Excellent MTS installed - See my profile for the mods installed ..... So much thanks for all the help given along the way by forum members.

Share this post


Link to post
Share on other sites

@@acidvertigo yes im using the latest version.

 

so the send email function works as written in a vanilla install? and actually sends the email?

 

many thanks


Now running on a fully modded, Mobile Friendly 2.3.4 Store with the Excellent MTS installed - See my profile for the mods installed ..... So much thanks for all the help given along the way by forum members.

Share this post


Link to post
Share on other sites

Ahhh ok, sorted it out now.

 

Although I had looked to make sure that the tep_mail function was there in the admin side, I had not checked in the catalog side, and had forgotton that some time ago I had changed the function tep_mail to tep_mail_attach (to send T&C attachements with mails) so I replaced tep_mail with tep_mail_attach in wishlist.php.

 

Now everything works as it should, the sun is shining and the world is a happier place :)

 

Thank you

Edited by Mort-lemur

Now running on a fully modded, Mobile Friendly 2.3.4 Store with the Excellent MTS installed - See my profile for the mods installed ..... So much thanks for all the help given along the way by forum members.

Share this post


Link to post
Share on other sites

@@Mort-lemur

 

Hi

 

I remember the issue with the T&C attachments :D . Generally I would never recommend to replace any of the default functions of osc by a new one since this could have unexpected implications elsewhere in the shop. It is always better to leave the existing function untouched and simply to add a possibly required additional function.

 

J.J.

Share this post


Link to post
Share on other sites

@@De Dokta Thats what I thought after my last post, so now I have changed the catalog/includes/functions/general.php to include both instances. ie:

 

tep_mail

tep_mail_attach

 

Many Thanks


Now running on a fully modded, Mobile Friendly 2.3.4 Store with the Excellent MTS installed - See my profile for the mods installed ..... So much thanks for all the help given along the way by forum members.

Share this post


Link to post
Share on other sites

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!

Register a new account

Sign in

Already have an account? Sign in here.

Sign In Now

×