= Description = If you jump to a page 1. with the "Search" form at the top, or 2. with selecting the name from a {{{[[FullSearch(t:blabla)]]}}} list, the search term gets highlighted. This is disadvantagous because of two reasons: 1. it´s stupid and surprising to highlight the title match in the whole text 2. it bypasses the caching. As these are (at least my) major navigation instruments and caching is important for the performance of the wiki, I consider this bug. == Steps to reproduce == ## Describe the steps needed to reproduce the bug. If we can't reproduce it, we probably can't fix it. Click on the link in {{{[[FullSearchCached(t:Title t:highlight)]]}}} and you will see all ''title'' and ''highlight'' being highlighted and caching is not performed. <> == Example == ##Add URL that show the bug, screenshot or test wiki markup that fail... ##URL: ##attachment:screenshot.png ##{{{ ##Example wiki markup that fail ##}}} == Details == Patch: {{{ Index: search.py =================================================================== RCS file: /cvs/TPS/moin/MoinMoin/search.py,v retrieving revision 1.1.1.2 diff -u -r1.1.1.2 search.py --- search.py 22 Mar 2006 09:35:59 -0000 1.1.1.2 +++ search.py 23 Nov 2006 19:27:37 -0000 @@ -309,7 +309,9 @@ return u'%s!"%s"' % (neg, unicode(self._pattern)) def highlight_re(self): - return u"(%s)" % self._pattern + # 2b, no highlight for title pattern (speedup) + return u"" + #return u"(%s)" % self._pattern def pageFilter(self): """ Page filter function for single title search """ }}} ## If you got a traceback, please save the traceback page as html and attach here: ## attachment:traceback.html ## if the bug is in this wiki, just kill the table and write: This Wiki. ## If a traceback is not available, please fill in the details here: || '''!MoinMoin Version''' || || || '''OS and Version''' || || || '''Python Version''' || || || '''Server Setup''' || || || '''Server Details''' || || || '''Language you are using the wiki in''' (set in the browser/UserPreferences) || || == Workaround == ## How to deal with the bug until it is fixed = Discussion = = Plan = ## This part is for Moin``Moin developers: * Priority: * Assigned to: * Status: fixed in 1.6 by http://hg.moinmo.in/moin/1.6/rev/0fcf23cee5e8 and in 1.7 by http://hg.moinmo.in/moin/1.7/rev/dc970f2d146c ---- ## If you are a moin core developer, replace the category to Category* in these cases: ## Category MoinMoinNoBug - if this is not a bug. ## Category MoinMoinBugConfirmed - if you can confirm the bug on current code. ## Category MoinMoinBugFixed - after the bug is fixed in current code. CategoryMoinMoinBugFixed