0

estoy actualizando el estado de un pedido, pero no se porque no consigo actualizar el registro, he hecho distintos formulario de edición, por ejemplo para editar un usuario y sin ningún tipo de problemas, pero aquí no hay forma, a ver si veis vosotros donde fallo.

Tengo una tabla que me arroja todos los resultados de los pedidos agrupados por un campo idpedido. En cada fila tengo un botón para poder ver el pedido, donde le paso por parámetro el idpedido para poder verlo. La ruta es la siguiente:

Route::get('admin/pedidos/ver_pedido/{idpedido}', 'AdminController@verPedido')->name('admin.ver_pedido');

Dentro de la ficha del pedido tengo un botón con un formulario para poder editar una columna del pedido.

@if ($numPedido->estado == 1 && $numPedido->factura == null)
   <form action="{{ route('admin.editarEstadoPedido', $numPedido->idpedido) }}" method="POST">
      @method('put')
      @csrf
      <input type="hidden" name="idpedido" value="{{$numPedido->idpedido}}">
      <button type="submit" class="btn facturar">Facturar</button>
   </form>
@endif

El if lo que hace es si el pedido esta sin facturar me aparece el botón, y si esta ya facturado no aparece.

Aquí la ruta para actualizar el pedido:

Route::put('admin/pedidos/{idpedido}', 'AdminController@editarEstadoPedido')->name('admin.editarEstadoPedido');

Y el controlador para actualizar:

public function editarEstadoPedido(Request $idpedido)
    {
        $tt = DB::table('orders')
            ->where('idpedido', '=', $idpedido)
            ->update([
                'factura' => '4234234'
             ]);
        $tt->save();

        return redirect('admin/pedidos');
    }

Por el momento le estoy pasando este valor 4234234 una vez me funcione ya generare lo que necesito. Hay que tener en cuenta que puede tener más de una fila el mismo número de pedido. Y bien me devuelve este error.

Call to a member function save() on int

introducir la descripción de la imagen aquí

2 respuestas 2

0

en el formulario en el action debes de enviarle el id del pedido no el campo idpedido

<form action="{{ route('admin.editarEstadoPedido', $numPedido->idpedido) }}"

y tu archivo de rutas

Route::put('admin/pedidos/{idpedido}', 'AdminController@editarEstadoPedido')->name('admin.editarEstadoPedido');

Como haces un metodo update en laravel del otro lado recibes dos cosas el Request y el idpedido

public function editarEstadoPedido(Request $request, $idpedido )
    {
   $idpedidoarr = Array($request->idpedido);
       //sabiendo que tienes un modelo de ORDER
       DB::update('update orders set idpedido = '. $idpedido .' where idpedido = ?', $idpedidoarr );

        return back();
    }

Con la intencion de que sigues correctamente y aprovechas al maximo laravel e construido o reconstrontruido la funcion

17
  • Pero aquí esta comparando el id autoincrementable, ese no es el que quiero comprobar @Dohko19 . Una pregunta, puede actualizar el registro y quedarse en la misma pagina?
    – Miguel
    Commented el 14 jul. 2020 a las 16:32
  • Solo harías un return back(); y es raro lo que pides ya que el Idpedido y siempre se busca por la PK y solo actualizas el campo que quieres
    – Dohko19
    Commented el 14 jul. 2020 a las 16:34
  • Piensa que al ser pedidos si piden más de una cosa se generan distintas filas, entonces están agrupados por un idpedido
    – Miguel
    Commented el 14 jul. 2020 a las 16:35
  • Entonces tu base de datos esta mal y deberias de hacer una relacion hasToMany para dichos pedidos, ya que si un pedido con diferentes cosas no deberias generar mas filas dentro de pedidos, sino una tabla aparte
    – Dohko19
    Commented el 14 jul. 2020 a las 16:36
  • Desconocia hacerlo de este modo que me comentas, seria interesante saberlo para proximos proyectos, este esta llegando a su fin
    – Miguel
    Commented el 14 jul. 2020 a las 16:37
0

La operación de base de datos lo estas haciendo a través del query builder. Cuando estas haciendo un update por esta via, no existe el metodo save. Al momento de llamar el update, esta va ejecutar la operación.

8
  • Hola @Philippe Thiers entonces entiendo que tengo que quitar $tt->save(); si es así probe ya, no me devuelve error pero no hace cambios
    – Miguel
    Commented el 14 jul. 2020 a las 16:04
  • puede que tu where de la query esten mal implementadas las condiciones, solo supongo que se llame id en vez de idpedido la columna en la DB Commented el 14 jul. 2020 a las 16:07
  • No estoy comparando el autoincrementable estoy comparando con otra columna que se llama idpedido
    – Miguel
    Commented el 14 jul. 2020 a las 16:07
  • me indicas la estructura de la tabla para poder ayudarte? Commented el 14 jul. 2020 a las 16:09
  • Te subi una captura! puede que no pueda comparar si no es una columna INT
    – Miguel
    Commented el 14 jul. 2020 a las 16:11

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.