1

Amigos, me podrían orientar con este problema que tengo, estoy recibiendo un array, pero cuando quiero actualizar los datos a mi base me arroja el siguiente error:

Attempt to assign property 'stock' of non-object

Mi código es el siguiente:

if($det=$request->detalles) {
    foreach ($det as $detalle=> $valor) {
        $detalle=Detalle::where('art_id', '=', $idArt)
            ->where('codigo_id', '=', $valor)
            ->exists();
    }
}

if($detalle==""){
    $doNeg=false;
    $errmesg="No existe articulo";
}

$detalle->stock=1;
$detalle->updated_at = Carbon::now();
if (Auth::guard('supervisor')->check()) {
    $detalle->admin_id= Auth::id();
} elseif (Auth::guard('bodeguero')->check()) {
    $detalle->admin_id= Auth::id();
}
$detalle->save();

Mi problema es cuando apretó el botón guardar y me dice que Attempt to assign property of non-object, ayuda porfis, porque no se en donde me caigo para poder actualizar mis datos

4
  • El error parece indicar que tienes un typo en algun lugar. No sera stock en lugar de sotck?
    – alanfcm
    Commented el 2 ago. 2018 a las 16:39
  • Si es stock, he modificado el texto, pero sigo sin encontrar cual es le error que hace que se me caiga y no guarde
    – Karli
    Commented el 2 ago. 2018 a las 16:40
  • si imprimes el array en el controlador que te muestra? Commented el 2 ago. 2018 a las 17:24
  • En el foreach pareces hacer una cosa algo rara: el índice se llama $detalle y se asigna una variable llamada $detalle a partir del valor. ¿No estará eso creando problemas? ¿Dónde se define la variable $detalle? ¿Cuál es su ámbito?
    – Alvaro Montoro
    Commented el 2 ago. 2018 a las 17:30

2 respuestas 2

2

Cuando estás llamando a $detalle->stock, $detalle no existe (a menos que estés omitiendo alguna parte del código).

En tu ciclo:

if($det=$request->detalles) {
    foreach ($det as $detalle=> $valor) {
        $detalle=Detalle::where('art_id', '=', $idArt)
            ->where('codigo_id', '=', $valor)
            ->exists();
    }
}

Lo único que haces es verificar si existe un Detalle que cumpla con las condiciones solicitadas y guardarlo en $detalle como true o false, porque exists() te devuelve verdadero o falso.

Si lo que quieres es instanciar tu $detalle con un tipo Detalle, tendrías que hacer:

$detalle=Detalle::where('art_id', '=', $idArt)
                ->where('codigo_id', '=', $valor)
                ->first();

Esto te devolverá una instancia de Detalle o NULL si no es encontrado.

Pero aún así, como lo haces dentro de un foreach, igualmente perderás el valor de $detalle al momento de terminar el ciclo. Tendrías que llamar a alguna función dentro del mismo ciclo después de instanciar $detalle. Por ejemplo:

if($det=$request->detalles) {
    foreach ($det as $detalle=> $valor) {
        $detalle=Detalle::where('art_id', '=', $idArt)
            ->where('codigo_id', '=', $valor)
            ->first();
        if($detalle) {
            self::hacerLoQueTienesQueHacer($detalle);
        }
    }
}
public static function hacerLoQueTienesQueHacer($detalle)
{
    $detalle->stock=1;
    $detalle->updated_at = Carbon::now();
    if (Auth::guard('supervisor')->check()) {
        $detalle->admin_id= Auth::id();
    } elseif (Auth::guard('bodeguero')->check()) {
        $detalle->admin_id= Auth::id();
    }
    $detalle->save();
}

Creé una función estática porque no tengo idea de dónde estás parado en tu código, pero si lo que quieres hacer es una tarea común de Detalle, lo ideal sería que la clase Detalle tuviera una función llamada hacerLoQueTienesQueHacer y en el ciclo simplemente harías:

if($det=$request->detalles) {
    foreach ($det as $detalle=> $valor) {
        $detalle=Detalle::where('art_id', '=', $idArt)
            ->where('codigo_id', '=', $valor)
            ->first();
        if($detalle) {
            $detalle->hacerLoQueTienesQueHacer();
        }
    }
}
2
  • Julio muchas gracias, mi error precisamente estaba en utilizar ->exists();, al modificarlo por un first(), me realizó los cambios que quería realizar. Es algo que jamás se me volverá a olvidar. Gracias nuevamente
    – Karli
    Commented el 2 ago. 2018 a las 19:22
  • Genial, me alegro de haberte podido ayudar, saludos.
    – Julio Lago
    Commented el 3 ago. 2018 a las 1:55
-1

he verificado que se debe incluir, en este caso; el

$detalle->save();

dentro del condicional if else

saludos

1
  • Tu respuesta es solo codigo. Por favor mira Cómo responder. Se pide que expliques que hace este código (mas alla que a ti vos te parezca obvio, puede que otro usuario no lo vea tan simple).
    – gbianchi
    Commented el 1 dic. 2020 a las 13: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.