mise à jour base mysql Fermé

Question

Bonjour,

Suite à un upload de fichier, je veux mettre un champ d'une base mysql à jour (adresse du fichier uploadé). Ce champ comporte déjà un enregistrement (adresse d'une image ou doc existante). Ce ne serait juste qu'une complémentation.
L'upload se passe bien, mais apparemment, la requête ne s'éxécute pas. Voilà ce que j'ai écrit :

$requete_update = "UPDATE manifestations SET lien_fichier='$lien_fichier' , fichier='$fichier' , extension='$extension' WHERE id='$id'";

J'ai l'impression que ce n'est pas la bonne requête !!!! ???

Quelle est celle que je dois utiliser pour complémenter un champ texte d'une base de donnée ?

Merci

giheller · Answer

bonsoir,

que fait la requ^te ? message d'erreur, rien ?

avez vous essayé en faisant un copier coller la requête en phpmyadmin ?

y a-t-il après 
$requete_update = "UPDATE manifestations SET lien_fichier='".$lien_fichier."' , fichier='".$fichier."' , extension=".'$extension."' WHERE id=".$id.'"; 

$resultat = mysql_update($requete_update);

InAme19 · Answer

hum, ça te met quoi comme erreur?

J'ai l'impression que cela provient de ta concaténation avec tes variables. Par exemple au lieu de mettre SET lien_fichier='$lien_fichier' 

tu mets : SET lien_fichier='.$lien_fichier.' 

Il faut des . pour concaténer une variable en php

t671 · Answer

Bonjour,  et merci à vous deux pour vos réponses !

Pour InAme19 : Si je concatène les variables, c'est pareils. Le résultat est le même que sans concaténation, aucun message d'erreur. Le "header" est éxécuté à la fin, le fichier est uploadé dans sa destination mais la base n'est pas mise à jour.

Pour giheller : donc, aucun message d'erreur. Le "header" est éxécuté à la fin, le fichier est uploadé dans sa destination mais la base n'est pas mise à jour. J'ai rajouté "$resultat = mysql_update($requete_update); 
", et j'ai un message : "Fatal error: Call to undefined function mysql_update() in D:\www\wamp\www\Kotoka\gestion\manifestations	est\add-manif-pj.php on line 50
".
Si je colle la requête dans phpmyadmin (mais je ne fait aucune sélection !!!??), j'ai comme message : You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '$requete_update = "UPDATE manifestations SET lien_fichier='.$lien_fichier.' , fi' at line 1.

J'ai pensé qu'il faudrait que je fasse un INSERT plutôt qu'un UPDATE ... !!!??? Je récupère les enregistrements écrit dans les champs concernés, je DELETE ces champs, et je fais un INSERT avec les enregistrements précédemment récupérés + le ou les nouveaux ! 

Vous en pensez quoi ???

Mon script de mise à jour de la base (actuel) :

<?php 
$host = 'localhost';
$user = '.......';
$pass = '........';
$db = '...........';

// connection à la DB
$link = mysql_connect ($host,$user,$pass) or die ('Erreur : '.mysql_error() );
mysql_select_db($db) or die ('Erreur :'.mysql_error());

$id = $_GET['id'];

// --------------------test upload fichier------------------------------

$dossier = './dossier_upload/';
$fichier = basename($_FILES['monfichier']['name']);
$taille_maxi = 2000000;
$taille = filesize($_FILES['monfichier']['tmp_name']);
$extensions = array('.gif', '.jpg', '.jpeg', '.doc', '.pdf', '.PDF', '.png', '.PNG', '.GIF', '.JPG', '.JPEG', '.DOC');
$extension = strrchr($_FILES['monfichier']['name'], '.'); 

.....................................etc...............

  		{
          echo 'Chargement effectué avec succès !';
                 
       // on écrit la requête sql .
$requete_update = "UPDATE manifestations SET lien_fichier='.$lien_fichier.' , fichier='.$fichier.' ,    extension='.$extension.' WHERE id='.$id.'";
$resultat = mysql_update($requete_update); 
	
    // on insère les informations du formulaire dans la table 
    mysql_query($requete_update) or die('Erreur SQL !'.$sql.mysql_error());
 
}
else
{
     echo $erreur;
}

    mysql_close();  // on ferme la connexion 

    header("Location: aff-manif.php");
exit;
?>

giheller · Answer

bonsoir,
tenter mysql_query($query_update)
mais mettre à jour des données ne peut se faire avec un insert

t671 · Answer

Si je met mysql_query($query_update), j'ai comme réponse "query is empty ..."

Je pense que c'est la requête UPDATE qui est mal écrite et qui ne s'exécute pas .... !!!!! Mais là .... ?????

giheller · Answer

Bonsoir,
j'ai ajouté quelques " voyez si cela va meiux.

$requete_update = "UPDATE manifestations SET lien_fichier='".$lien_fichier."' , fichier="'.$fichier."' , extension="'.$extension.'"  WHERE id="'.$id."'";

P@t@ch0n · Answer

C'est normal que tu le perdes, ton form utilise la method get.

Il faut donc que tu transmettes ton id via ton form, en ajoutant un champ caché dans celui-ci.

echo '<input type="hidden" name="id" value="' . (int)$_GET['id'] . '" />';

t671 · Answer

Merci P@t@ch0n ! J'ai donc modifier mon upload.php (qui contient également du html) ainsi : // connection à la DB $link = mysql_connect ($host,$user,$pass) or die ('Erreur : '.mysql_error() ); mysql_select_db($db) or die ('Erreur :'.mysql_error()); $id = $_GET["id"]; $result=mysql_query("SELECT * FROM manifestations WHERE id='$id'"); echo '

'; print_r($_GET); echo '

'; // visualiser l'id sélectionné en test ?> Upload fichier

t671 · Answer

Merci P@t@ch0n ! J'ai donc modifier mon upload.php (qui contient également du html) ainsi : // connection à la DB $link = mysql_connect ($host,$user,$pass) or die ('Erreur : '.mysql_error() ); mysql_select_db($db) or die ('Erreur :'.mysql_error()); $id = $_GET["id"]; $result=mysql_query("SELECT * FROM manifestations WHERE id='$id'"); echo '

'; print_r($_GET); echo '

'; // visualiser l'id sélectionné en test ?> Upload fichier

P@t@ch0n · Answer

C'est logique non ? Le code php doit toujours se trouver entre Donc : ... ...

P@t@ch0n · Answer

L'upload ne peut s'effectuer en methode get, post obligatoire.

J'avais pas fait attention à cela.
Donc l'id que tu devras récupérer sera dans $_POST, et plus dans get.

Et d'ailleurs, tu n'es plus obligé de le transmettre via un champ caché (cela est mieux quand même), tu peux le passer directement dans l'attribut action de ton form.

P@t@ch0n · Answer

Encore une fois, je dirais que l'on n'enregistre pas plusieurs valeurs dans un même champ.

Si une ligne de ta table manifestation peut posséder plusieurs fichiers, tu dois faire une table séparée pour ceux-ci.

Tu gagneras en facilité sur les futures manipulations.

t671 · Answer

1) Quentends-tu par "table séparée" ?

2) Si je veux complémenter ma table avec de nouvelles valeurs, comme je ne peux enregistrer plusieurs valeurs dans le même champs, dois-je créer à chaque fois un nouveau champs ? Et comment incrémenter le nouveau nom de ce nouveau champs ?

3) J'avais pensé récupérer l'enregistrement existant dans les champs, les renommés en "_old", et au moment de l'update, faire un genre de
$sql = "UPDATE manifestations SET lien_fichier='$lien_fichier_old $dossier.$fichier',fichier='$fichier_old $fichier',extension='$extension_old $extension' WHERE id='$id'";

Qu'en penses-tu ?

P@t@ch0n · Answer

« Qu'en penses-tu ? »

Que c'est une mauvaise idée.

J'entends par table séparée, une table destinée uniquement à enregistrer tes données concernant l'upload de tes fichiers.

t671 · Answer

Merci !!!! J'ai créé une 2° table, et je fais une jointure.

J'avance et je te tiens au courant .... ;o))

P@t@ch0n · Answer

« tous les enregistrements de la table manifestation s'affichent autant de fois (et avec) qu'il y a d'enregistrements dans la table upload.  »

C'est tout à fait normal, le sql ressort toujours les données brutes.
À toi de gérer l'affichage désiré avec php.

Préfère également les jointures normalisées.

SELECT les_colonnes  FROM manifestations m
JOIN  upload u
    ON m.id=u.id_liaison
ORDER BY date ASC

t671 · Answer

Bonjour, Je ne suis pas assez callé en php pour c omprendre ce que tu me dis !!!!! Voilà ce que j'ai écrit : $requete = 'SELECT * FROM manifestations, upload ORDER BY date ASC'; //--- le ORDER uniquement valable pour la table manifestation--- $result = mysql_query($requete,$link) or die ('Erreur : '.mysql_error() ); while($row = mysql_fetch_array($result)) { echo .nl2br($row['texte']).; //---données venant de la table manifestation--- if ($id == $id_liaison) //---id venant de la table manifestation, id_liaison venant de la table upload--- if (in_array($row['extension'], array( ".jpg", ".JPG", ".png", ".PNG", ".jpeg", ".JPEG", ".gif", ".GIF"))) { echo '

'; //données venant de la table upload } else if (in_array($row['extension'], array( ".doc", ".DOC", ".pdf", ".PDF", ".txt", ".TXT"))) { echo 'Télécharger

'; } } mysql_free_result($result); exit; ?> Je suis conscient que mon $requete est plutôt général, mais je ne sais pas comment l'écrire ???? J'ai rajouté des //--- pour des commentaires, que ce soit plus lisible.

P@t@ch0n · Answer

Salut, Comme ceci. // Ordonnancement par les ids en premiers et date en second $requete = 'SELECT texte, extension, lien_fichier FROM manifestations m JOIN upload u ON m.id = u.id_liaison ORDER BY u.id_liaison, m.date'; $result = mysql_query($requete,$link) or exit ('Erreur : '.mysql_error() ); $idEnCours = 0; while($row = mysql_fetch_array($result)) { if( $idEnCours != $row['id_liaison'] ) { $idEnCours = $row['id_liaison']; echo nl2br($row['texte']); } if (in_array($row['extension'], array( '.jpg', '.JPG', '.png', '.PNG', '.jpeg', '.JPEG', '.gif', '.GIF'))) { echo '

'; } elseif (in_array($row['extension'], array( '.doc', '.DOC', '.pdf', '.PDF', '.txt', '.TXT'))) { echo 'Télécharger

'; } } Autres commentaires, pourquoi n'as tu pas mit la casse des extensions en minuscules uniquement ? Le point des extensions n'a pas non plus besoin d'être enregistré dans la table.

t671 · Answer

Merci .... :o))

J'ai testé ce que tu me proposes ! Malheureusement, il n'y a que les enregistrements de la table manifestation ayant une liaison avec la table upload qui s'affichent ....   
Je pense que ça vient de $idEncours (ce que je ne comprends pas trop !!!!)
Alors je cherche , je cherche .....

Pour répondre à tes questions, je n'ai pas mis la casse des extensions uniquement en minuscule, car parfois les extensions sont écrites en majuscule, et je ne sais pas si php fait la différence .... (c'est vrai que j'aurais pu tester .... )

P@t@ch0n · Answer

« J'ai testé ce que tu me proposes ! Malheureusement, il n'y a que les enregistrements de la table manifestation ayant une liaison avec la table upload qui s'affichent .... »

Bah oui, c'est ce que la requete va chercher, je croyais que c'est ce que tu voulais.
Tu voudrais donc que toutes lignes de manifestation s'affichent ?

Si c'est ça, il suffit simplement d'utiliser un LEFT JOIN à la place du JOIN.

Ensuite, pour $idEncours, cette variable n'est là que pour générer un affichage sans répéter à chaque tour de boucle le contenu de texte.
Ce n'est pas ce que tu voulais non plus ?

Un peu de précisions donc :)

t671 · Answer

Merci P@t@ch0n ... ;o))

Maintenant, si je veux associé plusieurs éléments de la table upload à un unique enregistrement de la table manifestation (donc ces éléments de upload ont le même id_liaison qui font référence au même enregistrement de manifestation), à l'affichage, j'ai l'élément de manifestation qui s'affiche autant de fois qu'il a d'élément de upload associé.
Comment faire pour qu'à l'affichage, l'élément de manifestation s'affiche une seule fois avec tous les éléments de upload associés ?

J'espère avoir été assez clair ..... :o))
Merci ....

P@t@ch0n · Answer

Je te l'ai expliqué plus haut comment il faut faire. Le LEFT JOIN permettra de retourner tous les enregistrements de ta table manifestation, mais s'il n'y a pas de lignes affiliées à elles dans la table upload, les colonnes sélectionnées de la table upload auront pour valeur NULL. Ensuite pour la non-répétitions de chaque colonnes sélectionnées de la table manifestation qu'il y a de lignes dans la table upload, il suffit de stocker l'id précédent dans une variable, de comparer avec l'id actuel du tour de boucle et si différent, on affiche les colonnes de manifestation, et on change la valeur de cette variable par l'id courant. Donc niveau code, cela donne. // Ordonnancement par les ids en premiers et date en second ESSENTIEL POUR GÉNÉRER L'AFFICHAGE ORDONNÉ $requete = 'SELECT id, texte, id_liaison, extension, lien_fichier FROM manifestations LEFT JOIN upload ON id = id_liaison ORDER BY id, date'; $result = mysql_query($requete,$link) or exit ('Erreur : '.mysql_error() ); $idEnCours = 0; // On déclare cette variable de stockage avec une valeur ne pouvant être un id de la table, 0 donc. while($row = mysql_fetch_array($result)) { // Si la valeur de l'id de la table manifestation est différent de la valeur de $idEnCours // (ce qui sera forcément le cas lors du 1er tour de boucle) // On affiche les colonnes de la table manifestation, et on modifie la valeur de $idEnCours par la valeur de l'id de manifestation if( $idEnCours != $row['id'] ) { $idEnCours = $row['id_liaison']; echo nl2br($row['texte']); } // Pour savoir s'il y a des lignes correspondantes dans la table upload et le cas échéant les afficher, l'id_relation // (ou n'importe quelle autre colonne de la table upload) ne doit pas valoir NULL if( $row['id_liaison'] !== null ) { if (in_array($row['extension'], array( '.jpg', '.JPG', '.png', '.PNG', '.jpeg', '.JPEG', '.gif', '.GIF'))) { echo '

'; } elseif (in_array($row['extension'], array( '.doc', '.DOC', '.pdf', '.PDF', '.txt', '.TXT'))) { echo 'Télécharger

'; } } else { echo 'aucun fichier affilié à cette manifestation'; } }

t671 · Answer

Bonjour P@t@ch0n, J'arrive au bout du problème ..... :o)) Il me reste un problème d'affichage. Suite à l'affichage du texte, le premier élément de upload affilié au texte s'affiche en dessous, le tableau se ferme, et les autres éléments de upload affiliés au texte s'affichent également mais en dehors du tableau. Je n'arrive pas à rentrer "les autres éléments" dans le tableau, et à les afficher l'un à côté de l'autre. Voici mon code : $requete="SELECT manifestations.id, manifestations.jour , manifestations.heure ,manifestations.datefr ,manifestations.titre ,manifestations.texte , upload.id_liaison, upload.lien_fichier, upload.fichier, upload.extension FROM manifestations LEFT JOIN upload ON manifestations.id=upload.id_liaison ORDER BY manifestations.date ASC"; $result = mysql_query($requete,$link) or exit ('Erreur : '.mysql_error() ); $idEncours = 0; while($row = mysql_fetch_array($result)) { if( $idEnCours != $row['id'] ) { echo ''; echo ''; echo '

'.nl2br($row['texte']).'

'; } if ($idEnCours = $row['id_liaison']); if (in_array($row['extension'], array( ".jpg", ".JPG", ".png", ".PNG", ".jpeg", ".JPEG", ".gif", ".GIF"))) { echo '

'; } elseif (in_array($row['extension'], array( ".doc", ".DOC", ".pdf", ".PDF"))) { echo 'Télécharger ou afficher le document

'; } echo '

'; } Merci

Mise à jour base mysql

23 réponses

Discussions similaires