Since your looking for feedback on the mod I have some more:yes:
I have a Loyalty Point Mod on my store done with osC. How it works is the admin sets a certain range of dollar amounts and ties that to a percentage which is used to discount all future purchases that person makes when they reach that threshold. You can also set it to reset what they have spent over time back to 0.00 at 3, 6, 9, or 12 month intervals. I have mine set for yearly. It's a great system. I'm not saying I want something like that for this, but what It does is it is set by default to add the money spent to the yearly total when the status of Delivery is set.
With that code it looked for a certain Status ID not name. Since I don't use Delivery (I use Shipped) I was able to go into the code to where it checked for the status ID of 4 (which is delivered I think in OSC) I was able to set it to 8 which was equal to shipped for me.
I looked at the reward_point_functions.php hoping yours did the same thing, but I see it looks for the wording processed? If that is the case, this mod would not work for those who don't use the default status. Is there a way to change it to look for a certain status ID and then award the points when that status ID is met? This way if someone wants to use there own status on when they get awarded the points they can.
Doesn't have to be something you can set in admin, just something they can go into the script and change the status ID it looks for.
I just thought it would make your mod more flexible, that way your not tied to the default status of processed and could award it using something of the admins choice.
Zen Cart: 1.3.8a
PHP: 5.2.4
MySQL: 5.0.45
Pine Tree Candles