Discussion :

[none61]

Bonjour tout le monde,

Je viens poster ici ce soir car je butte sur la fonction php "move_uploaded_file ".

- En effet, je n'ai aucun message d'erreur mais l'upload ne se fait pas....

Voici mon code:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
 
<?php
 
$file = $_FILES['image1'];
 
$uploaddir = '/setup/';
$uploadfile = $uploaddir . basename($_FILES['image1']['name']);
 
 
echo '<pre>';
if (move_uploaded_file . basename($_FILES['image1']["name"], $uploadfile)) {
  echo "Fonctionne ok .
   Voici plus d'informations :\n";
} else {
  echo "Ne fonctionne pas.
      Voici plus d'informations :\n";
}
 
print_r($_FILES);
 
echo '</pre>';
echo "le nom du fichier selectionné est : ".basename($_FILES['image1']["name"]);?>
</br>
<?php
echo " le nouveau chemin de destination est : ".$uploadfile;
?>

Je ne vous ai pas mis le formulaire html...

- Mes écho fonctionnent bien,
- La page PHP m'affiche "Fonctionne ok"
- Je n'ai pour le moment pas fait de contrôle sur le type de fichier ni sur la taille etc... je voudrais déjà faire fonctionner cela simplement et petit a petit corser les choses

Je precise que mon serveur PHP est sur un Synology et que l'utilisateur http a bien les droits en écriture.
J'ai aussi fait un chmod 777 sur le dossier /setup ...

Je ne demande pas une solution toute faite mais plutôt quelques conseils sur cette fonction...

Merci a vous

[Invité]

• Tu fais une concaténation au lieu d'une fonction !
• Tu confonds "name" et "tmp_name"
• enlève basename()

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

if ( move_uploaded_file( ... ) ) {

[none61]

Bonjour et merci pour ta réponse.

J'ai testé avec ta ligne de code mais je passe dans le else..

[laurentSc]

Bonjour,

tu ne testes pas l'existence du fichier (comme indiqué ici : https://antoine-herault.developpez.c...ls/php/upload/). Or j'ai quelques doutes sur ta ligne 6 $uploaddir = '/setup/';. Y a pas un slash en trop ? $uploaddir = 'setup/';Sinon, garde bien la remarque de jreaux62 au post #2...

[Invité]

Bonjour,

il faut surtout :

• LIRE la DOC : move_uploaded_file()
• LIRE un TUTO : Upload de fichiers en PHP (3.1. L'upload *)

* Code d'upload de base

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
<?php
if(isset($_FILES['avatar']))
{ 
     $dossier = 'upload/';
     $fichier = basename($_FILES['avatar']['name']);
     if(move_uploaded_file($_FILES['avatar']['tmp_name'], $dossier . $fichier)) //Si la fonction renvoie TRUE, c'est que ça a fonctionné...
     {
          echo 'Upload effectué avec succès !';
     }
     else //Sinon (la fonction renvoie FALSE).
     {
          echo 'Echec de l\'upload !';
     }
}

[none61]

Bonjour tout le monde,

Merci pour vos réponses ;-)

Je vais essayer de vous répondre sans m'emmener les pinceaux ^^

- laurentSc => j'ai mis /setup/ car le dossier setup est a la racine de mon dossier web et mon script php ne l'est pas.. c'est pour éviter d'écrire ../../../setup/

Non je ne teste pas l'existence du fichier..je voudrais déjà comprendre et faire fonctionner le "noyau" pour ensuite l'étoffer avec toutes les options.. test existence, type, taille etc..

- jreaux62 => j'ai remarqué pour la concaténation..c'est vrai qu'a force de tout tester ...

Je confond peux être name et tmp_name enfin je ne pense pas avoir compris tmp_name car pour moi

name = c'est le nom du fichier uploadé
tmp_name = c'est le chemin d'accès a ce fichier dans le dossier temporaire du serveur php ce qui revient au meme que basename nan?

Si tel est le cas, je ne comprend pas

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
if(move_uploaded_file($_FILES['avatar']['tmp_name'], $dossier . $fichier))

on va faire le move sur le fichier associé a la requête "POST" qui a pour nom avatar et qui se trouve temporairement dans "tmp_name" ?

Donc on ne se sert pas du nom du fichier "$_FILES['avatar']['name']" mise a par pour le renommer a l'identique dans le dossier de destination?

Merci pour le temps que vous me consacrez

[none61]

J'ai copié/collé le code upload de base en ne modifiant que le nom de la requête "POST" ainsi que le chemin de destination.

J'ai testé avec "/setup/" et setup/ ... ^^

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
 
 
<?php
 
echo '<pre>';
if(isset($_FILES['image1']))
{ 
     $dossier = '/setup/';
     $fichier = basename($_FILES['image1']['name']);
     if(move_uploaded_file($_FILES['image1']['tmp_name'], $dossier . $fichier)) //Si la fonction renvoie TRUE, c'est que ça a fonctionné...
     {
          echo 'Upload effectué avec succès !';
     }
     else //Sinon (la fonction renvoie FALSE).
     {
          echo 'Echec de l\'upload !';
     }
}
 
print_r($_FILES);
 
echo '</pre>';
echo "le nom du fichier selectionné est : ".basename($_FILES['image1']["name"]);?>
</br>
<?php
echo " le nouveau dossier de destination est : ".$dossier;
?>

Voici mon retour :



Voici le formulaire html

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
 
<form action="upload.php" method="post" enctype="multipart/form-data">
    Select image to upload:
    <input type="file" name="image1">
    <input type="submit" value="Upload Image" name="submit">
</form>

[laurentSc]

Je sais pas si c'est important, mais dans ton formulaire, il manque une ligne <input type="hidden" name="MAX_FILE_SIZE" value="2000000">

[Invité]

...c'est pour éviter d'écrire ../../../setup/

• As-tu testé avec '../../../setup/' ??
• As-tu vérifié que le dossier a bien les droits en écriture (CHMOD) ?

on va faire le move sur le fichier associé a la requête "POST" qui a pour nom avatar et qui se trouve temporairement dans "tmp_name" ?
Donc on ne se sert pas du nom du fichier "$_FILES['avatar']['name']" mise a par pour le renommer a l'identique dans le dossier de destination?

Tout-à-fait.

On peut néanmoins tester l'extension du fichier :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

$file_Extension = strtolower(pathinfo($_FILES['image1']['name'],PATHINFO_EXTENSION));

et son type MIME RÉEL :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
$finfo = finfo_open(FILEINFO_MIME_TYPE); // Retourne le type mime
$file_MimeType = finfo_file($finfo, $_FILES['image1']['tmp_name']);
finfo_close($finfo);

Voir : Système de Gestion-Affichage de Nouvelles - Traitement Photo

[laurentSc]

Voici un code qui fonctionne (testé ; tu peux t'en inspirer).

Code php :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
<!DOCTYPE>
<html lang="fr">
<head>
	<title>upload</title>
	<meta http-equiv="content-type" content="text/html; charset=utf-8" />
 
</head>
<body>
 
<form method="POST" action="" enctype="multipart/form-data">
     <!-- On limite le fichier à 150Ko -->
     <input type="hidden" name="MAX_FILE_SIZE" value="150000">
     Fichier : <input type="file" name="avatar">
     <input type="submit" name="envoyer" value="Envoyer le fichier">
</form>
<?php
if (isset($_POST['envoyer'])) {	
 
    if(isset($_FILES['avatar']))
    { 
        $dossier = 'upload/';
        $fichier = basename($_FILES['avatar']['name']);
        if(move_uploaded_file($_FILES['avatar']['tmp_name'], $dossier . $fichier)) //Si la fonction renvoie TRUE, c'est que ça a fonctionné...
        {
            echo 'Upload effectué avec succès !';
        }
        else //Sinon (la fonction renvoie FALSE).
        {
            echo 'Echec de l\'upload !';
        }
    }
}
?>
 
</body>
</html>

arborescence :
upload.php
<dossier upload>

[none61]

ça fonctionne...enfin !!!

Donc: tout mes dossiers étés déjà en CHMOD 777.. j'ai été obligé de mettre /volume1/web en CHMOD 777.... c'est un peu nul que php n'ai pas le droit d'écrire par défaut à la racine de /web...

De plus en effet "/setup/" ne fonctionne pas... je suis obligé de faire "../../../setup" (c'est pas grave faut juste le savoir ^^)

Maintenant je vais voir pour affiner les droits d'écriture car 777 c'est en full pour tout le monde donc...pas sécur

laurentSc =>

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

 if(isset($_FILES['avatar']))

sert à contrôler l'existence du fichier lié à la requête "POST" avatar?

Mais a quoi sert

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

 if (isset($_POST['envoyer']))

?

Sinon je pensais à ça...il n'existe pas un fichier de "log" pour aider à débugger ce genre de problème? car si j'avais vu droit d'accès refuser ou un truc du genre ça m'aurai bien aidé

Encore merci à vous, vous m'avais bien aidé !!!!

[laurentSc]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

 if(isset($_FILES['avatar']))

sert à contrôler l'existence du fichier lié à la requête "POST" avatar?

oui, ça sert à vérifier que l'utilisateur a bien sélectionné un fichier à uploader.

Mais a quoi sert

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

 if (isset($_POST['envoyer']))

?
Dans mon code le traitement en PHP est dans le même fichier que le formulaire, mais le traitement ne doit être exécuté que si le formulaire a été rempli, et si et seulement si c'est le cas, cette variable existe.

[none61]

laurentSc Merci pour cette info ça me parait tellement évident maintenant que tu me l'a expliqué !!

C'est vrai qu'en règle générale, je faisais 2 pages pour les formulaires.ça m'embêtais que le script "tourne" pour rien a la première ouverture..

Je vais essayer ça ;-)

Encore merci !

[Invité]

• PRINCIPE de FORMULAIRE et TRAITEMENT sur la même page

[none61]

jreaux62 Merci pour le lien ;-)