Bugzilla – Bug 6047
Breadcrumb trail doesn't display correctly after performing a search
Last modified: 2007-11-07 08:58:32 UTC
In the default skin, if I click on Home->My Music->Search Music, the breadcrumb trail reads "Home > My Music > Search". If I click the Search button within the advanced search form, the breadcrumb trail changes to "Home > Advanced Search". It should be "Home > My Music > Search > Advanced Search".
One simple fix, with the difference being that I consider Advanced search as not being a sub-level of Search. The merging of the advanced with live/manual search is a fake, having the pwd_list would be another fake and not match with the other skins and would therefore require Default skin to have a duplicated advanced_search.html template just to handle the pwd. It might be worth considering merging the advanced and regular searches properly, adding support for search->advanced search for all skins if you want a more thorough fix (it would have the benefit of likely addressing other issues around the current advanced search in Default skin) The fix allows default to have the right info, and doesn't affect other skins as they do not use the homeCategory variable: Index: C:/slim/server/HTML/EN/advanced_search.html =================================================================== --- C:/slim/server/HTML/EN/advanced_search.html (revision 14394) +++ C:/slim/server/HTML/EN/advanced_search.html (working copy) @@ -1,4 +1,5 @@ [% pagetitle = 'ADVANCEDSEARCH' | string %] +[% homeCategory = 'MY_MUSIC' %] [% pageicon = 'ADVANCEDSEARCH' %] [% pwd_list = BLOCK %] <a href="advanced_search.html?player=[% playerURI %]">[% "ADVANCEDSEARCH" | string %]</a>
Alternatively make Advanced Search a new page, instead of a fake page within Search?
yes, which I have suggested. There are several ADV search bugs here, that should really jsut be a single "refactor search" bug, as they all relate to the same issue.
Change 14467