Skip to main content
adicionou 183 caracteres ao conteúdo
Fonte Link
Largato
  • 92,9mil
  • 18
  • 190
  • 261

Aqui eu incluí um isset(), para ver se houve envio:

$uploads_dir = 'images/';
$extok = array( 'jpg', 'jpeg', 'gif', 'png', 'bmp' );

$img = array();
if( isset( $_FILES['files'] ) ) {
   $fdata = $_FILES['files'];
   $count = count( $fdata['name'] ); 

   for ($i = 0; $i < $count; ++$i ) {
      $ext = pathinfo( $fdata['name'][$i], PATHINFO_EXTENSION);

      if( in_array( strtolower( $ext ), $extok ) ) { 
         $imagem_nome = md5(uniqid(time())).'.'.$ext;
         $temp_name = $fdata['tmp_name'][$i]; 

         move_uploaded_file($temp_name, $uploads_dir.$imagem_nome );
         $img[] = $imagem_nome;
      }
   }
}

Aproveitei e troquei a RegEx da extensão por uma função apropriada, e verificação das extensões aceitáveis.

O que precisa arrumar aí éTambém tirei o MD5 do uniquid. A única coisa que o uniqid faz de bom, é conseguir um Id único. Fazendo MD5 dele, acabou de perder issoestragar o uniqid. Seria bom revisar essa etapa

O time() dentro do uniqid() também não faz sentido, foi removido.

Se preferir, pode usar multiple="multiple", para não ficar um atributo vazio:

<input id="file-upload" name="files[]" type="file" multiple="multiple">

Aqui eu incluí um isset(), para ver se houve envio:

$uploads_dir = 'images/';
$img = array();
if( isset( $_FILES['files'] ) ) {
   $fdata = $_FILES['files'];
   $count = count( $fdata['name'] );
   for ($i = 0; $i < $count; ++$i ) {
      $ext = pathinfo( $fdata['name'][$i], PATHINFO_EXTENSION);
      $imagem_nome = md5(uniqid(time())).'.'.$ext;
      $temp_name = $fdata['tmp_name'][$i];
      move_uploaded_file($temp_name, $uploads_dir.$imagem_nome );
      $img[] = $imagem_nome;
   }
}

Aproveitei e troquei a RegEx da extensão por uma função apropriada.

O que precisa arrumar aí é o MD5 do uniquid. A única coisa que o uniqid faz de bom, é conseguir um Id único. Fazendo MD5 dele, acabou de perder isso. Seria bom revisar essa etapa.

Se preferir, pode usar multiple="multiple", para não ficar um atributo vazio:

<input id="file-upload" name="files[]" type="file" multiple="multiple">

Aqui eu incluí um isset(), para ver se houve envio:

$uploads_dir = 'images/';
$extok = array( 'jpg', 'jpeg', 'gif', 'png', 'bmp' );

$img = array();
if( isset( $_FILES['files'] ) ) {
   $fdata = $_FILES['files'];
   $count = count( $fdata['name'] ); 

   for ($i = 0; $i < $count; ++$i ) {
      $ext = pathinfo( $fdata['name'][$i], PATHINFO_EXTENSION);

      if( in_array( strtolower( $ext ), $extok ) ) { 
         $imagem_nome = uniqid().'.'.$ext;
         $temp_name = $fdata['tmp_name'][$i]; 

         move_uploaded_file($temp_name, $uploads_dir.$imagem_nome );
         $img[] = $imagem_nome;
      }
   }
}

Aproveitei e troquei a RegEx da extensão por uma função apropriada, e verificação das extensões aceitáveis.

Também tirei o MD5 do uniquid. A única coisa que o uniqid faz de bom, é conseguir um Id único. Fazendo MD5 dele, acabou de estragar o uniqid.

O time() dentro do uniqid() também não faz sentido, foi removido.

Se preferir, pode usar multiple="multiple", para não ficar um atributo vazio:

<input id="file-upload" name="files[]" type="file" multiple="multiple">
corrigida informação errada
Fonte Link
Largato
  • 92,9mil
  • 18
  • 190
  • 261

Primeiro, não seria o caso de usarAqui eu incluí um []isset() quando for um input só (não confundir com arquivos múltiplos):

<input id="file-upload" name="files" type="file" multiple>

Depois, seria o caso de testarpara ver se a variável contém algohouve envio:

$uploads_dir = 'images/';
$img = array();
if( isset( $_FILES['files'] ) ) {
   $fdata = $_FILES['files'];
   $count = count( $fdata['name'] );
   for ($i = 0; $i < $count; ++$i ) {
      $ext = pathinfo( $fdata['name'][$i], PATHINFO_EXTENSION);
      $imagem_nome = md5(uniqid(time())).'.'.$ext;
      $temp_name = $fdata['tmp_name'][$i];
      move_uploaded_file($temp_name, $uploads_dir.$imagem_nome );
      $img[] = $imagem_nome;
   }
}

Aproveitei e troquei a RegEx da extensão por uma função apropriada.

O que precisa arrumar aí é o MD5 do uniquid. A única coisa que o uniqid faz de bom, é conseguir um Id único. Fazendo MD5 dele, acabou de perder isso. Seria bom revisar essa etapa.

Se preferir, pode usar multiple="multiple", para não ficar um atributo vazio:

<input id="file-upload" name="files[]" type="file" multiple="multiple">

Primeiro, não seria o caso de usar [] quando for um input só (não confundir com arquivos múltiplos):

<input id="file-upload" name="files" type="file" multiple>

Depois, seria o caso de testar se a variável contém algo:

$uploads_dir = 'images/';
$img = array();
if( isset( $_FILES['files'] ) ) {
   $fdata = $_FILES['files'];
   $count = count( $fdata['name'] );
   for ($i = 0; $i < $count; ++$i ) {
      $ext = pathinfo( $fdata['name'][$i], PATHINFO_EXTENSION);
      $imagem_nome = md5(uniqid(time())).'.'.$ext;
      $temp_name = $fdata['tmp_name'][$i];
      move_uploaded_file($temp_name, $uploads_dir.$imagem_nome );
      $img[] = $imagem_nome;
   }
}

Aproveitei e troquei a RegEx da extensão por uma função apropriada.

O que precisa arrumar aí é o MD5 do uniquid. A única coisa que o uniqid faz de bom, é conseguir um Id único. Fazendo MD5 dele, acabou de perder isso. Seria bom revisar essa etapa.

Aqui eu incluí um isset(), para ver se houve envio:

$uploads_dir = 'images/';
$img = array();
if( isset( $_FILES['files'] ) ) {
   $fdata = $_FILES['files'];
   $count = count( $fdata['name'] );
   for ($i = 0; $i < $count; ++$i ) {
      $ext = pathinfo( $fdata['name'][$i], PATHINFO_EXTENSION);
      $imagem_nome = md5(uniqid(time())).'.'.$ext;
      $temp_name = $fdata['tmp_name'][$i];
      move_uploaded_file($temp_name, $uploads_dir.$imagem_nome );
      $img[] = $imagem_nome;
   }
}

Aproveitei e troquei a RegEx da extensão por uma função apropriada.

O que precisa arrumar aí é o MD5 do uniquid. A única coisa que o uniqid faz de bom, é conseguir um Id único. Fazendo MD5 dele, acabou de perder isso. Seria bom revisar essa etapa.

Se preferir, pode usar multiple="multiple", para não ficar um atributo vazio:

<input id="file-upload" name="files[]" type="file" multiple="multiple">
Fonte Link
Largato
  • 92,9mil
  • 18
  • 190
  • 261

Primeiro, não seria o caso de usar [] quando for um input só (não confundir com arquivos múltiplos):

<input id="file-upload" name="files" type="file" multiple>

Depois, seria o caso de testar se a variável contém algo:

$uploads_dir = 'images/';
$img = array();
if( isset( $_FILES['files'] ) ) {
   $fdata = $_FILES['files'];
   $count = count( $fdata['name'] );
   for ($i = 0; $i < $count; ++$i ) {
      $ext = pathinfo( $fdata['name'][$i], PATHINFO_EXTENSION);
      $imagem_nome = md5(uniqid(time())).'.'.$ext;
      $temp_name = $fdata['tmp_name'][$i];
      move_uploaded_file($temp_name, $uploads_dir.$imagem_nome );
      $img[] = $imagem_nome;
   }
}

Aproveitei e troquei a RegEx da extensão por uma função apropriada.

O que precisa arrumar aí é o MD5 do uniquid. A única coisa que o uniqid faz de bom, é conseguir um Id único. Fazendo MD5 dele, acabou de perder isso. Seria bom revisar essa etapa.