baskets rules, product remains free even if the rule is no longer valid

[Jean Francois G]

for version 1.5.4.1, in baskets rules when you define a product offered (gift) under conditions of purchase 5 products in a particular category, when you reach the 5th product basket is well prove a product offered 6th marked above with 'free'

 

Problem! then, If you remove one of the 5 products (so there are no more than 4), the offered product is still available!

Yet the rule cart is no longer respected.

 

This is a serious bug, because it is the loss of money.

 

 

en francais :

 

dans les règles paniers, lorsque vous définissez un produit offert (cadeau) sous conditions de l'achat de 5 produits d'une catégorie précise, une fois arrivé au 5eme produit le panier fait bien apparaitre un 6eme produit offert avec marqué dessus 'gratuit'

 

Problème ! Si vous enlevez alors un des 5 produits payant (donc il n'en reste plus que 4), le produit offert reste encore  offert !

Pourtant la règle panier n'est plus respectée.

 

C'est un bug grave, car il s'agit la de perte d'argent.

 

Une solution ?

[Gregory Roussac]

Bonjour,

 

Merci pour votre retour. Ce commit devrait régler ce souci

 

https://github.com/PrestaShop/PrestaShop/commit/824a6b90f33bdb255aa2d77d296c3381a28f1c50

 

EDIT : Non ce n’est pas le même souci. Je jette un oeil à celui ci et vous fait un retour.

 

Cordialement

Edited September 6, 2013 by Gregory Roussac (see edit history)

[Jean Francois G]

Bonjour,

changements effectués sur le fichier script. Aucune amélioration.

J'ai effacé le cache pour ne pas avoir de vieille trace du fichier, mais rien n'y fait.

Le bug reste identique

Autre chose à faire ?

[Gregory Roussac]

Rebonjour,

 

Laissez en place le premier correctif.

 

Pouvez vous essayer ensuite ce correctif partiel ?

 

https://github.com/PrestaShop/PrestaShop/commit/4c49048e391bbe1e2a68cb11f60fd86de2ceb637

 

Je corrigerais pour les marques, fabricants et catégories semaine prochaine.

 

Cordialement

[Jean Francois G]

net amélioration avec ce bout de code.. le panier réagit à la règle correctement

 

Alors à savoir que par défaut mes deux fichiers ne ressemblaient en rien à ceux que vous aviez dans le github.. Les lignes n'étant pas toutes présentes. Donc j'ai carrément fait un copier/coller du contenu complet des deux fichiers et remplacé le contenu des miens.

 

Vu que la structure des deux fichiers n'avaient rien à voir avec les vôtres, cela risque t'il de me poser un problème par la suite ?

 

Je vous communique par exemple mon fichier original cartrule.php,et cart-summary.js et vous verrez qu'il manque plein de lignes présentes d'origine dans le votre (avant correction).. C'est très grave de les avoir tout bonnement remplacé ?

 

merci d'avance

files origine.zip

Edited September 6, 2013 by Jean Francois G (see edit history)

[Gregory Roussac]

Ce sont les fichiers de la 1.5.5.0. Quelle version avez vous déjà ?

 

Ma foi pour ces deux la dans le cas présent, vous pouvez les copier en entier oui mais il ne faut en prendre l'habitude pour autant.

 

Cordialement