Jump to content

[SOLVED Bug on PS1.4.10] Dans order shipping, problème affichage adresses si modifie extra addresses


cactusman2

Recommended Posts

J'ai trouvé un bug sur PS1.4.10

J'ai fait un report de bug ici:

http://forge.prestas...owse/PSCFI-7157

 

En détails...

 

Ce bug est présent sur la version 1.4.00

A été testé en local et sur un serveur avec une installation de base.

Ce bug n'était pas présent sur la version 1.4.25.

 

Afin de me faire comprendre je vais décrire un exemple...

  • Un client ajoute un produit dans son panier et crée un compte avec une adresse 'adresse1'
  • il arrive sur la page order-address et décide d'avoir des adresses de livraison et de facturation différentes
  • il décoche 'Utiliser la même adresse pour la facturation'
  • Il appuie ensuite sur le bouton 'Ajouter une nouvelle adresse '
  • il arrive sur la page adresse et créé une nouvelle adresse 'adresse2'
  • il clique sur le bouton 'Valider' et se retrouve de nouveau sur order-address
  • il choisi ensuite 'adresse2' comme adresse de livraison et 'adresse1' comme adresse de facturation
  • Il remarque qu'il a fait une erreur et veux modifier cette nouvelle adresse 'adresse2'
  • il clique sur le lien 'Mettre à jour' situé en dessous de l'adresse de livraison
  • il arrive sur la page adresse et la modifie
  • il clique sur la bouton 'Valider' et se retrouve de nouveau sur la page order-address
  • et là, c'est le drame...

1er bug -> l'input 'Utiliser la même adresse pour la facturation' est coché

2ème bug -> si le client décoche de nouveau 'Utiliser la même adresse pour la facturation', les adresses de livraison et de facturation sont inversées.

 

J'ai bien essayé de trouver une solution mais jusqu'à présent aucun résultat.

 

Toute aide est la bienvenue :)

 

Loic

Edited by cactusman2 (see edit history)
Link to comment
Share on other sites

Ça semble être lié aux cookies

J'ai cherché un peu partout et j'ai ajouté {debug} à mon order-address.tpl pour voir les cookies

La première adresse créée durant la création de compte a pour id 8525

J'ai ensuite créé une nouvelle adresse depuis la page order-address

 

Cas 1: delivery address = 8525 et invoice address = 8526 et je met à jour delivery address

$cart->id_address_delivery = 8525

$cart->id_address_invoice = "8526"

Le input est décoché et tout est normal, la modification a été appliquée

 

Cas 2: delivery address = 8525 et invoice address = 8526 et je met à jour invoice address

$cart->id_address_delivery = "8525"

$cart->id_address_invoice = 8525

BUG: le input est coché cependant quand je le décoche je peux voir que la modification a été appliquée

 

Cas 3: delivery address = 8526 et invoice address = 8525 et je met à jour delivery address

$cart->id_address_delivery = 8525

$cart->id_address_invoice = "8525"

BUG: le input est coché, lorsque je le décoche je peux voir que la modification a été appliquée sur l'addresse 8526 cependant les adresses de livraison et de facturation sont inversées

 

Cas 4: delivery address = 8526 et invoice address = 8525 et je met à jour invoice address

$cart->id_address_delivery = "8526"

$cart->id_address_invoice = 8525

Le input est décoché et tout est normal, la modification a été appliquée

 

Conclusion:

C'est seulement lorsqu'on modifie l'adresse par défaut (la première adresse entrée lors de la création de compte) que tout fonctionne.

Si on modifie une adresse ajoutée ultèrieurement un bug apparait.

Link to comment
Share on other sites

J'ai trouvé une solution.

En espérant que ca n'impacte pas le reste de la boutique.

 

Il suffit de modifier classes/Address.php

- Commenter les fonctions update et _cleanCart vers les lignes 155 et 160

- Commenter

&& self::_cleanCart((int)$this->id)

dans la fonction delete vers ligne 180

 

 

En espérant que ce soit utile.

Edited by cactusman2 (see edit history)
Link to comment
Share on other sites

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!

Register a new account

Sign in

Already have an account? Sign in here.

Sign In Now
×
×
  • Create New...