Someone 'fixed' the bug  #6250, that jumpers don't wrok in applicaiton
context.

But it still works a bit enigmatic. I for example tested the jumper example
'myyahoo'. There one jumper is shipped in the apps1 'MyYahoo'.

In the current situation though, you _must_ include the contextpath in the
jumper's destination. Which I think is silly, because the contextpath can
change (e.g. be differnt in the test-environment), and be unknown (I don't
know in wich app-context people will install the distro).


I would propose to replace
  res.sentRedirect(url);        
with something like this
  res.sendRedirect(res.encodeRedirectURL(req.getContextPath() + url));
In JumpersFilter.java

Then jumpers work completely in context, so I would consider it a bugfix.
And config/applications/MyYahoo/jumpers.xml can include an example jumper
which works. But of course it is not entirely backwards compatible because
if people already used to include context in jumpers then that data gets
useless (on the other hand, perhaps before the fix of Keesj, jumpers did not
work altogether in context?)

I also fixed that JumperFilter does not completely crash mmbase if by change
jumpers builder is not yet install. 

I would now propose to switch on the jumper filter in the example web.xml.
The MyYahoo example will completely work out-of-the-box then.

Michiel



-- 
Michiel Meeuwissen 
Mediapark C101 Hilversum  
+31 (0)35 6772979
nl_NL eo_XX en_US
mihxil'
 [] ()

Reply via email to