Reward Points Full Suite v2.5 (for ZC v1.5x) Support Thread

DivaVocals:

Yes search AGAIN.. find the LANGUAGE file that contains the actual language definition.. You don't edit the file that USES the definition (as it it appears you are trying to do), you edit the language file that contains the text you want to replace..

Looks something like..

define, 'Total Reward Discount Amount';

REPLACE the text you DON'T want with the text you DO WANT..

Thanks, I will keep searching till I find correct one

Reinstalling this as part of a v1.5.4 upgrade for a client.. Can someone explain WHY the upgrade SQL contains the admin page delete statements only to RE-INSERT the same entries again later in the script??

/*
zencart 1.5 mods
*/
DELETE FROM admin_pages WHERE page_key='configRewardPoints';
DELETE FROM admin_pages WHERE page_key='GroupRewardRedeem';
DELETE FROM admin_pages WHERE page_key='RewardPoints';
DELETE FROM admin_pages WHERE page_key='CustomerRewardPoints';

SELECT @cgi:=configuration_group_id FROM configuration_group WHERE configuration_group_title = 'Reward Points';

INSERT INTO configuration
(configuration_id ,configuration_title ,configuration_key ,configuration_value ,configuration_description ,configuration_group_id ,sort_order ,last_modified ,date_added ,use_function ,set_function)
VALUES (NULL , 'Show Reward Points on Product Info Display Page', 'SHOW_REWARD_POINTS_PRODUCT', '0', 'Display Reward Points on product info display page?<br />0= No<br />1= Yes', @cgi, '1', NULL, now(), NULL , 'zen_cfg_select_option(array(''0'', ''1''), ');

INSERT INTO admin_pages (page_key,language_key,main_page,page_params,menu_key,display_on_menu,sort_order) VALUES ('configRewardPoints','BOX_CONFIGURATION_REWARD_POINTS','FILENAME_CONFIGURATION',CONCAT('gID=',@cgi), 'configuration', 'Y', @cgi);  
INSERT INTO admin_pages (page_key, language_key, main_page, page_params, menu_key, display_on_menu, sort_order) VALUES ('GroupRewardRedeem', 'BOX_GROUP_REWARD_POINTS_REDEEM', 'FILENAME_ADMIN_GROUP_REWARD_POINTS_REDEEM', '', 'customers', 'Y', 35); 
INSERT INTO admin_pages (page_key, language_key, main_page, page_params, menu_key, display_on_menu, sort_order) VALUES ('RewardPoints', 'BOX_REWARD_POINTS', 'FILENAME_ADMIN_REWARD_POINTS', '', 'catalog', 'Y', 36); 
INSERT INTO admin_pages (page_key, language_key, main_page, page_params, menu_key, display_on_menu, sort_order) VALUES ('CustomerRewardPoints', 'BOX_CUSTOMER_REWARD_POINTS', 'FILENAME_ADMIN_CUSTOMER_REWARD_POINTS', '', 'customers', 'Y', 37); 

As far as I can see the only REAL "upgrade/change" is the addition of one more configuration option.. Am I missing something here?????

more upgrade questions..

regarding the admin/orders.php file

on line 607 I see this change:

           '                <td align="right" class="'. str_replace('_', '-', $order->totals[$i]['class']) . '-Amount">' . $order->totals[$i]['text'] . '</td>' . "\n" .

the default code is this:

           '                <td align="right" class="'. str_replace('_', '-', $order->totals[$i]['class']) . '-Amount">' . $currencies->format($order->totals[$i]['value'], false) . '</td>' . "\n" .

Is this a Rewards points change?? (there are no code comments to help clarify this if it is) If it is a Rewards Points modification, I need some clarification as to what EXACTLY this change does???

Also because the orders.php file bundled in the Rewards Points module is from the last v1.5.4RC version and NOT the final release, it is MISSING a language file change that went into the last release v1.5.4RC version and eventually into the final release code.

line 978 reads:

      $contents[] = array('text' => 'Products Ordered: ' . sizeof($order->products) );

It should be:

      $contents[] = array('text' => TABLE_HEADING_PRODUCTS . ': ' . sizeof($order->products) );

I believe the change is to show the "text" field from the order_total database table directly instead of reading from the "value" and displaying based upon currency rules.

It has been a long time since I have looked at this module, But If I remeber correctly the order total module for reward points uses the "text" field to display the "points" and leaves the value at 0 for earned (and a monetary value if redeemed).

lhungil:

I believe the change is to show the "text" field from the order_total database table directly instead of reading from the "value" and displaying based upon currency rules.
Okay that makes sense.. I get the WHAT it does.. Now the question is WHY.. I am starting to suspect that this is some legacy thing that is still in this code.. because I left this change out and cannot see it making any difference.. Now I am have NEVER used this module before, so it could be I am just not getting it here, but I cannot see how this change is needed..

lhungil:

It has been a long time since I have looked at this module, But If I remeber correctly the order total module for reward points uses the "text" field to display the "points" and leaves the value at 0 for earned (and a monetary value if redeemed).so is the change to the admin/orders.php required for this to work??

Regarding my previous post..

DivaVocals:

DivaVocals:

more upgrade questions..

regarding the admin/orders.php file

on line 607 I see this change:

       '                <td align="right" class="'. str_replace('_', '-', $order->totals[$i]['class']) . '-Amount">' . $order->totals[$i]['text'] . '</td>' . "\n" .

> > 
> > the default code is this:
> > ```
           '                <td align="right" class="'. str_replace('_', '-', $order->totals[$i]['class']) . '-Amount">' . $currencies->format($order->totals[$i]['value'], false) . '</td>' . "\n" .

Is this a Rewards points change?? (there are no code comments to help clarify this if it is) If it is a Rewards Points modification, I need some clarification as to what EXACTLY this change does???

Also because the orders.php file bundled in the Rewards Points module is from the last v1.5.4RC version and NOT the final release, it is MISSING a language file change that went into the last release v1.5.4RC version and eventually into the final release code.

line 978 reads:

  $contents[] = array('text' => 'Products Ordered: ' . sizeof($order->products) );

> > 
> > It should be:
> > ```
      $contents[] = array('text' => TABLE_HEADING_PRODUCTS . ': ' . sizeof($order->products) );

lhungil:

I believe the change is to show the "text" field from the order_total database table directly instead of reading from the "value" and displaying based upon currency rules.

It has been a long time since I have looked at this module, But If I remeber correctly the order total module for reward points uses the "text" field to display the "points" and leaves the value at 0 for earned (and a monetary value if redeemed).

Now I see what happened and when this code was introduced into the base module.. Thanks to a little birdie, I found these posts from the old support thread:

izar74:

Hi again,

hard work bring issues as long as solutions ;-)

First the issue: I found that when i was going to the details of an order the Rewards point shown were always 0, but if you go on the invoice module the Rewards point magically appeared. Simply in the order details the point were not shown.

it seems that the problem could be with this line in /admin/order.php around line 666

<td align="right" class="'. str_replace('_', '-', $order->totals[$i]['class']) . '-Amount">' . $currencies->format($order->totals[$i]['value'], false) . '</td>' . "\n" . ``` > > > > If replaced with: > > ```php <td align="right" class="'. str_replace('_', '-', $order->totals[$i]['class']) . '-Amount">' . $order->totals[$i]['text'] . '</td>' . "\n" . ``` > > > > all seems fine. Someone else got this error? > > > > Zencart 1.51 > > Reward Point 2.1b > > > **lhungil:** > > > > Order Total Modules were designed to show prices added or subtracted from an order... While the changes mentioned earlier "may" work... They may also break pricing calculation for stores accepting more than one currency... The stock Zen Cart "/admin/order.php" uses the database entry for "value" (with any currency conversion and formatting) instead of "text". > > > > Instead of removing the currency calculations from EVERY order total module, one should probably add a modification to "/admin/order.php" specifically targeting the "Reward Points display" order total module (and using "text" when this module is encountered)... If going this route, one would also need to add an exemption to cause "Edit Orders" to NOT load the "Reward Points display" module (so it does not edit this line)... Or add "special" handling to address the non-standard use of the "value" and "text" but the "Reward Points display" order total module. > > > > The alternative is to stop using (and remove) the "Reward Points Display" order total module and add the reward points earned information directly to the various templates / pages involved. This avoids the issues caused by using the "orders_total" database table in a way not expected by the Zen Cart core code. > > > **lhungil:** > > > > Alright, dug into this the subject a little deeper today. Looks like Zen Cart 1.5.x out of box does not allow one to view orders in the admin with a currency other than the store default... I remember a discussion of this somewhere on the forums in the past, but do not remember what was decided... > > > > So your change should not make a difference for the majority of users of Zen Cart 1.5.x (only those who may have for some reason wanted to see localized pricing for a different country in the admin and made changes)... I still believe it would be even better to at least add some conditional code to check which "order total" module (in "/admin/orders.php") is being displayed and ONLY skip the format function on the "Reward Points display" order total module... Or replace the "Reward Points display" order total module completely. > > > > > > I currently have many other ongoing projects, so will not be able to spend any time on this project. Just was browsing this thread (have poked in here from time to time since it crossed my path) and decided to share some thoughts.

Now at the risk of offending anyone, IMHO this code should have NEVER found it's way into the base module.. I am not a fan of modules which include incomplete solutions which IMHO this code change is.. I think lhungil was on the right track on what the RIGHT solution SHOULD be. Perhaps the change to the orders.php file would have been better served as an instruction included in the readme as opposed to being made part of the code base. This way for those who are okay with this solution knowing the pros and cons of it can implement if they wish.. Or make this "bandaid" solution an admin on/off option..

DivaVocals:

more upgrade questions..

regarding the admin/orders.php file

on line 607 I see this change:

       '                <td align="right" class="'. str_replace('_', '-', $order->totals[$i]['class']) . '-Amount">' . $order->totals[$i]['text'] . '</td>' . "\n" .

> 
> the default code is this:
> ```
           '                <td align="right" class="'. str_replace('_', '-', $order->totals[$i]['class']) . '-Amount">' . $currencies->format($order->totals[$i]['value'], false) . '</td>' . "\n" .

Is this a Rewards points change?? (there are no code comments to help clarify this if it is) If it is a Rewards Points modification, I need some clarification as to what EXACTLY this change does???

Also because the orders.php file bundled in the Rewards Points module is from the last v1.5.4RC version and NOT the final release, it is MISSING a language file change that went into the last release v1.5.4RC version and eventually into the final release code.

line 978 reads:

  $contents[] = array('text' => 'Products Ordered: ' . sizeof($order->products) );

> 
> It should be:
> ```
      $contents[] = array('text' => TABLE_HEADING_PRODUCTS . ': ' . sizeof($order->products) );

Okay so after thinking about this I think I know what solution I would prefer to implement to achieve the objective of showing the actual points earned (the results of the order_total table's "text" column) on the order detail page..

Been messing around with stuff, and I think what would work while still keeping the currency support is the following:

Show ALL order totals from the order_total table where the class is not = to "ot_reward_points_display" display the "title" and the "value" columns

<?php for ($i = 0, $n = sizeof($order->totals); $i < $n; $i++) { echo ' <tr>' . "\n" . ' <td align="right" class="'. str_replace('_', '-', $order->totals[$i]['class']) . '-Text">' . $order->totals[$i]['title'] . '</td>' . "\n" . ' <td align="right" class="'. str_replace('_', '-', $order->totals[$i]['class']) . '-Amount">' . $currencies->format($order->totals[$i]['value'], false) . '</td>' . "\n" . ' </tr>' . "\n"; } ?>

BUT...

If there are order totals from the order_total table where the class is = to "ot_reward_points_display", show those totals JUST below the other orders totals, but display the values in the "title" and the "text" columns.

<?php
  for ($i = 0, $n = sizeof($order->totals); $i < $n; $i++) {
    echo '          <tr>' . "\n" .
         '            <td align="right" class="'. str_replace('_', '-', $order->totals[$i]['class']) . '-Text">' . $order->totals[$i]['title'] . '</td>' . "\n" .
         '            <td align="right" class="'. str_replace('_', '-', $order->totals[$i]['class']) . '-Amount">' . $order->totals[$i]['text'] . '</td>' . "\n" .
         '          </tr>' . "\n";
  }
?>

Now I've been messing around but I'm not sure how to do this..

I tried this:

<?php 
       if ($order->totals['class'] !== 'ot_reward_points_display') {
        
        code for showing default order totals code which supports currencies goes here. Will exclude Reward Points
        
        }
        
        if ($order->totals['class'] == 'ot_reward_points_display') {
        
        code for showing default order totals code which supports currencies goes here. Will include Reward Points

        }
?>

No dice..

I tried this:

<?php 
       if ($order->totals['class'] !== 'ot_reward_points_display') {
        
        code for showing order totals code which supports currencies goes here. Will exclude Reward Points
        
        }
?>
        
<?php
        if ($order->totals['class'] == 'ot_reward_points_display') {
        
        code for showing default order totals code which supports currencies goes here. Will include Reward Points

        }
?>

Again no dice..

Hoping a kind community member will jump in here..

Hi Diva you can redo and submit any changes you see fix. Was just trying to keep things simple from all the it you want to do this replace this with this. Want hurt my felling at all.....:smile:

countrycharm:

Hi Diva you can redo and submit any changes you see fix. Was just trying to keep things simple from all the it you want to do this replace this with this. Want hurt my felling at all.....:smile:

No worries my friend.. was mostly thinking out loud here.. :smile: If I come up with a solution that keeps the currency support intact, I will post here, and (with your blessings) update the module..:smile:

I am having an issue with the Rewards Points Full Suite that is driving me nuts:

Basically it has to do with Redeeming the points.

Situation is this:

When I a customer redeems points on their purchase to things happen;

1/ the points on their purchase is always incorrect
2/ the tax figure showing is incorrect not updating

The attached images will give you an idea what I mean

The points earned on this order should be 153 (not 150)
GST (tax) content of the order should be $19.89 (not $22.50)

I have worked out the the missing 3 points relates to the GST figure not updating collectly

If points aren't redeemed all works well as per attached:

I'm setting up on my test site which I have configured for NZ Tax's (GST) of 15% & have stripped all other Discount Modules of the site in case there was a conflict with them.

At the present time the development site which will become the live site has been set up with no tax's & it works with no problem at all EG

The Rewards Module is set up out of the box so to speak, but I have set the "Global Reward Point ratio:" at 1.1500 to account for the tax rate that we need to charge on Goods & Services & I quickly figured out that the points are calculated on the Net Product Price (ex tax)

Is there something that I need to do on the mod or is it a tax related issue that I should explore on the tax thread??

I would appreciate any help in sorting this out - Driving me insane - all works well except this issue.

I just installed and points are NOT being added to the customers account. My site is live on my server but not "live" to the public as of yet. I had 2 different testers try using both payment options and it didn't work :( My site is https://www.bellaornamenti.com Thank you

So doing some testing and found myself having to explain this to my client:

Advanced mode: Set the order status items that will trigger a transfer of the reward points between pending and earned. Points are transfered when the status changes between the "Pending" and "Earned". Status items set to "Ignore" will have no effect if the Order Status changes to it.

Honestly this is truly as clear as mud.. I changed it on my client's shop to read:

Advanced mode: Allows you to select the order status that will trigger a transfer of reward points from pending to earned. Points are transferred when the status changes between status items set to "Pending" and status items set to "Earned". Status items set to "Ignore" will have no effect on Order Status changes to it.

Wanted to share in case someone wants to make a similar change.. (or understand better what this means..:smile:)

There's no pagination on this page, and at the bottom of the page there's this text:
"No Customer (or invalid customer) selected. No action performed!"

Not sure what triggers that warning/error.. but there is no pagination or even sorting.. Yes I can search for the customer I am looking for, but some pagination and the ability to sort by first or last name would be a nice navigational touch..

Has anyone done this and if so would ya be willing to share what ya did..

samisoaps:

I just installed and points are NOT being added to the customers account. My site is live on my server but not "live" to the public as of yet. I had 2 different testers try using both payment options and it didn't work :( My site is www.bellaornamenti.com Thank you
Points are added when the order status changes..

DivaVocals:

Points are added when the order status changes..

I've changed the order status and it still didn't update the customers account. I've disabled it for now since I need to get a soft launch on the site this week.

samisoaps:

I've changed the order status and it still didn't update the customers account. I've disabled it for now since I need to get a soft launch on the site this week.

You are going to need to post ALL of your settings to get the RIGHT answer..

DivaVocals:

There's no pagination on this page, and at the bottom of the page there's this text:
"No Customer (or invalid customer) selected. No action performed!"

Not sure what triggers that warning/error.. but there is no pagination or even sorting.. Yes I can search for the customer I am looking for, but some pagination and the ability to sort by first or last name would be a nice navigational touch..

Has anyone done this and if so would ya be willing to share what ya did..

Diva put up a image of what you are seeing under customers/reward points. There should see a index across the top where you can search by A to Z

countrycharm:

Diva put up a image of what you are seeing under customers/reward points. There should see a index across the top where you can search by A to Z

and I DO see that.. But clicking on any letter doesn't show the full results for the letter clicked.. For example I clicked "J" to look for Johnson, and the list is truncated before I can even get to that part of the J's.. This is is where pagination would help, and so would sorting since what I would expect to see is the names arranged first by last name, and then by first name.. It truly is NOT a good user experience, and I'm surprised no one's grumbled about this before..

DivaVocals:

and I DO see that.. But clicking on any letter doesn't show the full results for the letter clicked.. For example I clicked "J" to look for Johnson, and the list is truncated before I can even get to that part of the J's.. This is is where pagination would help, and so would sorting since what I would expect to see is the names arranged first by last name, and then by first name.. It truly is NOT a good user experience, and I'm surprised no one's grumbled about this before..

I will look into getting this done. You have a good suggestion in how it should work, and I totally agree with you.
I will post back in a few days.

Hi,
Thank you for the great Mod.
I have just updated my ZC 1.3.9.h to 1.5.4 and the reward points are working fine. Except I have errors in my store/logs/ and wondered if anybody had any insight or have received similar errors?

[25-Feb-2015 16:45:03 America/New_York] 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 ((prp.scope_id=p.products_id AND prp.scope='2')
OR (p.products_i' at line 3 :: SELECT prp.point_ratio*p.products_price AS reward_points, prp.point_ratio, p.products_price, p.products_priced_by_attribute
FROM reward_master prp, products p, products_to_categories p2c
WHERE p.products_id='-1''
AND p2c.products_id='-1''
AND ((prp.scope_id=p.products_id AND prp.scope='2')
OR (p.products_id=p2c.products_id AND prp.scope_id=p2c.categories_id AND prp.scope='1')
OR (prp.scope='0'))
ORDER BY prp.scope DESC LIMIT 1; ==> (as called by) /home/xx/xx/store/includes/functions/extra_functions/reward_points_functions.php on line 62 <== in /home/xx/xx/store/includes/classes/db/mysql/query_factory.php on line 155

Any help would be much appreciated.