THELIA Forum

ta boucle produit est en une conditionnelle ? Et si tu boucles sur un ID en paramètre (id=#PRODUIT_ID) qui n'existe pas la condition est quand même remplie : elle affiche tous les produits du catalogue ?

tu peux me montrer la boucle ?

Ah ah ah !

La bonne blague!

En fait pour la référence, j'ai fais un test en dur donc ça doit donner la même chose qu'avec l'id...

Fichier substitproduits.php

if($ref)
	$tproduit->charger($ref);
else if($id_produit)
	$tproduit->charger_id($id_produit);
		
if( $ref || $id_produit)        
	$tproduitdesc->charger($tproduit->id, $_SESSION['navig']->lang);

là ça me sort rien donc les replace se font par du vide et donc ma boucle va tout chercher...

Exemple : http://www.distillerie-girardot.fr/prod … =04LIFRMIC

ça charge tout (et ça rame au passage je vais lui dire ^^)

En fait j'ai bien peur que c'est pareil partout, dès lors que tu spécifies une ref produit, un id produit, un id rubrique, un id contenu, un id dossier,... erroné, ça te charge tout ce que la boucle peut te sortir

Oui, moi aussi ça a failli faire plante Firefox ^^.

Sinon, non!
Car $ref ou $id_produit eux ils existent! (tout comme $id_rubrique etc)

Je dirais plutôt que dans les classes qui vont chercher les informations si aucun résultat n'est retourné alors on renvoie soit sur l'index soit sur une page erreur.php.

Last edited by Elyos (15-05-2009 00:47:26)

vas y je t'en prie

en gros faudrait faire
- soit dans la fonction getVars à la place de

return 0

mettre

header("Location: index.php");exit;

ou

header("Location: erreur.php");exit;

- soit dans les classes qui font appel à getVars à la place de

return $this->getVars("select * from $this->table where ref=\"$ref\"");

mettre

$result = $this->getVars("select * from $this->table where ref=\"$ref\"");
if(!$result || $result==0){
header("Location: index.php");exit; //ou erreur.php
}else{
return $result;
}

Last edited by Elyos (15-05-2009 01:20:32)

Oui car la première solution est utilisé à d'autres endroits.

Quoiqu'en fait, ce serait peut être même mieux de faire la vérification dans les substitutions globales avant de faire partir les boucles...

Yoan et Manu décideront, même si je vois déjà leur réponse : "Mais y'a pas d'intérêt à saisir une url fausse!" (nan ils sont pas comme ça )

Salut!

Oui oui en accès direct c'est pas possible... mais via un lien erroné c'est possible...et moi perso ça me gêne

Oui, c'est à ça qu'il sert le param num

Bon je reviens sur ce post après une petite idée ^^

Si vous voulez empêcher l'accès en cas de produit/rubrique/contenu inexistant

Boucle produit sur l'id :

<?php
$id_produit=#PRODUIT_ID;
if($id_produit==""){
 header("Location: index.php");exit;
}
?>

Boucle produit sur la référence:

<?php
$ref=#PRODUIT_REF;
if($ref==""{
 header("Location: index.php");exit;
}
?>

et ainsi de suite sur les rubriques et contenus.

Last edited by Elyos (19-05-2009 22:16:51)

Bon bah je suis passé par du javascript...

En fait, je récupère des infos pour générer le titre de chaque page.

Quand l'id produit n'existe pas (pareil pour les rubriques, les contenus etc...) la variable "titrepage" est vide et donc je remplace le contenu de ma page centrale par ce que je désire par du javascript.

Par contre, je ne fais des vérifications que sur les ids retournés que j'utilise.
Je ne fais aucune vérification sur les textes et les autres ids dans les urls mais ça c'est autre histoire

Last edited by Elyos (25-05-2009 17:40:54)

Je re-déterre le post, puisque j'ai exactement le même problème.
J'avais énormément de référence en doublons suite à une insertion par fichier CSV, des produits dans la BDD.
J'ai supprimé les doublons, malheureusement Google a référencé toutes les pages avant que je fasse le nettoyage.

Ce qui fait qu'une personne via Google peut tomber sur une page où l'ID du produit n'existe plus, ce qui fait que l'ensemble des produits existant dans la base de donnée est affiché.

Et j'ai remarqué , comme le message précédent, que sur le site démo il y a le même problème :
http://demo.thelia.net/?fond=produit&id … rubrique=3

Merci !