Level 16
Do it,
->orderByRaw('gp_products.position - gp_products.updated_at', 'DESC')
Sep 11, 2018
12
Level 1
Query is takes only first id
I wanted to show the products only from mentioned by using below query but which is showing only first one.
$productIds = "1,2,3,5,6,8";
$query = GP::table('gp_products')
->select('gp_products.*', 'gp_product_details.*', 'gp_product_status.*')
->join('gp_product_details', 'gp_product_details.product_id', '=', 'gp_products.product_id')
->leftJoin('gp_product_status', 'gp_product_status.product_id', '=', 'gp_products.product_id')
->whereIn('gp_products.product_id', [$productIds])
->where([
['gp_product_details.locale', '=', 'en_US'],
['gp_products.type', '=', 'dynamic'],
['gp_products.status', '=', 1]
])
->orderByRaw('gp_products.position - gp_products.updated_at DESC')
->get();
return $query;
Level 1
Thanks but the problem is here, because i removed that line all products are showing.
->whereIn('gp_products.product_id', [$productIds])
Level 54
You’ve defined your product ids as a string before passing that string to your query and wrapping it in brackets. Just create the array of ids first, and pass that in.
1 like
Level 16
Then do it,
$productIds = [1,2,3,5,6,8];
After that,
->whereIn('gp_products.product_id', $productIds)
1 like
Level 1
@NOMGUY I done that already same result.
Level 54
If you are only getting one record when passing in the array, check you actually have products with those ids in your database.
Installing a package such as Tracy will give you access to queries you are running in your UI which will allow scrutinise what’s running at db layer https://packagist.org/packages/recca0120/laravel-tracy
Level 1
@brightstormhq Yes, I have products.
Level 16
instead of
return $query;
try
echo "<pre>";
print_r($query);
die;
for getting the products. and tell what are you getting.
Level 1
I did following like said by @brightstormhq and it worked.
$productIds = "1,2,3,5,6,8";
$ids = explode(',', $productIds);
And
->whereIn('gp_products.product_id', $ids)
Level 54
@mkrahamath you can simplify the above by just setting your
$productIds = [1,2,3,5,6,8];
You are initially declaring a string of numbers and then turning that into an array when you could easily already define the array in the first place
Can you mark NOMGUYs answer as best answer so he picks up the credit for it?
Level 1
Thank you @NOMGUY @brightstormhq for answers.
Level 16
Thanks @brightstormhq No problem @mkrahamath
Please or to participate in this conversation.