Bonjour,

Ca a l'air de bien se tenir, il me semble que tu fais appel à une fonction trt() qui n'est nulle part dans tes scripts. Sinon c'est bien fait.

Quitte à pas réinventer la roue, tu as déjà tout ça dans prototype.js

Merci pour tous vos compliments ça fait toujours plaisir.

Quant aux évolutions, je serais impatient d'y jeter un ½il.

Bonjour,

Je n'ai pas eu trop le temps de répondre, ni de regarder tous les coms depuis un moment (et il y en a en masse : quel succès ^^), mais oui en effet il y a une erreur sur la fonction :

function ajouterArticle($numserie, $quantite, $montantHT = 0) {
if (!empty($numserie)) {
if (isset($this->article[$numserie])) $this->article[$numserie]['qte'] += $quantite;
else {
$this->article[$numserie]['qte'] = $quantite;
$this->nbarticle++;
}
if (isset($this->calculmontant) && $this->calculmontant == true) {
$this->article[$numserie]['prix']      = $montantHT;
$this->CalculMontantArticle($numserie, $this->article[$numserie]['prix'], $quantite);
$this->CalculTotal($this->article[$numserie]['montantHT']);
}
}
}

Il faut remplacer :
$this->CalculTotal($this->article[$numserie]['montantHT']);
par :
$this->CalculTotal($quantite * $montantHT);

Merci d'avoir fait remarquer cette erreur.

Stéph.
p.s.: vous pouvez éventuellement m'envoyer vos modifications, évolutions de la classe pour ceux qui en ont fait, ou tous les bugs répertioriés, je pourrais faire un mix et poster tout ça.

Bonjour,

L'intérêt de ce script était de pouvoir formater un menu déroulant au niveau affichage.
Ce qui permettait de gérer plusieurs colonnes.

Stéph.

Salut,
sans trop approfondir j'ai changé certaines lignes de ton code et ça marche sans les bugs sur ie, netscape et firefox (les dernières versions)

J'ai remplacé :

>> $tableur.= "<td class=\"clsCEL\"><input col=\"$j\" ligne=\"$i\" type=\"text\" name=\"c".$j."l".$i."\" class=\"clsTextBox\" OnKeyDown=\"txtKeyDown('c".$j."l".$i."');\" value =\"";


par :

$tableur.= "<td class=\"clsCEL\"><input type=\"text\" id=\"c_".$j."_".$i."\" name=\"c_".$j."_".$i."\" onblur=\"this.style.background='transparent'\" onfocus=\"this.style.background='#f2f292'\" class=\"clsTextBox\" onkeydown=\"txtKeyDown('c_".$j."_".$i."', event);\" value =\"";


et :
>> <script>
>>   var col_;
>>   var ligne_;
>>   // Code JavaScript pour la navigation entre celulles.
>>   // Quand on utilise les flèches, on change de celulle active.
>>   // Il semblerait qu'il y ait des problèmes de compatibilités...
>>   // Voilà donc la liste des explorateurs testés:
>>   // Testé sous:
>>   //   - FireFox  : Marche pas
>>   //   - IE       : Marche très bien!
>>   //   - NetScape : Marche pas
>>   //   - Opera    : Marche pas
>>   // Ceci est sans doute dut aux deux attribus factices ('col' et 'ligne')
>>   // L'idée n'était pas mauvaise! Mais, à creuser! Mdr!
>>   function txtKeyDown(txt){
>>            col_   = document.getElementById(txt).col;
>>            ligne_ = document.getElementById(txt).ligne;
>>            // On presse la flèche de gauche
>>            if(event.keyCode==37){
>>               col_--;
>>               focuss();
>>            }
>>            // On presse la flèche du haut
>>            else if(event.keyCode==38){
>>               ligne_--;
>>               focuss();
>>            }
>>            // On presse la flèche de droite
>>            else if(event.keyCode==39){
>>               col_++;
>>               focuss();
>>            }
>>            // On presse la flèche du bas
>>            else if(event.keyCode==40){
>>               ligne_++;
>>               focuss();
>>            }
>>   }
>>   function focuss(){
>>           // Normalement, ici on met la gestion de l'erreur
>>           // Mais je n'ai pas encore trouvé comment.
>>           // Bug: Si on est en tout début ou en fin de tableau,
>>           // Et qu'il n'y a pas de suivante, il y a une erreur.
>>           // Par exemple, mettez vous sur 'A1' et pressez la flèche du haut.
>>           if(document.getElementById("c"+col_+"l"+ligne_).type=="text"){
>>              document.getElementById("c"+col_+"l"+ligne_).focus();
>>           }
>>
>>   }
>> </script>

par :

<script>
// Code JavaScript pour la navigation entre celulles.
// Quand on utilise les flèches, on change de celulle active.
// Il semblerait qu'il y ait des problèmes de compatibilités...
// Voilà donc la liste des explorateurs testés:
// Testé sous:
//   - FireFox  : Marche pas
//   - IE       : Marche très bien!
//   - NetScape : Marche pas
//   - Opera    : Marche pas
// Ceci est sans doute dut aux deux attribus factices ('col' et 'ligne')
// L'idée n'était pas mauvaise! Mais, à creuser! Mdr!
function txtKeyDown(txt, evt) {
bouge = false;
obj    = document.getElementById(txt);
nomId  = obj.id;
nomId  = nomId.split("_");
col_   = nomId[1];
ligne_ = nomId[2];
codeClavier = -1;

if (document.all) {
codeClavier = event.keyCode;
codeClavier = event.keyCode;
} else {
codeClavier = evt.which;
codeClavier = evt.which;
}
// On presse la flèche de gauche
if (codeClavier == 37) {
col_--;
bouge = true;
}
// On presse la flèche du haut
else if(codeClavier == 38){
ligne_--;
bouge = true;
}
// On presse la flèche de droite
else if(codeClavier == 39){
col_++;
bouge = true;
}
// On presse la flèche du bas
else if(codeClavier == 40){
ligne_++;
bouge = true;
}
if (bouge && document.getElementById("c_"+col_+"_"+ligne_) != null) {
objDest = document.getElementById("c_"+col_+"_"+ligne_);
objDest.focus();
}
}
</script>

Cliques sur télécharger le zip, tu verras qu'il y est quand même et que le zip contient bien les fichiers, mais je ne sais pas pourquoi le contenu n'apparaît pas en visuel sur le site.

Salut,

Initié pour ton bout de code c fortement exagéré.

Petite modif,
pour éviter le bug lors du vidage du panier.

Stéph.

Bonjour,

Je pense qu'il faut rajouter un 'isset' sur l'existence de la variable à la ligne 146, pour qu'il n'y ait plus d'erreur. Je mettrais ça à jour le plus rapidement possible dès que j'en aurais l'occasion.

En attendant, essaye de remplacer à la ligne 146 :

ça :
   if ($this->article[$numserie]) $this->article[$numserie]['qte'] += $quantite;

par ça :
   if (isset($this->article[$numserie])) $this->article[$numserie]['qte'] += $quantite;

et dis moi si c bon.

Stéph.

Salut et merci,

Dans le cas que tu précises il aurait d'ailleurs fallu mettre $_GET étant donnée que les variables sont récupérées d'un lien.

Stéph