THELIA Forum

Salut

Encore une question, cette fois-ci je ne comprend pas la logique de compte_modifiererr.html dans la template Thelia. Quand on est loggé sur son compte et qu'on clique sur "mon compte" puis sur "modifier mes coordonnées" on arrive sur une page compte_modifier.html.

Mettons que je clique sur le premier "valider" sous les deux champs qui donnent la possibilité de changer le mot de passe, sans rien entrer dans ces champs ou en faisant volontairement une erreur (- de 6 car ou 2 mots de passe différents). On se retrouve sur une page compte_modifiererr.html qui signale l'erreur mais qui demande maintenant "Mon email" et "mon mot de passe".

Et là, que doit-on faire ? si on saisit effectivement le email et le bon mot de passe on reste sur le même ecran sans message d'erreur à ce niveau, on ne sait pas vraiment ce qu'on doit faire ... on arrive à revenir à l'écran de modif des mots de passe en refaisant tout le chemin ci-dessus, mais je dois avouer que je ne comprends pas la logique. Pour moi, si erreur sur le mot de passe (- de 6 carac ou discordance) on devrait se retrouver sur un écran qui demande de recommencer la saisie de 2 mots de passe ...

Je pose cette question mais je commence à douter car après 2 jours de dev, je commence à coincer dans tous les coins, sur des pbms qui me paraissent basiques mais dont je ne vois pas la solution. On peut effectivement se replonger dans le code (j'ai déjà été assez loin pour mon pbm non résolu de livraison_modifier), mais bon, le but d'utiliser ce genre d'outil c'est justement d'éviter ça. Je précise que je conduis tous les tests sur les templates de Thelia en parallèle des miennes, donc pour l'instant j'écarte mes erreurs !
Je serai même prêt à envisager du support payant s'il existe pour éviter de coincer bêtement, c'est dire à quel point je prend au sérieux le travail d'Octolys sur le sujet ... en plus j'ai pas l'impression qu'une formation changerait grand chose, j'ai plus un problème de bugs ou de mauvaise compréhension de points très spécifiques.

Enfin bref, si quelqu'un voit les mêmes pbms que moi, ça m'intéresse, au moins je saurai que c'est pas moi le pbm !

Oui,
Il y a effectivement un côté déroutant dans la logique des fichiers d'erreurs. J'ai fait la remarque à YOAN, en lui demandant à quel niveau il faut intervenir pour effectuer son propre chemin. Donc j'ai modifié le fichier action.php, c'est vers la fin du fichier qu'il faut modifier la redirection en cas d'erreur dans la modification du mot de passe. Ensuite j'ai créé mon propre fichier d'erreur. Allez vous en avez pour 3 minutes pour modifier cela, courage. Il faut savoir que dans la structure des templates de base, si les mdp sont bons on est redirigé sur la dernière page consultée (pas sur mais presque).

Mci, la modif marche bien. J'ai d'ailleurs poussé le concept en créant aussi un compte_modifierok.php/html pour appliquer le même principe à la modif de compte, avec modif supplémentaire dans le redirige de action.php.

Ca fait bcp de fichiers pour de simples modifs de mot de passe et de comptes ... ajax serait peut-être la solution.
Pierre.

En Ajax je pense qu'on pourrait tout traiter depuis un seul fichier php plutôt que 3 ... bien sûr en contrepartie il faudrait un traitement ajax.
Pierre.

Peux-tu publier le code de ta fonction "GiB_passwordChange" pour compléter ton idée ? Ce qu'a fait Cécile marchait bien et ta solution permet de n'avoir qu'un couple php/html au lieu de 3, ce qui est plus simple.
C'est vrai qu'entre temps est apparue la possibilité de parser du php, ce que n'avait pas Cécile ... ajax était une autre solution avant cette possibilité pour tout réduire en 2 fichiers, mais c'est le genre de problème sur lequel on essaye d'éviter de passer des heures et des heures, il y a plus important !
En tout cas merci d'avoir publié ça !

Je n'avais pas inclus le code de ma fonction passwordChange car c'est pratiquement la même que celle de Cecile.

Mais la voici:

function GiB_passwordChange ($motdepasse1, $motdepasse2) {
	$client = New Client();
	$client->charger_id($_SESSION['navig']->client->id);
	if ( ($motdepasse1 == $motdepasse2) && (strlen(trim($motdepasse1)) >= 6) ) {
	   $client->motdepasse = $motdepasse1;
	   $client->crypter();
	   $client->maj();
	   $_SESSION['navig']->client = $client;
	   redirige("password-change.php?ok=1");
	}
	elseif($motdepasse1 != $motdepasse2)  {
	   $_SESSION['navig']->formcli->motdepasse = "";
	   redirige("password-change.php?error=1"); // les passwords 1 et 2 ne correspondent pas
	}
	elseif (strlen(trim($motdepasse1)) < 6)  {
	   $_SESSION['navig']->formcli->motdepasse = "";
	   redirige("password-change.php?error=2"); // le password est trop court
	}
}

Oui, c'est vrai que #VARIABLE est fait pour ça mais je trouve plus pratique de gérer le multi-langue dans des fichiers textes, plus souple à mettre à jour que des entrée multiples dans une base de données, surtout via une interface d'admin...