Follow us on Twitter X-Cart on Facebook Wiki
Shopping cart software Solutions for online shops and malls

Review Reminder Email

 
Reply
   X-Cart forums > X-Cart 4 > Dev Questions > Changing design
 
Thread Tools Search this Thread
  #21  
Old 01-23-2013, 06:19 AM
 
carpeperdiem carpeperdiem is offline
 

X-Guru
  
Join Date: Jul 2006
Location: New York City, USA
Posts: 5,399
 

Default Re: Review Reminder Email

Quote:
Originally Posted by DavyMac
Within the review reminder email I'd like to include the customer's email address in the link to the add_review.php page so that as well as automatically adding their name to the author field it will add their email address to the email address field for them.

Does anybody see a way of doing this?

I solved this by using the mod, "Remember Me" by WCM
http://www.websitecm.com/x-cart-mods/remember-me.html

If a customer is logged in, then the fields on the write review page will automatically fill using this code (not sure if I added it or if it's stock):

file:
/common_files/modules/Advanced_Customer_Reviews/customer_add_review.tpl

Code:
<input type="text" name="review[author]" id="author" value="{if $login ne ''}{$fullname}{elseif $stored_review.author}{$stored_review.author|escape}{/if}" size="40" />

(see the {if $login ne ''}, etc....)

and:
Code:
<input type="text" class="input-email" name="review[email]" id="email" value="{if $login ne ''}{$login}{else $stored_review.email|escape}{/if}" size="40" />

If the customer is not logged in, I'm having a hard time justifying the expectation of pre-populating form fields. Does this work better for you?
__________________
xcart 4.5.4 gold+ w/x-payments 1.0.6; xcart gold 4.4.4
Reply With Quote

The following user thanks carpeperdiem for this useful post:
DavyMac (02-17-2013)
  #22  
Old 02-18-2013, 08:06 AM
 
DavyMac DavyMac is offline
 

eXpert
  
Join Date: Apr 2007
Location: Poole, UK
Posts: 333
 

Default Re: Review Reminder Email

In Test Mode emails are being sent for orders that failed, not finished, payment authorised and awaiting payment yet in the settings "Send review reminder to customer (if order is completed) " is set true?

Not only that but customers that had multiple not finished orders have recieved a review request for each incomplete order!

Has anyone else had this?
__________________
X-Cart GOLD Plus v4.5.4
CDSEO Pro
ewdhosting Dedicated Server
Dedicated SSL
Stone Edge OM
Reply With Quote
  #23  
Old 02-18-2013, 10:48 AM
 
DavyMac DavyMac is offline
 

eXpert
  
Join Date: Apr 2007
Location: Poole, UK
Posts: 333
 

Default Re: Review Reminder Email

This is very odd behaviour indeed. Of the 148 orders that were checked to see if a reminder should be emailed to the customer their status was as follows:
8 Awaiting Payment
12 Not Finished
8 Failed
1 Cancelled
60 Dispatched
59 Payment Authorised.

Except for 1 Not Finished and 1 payment Authorised ALL orders were sent an email.
__________________
X-Cart GOLD Plus v4.5.4
CDSEO Pro
ewdhosting Dedicated Server
Dedicated SSL
Stone Edge OM
Reply With Quote
  #24  
Old 02-20-2013, 10:55 AM
 
DavyMac DavyMac is offline
 

eXpert
  
Join Date: Apr 2007
Location: Poole, UK
Posts: 333
 

Default Re: Review Reminder Email

Well, apparently the odd behaviour I witnessed is exactly what Test mode is meant to do - it simply provides you with a method of testing the format of the email sent, which I guess is fair enough. Would have been more reassuring to have it run the algorithm that determines who gets a reminder.
__________________
X-Cart GOLD Plus v4.5.4
CDSEO Pro
ewdhosting Dedicated Server
Dedicated SSL
Stone Edge OM
Reply With Quote
  #25  
Old 02-20-2013, 11:00 AM
 
DavyMac DavyMac is offline
 

eXpert
  
Join Date: Apr 2007
Location: Poole, UK
Posts: 333
 

Default Re: Review Reminder Email

Now that I'm out of test mode I've experienced the following SQL error:
Remote IP :
Logged as :
SQL query : SELECT product, descr, fulldescr FROM WHERE productid = '2404' LIMIT 1

Error code : 1064

Description :You have an error in your SQL syntax; check the manual that corresponds to yourMySQL server version for the right syntax to use near 'WHERE productid = '2404'LIMIT 1' at line 1 Request URI: /send_review_reminders.php?key=

Backtrace:

/home/xxx/public_html/include/func/func.db.php:309
/home/xxx/public_html/include/func/func.db.php:209
/home/xxx/public_html/include/func/func.db.php:522
/home/xxx/public_html/include/func/func.order.php:3144
/home/xxx/public_html/modules/Advanced_Customer_Reviews/func.php:525
/home/xxx/public_html/modules/Advanced_Customer_Reviews/func.php:593
/home/xxx/public_html/send_review_reminders.php:92



Does anyone have any ideas please?
Since I have not modified the code I didn't expect to see SQL errors.
__________________
X-Cart GOLD Plus v4.5.4
CDSEO Pro
ewdhosting Dedicated Server
Dedicated SSL
Stone Edge OM
Reply With Quote
  #26  
Old 02-20-2013, 12:00 PM
 
DavyMac DavyMac is offline
 

eXpert
  
Join Date: Apr 2007
Location: Poole, UK
Posts: 333
 

Default Re: Review Reminder Email

What can I say? I changed the "Send review reminder to customer in N days " from 9 to 3 and the SQL errors went away.
__________________
X-Cart GOLD Plus v4.5.4
CDSEO Pro
ewdhosting Dedicated Server
Dedicated SSL
Stone Edge OM
Reply With Quote
  #27  
Old 02-20-2013, 12:01 PM
  karina's Avatar 
karina karina is offline
 

X-Cart team
  
Join Date: Jul 2009
Posts: 123
 

Default Re: Review Reminder Email

Quote:
Originally Posted by DavyMac
In Test Mode emails are being sent for orders that failed, not finished, payment authorised and awaiting payment yet in the settings "Send review reminder to customer (if order is completed) " is set true?

Not only that but customers that had multiple not finished orders have recieved a review request for each incomplete order!

Has anyone else had this?

In the test mode the order status is not taken into account.

Quote:
Originally Posted by DavyMac
Well, apparently the odd behaviour I witnessed is exactly what Test mode is meant to do - it simply provides you with a method of testing the format of the email sent, which I guess is fair enough. Would have been more reassuring to have it run the algorithm that determines who gets a reminder.

I've added the new condition for test mode: TEST_EMAIL is required now if TEST_MODE is TRUE. If TEST_EMAIL is empty then review reminders will not be sent at all. It will be in the updated version of the module. In this case store owner/developer can specify his/her own email for testing purpose and get test review reminders.

Quote:
Originally Posted by DavyMac
SQL query : SELECT product, descr, fulldescr FROM WHERE productid = '2404' LIMIT 1

Error code : 1064

Description :You have an error in your SQL syntax; check the manual that corresponds to yourMySQL server version for the right syntax to use near 'WHERE productid = '2404'LIMIT 1' at line 1 Request URI: /send_review_reminders.php?key=

It seems that $sql_tbl array is not defined for some reason. There are two functions for review reminder in the 'modules/Advanced_Customer_Reviews/func.php' file:
func_acr_send_review_reminder()
func_acr_send_review_reminders()

Both of them should contain part of code like this:
PHP Code:
global $sql_tbl

Could you please check your file and check if global definition of $sql_tbl is there?
__________________
Karina Lipnyagova

The "Advanced Customer Reviews", "Root Categories" and "Pop-up Anywhere" author
Reply With Quote
  #28  
Old 02-21-2013, 04:25 AM
  karina's Avatar 
karina karina is offline
 

X-Cart team
  
Join Date: Jul 2009
Posts: 123
 

Default Re: Review Reminder Email

Quote:
Originally Posted by DavyMac
What can I say? I changed the "Send review reminder to customer in N days " from 9 to 3 and the SQL errors went away.

Quote:
Originally Posted by DavyMac
SQL query : SELECT product, descr, fulldescr FROM WHERE productid = '2404' LIMIT 1

Error code : 1064

Description :You have an error in your SQL syntax; check the manual that corresponds to yourMySQL server version for the right syntax to use near 'WHERE productid = '2404'LIMIT 1' at line 1 Request URI: /send_review_reminders.php?key=

This SQL error should not be connected with the "Send review reminder to customer in N days " setting. I've failed to reproduce the same SQL error. However I've investigated this issue and found another possible reason.

Please check your '<xcart_dir>/send_review_reminders.php' file and found this part of code:

PHP Code:
define('SKIP_CHECK_REQUIREMENTS.PHP'true);

require 
'./init.php'
 
$argv $_SERVER['argv']; 

and replace it with:

PHP Code:
define('SKIP_CHECK_REQUIREMENTS.PHP'true);

require 
'./init.php'
require 
'./include/get_language.php';
 
$argv $_SERVER['argv']; 
__________________
Karina Lipnyagova

The "Advanced Customer Reviews", "Root Categories" and "Pop-up Anywhere" author
Reply With Quote
  #29  
Old 02-21-2013, 04:33 AM
 
carpeperdiem carpeperdiem is offline
 

X-Guru
  
Join Date: Jul 2006
Location: New York City, USA
Posts: 5,399
 

Default Re: Review Reminder Email

Quote:
Originally Posted by karina
However I've investigated this issue and found another possible reason.

Please check your '<xcart_dir>/send_review_reminders.php' file and found this part of code:

PHP Code:
define('SKIP_CHECK_REQUIREMENTS.PHP'true);

require 
'./init.php'
 
$argv $_SERVER['argv']; 

and replace it with:

PHP Code:
define('SKIP_CHECK_REQUIREMENTS.PHP'true);

require 
'./init.php'
require 
'./include/get_language.php';
 
$argv $_SERVER['argv']; 

Hi Karina,
Is this an edit you recommend to all ACR installs? Or is this something only needed for stores with more than one language?
Thanks for clarifying and thanks for the constant monitoring and improvements of ACR.
Jeremy
__________________
xcart 4.5.4 gold+ w/x-payments 1.0.6; xcart gold 4.4.4
Reply With Quote
  #30  
Old 02-21-2013, 04:51 AM
  karina's Avatar 
karina karina is offline
 

X-Cart team
  
Join Date: Jul 2009
Posts: 123
 

Default Re: Review Reminder Email

Quote:
Originally Posted by carpeperdiem
Hi Karina,
Is this an edit you recommend to all ACR installs? Or is this something only needed for stores with more than one language?
Thanks for clarifying and thanks for the constant monitoring and improvements of ACR.
Jeremy

Jeremy, yes, it's better to apply this fix to all ACR installs.

More information: I failed to reproduce the SQL error from DavyMac post, but I've faced with another small issue: wrong encoding of review reminder subject line. I've fixed it with the code from my previous post. And I guess my issue and issue from DavyMac post are connected. This issue is actual for module versions for X-Cart v4.5.3, 4.5.4, 4.5.5 only.
__________________
Karina Lipnyagova

The "Advanced Customer Reviews", "Root Categories" and "Pop-up Anywhere" author
Reply With Quote

The following user thanks karina for this useful post:
DavyMac (02-21-2013)
Reply
   X-Cart forums > X-Cart 4 > Dev Questions > Changing design


Thread Tools Search this Thread
Search this Thread:

Advanced Search

Posting Rules
You may not post new threads
You may not post replies
You may not post attachments
You may not edit your posts

vB code is On
Smilies are On
[IMG] code is On
HTML code is Off
Forum Jump


All times are GMT -8. The time now is 06:04 PM.

   

 
X-Cart forums © 2001-2020