Bogus 'free delivery' option on vouchers list after old cart rule delete. PSCSX-5178

[a_smith]

Hello

 

The problem has been reported here but no help so far:

http://forge.prestashop.com/browse/PSCSX-5178

 

 

On BO old cart tules can be deleted from the list (database).

 

After cart rule deleting from the list, customer sees 'free delivery' false offer on their vouchers list on their accounts.

Those 'free delivery' is offered for orders for which deleted cart rules had been related.

Those records has no other information on the table.... All is confusing to customers who thinks its true offer and wants to use it.. etc.. etc...

 

I urgently needed to get rid of those bogus entries and the only one way I found is this:
When deleting cart rules from BO, ps_loyalty table should be purged in the same time from all those records related to deleted id_cart_rule.
If id_cart_rule in ps_loyalty table include deleted/non-existent cart rules ID, customers will see FREE SHIPPING offer on their accounts (see screenshot).

 

PS Dev, to replicate

- make test order as a registered customer

- via BO confirm your test order to make loyalty points for this order usable

- on FO test customer can now exchange those loyalty points into voucher. Do so and use generated voucher for another order.

- on FO test customer will see those loyalty points as 'already used' on the loyalty/order history (upper table)

- on BO delete this voucher (cart rule) used for second test order

- now customer will see this false 'free delivery' entry on vouchers list (bottom table)

 

Its not a problem if you have 10 customers and you delete 3 old cart rules.

BUt it is a MASSIVE problem if you have 10k customers and just deleted about 3k of old/expired/used cart rules - as we did recently.

Now we get 50 emails/hr from confused customers asking 'how to use free shipping offer' and we make them even more upset by seying ,oh sorry is just system fault not true offer'.....

 

Presta Team.... get in charge please

Edited March 31, 2015 by a_smith (see edit history)