Lo que intenté hacer es lo siguiente:
Si el usuario es "Admin" muestro todas las plantas.
Si el usuario es "User1" muestro únicamente su planta.
Si no se ha selecciona ningún área, selecciono la "0", la cual pertenece a la opción "All Areas" de mi formulario.
Si se ha seleccionado "All Areas", cogo todas las áreas de la planta seleccionada y luego todos los equipos de esas áreas.
Es obligatorio seleccionar una planta (no existe la opción "0", es decir, "All Plants").
¿Se puede mejorar este código? a simple vista no me parece muy profesional, me gustaría que me den consejos para poder aprender.
public function index(Request $request)
{
$plant = Plant::query();
$areas = Area::query();
$areas_id = [];
$area_id = 0;
$plant_id = 0;
$equipment;
if(Auth::user()->hasRole('admin')){
$plant_id = (!empty($request->plant_id))? $request->plant_id: $plant->get()->first()->id;
$area_id = (!empty($request->area_id))? $request->area_id: 0;
}
elseif(Auth::user()->hasRole('user1')){
$plant_id = Auth::user()->plant->id;
$area_id = (!empty($request->area_id))? $request->area_id: 0;
}
if(!empty($plant_id)){
$areas_id = DB::table('areas')->select('id')->where('plant_id', $plant_id)->get();
$areas_id = json_decode(json_encode($areas_id), true);
}
//Lleno los select de mi formulario
$plants = Plant::get();
$areas = $areas->where('plant_id', $plant_id)->get();
if(Auth::user()->hasRole('user1')){
$plants = Plant::where('id', Auth::user()->plant->id);
}
if($area_id <> 0)
$equipment = DB::table('equipment')->whereIn('area_id', $areas_id)->paginate(10);
else
$equipment = Equipment::latest()->paginate(10);
return view('equipment.index', compact('equipment', 'areas', 'plants', 'plant_id', 'area_id'));
}