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

pagination in the backend generates wrong URLs #102

Closed
fiammybe opened this Issue May 17, 2017 · 3 comments

Comments

Projects
2 participants
@fiammybe
Copy link
Member

fiammybe commented May 17, 2017

In ImpressCMS 1.3.10, I go for example to the blocks page. There, I see that the blocks take 4 pages. When I click on the '2' to go to the second page, I get thrown back to the ACP front page.

When I do the same on 1.3.9, I am merrily taken to the second page.

The problem was checked on other pages with pagination, and is present on 1.3.10, but not on 1.3.9.

@fiammybe fiammybe added the bug label May 17, 2017

@fiammybe

This comment has been minimized.

Copy link
Member

fiammybe commented May 17, 2017

When I investigate this a bit more, the URL generated by the 1.3.9 site is
"/modules/system/admin.php?fct=blocksadmin&sortsel=title&ordersel=ASC&limitsel=15&startbid=15"

whereas the URL generated by 1.3.10 is
"/modules/system/admin.php?sortsel=title&ordersel=ASC&limitsel=15&startbid=15"

It seems the fct=blocksadmin is missing (and the fct=somethin in the other cases as well.

@skenow

This comment has been minimized.

Copy link
Contributor

skenow commented Jun 25, 2017

It must be something from the added filtering in 1.3.10 - I'll sort it out.

@skenow skenow self-assigned this Jun 25, 2017

@skenow skenow added this to the v1.3.11 milestone Jun 25, 2017

skenow added a commit to skenow/impresscms that referenced this issue Jun 26, 2017

@fiammybe fiammybe added this to In Progress in v1.3.11 Jun 26, 2017

fiammybe added a commit that referenced this issue Jun 27, 2017

Merge pull request #104 from skenow/dev/1.3.11
Adding admin function to allowed input variables test #102.
@fiammybe

This comment has been minimized.

Copy link
Member

fiammybe commented Jun 27, 2017

Ok for 1.3.11

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment