Skip to content

Commit

Permalink
the advanced_search input is present twice
Browse files Browse the repository at this point in the history
The second occurence is [there](https://github.com/plone/Products.CMFPlone/blob/master/Products/CMFPlone/browser/templates/search.pt#L64-L65).

I think the first one shouldn't be there. We have the advanced_query parameter in double with both False and True as a value if we keep them both.
  • Loading branch information
Gagaro committed Jul 7, 2016
1 parent 0f37d19 commit a1f15c0
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
5 changes: 4 additions & 1 deletion CHANGES.rst
@@ -1,4 +1,4 @@
.. This file should contain the changes for the last release only, which
[6~].. This file should contain the changes for the last release only, which
will be included on the package's page on pypi. All older entries are
kept in HISTORY.txt

Expand Down Expand Up @@ -156,6 +156,9 @@ Bug fixes:
- Use zope.interface decorator.
[gforcada]

- Remove advanced_search input which is in double.
[Gagaro]

5.1a1 (2016-03-31)
------------------

Expand Down
1 change: 0 additions & 1 deletion Products/CMFPlone/browser/templates/search.pt
Expand Up @@ -37,7 +37,6 @@
DateTime python:modules['DateTime'].DateTime;
navigation_root_url view/navroot_url;">

<input type="hidden" name="advanced_search" value="False" />
<input type="hidden" name="sort_on" value="" />
<input type="hidden" name="sort_order" value="" />

Expand Down

0 comments on commit a1f15c0

Please sign in to comment.