ChDUP Posted December 23, 2012 Share Posted December 23, 2012 Bonjour à tous. Je me suis aperçu que lorsqu'un visiteur entre une espelette dans un champ de personnalisation, alors il obtient un message "Merci de remplir tous les champs" lors de l'ajout au panier. Le problème vient de la fonction checkCustomizations dans le fichier js/tools.js if (parseInt(customizationFields[i][1]) == 1 && ($('#' + customizationFields[i][0]).html() == '' || [b]$('#' + customizationFields[i][0]).html() != $('#' + customizationFields[i][0]).val())[/b] && !pattern.test($('#' + customizationFields[i][0]).attr('class'))) Dans le cas de l'espelette $('#' + customizationFields[0]).html() renvoie & tandis que $('#' + customizationFields[0]).val()) renvoie & d'où l'erreur Pour ma part, j'ai remplacé toute la condition par if (parseInt(customizationFields[i][1]) == 1 && ($('#' + customizationFields[i][0]).html() == '' )) Ce qui fonctionne mais entraine bien sur des problèmes de sécurité. Comment faudrait-il faire pour "contourner" ce problème proprement ? Link to comment Share on other sites More sharing options...
micheldmo Posted April 15, 2013 Share Posted April 15, 2013 Ami de l'espelette bonjour, J'ai le même problème que toi. Aurais-tu trouvé une solution par toi-même ou bien une âme charitable qui t'aurait dépanné par un autre biais que cette discussion-ci ? La mise à jour de 1.4 -> 1.5 corrigerait-elle le pb ? Donc : est-ce qu'un utilisateur de la 1.5 a ce même problème ? Par avance je te remercie, Michel. Link to comment Share on other sites More sharing options...
ChDUP Posted April 16, 2013 Author Share Posted April 16, 2013 non désolé je n'ai pas avancé par rapport à mon message d'origine. Link to comment Share on other sites More sharing options...
micheldmo Posted May 10, 2013 Share Posted May 10, 2013 Ok merci, je follow this topic et j'informe dès que y'a du neuf. Link to comment Share on other sites More sharing options...
Recommended Posts
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 accountSign in
Already have an account? Sign in here.
Sign In Now