GENERATEUR DE PLAQUETTE GRAPHIQUE V1.1

Commentaire de Tit0x le 29/06/2006 23:24:37

hello,

J'ai juste regarder le code. Voilà ce que j'ai retenu :

- <?php au lieu <? pour ouvrir tes balises
- ?><? Pourquoi la fermer pour la réouvrir après ?
- A paramétrer c'est chiant ^^ 36000 fois mysql_connect.. mais bon tu l'as dit toi même " Promis ca va bientot etre corrigé " :)
- Tu ne testes pas tes GET et POST, pas cool :/
- HTML : manque du doctype, du body,...
  
Voilà j'ai vu ça pour l'instant :)

Commentaire de Tit0x le 29/06/2006 23:37:43

hello,

Pourquoi <?php au lieu de <?, c'est parcequ'il me semble que XML utilise aussi <? et donc c'est pour les différencier. Je pense que c'est pour ça, j'avais un post sur ça un jour.. :)

>> mais en meme temps quand on est a la bourre on est a la bourre :D

C'est sur ^^

Commentaire de Tit0x le 29/06/2006 23:42:47

Tes requêtes, je les trouve bizar lol

$sql="UPDATE `plaquette` SET `titre`='".$_POST['titre']."' , `rvb`='".$_POST['couleur']."'
, `hex`='".$hex."' , `description`='".$_POST['justifier']."' , `cmjn`='".$_POST['cmjn']."'
WHERE `ID`=".$_GET['modif'];


J'aurais plutôt mis ça :

$sql='UPDATE plaquette SET titre='$_POST['titre']' , rvb='$_POST['couleur']' , hex='$hex' , description='$_POST['justifier']' , cmjn='$_POST['cmjn']' WHERE ID='$_GET['modif']'';

Commentaire de kankrelune le 30/06/2006 04:33:10

"Oui c'est bien pour differencier du xml mais maintenant le <? est rentré dans les standards pour les serveurs avec php"

Non <? ne fait pas spécialement partit des standards... si short_open_tags est à off dans la config php ton script ne marchera pas... .. . ;o)

Et puis, comme disait quelqu'un (je sais plus qui), si tu met <? c'est comme si tu disait "je parle le" alors qu'avec <?php ça donne "je parle le français"... .. .

Sinon je n'ais pas regardé le code, je ne le commenterais donc pas mais qu'est ce que tu appel "PLAQUETTE GRAPHIQUE"... .. ?

@ tchaOo°

Commentaire de cedriclomb le 30/06/2006 09:44:09

Merci  kankrelune pour tes conseils secu j'avoue que je n'avais pas capté ce point.
Donc pour etre sur que je ne fasse pas de conneries en plus comment tu filtres les GET et POST

Pour le a quoi ca sert ? Eh bien nous en SRC quand on a un projet multimédia on doit rendre un charte graphique avec les carrés de couleurs, les commentaires... Et la methode pour nous c'était la gualère car word pour faire des tableaux avec des carrées contenant les couleurs web c'est ce que l'on qualifie de galere.

Commentaire de JoJo738 le 30/06/2006 13:14:19

Titox > " $sql='UPDATE plaquette SET titre='$_POST['titre']' , rvb='$_POST['couleur']' , hex='$hex' , description='$_POST['justifier']' , cmjn='$_POST['cmjn']' WHERE ID='$_GET['modif']''; "

Euh .... t'es sûr que ca va marcher ???

$sql=' UPDATE plaquette SET titre="' . $_POST['titre'] . '" , rvb="' . $_POST['couleur'] . '" , hex="' . $hex . '" , description="' . $_POST['justifier'] . '" , cmjn="' . $_POST['cmjn'] . '" WHERE ID="' . $_GET['modif'] . '"';

Commentaire de kankrelune le 30/06/2006 13:44:42

@ cedriclomb... tu crois pas que j'allais donner la bonne requete... .. . ;o)

Mais crois moi c'est facilement faisable... il suffit de fermer la requete update avec la première variable $_POST puis de faire le drop table et de commenter la fin pour qu'elle ne soit pas prise en compte... .. .

tiens... un peu de lecture...

http://www.phpsecure.info/v2/article/InjSql.php

^_^

@ tchaOo°

Commentaire de kankrelune le 03/07/2006 22:07:45

@ Diti2330 comment tu peux noter quelque chose dont tu ne connait pas l'utilité... je trouve ça fort... .. .

@ tchaOo°