[Tickets #12572] Re: Search from top navigation broken with git HEAD

[noreply at bugs.horde.org]

DO NOT REPLY TO THIS MESSAGE. THIS EMAIL ADDRESS IS NOT MONITORED.

Ticket URL: http://bugs.horde.org/ticket/12572
------------------------------------------------------------------------------
  Ticket             | 12572
  Updated By         | Jan Schneider <jan at horde.org>
  Summary            | Search from top navigation broken with git HEAD
  Queue              | IMP
  Version            | Git master
  Type               | Bug
  State              | Resolved
  Priority           | 1. Low
  Milestone          |
  Patch              |
  Owners             | Jan Schneider
------------------------------------------------------------------------------


Jan Schneider <jan at horde.org> (2013-08-26 18:24) wrote:

>> But removing the string cast from Horde_Tree is a BC break,
>> so we have to deal with raw URLs even in HTML renderers. At least
>> until H6.
>
> What about fixing this for Topbar generation only?   
> Horde_Tree_Renderer allows a Horde_Tree object to be specified in  
> the constructor... so create a class that extends Horde_Tree with  
> the fix, and then use this tree object explicitly with the topbar.

Not worth it. Worst thing that happens now is that we have unencoded  
URLs in the initial topbar menu, which is handled fine by all browsers  
AFAIK.