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.

Sign up for GitHub

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

Jump to bottom

Using "q" as the Search Index triggers a 404 error #24

Open
beau-gosse opened this issue Dec 8, 2015 · 1 comment
Open

Using "q" as the Search Index triggers a 404 error #24

beau-gosse opened this issue Dec 8, 2015 · 1 comment

Comments

@beau-gosse
Copy link

SimpleSearch doesn't accept q as the search index. The 404 error page is thrown instead of showing results.

Steps to reproduce

  1. Set the &searchIndex property to q in both SimpleSearch and SimpleSearchForm.
  2. Clear the cache, perform a search in the front end.

Why would one use q instead of search as index? IMO it's visually more efficient and it looks less redundant. /search?q=modx+is+awesome+ VS /search?search=modx+is+awesome+ . See what I mean?
@bwente
Copy link

bwente commented Apr 27, 2016

I believe it is a conflict with the system setting.

Request Alias Parameter I changed mine to "mq" to avoid the conflict.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@bwente @beau-gosse