Petit bug sur le script

Bonjour la communauté
Je reviens vers vous cars j ai un petit bug
que voici:
Parse error: syntax error, unexpected '"' en ligne 26
Voici la ligne incriminée
ligne 26 -
[code ]
function FILE_UPLOADER($num_of_uploads=1, $file_types_array=array('mp3'), $max_file_size=26214400, j ai juste changé $upload_dir="../dossier") par $upload_dir="$_POST['folder']")
{

if(!is_numeric($max_file_size)){
$max_file_size = 26214400;
}
$max_file_size_Mo = $max_file_size/26214400;
if(!isset($_POST['submitted'])){
$form = '<form action="'.$PHP_SELF.'" method="post" enctype="multipart/form-data">Telechargement de fichier:<br /><input type="hidden" name="submitted" value="TRUE" id="'.time().'"><input type="hidden" name="MAX_FILE_SIZE" value="'.$max_file_size.'">';
for($x=0;$x<$num_of_uploads;$x++){
$form .= '<input type="file" name="file[]"><font color="red">*</font><br /><br />';
}
$form .= '<label>Destination
<select name="folder" id="folder">
<option value="http://www.http://www.toto.com/zip">Zip</option>
<option value="http://www.http://www.toto.com/mp3">Mp3</option>
</select>';
$form .= '<input type="submit" value="Telecharger"><br /><font color="red">*</font> Type(s) de fichiers autorisés: ';
$y=count($file_types_array);
for($x=0;$x<$y;$x++){
if($x<$y-1){
$form .= $file_types_array[$x].', ';
}else{
$form .= $file_types_array[$x].'.';
}
}
$form .= '</form>';
echo($form);
}
[/code ]
Pour recupere l information d'une selection
j ai besoin de $_POST['folder']" pour pouvoir là recuperer
folder etant le nom de mon select

que voici
<select name="folder" id="folder">
<option value="zip">Zip</option>
<option value="mp3">Mp3</option>
</select>

une petite idée
Merci de votre aide
Modifié par william_13 (09 Aug 2011 - 13:28)

Bonjour, si tu pouvais mettre ton code dans des balises codes ça rendrait le code plus clair.
Je ne comprends pas très bien ce que tu fais, tu pourrais poster un peu plus de code.
C'est quand même étrange de déclarer une fonction avec des paramètres que tu initialises au moment de la déclaration.
Bref .

//appel de la fonction
FILE_UPLOADER($_POST['folder']);


//déclaration de la fonction
function FILE_UPLOADER($upload_dir)
{
      $num_of_uploads = 1;
      $file_types_array = array('mp3','zip');
      $max_file_size = 26214400;
      //ce que tu veux faire après ?
     
}

De plus je suppose que ton formulaire est un formulaire d'upload avec un champ de type "file". Sache que tu peux connaitre le type du fichier, sans le demander à l'utilisateur il est contenu dans la variable :
$_FILES['le_nom_de_ton_champ']['type']
cf : http://phpcodeur.net/articles/php/upload
http://fr.wikipedia.org/wiki/Type_MIME
Modifié par Su4p (09 Aug 2011 - 11:36)

Rebonjour

je te rajoute les explications au niveau du script


**************

$num_uploads = Nombre de champs d'upload à afficher et à gerer simultanément.

**************

$file_types_array = Un tableau regroupant toutes les extentions autorisées. par defaut, seuls les fichiers ('mp3') sont autorisés.

*************
$max_file_size = poids maximun de chaque fichier à uploader. pour toute valeur autre que numerique aura comme consequense de fixer cette variable à sa valeur par defaut de 1Mo .

*************

$upload_dir = Chemin menant au repertoire de destination du fichier à uploader.

*************

La fonction affiche le formulaire et gere les données soumises par elle-même, pour uploader un ou plusieurs fichiers je n ai qu'à modifier les parametres d'entrée selon mes convenances, elle se charge du reste.

Pour le transfert d'un seul fichier avec les parametres par defaut il faut juste a appeler la fonction comme ceci: <?php FILE_UPLOADER(); ?>

si besoin du code complet ,pas de probléme

Merci encore

L'erreur que tu obtiens se situe bien ici $upload_dir = "$_POST['folder']".

Pourquoi ? Parce que les valeurs des paramètres optionnels doivent être

sous forme "constantes" : booléennes (TRUE, FALSE), entières, décimales, null, une chaîne de caractères

mais pas le résultat d'une opération par exemple (concaténation, addition, ...), fonctions ou autres.De plus les paramètres optionnels doivent être les derniers de la liste des paramètres

(ce qui est logique si on y réfléchit).

Donc si on reprends ton exemple il faudrait : function FILE_UPLOADER($upload_dir,$num_of_uploads=1, $file_types_array=array('mp3'), $max_file_size=26214400)

Car $upload_dir n'est pas un paramétre optionel avec une valeur par default car sa valeur est le resultat d'une opération

Tu appeleras donc ta fonction FILE_UPLOADER($_POST['folder']).

<?php
/* 
	L'erreur que tu obtiens se situe bien ici $upload_dir = "$_POST['folder']".
	Pourquoi ? Parce que les valeurs doivent être 
	sous forme "constantes" : booléennes (TRUE, FALSE), entières, décimales, null, une chaîne de caractères 
	mais pas le résultat d'une opération par exemple (concaténation, addition, ...), fonctions ou autres.De plus les paramètres optionnels doivent être les derniers de la liste des paramètres 
	(ce qui est logique si on y réfléchit). 
	Donc si on reprends ton exemple il faudrait : function FILE_UPLOADER($upload_dir,$num_of_uploads=1, $file_types_array=array('mp3'), $max_file_size=26214400)
	Car $upload_dir n'est pas un paramétre optionel avec une valeur par default car sa valeur est le resultat d'une opération
	Tu appeleras donc ta fonction FILE_UPLOADER($_POST['folder']).
	
*/
function FILE_UPLOADER($num_of_uploads=1, $file_types_array=array('mp3'), $max_file_size=26214400, $upload_dir = "$_POST['folder']")
{

	if(!is_numeric($max_file_size)){
		$max_file_size = 26214400;
	}
	$max_file_size_Mo = $max_file_size/26214400;
	if(!isset($_POST['submitted'])){
		$form = '<form action="'.$PHP_SELF.'" method="post" enctype="multipart/form-data">Telechargement de fichier:<br /><input type="hidden" name="submitted" value="TRUE" id="'.time().'"><input type="hidden" name="MAX_FILE_SIZE" value="'.$max_file_size.'">';
		for($x=0;$x<$num_of_uploads;$x++){
		$form .= '<input type="file" name="file[]"><font color="red">*</font><br /><br />';
	}
	$form .= '<label>Destination
	<select name="folder" id="folder">
	<option value="http://www.http://www.toto.com/zip">Zip</option>
	<option value="http://www.http://www.toto.com/mp3">Mp3</option>
	</select>';
	$form .= '<input type="submit" value="Telecharger"><br /><font color="red">*</font> Type(s) de fichiers autorisés: ';
	$y=count($file_types_array);
	for($x=0 ; $x<$y ; $x++ ){

		if($x<$y-1){
			$form .= $file_types_array[$x].', ';
		}else
		{
			$form .= $file_types_array[$x].'.';
		}
	}
	$form .= '</form>';
	/*
		c'est dommage en mettant ton formulaire dans une varibale php tu perds la coloration syntaxique du html
	*/
	echo($form);
	}
}

Modifié par Su4p (09 Aug 2011 - 14:53)

ReREbonjour

Merci pour tes explications

donc $upload_dir = Chemin menant au repertoire de destination du fichier à uploader ne prend pas en compte la valeur de select!!
Si j ajoute comme celà $upload_dir = "$_POST['folder']" je me retrouve avec le probléme de départ.
et si je laisse comme sur le code si dessous
Je me retrouve avec ceci !
Notice: Undefined index: folder on line 25

Merci pour ta patience mais là j ai du mal à cogiter!
Pour faire simple

voici le code complet modifié si j ai bien compris

<?php
FILE_UPLOADER($_POST['folder']);?> ligne 25

//non modifier ancien code
function FILE_UPLOADER($num_of_uploads=1, $file_types_array=array('mp3'), $max_file_size=26214400, j ai juste changé $upload_dir="../dossier") par $upload_dir="$_POST['folder']")

//La modification que tu m as ecrite si j ai saisi !
<?php
function FILE_UPLOADER($upload_dir , $num_of_uploads=1, $file_types_array=array('mp3'), $max_file_size=26214400)

{

	if(!is_numeric($max_file_size)){
		$max_file_size = 26214400;
	}
	$max_file_size_Mo = $max_file_size/26214400;
	if(!isset($_POST['submitted'])){
		$form = '<form action="_1.php" method="post" enctype="multipart/form-data">Telechargement de fichier:<br /><input type="hidden" name="submitted" value="TRUE" id="'.time().'"><input type="hidden" name="MAX_FILE_SIZE" value="'.$max_file_size.'">';
		for($x=0;$x<$num_of_uploads;$x++){
		$form .= '<input type="file" name="file[]"><font color="red">*</font><br /><br />';
	}
	$form .= '<label>Destination
	<select name="folder" id="folder">
	<option value="http://www.toto.com//zip">Zip</option>
	<option value="http://www.toto.com//mp3">Mp3</option>
	</select>';
	$form .= '<input type="submit" value="Telecharger"><br /><font color="red">*</font> Type(s) de fichiers autorisés: ';
	$y=count($file_types_array);
	for($x=0 ; $x<$y ; $x++ ){

		if($x<$y-1){
			$form .= $file_types_array[$x].', ';
		}else
		{
			$form .= $file_types_array[$x].'.';
		}
	}
	$form .= '</form>';
  }else{
    foreach($_FILES['file']['error'] as $key => $value){
      if($_FILES['file']['name'][$key]!=""){
        if($value==UPLOAD_ERR_OK){
          $origfilename = $_FILES['file']['name'][$key];
          $filename = explode('.', $_FILES['file']['name'][$key]);
         $filenameext = $filename[count($filename)-1];
          unset($filename[count($filename)-1]);
          $filename = implode('.', $filename);
          $filename = substr($filename, 0, 15).'.'.$filenameext;
          $file_ext_allow = FALSE;//par mesure de securité on suppose l'extenion du fichier fausse
		  //verifions si notre fichier fait partie des types autorisés
		  if(false !== ($iClef = array_search($filenameext, $file_types_array))) {
 $file_ext_allow = TRUE;
}
          if($file_ext_allow){
            if($_FILES['file']['size'][$key]<$max_file_size){
              if(move_uploaded_file($_FILES['file']['tmp_name'][$key], $upload_dir.$filename)){
                echo('Transfert de fichier effectué avec succès. - <a href="'.$upload_dir.$filename.'" target="_blank">'.$filename.'</a><br />');/*evidemment plutot que d'afficher ici le lien vers le fichier uploader sur le serveur vous pouvez proceder à une redirection vers une autre page*/
              }else{
                echo('Une erreur est survenue lors du transfert de '.'<strong>'.$origfilename.'</strong><br />');
              }
            }else{
              echo('La taille du fichier '.'<strong>'.$origfilename.'</strong>'.' excède les '.$max_file_size_Mo.' Mo autorisé(s), ERREUR DE TRANSFERT !<br />');
            }
          }else{
            echo('Le fichier '.'<strong>'.$origfilename.'</strong>'.'  a une extension invalide, ERREUR DE TRANSFERT !<br />');
        }       }else{
          echo('Une erreur est survenue lors du transfert de '.'<strong>'.$origfilename.'</strong><br />');
        }
      }
    }
  }
}

Modifié par william_13 (09 Aug 2011 - 17:55)

t'as ligne 25 c'est laquelle ? Notice: Undefined index: folder on line 25 ?

Perso de mon coté ton code a l'air de marché je me suis envoyé un fichier il a dit mauvaise extension, j'ai donc envoyé un mp3 et ça c'est passé ...
Modifié par Su4p (09 Aug 2011 - 17:13)

Bonsoir

Merci de ta patience

en fait c est ici
<,php
FILE_UPLOADER($_POST['folder']);?> ligne 25
celà appelle la fonction

<?php
function FILE_UPLOADER($upload_dir , $num_of_uploads=1, $file_types_array=array('mp3'), $max_file_size=26214400)....etc

pour le moment le formulaire n apparait plus!

j ai changé la function
voici ce que j ai mis

<?php FILE_UPLOADER(); ?>
<?php
function FILE_UPLOADER($num_of_uploads=1, $file_types_array=array('mp3'), $max_file_size=26214400, $upload_dir='../$POST_folder/'){

le formulaire est de nouveau opérationnel
le probléme , c est que le fichier est bien uplodé mais à la racine et pas dans un des dossiers selectionnés dans le formulaire

Merci encore à toi pour ta gentillesse à vouloir m aider à démeler ce probléme



William_13
Modifié par william_13 (09 Aug 2011 - 19:05)

La syntaxe PHP de base présente les règles quant aux guillemets simples / doubles, ainsi que l'utilisation des variables globales, et des tableaux. Tu as accumulé les problèmes:
- $upload_dir='../$POST_folder/' n'a aucun sens.

Déjà, $POST_folder, je doute que cette variable existe. Même en-dehors de ça, là, tu assignes juste la valeur littérale "../$POST_folder" (et pas la valeur d'une quelconque variable) à ton $upload_dir.
Comme l'a précisé Su4p plusieurs fois, ce n'est pas ainsi qu'on utilise une fonction. Des paramètres variables par défaut, ce n'est pas ce genre de cas.
- tes manips explode / implode / unset sur le $filename, en plus d'être peu utiles, vont te scraper gentiment ton nom de fichier et faire apparaître de "..". Définis-toi même le format du nom de fichier, ou laisses-le tel quel.

Corriges ces soucis, et tu verras que tu n'auras plus ce problème

Bonjour
Merci pour ce conseil:

quand je fais

<?php 
echo '<pre>' ; 
print_r($_POST) ; 
echo '</pre>' ; 
?> 
 j ai bien 
Array
(
    [submitted] => TRUE
    [MAX_FILE_SIZE] => 26214400
    [folder] =>  http://www.toto.com/zip/
 
)

lorsque j envoi le formulaire avec le dossier zip selectionné.

En fait c'est ce là que je voudrais recupérer
=> http://www.toto.com/zip/

dans sa 1er version
upload_dir était comme celà $upload_dir="../swf/" et tout fonctionnait parfaitement

ayant creé un select pour choisir le repertoire de destination
je pensai que mettre à la place de (swf) $_POST ['folder '](nom du select)

celà aurait donné la solution escomptée.

Merci de m avoir répondu

nos messages se sont croisés!
Je vais essayer de suivre ton conseil
Merci
Modifié par william_13 (10 Aug 2011 - 15:34)

Certes. Donc:
- $_POST['folder'] existe bel et bien
- $POST_folder n'existe pas pour autant
- le setting dynamique d'arguments par défaut en PHP (comme dans tout langage) est déconseillé, et ne se fait pas de cette façon

C'est bel et bien le prototype de ta fonction qu'il te faut rectifier.

Attention, en passant, c'est les chemins réels qu'il te faut indiquer ( /var/www ... ), et non les chemins virtuels ( http://www. ... ).

Bonsoir la communauté

Merci à vous pour avoir pris un peu de votre temps

j ai trouvé la solution
le probléme est résolu

Merci
William_13