Be part of JetBrains PHPverse 2026 on June 9 – a free online event bringing PHP devs worldwide together.
Could you tell me, please, is it possible to add an "if" condition to the query? I have a request:
public static function getAllAvailableAttributes ($categoryTitle, $attrBrand, $attrColor) {
return static::select(['brand', 'color', 'size'])
->where([
['stock', 'in-stock'],
['brand', '!=', ''],
['color', '!=', ''],
])
->where(function ($query) use ($categoryTitle) {
foreach ($categoryTitle as $item) {
$query->orWhere('category_title', $item);
}
})
->where(function ($query) use ($attrBrand) {
foreach ($attrBrand as $id => $item) {
$query->orWhere('brand', $item->title);
}
})
->where(function ($query) use ($attrColor) {
foreach ($attrColoras $id => $item) {
$query->orWhere('color', $item->title);
}
})
->get();
But the problem is that $attrBrand and $attrColor values do not always come. Sometimes they are null. how can I check if the value is null or not before running the condition? This method is certainly not correct, but for clarity, I will show what I need:
if($attrColor != null) {
->where(function ($query) use ($attrColor) {
foreach ($attrColoras $id => $item) {
$query->orWhere('color', $item->title);
}
})
}
Use the when Builder method, e.g.
->when($attrBrand, (function ($query, $attrBrand) {
foreach ($attrBrand as $brand) {
$query->orWhere('brand', $brand->title);
}
})
https://laravel.com/docs/9.x/queries#conditional-clauses
I assume the parameter should have a default if it is not always passed???
->where(function ($query) use ($attrColor) {
if ($attrColor!= null) {
foreach ($attrColoras $id => $item) {
$query->orWhere('color', $item->title);
}
}
})
Can it be done this way? I did and it seems to have worked.
@Rediska what is $attrVozrast - it is not even in scope?
Edit: you changed it... so if $attrColor is an array; an empty array, then even the if block is unnecessary.
@tykus I made a typo.
@Rediska you don't need the if block if you ensure the $attrColor variable defaults to an empty array.
public static function getAllAvailableAttributes ($categoryTitle, $attrBrand, $attrColor = []) {
@tykus I tried to follow your example and got an error: ErrorException Invalid argument supplied for foreach()
Default value is always null
@Rediska so you're literally passing null into the static method?
@tykus Yes, I made a mistake. If you fix it to [], then everything works correctly! This is about me =) Thank you all!
Note that all your where statements are AND conditions. Is this what you intended?
@Snapey I'm still trying to figure out the product filter. I determine from the URL which filters were requested. And in order not to display empty attributes, I'm trying to make such a check =)))
You can do it like this
$query = static::select(['brand', 'color', 'size'])
->where([
['stock', 'in-stock'],
['brand', '!=', ''],
['color', '!=', ''],
]);
if($request->categoryTitle)->filled()) {
$query->where(function ($query) use ($categoryTitle) {
foreach ($categoryTitle as $item) {
$query->orWhere('category_title', $item);
}
});
if($request->attrBrand)->filled()) {
$query->where(function ($query) use ($attrBrand) {
foreach ($attrBrand as $id => $item) {
$query->orWhere('brand', $item->title);
}
});
if($request->attrColor)->filled()) {
$query->where(function ($query) use ($attrColor) {
foreach ($attrColor as $id => $item) {
$query->orWhere('color', $item->title);
}
})
return $query->get();
So, building up the query if filters are set.
What @tykus is saying is that there is a when method that does the same but allows it to be constructed as one fluent eloquent statement;
return static::select(['brand', 'color', 'size'])
->where([
['stock', 'in-stock'],
['brand', '!=', ''],
['color', '!=', ''],
])
->when($request->categoryTitle)->filled()) {
$query->where(function ($query) use ($categoryTitle) {
foreach ($categoryTitle as $item) {
$query->orWhere('category_title', $item);
}
})
->when($request->attrBrand)->filled()) {
$query->where(function ($query) use ($attrBrand) {
foreach ($attrBrand as $id => $item) {
$query->orWhere('brand', $item->title);
}
})
->when($request->attrColor)->filled()) {
$query->where(function ($query) use ($attrColor) {
foreach ($attrColor as $id => $item) {
$query->orWhere('color', $item->title);
}
})
->get()
Note I notice your filters look off? item on category and item->name on the others? What does your form return as filters? Surely just an array of selected filters or checkboxes?
Please or to participate in this conversation.