Navigation Menu

Skip to content
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

Later pages of product lists won't show when they include deleted products #6005

Closed
ecerta opened this issue Feb 4, 2015 · 3 comments
Closed

Comments

@ecerta
Copy link
Contributor

ecerta commented Feb 4, 2015

I followed the instructions on this page to display our list of deleted and active products on our website. We have a large number of deleted projects, such that a page 2 is required to list all of them along with our current products. However, when I click the links for page 2 or later, I am redirected to a page that says "NO PRODUCTS FOUND, ADD ONE!"

The only way to get to page 2 or beyond showing these products seems to be manually adjusting the querystring. Here's the URL for page 1 (works automatically)

https://ourwebsite.com/admin/products?utf8=%E2%9C%93&q%5Bname_cont%5D=&q%5Bvariants_including_master_sku_cont%5D=&q%5Bdeleted_at_null%5D=1&q%5Bdeleted_at_null%5D=0&button=

Here's what comes up when I try to click the link for page 2 (doesn't work)

https://ourwebsite.com/admin/products?button=&page=2&q%5Bname_cont%5D=&q%5Bs%5D=name+asc&q%5Bvariants_including_master_sku_cont%5D=&utf8=%E2%9C%93

Here's how we manually adjust the querystring to get to page 2. We just added &page=2 to page 1's querystring (works)

https://ourwebsite.com/admin/products?utf8=%E2%9C%93&q%5Bname_cont%5D=&q%5Bvariants_including_master_sku_cont%5D=&q%5Bdeleted_at_null%5D=1&q%5Bdeleted_at_null%5D=0&button=&page=2

We're using spree 2-4-stable, and while we've put some custom development into our Spree store, none of it is anywhere near these pages.

TrangPham added a commit to TrangPham/spree that referenced this issue Feb 12, 2015
@TrangPham
Copy link

@ecerta I have created a test spec to reproduce the issue https://github.com/TrangPham/spree/pull/1/files

Though in my test if I check page.html after selecting "Show Deleted" I see that the pagination urls both don't have deleted_at_null=0 in the url. Thus even clicking on "1" in the pagination list returns "NO PRODUCTS FOUND, ADD ONE!"

I'm also not sure why your link has both deleted_at_null=1 and deleted_at_null=0

@ecerta
Copy link
Contributor Author

ecerta commented Feb 12, 2015

I'm no expert on Spree's API, so pardon me if this is a really off-base guess: maybe my example has deleted_at_null=1 and deleted_at_null=0 is because I have both non-deleted products and deleted products in my list, whereas your test only has deleted products.

@priyank-gupta
Copy link
Contributor

@ecerta This is not longer an issue with 3-0-stable and master branch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants