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">