Lo que posiblemente esté haciendo que la consulta ignore las demás cláusulas where
, es que tienes un orWhere
en el ámbito global de la consulta. Me refiero a esta línea:
$ads = $ads->where('title', 'like', '%'.$request->b.'%')
->orWhere('description', 'like', '%'.$request->b.'%');
Eso va a hacer que independientemente de lo que pongas en otras cláusulas, la consulta te devuelva todos los registros que cumplan la condición orWhere
.
Para resolverlo puedes agrupar esa cláusula orWhere
. Puedes hacerlo así:
$ads = $ads->where(function($query) use ($request) {
$query->where('title', 'like', '%'.$request->b.'%')
->orWhere('description', 'like', '%'.$request->b.'%');
});
Referencia: Parameter Grouping
Específicamente esta nota: "Siempre debe agrupar las llamadas orWhere
para evitar comportamientos inesperados cuando se aplican ámbitos globales."
Como sugerencia, para no poner if
s la consulta, puedes hacerlo de la "forma laravel" con Conditional Clauses.
En ocasiones, es posible que desee que se apliquen determinadas cláusulas de consulta a una consulta en función de otra condición. Por ejemplo, es posible que solo desee aplicar una instrucción
where
si un valor de entrada dado está presente en la solicitud HTTP entrante. Puede lograr esto usando el métodowhen
.
Por ejemplo:
$b = $request->b;
$departamento = $request->departamento;
$provincia = $request->provincia;
$distrito = $request->distrito;
$ads = Ad::where('adtype_id', $b_tipo)->orderBy('date_created', 'DESC')
->when($b, function ($query, $b) {
return $query->where(function($q) use ($b) {
$q->where('title', 'like', '%'.$b.'%')
->orWhere('description', 'like', '%'.$b.'%');
});
})
->when($departamento, function ($query, $departamento) {
return $query->where('department_id', '=', $departamento);
})
->when($provincia, function ($query, $provincia) {
return $query->where('province_id', $request->provincia$provincia);
})
->when($distrito, function ($query, $distrito) {
return $query->where('district_id', $distrito);
})
->paginate(10);