377298 - [1.1] Add @validate on submission element

Status: RESOLVED FIXED

(Core Graveyard :: XForms, enhancement)

Description

[Steve Speicher]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1.3) Gecko/20070309 Firefox/2.0.0.3
Build Identifier: 

Currently in XForms 1.0 submissions are blocked due to data being invalid.

In 1.1 it is possible to disable this behavior by using:
  <submission validate="false" ... />


Reproducible: Always

Comment 1

[Steve Speicher]

Attachment: test case

Comment 2

[Steve Speicher]

See bug 278762 that is intended to put explicit ValidateDocument() call in.  Currently submission will fail if there is node marked invalid (where a check will need to be made for the new @validate)

Comment 3

[Steve Speicher]

Attachment: patch

I don't put out any validation warning messages when @validate=false.  I assume that is what would be desired.

Comment 4

[aaronr]

Comment on attachment 261467 [details] [diff] [review]
patch

>Index: nsXFormsSubmissionElement.cpp
>===================================================================

>@@ -1428,20 +1433,23 @@ nsXFormsSubmissionElement::CopyChildren(
>          *  SUBMIT_SKIP_NODE        - node is not to be serialized
>          *  SUBMIT_ABORT_SUBMISSION - abort submission (invalid node or empty required node)
>          */
>         if (handleNodeResult == nsIModelElementPrivate::SUBMIT_SKIP_NODE) {
>           // skip node and subtree
>           currentNode->GetNextSibling(getter_AddRefs(node));
>           currentNode.swap(node);
>           continue;
>-        } else if (handleNodeResult == nsIModelElementPrivate::SUBMIT_ABORT_SUBMISSION) {
>+        } else if (validate &&
>+                   handleNodeResult ==
>+                     nsIModelElementPrivate::SUBMIT_ABORT_SUBMISSION) {
>           // abort
>           nsXFormsUtils::ReportError(NS_LITERAL_STRING("warnSubmitInvalidNode"),
>                                      currentNode, nsIScriptError::warningFlag);
>+
>           return NS_ERROR_ILLEGAL_VALUE;
>         }

nit: please add a comment here spelling out that if validate is false we won't worry about  HandleInstanceDataNode returning  SUBMIT_ABORT_SUBMISSION for required but empty nodes or nodes that are invalid.  And that we assume that these are the only cases where HandleInstanceDataNode CAN return SUBMIT_ABORT_SUBMISSION.

with that, r=me

Comment 5

[Steve Speicher]

Attachment: patch with Aaron's requested comment added

Comment 6

[aaronr]

checked into trunk for sspeiche

Comment 7

[aaronr]

checked into 1.8 and 1.8.0 branches