THELIA Forum

Bonjour,

Ce petit message pour remettre le bug des caractéristiques au goût du jour !

http://forum.thelia.fr/viewtopic.php?id=47
https://sourceforge.net/tracker/?func=b … tid=906132

Les carctérisiques ont toutes le même nom dans le gestionnaire et la même id ce que je peux vérifier dans la table "rubcaracteristique".

Synoptase, as-tu réussi à résoudre ce problème d'affichage ?

Un commentaire a été rajouté sur le bug tracker de Thelia  mais ce commentaire n'est pas clair du tout (ohhhh loin de là )

merci

J'ai passé + de 2 heures pour debugger ce problème

J'ai tout testé, mis en place des fonctions de déserialisation afin de voir d'où vient le prob (je suis remonté jusquà la classe Json.php)

en fait, j'ai bien ciblé le problème mais c'est à n'y rien comprendre

dans la boucle while() (fonction charger() de la page rubcaractéristique.php) si on met un echo à chaque itération (echo $caracteristiquedesc->titre)

cela s'affiche bien...

donc je teste le array_push($tab,$caracteristiquedesc);
ou tab[] = $caracteristiquedesc ;
....

et bien j'ai bien à retourner le prob dans tout les sens :  $caracteristiquedesc qui est une instance de la classe Caracteristiquedesc() est changé dans chaque item de $tab

comme s'il y avait un global ... c'est à n'y rien comprendre !!

L'erreur se situe au niveau javascript je pense.

J'ai eu ce problème, et ça venait d'une fonction de conversion de caractères.. jettez voir un oeil aux autres sujets à propos des caractéristiques..

C'est une faute fréquente de conception orienté objet à mon avis.

Dans la boucle while() qui charge le tableau, les affectaction de tab[$i] de toutes les itérations sont faites avec la même instance de caracteristiquedesc, et donc, en fin de boucle, toutes les case du tableau pointent vers cette seule et unique instance. Les valeurs d'attributs de celle-ci changent lorsqu'on fait caracteristiquedesc->charger(), mais ça reste la même instance. Donc, en fin de boucle, toutes les cases du tableau permettent d'accéder aux même valeurs, celle de la dernière itération qui a "rechargé" l'instance en question.

Pour corriger cela, il faut affecter des instances différentes aux éléments du tableau (ou revoir complètement la façons dont on s'en sert en n'y enregistrant que les références et les titres par exemple). Une solution bête et méchante peut être simplement d'ajouter une instanciation de caracteristiquedesc avant de "recharger" :

$caracteristiquedesc = new Caracteristiquedesc();
$caracteristiquedesc->charger($row->id);
$caracteristiquedesc->titre = htmlentities($caracteristiquedesc->titre);
$tab[$i] = $caracteristiquedesc;
$i++;

[edit] Cette modif est bien à faire dans le fichier admin/rubcaracteristique.php, dans la seconde boucle de la fonction charger().

Last edited by 3.14R (30-05-2007 13:55:11)

Merci pour la correction, ce bug trainais depuis un bail.

Dans la fonction charger(), essaie de remplacer la 2ème boucle while :

while($row = mysql_fetch_object($resul)){                		

                		$res = $rubcaracteristique->charger($rubrique, $row->id);
                		if(($type=="h" && $res) || ($type=="d" && !$res)) continue;

                        $caracteristiquedesc->charger($row->id);
                        $caracteristiquedesc->titre = htmlentities($caracteristiquedesc->titre);
			$tab[$i] = new Caracteristiquedesc();
                        $tab[$i++] = $caracteristiquedesc;
                }

par celle-ci :

while($row = mysql_fetch_object($resul)){                		

                		$res = $rubcaracteristique->charger($rubrique, $row->id);
                		if(($type=="h" && $res) || ($type=="d" && !$res)) continue;

						$caracteristiquedesc = new Caracteristiquedesc();
                        $caracteristiquedesc->charger($row->id);
                        $caracteristiquedesc->titre = htmlentities($caracteristiquedesc->titre);
						$tab[$i] = $caracteristiquedesc;
						$i++;
						
                }

Ca devrait fonctionner

Thelia Version 1.2.2

Idem pour les déclinaisons dans admin/rubdeclinaison.php

Fonction charger()

Remplacer la boucle while  par celle-ci :

                while($row = mysql_fetch_object($resul)){	
                	$res = $rubdeclinaison->charger($rubrique, $row->id);
                	if(($type=="h" && $res) || ($type=="d" && !$res)) continue;
                        $declinaisondesc = new Declinaisondesc();
                        $declinaisondesc->charger($row->id);
                        $declinaisondesc->titre = htmlentities($declinaisondesc->titre);
                        $tab[$i] = $declinaisondesc;
                        $i++;
                }