Ivan

Houla ! ... je l'avais oublié, celle-là ! ;)

• Cette page (achat/RECHERCHE.PHP) fonctionne très mal, voire plante carrément quand on a plusieurs "pages" de résultats trouvées ! ... le lien "Page suivante" donne un beau "404-page non trouvée" (...ou "No input file specified" en PHP) et les "Pages : 1 2 3..." ne font absolument rien.

Joliiii :(

• Alors voilà les corrections à apporter à cette page pour qu'elle fonctionne pleinement :
---------- 1°) Donner les bon liens
- dans le fichier original PEEL v2.9.3, vous avez : (en fin de fichier)

print("<a class=\"petit\" href=\"recherche.php?motclef=".$_GET['motclef']."&start=".($start-$nb)."\">Page précédente</a>"); } $result=mysql_query("SELECT COUNT(*) FROM peel_produits WHERE nom LIKE '%" . addslashes($_GET['motclef']) . "%' "); $row=mysql_fetch_row($result); if($row[0]>($start+$nb)) { if($start) {print(" / ");} print("<a class=\"petit\" href=\"recherche.php?motclef=".$_GET['motclef']."&start=".($start+$nb)."\">Page suivante</a>"); >> Vous complètez comme suit : (en gras) print("<a class=\"petit\" href=\"$wwwroot/achat/recherche.php?motclef=.$_GET['motclef]."&start=".($start-$nb)."\">Page précédente</a>"); } $result=mysql_query("SELECT COUNT(*) FROM peel_produits WHERE nom LIKE '%" . addslashes($_GET['motclef']) . "%' "); $row=mysql_fetch_row($result); if($row[0]>($start+$nb)) { if($start) {print(" / ");} print("<a class=\"petit\" href=\"$wwwroot/achat/recherche.php?motclef=.$_GET['motclef]."&start=".($start+$nb)."\">Page suivante</a>"); ---------- 2°) Ça ne plante plus ... mais ça ne marche toujours pas ! Le lien "Page suivante" ne plante plus, mais ne fait rien ... et "1 2 3..." font toujours rien ! :rolleyes: ... ben alors, oukilé le pb ? :huh: Il est dans l'initialisation de la variable $start, vers le début du fichier ... - dans le fichier original PEEL v2.9.3, vous avez : $nb=10; //nombre d'enregistrement par page if(!isset($start)) {$start=0;} >> C'est pas bon, $start vaut toujours 0 ... vous remplacez par : if (!isset($_GET['start'])) {$start=0;} else {$start = $_REQUEST['start'];} ---------- 3°) Soyons logiques et cohérents ! - dans le fichier original PEEL v2.9.3, vous avez : **********(juste sous la ligne if(!isset.... qu'on vient de modifier)********** $resultat = mysql_query("SELECT id, reference, nom FROM peel_produits WHERE etat = '1' AND nom LIKE '%" . addslashes($_GET['motclef']) . "%' OR descriptif LIKE '%" . addslashes($_GET['motclef']) . "%' LIMIT $start,$nb"); ****... et vers la fin du fichier, juste avant //Affichage des pages, qu'on a modifié au 1°)**** $result=mysql_query("SELECT COUNT(*) FROM peel_produits WHERE nom LIKE '%" . addslashes($_GET['motclef']) . "%' "); ... C'est pas logique : le 1ère requête (plus générale car tenant compte des descriptifs produits) trouve PLUS de résultats qu'il n'en est compté par la 2ème (moins générale, pas les descriptifs) pour l'affichage des différentes pages ! :)
>> Vous complètez la 2ème requête comme suit (comme la 1ère, en fait) : (en gras)

$result=mysql_query("SELECT COUNT(*) FROM peel_produits WHERE etat = '1' AND nom LIKE '%" . addslashes($_GET['motclef']) . "%' OR descriptif LIKE '%" . addslashes($_GET['motclef']) . "%' ");

---------- 4°) Les cerises sur le gateau : :)
• Bon, maintenant c'est OK ... les liens de changement de page fonctionnent ... sauf que ce ne sont pas des pages que l'on fait défiler, mais des "paquets" de (x) produits parmis (n). Alors il serait quand même plus indiqué de remplacer les intitulés "Page suivante" & "Page précédente" par "Produits suivants" & "Produits précédents", non ?
>> Alors rendez-vous au 1°) pour changer ces intitulés

• Enfin, il m'a semblé plus qu'intéressant d'indiquer "en clair" le total des produits trouvés, vous ne trouvez pas ? ;)
- Pour cela, sous la 1ère requête vue au 3°) (qui affecte $resultat), vous rajoutez la même requête mais plus limitée à $nb produits (on enlève le paramètre LIMIT final) que vous affectez à une variable ($resultatot) :
$resultat = mysql_query("SELECT id, reference, nom FROM peel_produits WHERE etat = '1' AND nom LIKE '%" . addslashes($_GET['motclef']) . "%' OR descriptif LIKE '%" . addslashes($_GET['motclef']) . "%' LIMIT $start,$nb");
$resultatot = mysql_query("SELECT id, reference, nom FROM peel_produits WHERE etat = '1' AND nom LIKE '%" . addslashes($_GET['motclef']) . "%' OR descriptif LIKE '%" . addslashes($_GET['motclef']) . "%' ");

- Ensuite, sous le tableau d'affichage des résultats de la recherche et juste avant les liens de navigation "Produits suivants/précédents" , vous insérez la ligne suivante (indiquée en gras, en italique ce sont les lignes existantes) :
<?php
//Boutons précédent et suivant
echo "<font class=\"normal\">[<b>".mysql_num_rows($resultatot)."</b> produit(s) trouvé(s) contenant <em>' ".addslashes($_GET['motclef'])." '</em>  dans le Nom ou le Descriptif]<br /><br /></font>";
if($start)
{
>> Cela affichera systématiquement (qu'il y ait des pages de suite ou non), sous le tableau d'affichage des résultats de recherche, le nombre total de résultats trouvés, avec une phrase comme suit :
[x produit(s) trouvé(s) contenant 'mot-clé' dans le Nom ou le Descriptif]
--------------------------------------------------------------------------------------------------------------

• Voilà encore un post bavard ;) mais qui devrait convenir à tout le monde, non ?
Ivan

Bonjour,

Un petit souci dans [Administer/Produits] ...

Qd on change le taux de TVA d'un produit et que l'on revient sur la fiche du produit concerné, le taux de TVA affiché dans le formulaire de choix est par défaut toujours le même (19,6%, la plus gde valeur), et pas le nouveau.

Comment faire pour que l'affichage dans le champ du formulaire TVA soit par défaut réglé sur la vraie valeur en cours ?

Merci

*** NE PAS TENIR COMPTE DE CE MESSAGE ! ... pb dû à une grosse erreur de ma part sur la base (peel_produits) ! ;)

(Il faudrait effacer ce message)

Bonjour,

Une petite idée que je soumets à la sagacité de tous ...

• Il pourrait être sympa de mettre un lien "Retour haut de page" en bas de chaque page pour faciliter la navigation dans la boutique.

Je pense logiquement qu'il faudrait déclarer l'ancre en début de <body> dans (haut.php) et placer le lien en fin de <body> dans (bas.php), mais le problème consiste à définir l'id de cette ancre pour qu'elle soit unique pour chaque page ... et là, je sèche ! :D

... si qqun a une idée ?

Merci d'avance à tous les curieux titillés par ce petit exercice :(

Ivan

Aller, encore un truc ... y'en a qui bossent ! B)

Dans l'affichage du détail du caddie, vous avez un champ "Quantité" que vous pouvez modifier à volonté, quitte à y saisir n'importe quoi car il n'est absolument pas contrôlé ! ... ça avait plus que tendance à m'énerver ! :(
• Rien n'empêche d'y saisir 550873 et là, bonjour la gestion des stocks !
• Rien n'empêche d'y saisir y2b4 et là, essayez de faire "METTRE à JOUR" pour voir ce qu'il se passe ! :)

Enfin bref, ce champ doit être contrôlé, déjà au minimum pour que ça soit un champ numérique. Le fichier à modifier est (.../achat/modeles/caddie.php), au niveau du champ <input ... name="quantite[]" ... > La ligne originale de PEEL-SHOPPING 2.9.3 est la suivante :


• 1°) Contrôle champ numérique:
Bien mieux qu'un n-ième message d'avertissement à postériori "Le champ tartempion doit être...", il faut contrôler le champ directement à la saisie en contrôlant les touches-clavier frappées (contrôle des valeurs décimales des touches). Pour ça, utiliser les events (onKeypress & onKeydown) event.keycode pour IE et event.which pour Netscape... (ils font exactement la même chose, mais ça aurait été trop simple d'avoir une même syntaxe pour les deux navigateurs ! :D )

Il faut dons rajouter dans le champ input :

... bon, déjà là, c'est pas mal : dès qu'on frappe autre chose qu'un chiffre, le caractère n'est pas tapé ! :D

• 2°) Empêcher la frappe de la touche SUPPRIME:
Mais un petit piège : rien n'empêche le fonctionnement de la touche "suppression" du clavier ... il y a danger, car le champ peut être vide et, là aussi, essayez de faire METTRE à JOUR pour voir ce qu'il se passe dans ce cas ! ... de manière radicale, on va donc empêcher la frappe de la touche SUPPR (code décimal 46)

Rajouter dans le champ input :

... pour modifier le champ Quantité, il faut donc procéder uniquement par "sélection/frappe" et non plus par "suppression/frappe" et le champ ne peut ainsi plus être vide ! ... bien, on avance...

• 3°) Limiter le nbre de caractères du champ:
Ici classiquement, utiliser le paramètre maxlength="x" des champs input. De base, il n'a pas de limite dans (caddie.php) => la fixer à x=2 ou 3, ce qui permet déjà de commander ou 99 ou 999 articles d'un même type !! :o

Pour limite à 2, rajouter dans le champ input :


Ce qui nous donne le champ input global :


====================
Personnellement, je vais fixer maxlength=1 ... je ne veux pas que les clients puissent "facilement" commander plus de 9 fois un même article (moyenne 20Kg et 300€). Ma ligne input est donc la suivante:


j'élimine donc en plus la touche/valeur 0 (code décimal 48) pour que la quantité ne soit jamais nulle.
====================

• Voilà encore un truc utile, je pense ... on peut sûrement faire mieux, mais c'est déjà mieux que RIEN ! ... :D ... mais ça serait qd même bien qu'un minimum de contrôle sur ce champ Quantité (au moins numérique) soit fait dans PEEL-SHOPPING de base, non ?

Aller, bon developpement à tous,
@+

PS : Ces contrôles agissent directement sur le champ Quantité de la page (caddie.php), mais elles n'empêchent pas le client "volontaire" de commander PLUS d'objets que la limite imposée ! ... En effet, il peut toujours, sans que le résultat n'en soit affecté dans le caddie, cliquer autant de fois qu'il le veut sur "Ajouter au Caddie" sur un produit dans le catalogue; le résultat dans le caddie sera correct ... mais je ne pense pas ainsi qu'il lui vienne l'envie de cliquer 550873 fois ! :D

Un petit truc, pour éviter les déconvenues :(

• Pour interdire les adresses Hotmail à l'enregistrement des Comptes-Clients, agir sur (enregistrement.php) en rajoutant ces lignes dans les tests de validation de l'e-mail :

... je conseille d'ailleurs de faire de même avec les adresses Caramail ! :D

À bon entendeur ...