1

Bom to com um problema aqui que ja tentei resolver de varias maneiras sem exito. Segue parte que esta pegando

$stmt->execute();  -------> final do insert até aki ok
if (DB::lastInsertId()) {
// pego o ultimo id
$lastId = DB::lastInsertId();
// crio o diretorio
mkdir ("../img/$lastId", 0755 ); ---> ate aqui ok o diretorio e criado usando o ultimo id
---------------------> a partir daqui que esta o problema
// upload e update
// altero o nome
$filename = time() . '_' . $_FILES["imagem"]["name"];
$diretorio = '../img/$lastId/';
$filepath = '$diretorio' . $filename;
move_uploaded_file($_FILES["imagem"]["tmp_name"], $filepath)
// insere no bd
$sqlInsert2 = "UPDATE noticias SET imagem=$filename WHERE idnoticia=$lastId";
$stmt = DB::prepare($sqlInsert2);
$stmt->bindParam("imagem", $filename);
$stmt->execute();


 setMessage("Notícia $lastId $filename cadastrado com sucesso.");
    redirect("noticiasListar.php");

os demais itens das noticias estao sendo inseridos, a pasta esta sendo criada, só não esta fazendo o upload da imagem e nem o update da imagem no bd

alguma solução?

1
  • 1
    Se cada noticia tem uma imagem única, por que não salva o arquivo nomeado com o id da noticia, ou ainda dentro da pasta que você criou usando o id, porém com um nome padrão? 26/02/2015 às 20:20

1 Resposta 1

3

O problema está aqui:

$diretorio = '../img/$lastId/';
$filepath = '$diretorio' . $filename;

Repare nas aspas.

Ou você troca para aspas duplas ou concatena:

$diretorio = '../img/' . $lastId . '/';
$filepath = $diretorio . $filename;

Tem que colocar aspas na query e concatenar também:

$sqlInsert2 = 'UPDATE noticias SET imagem = "' . $filename . '" WHERE idnoticia = "' . $lastId . '"';
9
  • vou efetuar a alteração aqui e qq coisa retorno 26/02/2015 às 20:05
  • 1
    Ótimo. Não sei se tem como pegar o próximo ID, mas mesmo se tiver, não acho uma boa prática porque alguém pode inserir um registro após pegar o próximo ID e antes de inserir na tabela
    – Lucas
    26/02/2015 às 20:19
  • 2
    Eu costumo usar transaction (php.net/manual/pt_BR/pdo.begintransaction.php) para evitar inconsistencias no banco de dados 26/02/2015 às 20:24
  • 1
    @ArsomNolasco por que não pega o nome do arquivo antes do insert? 26/02/2015 às 20:26
  • 1
    @ArsomNolasco no seu caso pode pegar antes, você não faz nenhuma validação se o diretório foi criado e o arquivo movido ou não, porém deveria fazer tais. Ps: quando direcionar a resposta para alguém marque a mesma, assim aparecerá uma notificação na inbox. 26/02/2015 às 21:03

Você deve fazer log-in para responder a esta pergunta.

Esta não é a resposta que você está procurando? Pesquise outras perguntas com a tag .