0

Estou tentando fazer com que seja informado uma mensagem de sucesso em uma div ao ser realizado uma ação, porem não consegui ainda, o erro que aparece é:

(3/3) ErrorException Undefined variable: sucess (View: /opt/lampp/htdocs/projetoslaravel/clinica-beta/resources/views/layouts/app.blade.php) (View: /opt/lampp/htdocs/projetoslaravel/clinica-beta/resources/views/layouts/app.blade.php)

meu código está da seguinte forma no layout da pagina

@if(isset($errors) and count($errors) > 0)
     <div id="msg" class="alert alert-error">                
                <p>{{$error}}</p>
     </div>           
@elseif(isset($sucess) and count($sucess) > 0)
     <div id="msg" class="alert alert-sucess">                
                <p>{{$sucess}}</p>
     </div>
@endif

na controller esta da seguinte forma

public function ativar($id) 
{
    $apt = $this->ModelApartamento->find($id);
    $liberacao = $apt->verificarAtivar();
    if ($liberacao) {
        $dados = ['status' => 'L'];
        //dd($dados);
        $update = $apt->update($dados);
        if ($update) {
            return redirect()
                   ->back()
                   ->with('sucess','Apartamento ativado com sucesso'); 
        } else {
            return redirect()
                   ->back()
                   ->with('errors', 'Ocorreu um erro ao tentar ativar apartamento');
        }
    } else {
        return redirect()
                   ->back()
                   ->with('errors', 'Apartamento está com paciente no momento');
    }
}
1
  • A variavel não foi definida, ela é definida no controller se houver sucesso do update, então o correto no seu templete é fazer "se for error mostre senão mostre o sucesso" e você esta perguntando para erro e sucesso. 5/09/2017 às 19:35

1 Resposta 1

0

Existe várias formas de fazer um status, ao meu ver o seu é errado, inclusive porque o seu teste no if não testa corretamente as variáveis e poderia ser feito por exemplo da seguinte forma:

Crie uma variável de um classe predefinida no PHP, ou seja, stdClass, e no código do controller faça essas alterações:

public function ativar($id) 
{
    $apt = $this->ModelApartamento->find($id);
    $liberacao = $apt->verificarAtivar();

    $st = new StdClass;

    if ($liberacao) 
    {
        $dados = ['status' => 'L'];        
        $update = $apt->update($dados);
        if ($update) 
        {
            $st->status = true;
            $st->message = "Apartamento ativado com sucesso";
            return redirect()
                   ->back()
                   ->with('st', $st); 
        } 
        else 
        {
            $st->status = false;
            $st->message = "Ocorreu um erro ao tentar ativar apartamento";
            return redirect()
                   ->back()
                   ->with('st', $st);
        }
    } 
    else 
    {
        $st->status = false;
        $st->message = "Apartamento está com paciente no momento";
        return redirect()
                   ->back()
                   ->with('st', $st);
    }
}

e na View também faça essas alterações:

@if(session()->has('st'))
@if(!session()->get('st')->status)
    <div id="msg" class="alert alert-error">                
        <p>{{$st->message}}</p>
    </div>           
@else
    <div id="msg" class="alert alert-sucess">                
        <p>{{$st->message}}</p>
    </div>
@endif
@endif

poderia ainda ficar melhor na sua View da seguinte forma:

@if(session()->has('st'))
<div 
  id="msg" 
  class="@if(session()->get('st')->status){{'alert alert-sucess'}}@else{{'alert alert-error'}}@endif">
    <p>{{$st->message}}</p>
</div>
@endif

essa seria uma das formas.

Referencias

10
  • @GabrielRodrigues não neh! se for true ele inverte e cai no else que no caso da resposta é sucesso? correto?
    – novic
    5/09/2017 às 19:57
  • Não estou conseguindo pegar o valor dessa variavel na view , no caso ela esta redirecionando para a index.blade.php , tem alguma outra forma em que eu não precise ficar sempre criando uma instancia de StdClass? no caso estou pensando nisso para um sistema maior com muitos cruds e muitas mensagens de sucesso ou error 5/09/2017 às 20:13
  • @MárcioCésar não precisa mudar essa forma, talvez o contexto seu é maior e isso que atrapalha, por exemplo como você utiliza o código e porque está usando back()?
    – novic
    5/09/2017 às 20:17
  • esta ação ativar esta sendo mostrada na index , utilizo o back() para voltar a index() porem quando envio o with('st',$st) no template.blade.php ele diz que a variavel não esta definida o template esta sendo chamado no arquivo index tbm. 5/09/2017 às 20:20
  • tente assim eu edite @MárcioCésar (withInput) dá uma olhada na resposta
    – novic
    5/09/2017 às 20:23

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 .