Reward Points Module for ZC v13x

[southshorepizza]

I don't think that is the issue. I just downloaded that file and uploaded it to the site.
I got this:
[29-Mar-2014 17:36:59] PHP Fatal error: 1064:You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'AND status=0 ORDER BY date DESC LIMIT 12' at line 1 :: SELECT rewards_id, orders_id, date, reward_points, status FROM zen_reward_status_track WHERE customers_id= AND status=0 ORDER BY date DESC LIMIT 12; in /home4/w57dsjmm/public_html/order/includes/classes/db/mysql/query_factory.php on line 120

Sorry it didn't work.

[lhungil]

... PHP Fatal error: 1064:You have an error in your SQL syntax; ... SELECT rewards_id, orders_id, date, reward_points, status FROM zen_reward_status_track WHERE customers_id= AND status=0 ORDER BY date DESC LIMIT 12; ...

I believe an issue with a missing customer_id in the SQL statement has been reported before in this thread... Not sure if a solution was posted... If you install this modification by lat9, you can track down where in the reward points module the problematic SQL query is being built.

Hopefully once we all know where the SQL statement causing the error is being created, someone will be able to help further diagnose and fix the issue.

[mnitescu]

hi there
I have the 1.3.9. version of reward points.
Now I updgraded from zen cart 1.3.9. to 1.5.1 and used the 1.5.0 upgrade sql.
So I have the 1.3.9 version of reward points with 1.5.0 upgrade, which I think is not quite right.
When I go to Customer>Reward Points and try to edit anything all I get is a blank page.
What should I do?

[DivaVocals]

What should I do?

Blank page = ERROR LOG

Go to your error log and tell us what the error log says..

[southshorepizza]

I believe an issue with a missing customer_id in the SQL statement has been reported before in this thread... Not sure if a solution was posted... If you install this modification by lat9, you can track down where in the reward points module the problematic SQL query is being built.

Hopefully once we all know where the SQL statement causing the error is being created, someone will be able to help further diagnose and fix the issue.

I have installed it and will report back soon. Thank you for this tip!

[southshorepizza]

I installed the modification by lat9 as suggested. Now my error log reads:

[05-Apr-2014 04:20:36] PHP Fatal error:  1064:You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'AND status=0 ORDER BY date DESC LIMIT 12' at line 1 :: SELECT rewards_id, orders_id, date, reward_points, status FROM zen_reward_status_track WHERE customers_id= AND status=0 ORDER BY date DESC LIMIT 12; ==> (as called by) /home4/xxxx/public_html/order/admin/customers_reward_points.php on line 358 <== in /home4/xxxx/public_html/order/includes/classes/db/mysql/query_factory.php on line 130

Any help would be greatly appreciated!

[lhungil]

... PHP Fatal error: 1064:You have an error in your SQL syntax; ... /admin/customers_reward_points.php on line 358 ...

Any help would be greatly appreciated!

Looks like the code in that file ASSUMES a valid customer id will be passed in the URL when certain actions are performed. If the customer id is not valid when trying to edit, update, or process reward points this error will occur.

As a temporary way to avoid this error, one "could" wrap the entire switch statement (performing the action - starts around line 318) with something similar to:

if(zen_not_null($current_customer)) {

Closing the if statement after the switch (around line 381) using something similar to:

} else { echo '<div class="alert">No Customer (or invalid customer) selected. No action performed!</div>'; }

[izar74]

Hi lhungil,
thanks for the advice. I sometimes got the error but dunno where it was generated. Just to be sure not to do stupid changes could you please confirm that the following code is correct:
Unmodified file customers_reward_points.php

  switch ($action) //line 317
  {                     //line 318
    case 'edit':      //line 319
    case 'update': 

Modified:

if(zen_not_null($current_customer)) { //line 316 added by lhungil
switch ($action)  //line 317
  {                     //line 318
    case 'edit':      //line 319
    case 'update': 

Then in the end:
Unmodified

if((zen_not_null($heading)) && (zen_not_null($contents))) //line 381
  {                                                                           //line 382
    echo '            <td width="25%" valign="top">' . "\n";    //line 383 

modified:

} else { echo '<div class="alert">No Customer (or invalid customer) selected. No action performed!</div>'; } //line 380
if((zen_not_null($heading)) && (zen_not_null($contents))) //line 381
  {                                                                           //line 382
    echo '            <td width="25%" valign="top">' . "\n";    //line 383 

Sorry but that "around line xxx" put me in danger zone, i prefer to be sure before making any modification (i know it for dummy but be patient )

[bvgolf]

Hi,

I would like to show the reward points below product price on the main page, categories, featured items, new products, etc., but not for "call for price" items. Do you think that this is possible?

Thank you,

[lhungil]

... could you please confirm that the following code is correct: ...

"/admin/customers_reward_points.php" (changes in red):

315:	$contents = array();
316:
	  if(zen_not_null($current_customer)) {
317:	    switch ($action) { // Combined w/ line 318
319:	      case 'edit':
320:	      case 'update':
321:	      case 'handle_point_list':

...

376:	          $record_contents[]=array('text' => '</form>');
377:	        }
378:	        break;
379:	    } // End of switch statement
	  } else {
	    echo '<div class="alert">No Customer (or invalid customer) selected. No action performed!</div>';
	  }
380:
381:	  if((zen_not_null($heading)) && (zen_not_null($contents)))