Forum d'entraide PHPFrance

Bonjour, voici mon problème j'ai un formulaire de modification qui me sert a modifier un produit.
Ce produit a une image qui a été uploadé lors de l'ajout d'un produit.
Si je modifie le produit sans modifier l'image je veut gardé l'image précédente donc je n'update pas l'image
Si j'update une image celle ci remplacera l'ancienne.
Or sa ne fonctionne pas et je commence a m'arraché les cheveux voici mon code Formulaire + action PHP.

Voici mon code : Formulaire : et mon Traitement PHP :

<?php
    require("session.class.php");
    $Session = new Session(); 
    //connection au serveur
    include ('includes/connexion_bdd2.php'); 
    extract($_POST);
            $id = $_GET['page'];
            $sql =  "UPDATE produits SET nom='$nom', prix='$prix', quantiter='$quantiter', description='$description', id_consoles='$id_consoles', id_genres='$id_genres', id_types='$id_types' WHERE id_produits=$id";         
            mysql_query($sql);
if(!empty($_FILES))
{
    $image = $_FILES['image'];
    $image_name = $image['name'];
    $ext = strtolower(substr(strrchr($image_name,'.'),1));
    $ext_aut = array('jpg','jpeg','png','gif');
     
    function check_extension($ext,$ext_aut)
    {
        if(in_array($ext,$ext_aut))
        {
            return true;
        }
    }
     
    $valid = (!check_extension($ext,$ext_aut)) ? false : true;
    $erreur = (!check_extension($ext,$ext_aut)) ? 'Veuillez charger une image' : '';
     
    if($valid)
    {
        $max_size = 2000000;
        if($image['size']>$max_size)
        {
            $valid = false;
            $erreur = 'Fichier trop gros';
        }
    }
     
    if($valid)
    {
        if($image['error']>0)
        {
            $valid = false;
            $erreur = 'Erreur lors du transfert';
        }
    }
     
    if($valid)
    {
        $path_to_image = 'images/fullsize/';
        $path_to_min = 'images/min/';
         
        $filename = sha1(uniqid($image_name));
         
        $source = $image['tmp_name'];
        $target = $path_to_image . $filename. '.'. $ext;
         
        move_uploaded_file($source,$target);
         
        if($ext == 'jpg' || $ext == 'jpeg') {$im = imagecreatefromjpeg($path_to_image.$filename.'.'.$ext);}
        if($ext == 'png') {$im = imagecreatefrompng($path_to_image.$filename.'.'.$ext);}
        if($ext == 'gif') { $im = imagecreatefromgif($path_to_image.$filename.'.'.$ext);}
         
        $ox = imagesx($im);
        $oy = imagesy($im);
         
        $nx = 150;
        $ny = floor($oy *($nx/$ox));
         
        $nm = imagecreatetruecolor($nx,$ny);
         
        imagecopyresized($nm, $im, 0,0,0,0, $nx,$ny,$ox,$oy);
         
        imagejpeg($nm, $path_to_min.$filename.'.'.$ext);
         
        $nom_image = $filename.'.'.$ext;
 
    } 
            extract($_POST);
            $id = $_GET['page'];
            $sql =  "UPDATE produits SET image='$nom_image' WHERE id_produits=$id";         
            mysql_query($sql); 
    }
            // message d'alerte
            $Session->setFlash('La modification du produits a été effectuer','success');
            // retour a la page d'acceuil
            header('Location:admin_produits.php');
 
 
?>

salut,


qu'est ce qui ne fonctionne pas exactement ?

tu peux commencer par simplifier la validation afin de ne garder que l'essentiel pour tester la chose (ensuite tu ajoute les contrôles nécessaire quand tu a validé la logique de base).

tu peux aussi virer le extract et utiliser les variable contenu dans $_POST.
si c'est la formation de la requête qui te gêne utilise la concaténation

ton script est ouvert au faille sql (injection) utiliser mysql_real_escape_string pour protéger tes données.

tu peux aussi éviter d'ajouter un paramètre dans l'url action du formulaire, utilise simplement un champs caché (input type hidden) dans le formulaire pour passe ton id

Pour ce qui est de l'upload c'est pareil, évite la copie de variable c'es inutile et se perd à a lecture du code.
As tu regardé ce que contenait $_FILES lorsque tu ne n'utilise pas ce champ ?

$valid = (!check_extension($ext,$ext_aut)) ? false : true;
$erreur = (!check_extension($ext,$ext_aut)) ? 'Veuillez charger une image' : '';
c'est exactement la même chose pourquoi deux tests ?
d'ailleurs le message d'erreur n'est pas clair c'est un fichier dont l'extension n'est pas autorisé (par exemple bmp, raw c'est des images ).

deux fois if($valid)) c'est inutile

simplifie ton code tu y verra plus clair et tu devrais t'en sortir


@+

J'ai l'impression que ce code est du copié/collé de différents tutoriels. Pourquoi utilises-tu d'un côté l'extension MySQL et de l'autre PDO ?
Avec un niveau pareil, je te déconseille fortement de jouer à uploader des fichiers. C'est dangereux pour ton serveur.

c'est pas du copié collé mdr XD au début j'utilisé mysqlconnect ensuite j'ai appris a codé avec PDO et comme je m’étais déjà avancé je voulais pas tout refaire donc je jongle avec les 2
il n'y a que pour l'upload de d'image que j'ai utilisé un tuto

J'ai changé mon code en optant pour une image par url c'est moin compliqué qu'un uploader d'image et sa fonctionne nickel

<?php
    require("session.class.php");
    $Session = new Session(); 
    //connection au serveur
    include ('includes/connexion_bdd2.php'); 
			extract($_POST);
            $id = $_GET['page'];
            $sql =  "UPDATE produits SET nom='$nom', prix='$prix', quantiter='$quantiter', description='$description', id_consoles='$id_consoles', id_genres='$id_genres', id_types='$id_types' WHERE id_produits=$id";         
            mysql_query($sql);
if(!empty($_POST['image']))
{
            $id = $_GET['page'];
            $sql =  "UPDATE produits SET image='$image' WHERE id_produits=$id";         
            mysql_query($sql); 
    }
            // message d'alerte
            $Session->setFlash('La modification du produits a été effectuer','success');
            // retour a la page d'acceuil
            header('Location:admin_produits.php');
 
 
?>