[Tickets #9903] Re: Message Source stopped working

[bugs at horde.org]

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

Ticket URL: http://bugs.horde.org/ticket/9903
------------------------------------------------------------------------------
  Ticket             | 9903
  Updated By         | Michael Slusarz <slusarz at horde.org>
  Summary            | Message Source stopped working
  Queue              | IMP
  Version            | 5.0
  Type               | Bug
  State              | Feedback
  Priority           | 1. Low
  Milestone          |
  Patch              |
-Owners             |
+Owners             | Michael Slusarz
------------------------------------------------------------------------------


Michael Slusarz <slusarz at horde.org> (2011-04-13 20:43) wrote:

> <a  
> onclick="void(Horde.popup('%7B%22url%22%3A%22%5C%2Fhorde%5C%2Fimp%5C%2Fview.php%22%2C%22menu%22%3A1%2C%22params%22%3A%22actionID%3Dview_source%26amp%3Bid%3D1%26amp%3Buid%3D44691%26amp%3Bmailbox%3DINBOX%22%7D'));" class="widget" accesskey="M"><span class="accessKey">M</span>essage  
> Source</a>

That *is* an invalid string - '[...]view_source%26amp%3Bid[...]'  
should be '[...]view_source%3Bid[...]'.  But the problem is neither I  
(nor Jan) can reproduce - we both get the 2nd encoded version while  
you are getting the 1st encoded version.

In imp/lib/Contents.php, in the function linkViewJS(), can you put a  
Horde::debug($url) call in after the line that begins '$url =  
Horde::popupJs(...'

Horde::debug() documentation: http://wiki.horde.org/Doc/Dev/DebugH4

My guess is that your system is auto-appending some kind of parameters  
to all URLs that is not happening here, so the ampersand  
auto-detection is failing in Horde_Url.