0

Olá, nesse código abaixo eu faço um delete no banco de dados dos registros selecionados em uma lista de chekboxes.

Tenho um array de Ids que é montado da seguinte forma:

Array (
    [0] => 810
    [1] => 811
)

Em cima desse array eu faço um foreach busco cada registro e chamo o método de exclusão, código completo abaixo:

public function destroy_download($id = 0){
    $ids = ($id >= 1) ? array($id) : Input::get('ids');

    if( ! empty($ids) ){
        foreach ($ids as $id) {
            $download = Download::find($id);
            $download->delete();
        }
    }
}

Isso deveria funcionar corretamente, no entanto dá o seguinte erro:

Symfony \ Component \ Debug \ Exception \ FatalErrorException Call to a member function delete() on a non-object

na linha:

$download->delete();

É como se o registro não tivesse sido encontrado.

Ele remove o registro no banco de dados só que acontece esse erro, alguém consegue imaginar o que seja?

O Array está correto, já debuguei mas sempre dá isso.

2 Respostas 2

1

Tente o seguinte:

public function destroy_download($id = 0){
    $ids = ($id >= 1) ? array($id) : Input::get('ids');

    if( ! empty($ids) ){
        foreach ($ids as $id) {
            try {
                Download::findOrFail($id)->delete();
            } catch (ModelNotFoundException $e){
                continue;
            }
        }
    }
}

Esse código tentará excluir o id, caso não encontre esse registro, segue para o próximo.

Sugestão: Acho uma boa rever esse método, da maneira que está temos verificações desnecessárias da variável $ids

public function destroyDownloads(){
    $ids = Input::get('ids');

    // Não é um array? Aborta a função
    if(!is_array($ids)){
        return false;
    }

    foreach ($ids as $id){
        $this->deleteDownload($id);
    }

    // Ou returne um Response::make()
    return true;

}

private function deleteDownload($id){

    try {
        Download::findOrFail($id)->delete();
    } catch (ModelNotFoundException $e){
        // Caso queira verificar erros no futuro, basta verificar com um if
        // o retorno de $this->deleteDownload($id), se for falso o registro não existia
        return false;
    }

    return true;
}
5
  • Eu tentei usar o try catch, mas ele não lança uma exceção ModelNotFoundException. Ele dá erro no $download->delete(), e não entra no catch. Tá bizarro isso. Segue um printa pra você ver: imageshack.com/a/img600/2163/dfzi.png
    – Jarbas
    Commented 10/04/2014 às 17:40
  • O mais estranho é que se ao invés de eu chamar o método $download->delete(), eu fizer echo $download->name funciona, não dá erro algum, ele exibe os nomes.
    – Jarbas
    Commented 10/04/2014 às 17:48
  • @Jarbas tente postar um var_dump do objeto $download e postar o seu model Download
    – gmsantos
    Commented 10/04/2014 às 18:52
  • postei no pastebin. O Dump é do objeto sem chamar o ->delete(), porque ai não dá erro. Objeto $download: pastebin.com/dwGcStJg Model: pastebin.com/vSDGsxt7
    – Jarbas
    Commented 10/04/2014 às 19:03
  • Estranho... vou tentar reproduzir o erro aqui
    – gmsantos
    Commented 14/04/2014 às 11:19
0

Confirme se o ID do objeto existe, interpretando um pouco o resultado, parece a função Download::find($id); retorna uma string/null quando o Id não existe, consequentemente quando voce tenta chamar o método delete() gera esse erro.

Crie uma verificação que confirme se o ID realmente existe antes de chamar a delete().

3
  • Infelizmente não é isso, eu testei.
    – Jarbas
    Commented 9/04/2014 às 21:28
  • @Jarbas, atualizei a resposta Commented 9/04/2014 às 21:53
  • Rogerio, eu já fiz todos os testes possíveis, testei se o objeto existe e etc. Esses ids vem de um grid que lista os registros dessa tabela. O Problema é que ele deleta o registro que dá erro.
    – Jarbas
    Commented 10/04/2014 às 17:34

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 .