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

search tool is building an outdated search URL #394

Closed
schplurtz opened this issue Oct 21, 2018 · 1 comment
Closed

search tool is building an outdated search URL #394

schplurtz opened this issue Oct 21, 2018 · 1 comment

Comments

@schplurtz
Copy link
Contributor

Description

Hi, this is a 2 in one issue.

bootstrap3_searchform() builds an outdated search URL. The URL currently built by bootstrap3 uses id=searched-test&do=search. Current DokuWiki needs q=searched-text&do=search. DokuWiki itself detects this condition and sends a redirect, so most of the time, this goes unnoticed. There are conditions where this can lead to DokuWiki not searching anything, but just showing the start page.

This has been discussed on DW forum, thread 16375 with andi @splitbrain.

Also, this same function bootstrap3_searchform() "looses" the current page, since it redirects to the start page of the wiki. Say you are viewing page "some:page", then search something, you get to page "start" and showing search results. Current DokuWiki with its default template would leave you on page "some:page" and show search results.

Steps to reproduce

  1. Install fresh DokuWiki version greebo
  2. Add bootstrap3 template, and activate it.
  3. use the search tool

Expected behavior:
browser should send page:id?q=searched-text&do=search (or if not using nice URL doku.php?id=page:id&q=searched-text&do=search)

Actual behavior: [What actually happened]
browser sends start?id=searched-text&do=search

Versions

  • [Bootstrap3 Template] current
  • [DokuWiki] greebo
  • [Plugins] a lot...
  • [Browser] FF 62.0.3

Screenshots or Logs

image

@giterlizzi
Copy link
Owner

Great job, PR merged 👍

Thanks a lot!

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

2 participants