0

Quiero validar que otros usuarios no puedan editar un producto si no les pertenece estoy usando políticas de laravel pero tengo que verificar que empresa esta usando ese usuario y luego si validar si a esa empresa pertenece el producto este es mi código

public function update(User $user, Productos $productos)
    {
        $id = Auth::id();
        $id_empresa = Empresa::where('fk_id_user', $id)->get();
       $empresa = $id_empresa['id_empre'];
        return $empresa == $productos;
    }

pero siempre marca que no tengo autorización.

8
  • No veo dónde está definida la variable $producto.
    – Shaz
    Commented el 27 mar. 2018 a las 21:43
  • el no debería recibir la variable del el método del controlador Commented el 27 mar. 2018 a las 22:00
  • $productos= Productos::find($request->id_p); $this->authorize('update',$productos); ]); Commented el 27 mar. 2018 a las 22:01
  • Estoy hablando de $producto, en singular, no de $productos
    – Shaz
    Commented el 27 mar. 2018 a las 22:10
  • fue un error aunque ya lo corregí y sigue sin funcionar Commented el 27 mar. 2018 a las 22:13

2 respuestas 2

1

Puede ser varias cosas,

1- El primer problema es en empresa el get() te devuelve una colección de modelos. Para obtener el modelo usa first().

2- $productos , imagino, que es una colección de modelos, lo mismo hay que seleccionar el producto first() y comparar solo el id de empresa, yo le he llamado id_empre, pero no se si producto tiene ese atributo

public function update(User $user, Productos $productos)
    {
        $empresa = Empresa::where('fk_id_user', \Auth::id())->first(); 
        return $empresa->id_empre == $productos->first()->id_empre;
    }
0

Hay que fijarse que dependiendo de la relación si es una hasMany-belongsTo (1:N)(N:1) o belongsToMany (N:M).

Where Has , no usar el id a pelo

Eloquent con whereHas, si están bien formadas las relaciones en un caso accederá a la tabla del usuario , o a la tabla pivote , de forma automññatica gracias a eloquent , por eso es más recomendable usar whereHas que hacerlo con id's a pelo , porque si cambias la relación por requisitos del cliente , tienes que rehacer las consultas , mientras que con whereHas no tienes porqué , solamente la forma de asociarlo tanto con $usuarios->empresa()->attach($empresa) o $usuario->empresa()->associate($empresa);.

public function update(User $user, Productos $productos)
{
    $empresa = Empresa::whereHas('usuario',function($qUsuario){
       $qUsuario->where('id_user',Auth::id());
    })->first();

    $productoEmpresa = $productos::whereHas('empresa',function($qEmpresa) use ($empresa){
       $qEmpresa->where('id_empre',$empresa->id_empre);
    })->first();
    if($empresa == null || $productoEmpresa == null){
      return false;
    }else{
      return $empresa->id_empre == $producto->id_empre;
    }
}

Podra haber problemas de foraneas y demás , que no se el nombre de dichos ids , pero espero que cogas el concepto.

First y get

Con first, llamas al primer elemento existente , si es una relación de varios , cogerás el primero de los N-elementos que encuentres en la busqueda. Con get, como dice mi compañero coges todos los elementos y te lo devuelve en un array de colecciones.

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.