New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Search REST API returns wrong total_count #17295
Comments
Hi @mohamed-karam. Thank you for your report.
Please make sure that the issue is reproducible on the vanilla Magento instance following Steps to reproduce. To deploy vanilla Magento instance on our environment, please, add a comment to the issue:
where @mohamed-karam do you confirm that you was able to reproduce the issue on vanilla Magento instance following steps to reproduce?
|
Hi @mohamed-karam I'm not able to reproduce this issue by following the steps you provided. |
@mohamed-karam you need to use together, like this-> |
I think I need to elaborate more. If this search query for example has 15 items, the |
@mohamed-karam in your case currentPage=1 pageSize=10, so if you have a 100 result items it be a -> |
That isn't what happens with other endpoints that have searchCriteria. For example if you call this endpoint |
@mohamed-karam , thank you for your report . |
@engcom-backlog-andrii Please Update? I Will also face this issue. |
Hi @ronak2ram. Thank you for working on this issue.
|
@magento-engcom-team give me 2.3-develop instance |
Hi @ronak2ram. Thank you for your request. I'm working on Magento 2.3-develop instance for you |
Hi @ronak2ram, here is your Magento instance. |
Hello @magento-engcom-team |
Dear Ronak Patel : Please Changed url and check on rest/V1/search.
…On Sat, Jan 12, 2019 at 12:20 PM Ronak Patel ***@***.***> wrote:
Hello @magento-engcom-team <https://github.com/magento-engcom-team>
I can't find this issue in 2.3-develop instance.
[image: screenshot from 2019-01-12 12-21-22]
<https://user-images.githubusercontent.com/11473750/51070511-7f51fc00-1668-11e9-8ac9-0cd6529e04fd.png>
[image: screenshot from 2019-01-12 12-20-52]
<https://user-images.githubusercontent.com/11473750/51070512-7fea9280-1668-11e9-88d4-d8fbbec25dc6.png>
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#17295 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/Ano0DzQyzzEIAqvUUyCuI183R-IM8xc2ks5vCYyxgaJpZM4VosU6>
.
|
@abdulmalik422 You right there is an issue. Let me try to find a solution |
Hi @mohamed-karam. Thank you for your report.
The fix will be available with the upcoming 2.3.2 release. |
Preconditions
Steps to reproduce
GET /V1/search
and providesearchCriteria[pageSize]
.searchCiteria[currentPage]
= 2Expected result
searchCriteria[pageSize]
in the first timesearchCriteria[pageSize]
in the second timetotal_count
is the same on both requestsActual result
The value of
total_count
equalssearchCriteria[pageSize]
not the total count of the search result items.The text was updated successfully, but these errors were encountered: