Jump to content

Signalement d'un problème sur /themes/prestashop/js/tools.js


Recommended Posts

Bonjour

 

je ne poste pas ce problème dans rapport de bugs car apparemment il faut être plusieurs à attester qu'il existe. Et visiblement à part moi :huh: personne n'a été confronté à ce souci.

 

Du jour au lendemain la commande en 1 clic sur une boutique en fin de développement s'est bloquée :

lors de l'authentification dans la page de Commande rapide en tant que client déjà existant (Zone Compte : déjà inscrit, cliquez-ici), les scripts d'affichage des méthodes de livraison moulinaient à l'infini, comme le montre l'image jointe. Toute poursuite de commande était impossible.

 

Dans la console d'erreurs Opera, il m'était signalé ceci (à l'infini) :

 

Uncaught exception: TypeError: 'price.toFixed' is not a function

Error thrown at line 86, column 1 in formatCurrency(price, currencyFormat, currencySign, currencyBlank) in /themes/prestashop/js/tools.js:

 

price = parseFloat(price.toFixed(6));

 

J'ai donc remplacé la ligne 86 incriminée de themes/prestashop/js/tools.js

 

price = parseFloat(price.toFixed(6));

 

par celles-ci

 

 

price = parseFloat(price);
price = price.toFixed(6);

 

et maintenant tout marche.

 

Néammoins si un pro du Javascript pouvait me donner une explication... j'aimerais bien en avoir une, histoire de comprendre un peu le pourquoi du comment. (pourquoi d'un coup, pourquoi devoir parser ce script en 2 lignes, pourquoi ce problème que chez moi, ou pas que chez moi d'ailleurs ????)

 

En attendant, si ça peut servir à quelqu'un.

 

Amicalement

post-147051-0-06249500-1344347524_thumb.gif

Link to comment
Share on other sites

  • 3 years later...

Je laisse ici ma solution au cas ou quelqu'un serait intéressé

 

Dans tools.js remplacer la ligne suivante : price = parseFloat(price.toFixed(10));

 

par :

 

if(!price)
price = price.toFixed(10);

price = parseFloat(price); 

  • Like 1
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...