Item14438: EDDITTABLE and COMMENT Plugin compatibility issue

pencil
Priority: Normal
Current State: Confirmed
Released In: n/a
Target Release:
Applies To: Extension
Component: CommentPlugin
Branches:
Reported By: DiegoBetancur
Waiting For:
Last Change By: DiegoBetancur
I replicated the issue here: https://foswiki.org/Sandbox/TestIssuePlugins

The issue occurs when there is a EDITTABLE and a COMMENT macro on the topic. An example of the wiki text to replicate the issue in a topic would be:
%EDITTABLE{ }%
|*Key*|*Value*|
| Name: | John Smith |

% COMMENT{type="toctalk"}%

-- DiegoBetancur - 17 Jul 2017

(Added a space to break the above comment macro to prevent it from breaking the comments inserted into this topic)

As you didn't actually explain what's going wrong, I'm guessing that the issue is the Comment action always causes a 403 "Validation Error"? That's what seems to be happening in the TestIssuePlugins topic.

If this is the issue you are reporting, the bug seems to be in the sequence of the generated HTML form statements. There appears to be a nested related to the EditRowPlugin embedded into the form emitted by the COMMENT macro.

<form method="post" action="https://foswiki.org/bin/rest/CommentPlugin/comment" enctype="multipart/form-data" class="commentPluginForm" id="toctalk0" name="toctalk0" onsubmit="StrikeOne.submit(this)"><input type='hidden' name='validation_key' value='?d523874403a2e3dd1cf277fa62e07ee2' />
<!-- STARTINCLUDE Sandbox.TestIssuePlugins:T1 -->
<form action='valid' method='POST' name='erp_form_TABLE_1' onsubmit="StrikeOne.submit(this)"><input type='hidden' name='validation_key' value='?58578eb509470eaecf7c03fa9a514fd5' />
<a name='erp_TABLE_1'></a>

-- GeorgeClark - 17 Jul 2017

You are correct. Apologies for not providing more details. It causes a '403 Validation Error'.

Thanks for the extra details on the bug.

-- DiegoBetancur - 17 Jul 2017
 

ItemTemplate edit

Summary EDDITTABLE and COMMENT Plugin compatibility issue
ReportedBy DiegoBetancur
Codebase 2.1.4
SVN Range
AppliesTo Extension
Component CommentPlugin
Priority Normal
CurrentState Confirmed
WaitingFor
Checkins
ReleasedIn n/a
CheckinsOnBranches
trunkCheckins
masterCheckins
ItemBranchCheckins
Release02x01Checkins
Release02x00Checkins
Release01x01Checkins
Topic revision: r3 - 17 Jul 2017, DiegoBetancur
The copyright of the content on this website is held by the contributing authors, except where stated elsewhere. See Copyright Statement. Creative Commons License    Legal Imprint    Privacy Policy