creation d'un module pour mon CMS

[julkien]

Bonjour tous le monde les gens !

 

Je développe un module pour mon CMS KWSphp, et j'ai un soucie, que je n'arrive pas a trouver.

 

Voila un code, c'est celui (a la base) du traitement d'une édition d'une fiche.

 

<?php
// --------------------------------------------------------------------
// ----
// --------------------------------------------------------------------

require('modules/km_club/securite.php');

if(get_grade() >= GRD_MEMBR)
bloc_head('Editer sa fiche :');

{
	$id = $_POST['id'];
	echo "$newklm";	 // vérification de la valeur du nouveau nombre de klm   (la valeur arrive bien)
	echo "$id";		 // id de la page concerné				(l'id arrive bien)

$req = mysql_query("SELECT `klm`, `nbr_klm`, FROM `klmclub` WHERE `id`='".$id."';");
$row = mysql_fetch_array($req);

$valeur_total = $df_ligne->klm;

$nombre_de_saisie = ($df_ligne->nbr_klm +1);

$klm_final = $valeur_total+$newklm;


mysql_query('UPDATE `klmclub` SET `klm`="'.$klm_final.'", `nbr_klm`="'.$nombre_de_saisie.'" WHERE id="'.$id.'";');
header("Location: index.php?mod=km_club&ac=detail&id=".$id."");



}

close_table();
?>

 

la variable $newklm qui est rentré dans une autre page, via un formulaire s'affiche bien dans l'echo "$newklm"

 

Mais ce code (si dessus) est censé updaté la table (et plus particulièrement les champs `klm`& `nbr_klm` )

 

Mais en faite se code écrase les champs, et remplace par la valeur $newklm et reste a 1 pour $nbr_klm alors qu'a chaque enregistrement il est censé faire +1.

 

l'id correspond bien (vue que c'est le bon champ du bon pseudo qui est updater[updater,ouais enfin écraser]).

 

Merci a ceux qui sauront trouvé le soucie.

Modifié le 4 avril 2008 par julkien

[Greywolf]

tu as essayé de faire un echo des différentes variables que tu calcules? (avant de faire la requête UPDATE; que tu peux commenter ou afficher)

[julkien]

tu as essayé de faire un echo des différentes variables que tu calcules?

 

Salut Greywolf, je test tout de suite.

 

echo "$df_ligne->klm"; me ramène a rien (donc problème)

echo "$klm_final"; me ramène a $newklm (donc problème)

echo "$nombre_de_saisie"; me ramène a 1 donc soucie, quoi que c'est un +1 simplement.

 

Bon toujours est-il que je débute depuis un bon mois en PHP, donc je bidouille avec mes maigres connaissances. (un peu comme mon français)

 

Voili voilou, en espérant que tu puisse m'aider ^^

[KewlCat]

(...)
  $row = mysql_fetch_array($req);

$valeur_total = $df_ligne->klm;
(...)

C'est moi, ou le "$df_ligne" de la seconde ligne de cet extrait devrait être "$row" ? (ou l'inverse)

 

Hum...

 

Je n'ai pas compris pourquoi tu as des accolades après ton if (vu que le if ne concerne que la ligne qui suit, vu comment c'est construit)...

 

Je sais que les programmeurs PHP sont friands d'inclusions de quotes / guillemets / backquotes en tous genres et dans tous les sens, mais par chez moi dans une requête SQL il ne faut pas de backquote (ni quoi que ce soit) autour des noms de table et de colonnes, et il n'en faut surtout pas autour des valeurs numériques...

 

Mon premier réflexe serait de changer cette ligne...

mysql_query('UPDATE `klmclub` SET `klm`="'.$klm_final.'", `nbr_klm`="'.$nombre_de_saisie.'" WHERE id="'.$id.'";');

... en ceci ...

mysql_query("UPDATE klmclub SET klm = " . $klm_final . ", nbr_klm = " . $nombre_de_saisie . " WHERE id = " . $id);

 

Ensuite, j'éviterais d'utiliser des variables temporaires inutiles.

	$nombre_de_saisie = $df_ligne->nbr_klm + 1;
$klm_final = $df_ligne->klm + $newklm;

Et je pousserais même le bouchon plus loin en zappant complètement ces variables...

mysql_query("UPDATE klmclub SET klm = " . ($df_ligne->klm + $newklm) . ", nbr_klm = " . $df_ligne->nbr_klm . " + 1 WHERE id = " . $id);

 

Et si j'étais vraiment mesquin j'éviterais carrément le SELECT :

mysql_query("UPDATE klmclub SET klm = klm + " . $newklm . ", nbr_klm = nbr_klm + 1 WHERE id = " . $id);

 

Soit, au final

<?php
// --------------------------------------------------------------------
// ----
// --------------------------------------------------------------------

require('modules/km_club/securite.php');

if(get_grade() >= GRD_MEMBR) {
bloc_head('Editer sa fiche :');
}

$id = $_POST['id'];
echo "$newklm";	 // vérification de la valeur du nouveau nombre de klm   (la valeur arrive bien)
echo "$id";		 // id de la page concerné				(l'id arrive bien)

mysql_query("UPDATE klmclub SET klm = klm + " . $newklm . ", nbr_klm = nbr_klm + 1 WHERE id = " . $id);

close_table();
?>

 

Maintenant, le souci c'est peut-être le manque de vérification d'existence d'un enregistrement portant l'id $id ?

[julkien]

ha heuu $row c'et bien possible

 

je tente ton code Kewlcat ou tu le refait avec des $row ?

[KewlCat]

Commence par remplacer "df_ligne" par "row", va...

Le reste, c'était juste histoire de parler "si jamais ça te prenait de tout modifier"

[julkien]

ha bha ton code juste au dessus fonctionne Kelwcat, donc c'est cool

 

Je vous remercie tous de votre aide...

[julkien]

Nouvelle Question :

 

j'ai rajouter un champs dans la table klmclub se nomant `description`. ( `description` text NOT NULL, )

 

J'ai rajouter sur la page d'édition, ceci :

 

echo "<input type='text' name='newdesc' maxlength='400' size='110' value='".$newdesc."' />";

 

je n'est pas mis de textearea pour une histoire de présentation...

 

j'ai ensuite repris ton code Kewlcat, que j'ai mis a la place du mien hier, et j'ai modifier comme ceci :

 

mysql_query("UPDATE klmclub SET klm = klm + " . $newklm . ", nbr_klm = nbr_klm + 1, description = " . $newdesc . " WHERE id = " . $id);

 

Mais plus rien ne rentre dans la table, si je laisse comme ça...

 

Mais si je vire mon descrition = " . $newdesc . " alors sa refonctionne, pourtant je ne voit rien d'anormal

 

Il est ou le soucie, parce que je cherche depuis quelques heures et je trouve pas...

 

Merci de ton aide.

Modifié le 27 mars 2008 par julkien

[KewlCat]

dans une requête SQL il ne faut pas de backquote (ni quoi que ce soit) autour des noms de table et de colonnes, et il n'en faut surtout pas autour des valeurs numériques...

Aurais-je oublié de rappeler que c'est cependant absolument indispensable autour des chaînes de caractères ?

 

mysql_query("UPDATE klmclub SET klm = klm + " . $newklm . ", nbr_klm = nbr_klm + 1, description = '" . $newdesc . "' WHERE id = " . $id);

Autre chose, ce n'est pas prudent de prendre le contenu saisi par un utilisateur et le coller tel-quel dans une requête. Je ne sais pas quels filtres tu as prévu (magic_quotes et tout le bazar) mais tu devrais attendre qu'un pro du PHP t'explique comment faire. En attendant, évite de saisir une apostrophe dans ton champ de saisie

 

Ah, un autre truc...

echo "<input type='text' name='newdesc' maxlength='400' size='110' value='".$newdesc."' />";

En HTML, on utilise des guillemets (impérativement) pour délimiter les valeurs des attributs. Il faut donc profiter de ce que PHP n'est pas rigoureux à ce niveau-là pour écrire plutôt :

echo '<input type="text" name="newdesc" maxlength="400" size="110" value="' . $newdesc . '" />';

[julkien]

merci Kewlcat ^^

 

Sinon je vient de me rendre compte que je n'est pas assoscier la table klmclub_final a ma requete, qui doit donc diminuer au contraire e klmclub, enfin sur une variable.

 

Je m'explique,

 

la table klmclub-final contient plusieurs champ (mais un seul m'interesse pour l'instant) c'est le champ `global` qui est en faite le total prévu par le club, et qui diminue en fonction de chaque users qui rajoute ses nouveaux Km.

 

voilà donc ce que j'ai penser faire mais sa fonctionne pas : (ps j'ai pas encore mis ton code si dessus)

mysql_query("UPDATE klmclub SET klm = klm + " . $newklm . ", nbr_klm = nbr_klm + 1 WHERE id = " . $id);
mysql_query("UPDATE klmclub_final SET global = klm - " . $newklm . " WHERE id");
header("Location: index.php?mod=km_club&ac=detail&id=".$id."");

 

je pense qu'il y'a comme une histoire de AND quelque part ou autre chose, mais comme je l'est dit nettement plus haut, je débute et j'aime apprendre sur le tas, donc bon...

 

Merci de ton aide en tout cas ^^

 

EDIT: je vient de mettre ton code, et après recherche (car il fonctionnais pas) j'ai trouver, y'avais une inversion de quote et doublequote sur une de mes lignes...

Modifié le 27 mars 2008 par julkien