[Tickets #9612] Re: Onclick handler for Horde_Tree broken

Tue Mar 1 18:26:01 UTC 2011

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

Ticket URL: http://bugs.horde.org/ticket/9612
------------------------------------------------------------------------------
  Ticket             | 9612
  Updated By         | Michael Slusarz <slusarz at horde.org>
  Summary            | Onclick handler for Horde_Tree broken
  Queue              | Horde Base
  Version            | Git master
  Type               | Bug
  State              | Assigned
  Priority           | 2. Medium
  Milestone          |
  Patch              |
  Owners             | Michael Slusarz
------------------------------------------------------------------------------

Michael Slusarz <slusarz at horde.org> (2011-03-01 18:26) wrote:

The onclick element should be completely removed.  Going forward, we  
should not be running any javascript via HTML on* parameters.

The *one* place where using embedded javascript may be acceptable is  
to open a popup window.  But this should be done via the url parameter  
rather than onclick anyway.