Priority: Normal
Current State: Closed
Released In: 1.2.0
Target Release: n/a
CommentPlugin redirectto
combined with
target
works on Release1.1, fails with trunk version.
%<nop>COMMENT{target="Sandbox/SomeTopic" type="returntab"}%
%TMPL:DEF{returnpromptboxtab}%
<input type="hidden" name="redirectto" value="%BASEWEB%.%BASETOPIC%?tab=discuss" />
%TMPL:P{promptbox}%
%TMPL:END%
%TMPL:DEF{PROMPT:returntab}%%TMPL:P{returnpromptboxtab}%%TMPL:END%
%TMPL:DEF{OUTPUT:returntab}%%POS:BEFORE%%TMPL:P{OUTPUT:threadmode}%%TMPL:END%
On Release 1.1, flow captured is:
- POST
bin/save/Sandbox/SomeTopic
parameter dedirectto=TheWeb/TheTopic?tab=discuss
- GET
TheWeb/TheTopic?tab=discuss
On Trunk, flow captured is:
- POST
bin/rest/CommentPlugin/comment
param redirectto=TheWeb/TheTopic?tab=discuss
param topic=Sandbox/SomeTopic
- GET
Sandbox/SomeTopic
--
GeorgeClark - 14 Jan 2012
Looking at the new
CommentPlugin code, redirectto is not implemented.
--
GeorgeClark - 14 Jan 2012
Solution - Comment.pm needs to set the $endPoint parm passed to rest to the redirectto location.
--
GeorgeClark - 14 Jan 2012
Fixed - but not sure if anchor needs to be sanitized or encoded.
--
GeorgeClark - 14 Jan 2012
Needed more work. The rest script doesn't understand
redirectto
- it wants
endPoint
. Added more unit tests, and changed the rest save handler to replace endPoint with redirectto parameter if it's provided.
--
GeorgeClark - 15 Jan 2012