#5187 closed defect (bug) (fixed)
Search Widget Redirect Problem
Reported by: |
|
Owned by: | |
---|---|---|---|
Milestone: | 2.7 | Priority: | low |
Severity: | minor | Version: | 2.2.3 |
Component: | Widgets | Keywords: | needs-patch dev-feedback |
Focuses: | Cc: |
Description ¶
On my install, bloginfo('url') does not return a trailing slash so when I use the default search widget, the form action url is malformed. The search still works but it redirects to www.mydomain.com rather than mydomain.com. My patch simply added a trailing slash after the bloginfo call. Diff attached.
Pull Requests
- Loading…
Change History (5)
#1
@
16 years ago
- Keywords needs-patch 2nd-opinion added
I'm not convinced that the best thing to do here is to patch the search widget. After all, the issue is that the blog url is not being returned with the trailing slash. IMO It'd be better to make bloginfo("url") return with a trailing slash - especially so since that's where the browser gets redirected to.
(In general though the situation still applies, though I'm not convinced it is the actual cause of the problem, or even that this is a problem, as the search form works fine for me regardless..)
Still current at 2.7
#2
@
16 years ago
- Keywords dev-feedback added; 2nd-opinion removed
err... in the latest 2.7 branch, get_search_form() users get_option('home') . /
we can probably close this one as invalid or fixed.
Search Widget Patch