Priority: Normal
Current State: Closed
Released In: 1.2.0
Target Release: n/a
The last two
noform
examples in
CommentPluginExamples don't work on the new version of the plugin. The examples still use the POST to the save script for their implementation.
I have one fix working - I think that there are probably a couple.
Just changing as a post to rest ran into issues with the topic= parameter prompted in the form. It's validated early in the rest script before the handler gets control. If it's not right, fully qualified with a web, the rest scripts throws a 400 error.
In order to move user input away from the topic= parameter so it's not user entered:
- Added a
comment_target
field consistent with the other comment_
variables.
- Rest handler validates
comment_target
and overrides the web and topic using it.
- change form to post to
rest/CommentPlugin/comment
, send topic="%BASEWEB%.%BASETOPIC%"
, and use comment_target
for user input.
Another solution would be to use javascript to validate the user input so that topic= is always valid to prevent rest failures.
--
GeorgeClark - 16 Jan 2012
I like that. JS is the right approach IMHO.
--
CrawfordCurrie - 28 Jan 2012
The
comment_target
option has been added to the examples and they are all working on 1.1 using the plugin I release to Extensions/Testing.
--
GeorgeClark - 30 Jan 2012
Cool. You have control.
--
CrawfordCurrie - 30 Jan 2012