0

Tengo una vista en blade, con una tabla, que a través de un método, me carga mediante un foreach todos los datos de la base de datos, que son solicitudes, esas solicitudes tienen que ser aprobadas(cambiando el campo de la base de datos aceptado, a 1) o denegadas, lo cual eliminaría la solicitud sin mas, mi problema viene en que no soy capaz de pasar al controlador los parámetros necesarios para que actualice la solicitud y desaparezca, he probado distintas formas y visto muchos posts , pero ningún se adecua exactamente a lo que yo pretendo hacer, aquí les pongo el código:

Vista:

 @if($vacations)
                  <table class="table table-striped table-bordered scrolling-dataTable">
                    <thead>
                      <tr>
                       <th>Nombre</th>
                       <th>Tipo</th>
                       <th>Fecha inicio</th>
                       <th>Fecha fin</th>
                       <th>Observaciones</th>
                       <th>Pendiente</th>
                       <th>Aprobar</th>
                      </tr>
                    </thead>
                    <tbody>
                    @foreach ($vacations as $vacation)
                    <tr>
                      <td>{{ $vacation->name }}</td>
                      <td>{{ $vacation->type }}</td>
                      <td>{{ $vacation->date_from }}</td>
                      <td>{{ $vacation->date_to }}</td>
                      <td>{{ $vacation->observations }}</td>
                      <td>{{ $vacation->aceptado }}</td>

                        <td>  <!-- Single button definir boton para aprobar vacaciones -->
                        <div role="group" class="btn-group">
                          <button id="btnGroupDrop1" type="button" data-toggle="dropdown" aria-haspopup="true" aria-expanded="false" class="btn btn-outline-primary dropdown-toggle dropdown-menu-right"><i class="icon-cog3 icon-left"></i> Settings</button>
                          <div aria-labelledby="btnGroupDrop1" class="dropdown-menu">
                            <a href="{{ url ('/vacation/aceptado',[$vacation->id]) }}" class="dropdown-item">Aprobar</a>
                            <a href="{{url('/vacation/deny')}}" class="dropdown-item">Denegar</a></div>
                          </div>
                         </div>
                       </td>
                   </tr>
               @endforeach

Controlador:

 public function update(Request $request, Vacation $vacations)
{
     $this->validate($request, [
        'aceptado' => 'required',
    ]);
    //$inputs=Input::all();
    //$vacation = new Vacation();
    $vacation=Vacation::findOrFail($id);
    $vacation-> aceptado = '1';
    //$vacation-> id = $inputs['id'];

    $vacation->save();

    Mail::send("correo.solicitud", $data, function($message) use ($data){
        $message->to('email','user')
        ->subject("Respuesta a su solicitud");
    });

    return redirect('/vacation/calendar');

}

y la ruta:

 Route::post('/vacation/update', 'VacationController@update')->name('aceptado');

cuando le doy al botón para aceptar la solicitud me devuelve:

NotFoundHttpException in RouteCollection.php line 161:

2
  • no entiendo en la linea del controlador "public function update(Request $request, Vacation $vacations)" esto Vacation $vacations y despues "$vacation=Vacation::findOrFail($id);" de donde sacas la variable $id?? deberia ser public function update(Request $request, $id) el 30 ene. 2018 a las 15:47
  • LO he corregido segun me dijo Dev. Joel tal cual su respuesta, gracias por contestar
    – Peisou
    el 30 ene. 2018 a las 16:13

2 respuestas 2

2

El metodo correcto para el update deberia de ser PUT

Route::put('/vacation/{id}', 'VacationController@update')->name('aceptado');

En Blade , el formulario debe de tener tanto el CRSF y el verbo correcto PUT

{!! Form::open(array('url' => ['/vacation/update'], 'method' => 'PUT', 'class' =>'form-horizontal')) !!}

{{ csrf_field() }} 

Y en el controlador

public function update(Request $request,$id) {...}
4
  • Hola Unai. Una respuesta con muy buen formato ^^ Recuerda leer el recorrido de bienvenida para aprender como funciona el sitio y llevarte otra medalla. Un saludo
    – lois6b
    el 31 ene. 2018 a las 12:25
  • 1
    Gracias @lois6b, estoy intentando aprender ha usar esto. el 31 ene. 2018 a las 12:28
  • Lo probare unai muchas gracias por tomarte la molestia, saludos!
    – Peisou
    el 1 feb. 2018 a las 13:38
  • @UnaiSusperregi he probado a añadirlo, pero sigue sin funcionarme, como le comentaba a Dev Joel debo de tener algún problema en el método del controlador.
    – Peisou
    el 5 feb. 2018 a las 15:21
1

El error es porque no encuentra una ruta con esa URL , como estás definiendo la ruta con name al final, puedes hacer uso de esta pero le hace falta pasarle el parámetro $id a la ruta , Al hacer click en el enlace , este se envía por GET , y no por POST, debería cambiar el verbo desde la ruta.

Route::get('/vacation/update/{id}', 'VacationController@update')->name('aceptado');

En tu vista llamarías a esta ruta , con el helper route pasando el parámetro, nombre de ruta y el $id

<a href="{{ route('aceptado',$vacation->id)}}"
         class="dropdown-item">Aprobar</a>

Y en tu controlador deberías recibir este Id como segundo parámetro

public function update(Request $request,$id) {...}
9
  • Gracias por tu pronto comentario, lo he probado pero basicamente me sigue diciendo lo mismo NotFoundHtttpException
    – Peisou
    el 30 ene. 2018 a las 13:56
  • @DavidPazóLópez actualicé mi respuesta, tiene que cambiar el verbo de la ruta, verifique 3:)
    – Dev. Joel
    el 30 ene. 2018 a las 13:58
  • gracias por la correcion, lo he probado y nada sigue igual
    – Peisou
    el 30 ene. 2018 a las 14:11
  • Haga un dd($id) en el controlador, llega el valor? Lo he probado localmente y funciona, llega el id al controlador.
    – Dev. Joel
    el 30 ene. 2018 a las 14:15
  • me salta directamente al error, no tengo manera de visualizar ese dd, sera posible que en local se me haya quedado enganchado el xampp?(es el que uso, y es por decirlo de alguna manera..)
    – Peisou
    el 30 ene. 2018 a las 14:17

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.