Last Comment Bug 278762 - Hook up schema validation to xforms submission
: Hook up schema validation to xforms submission
Product: Core Graveyard
Classification: Graveyard
Component: XForms (show other bugs)
: Trunk
: All All
-- normal with 1 vote (vote)
: ---
Assigned To: xforms
: Stephen Pride
Depends on: 274083
Blocks: 263086 326372 326373 338314
  Show dependency treegraph
Reported: 2005-01-17 14:22 PST by Darin Fisher
Modified: 2016-07-15 14:46 PDT (History)
4 users (show)
See Also:
QA Whiteboard:
Iteration: ---
Points: ---

Testcase (2.17 KB, application/xhtml+xml)
2006-05-17 07:41 PDT, Allan Beaufour
no flags Details

Description User image Darin Fisher 2005-01-17 14:22:41 PST
Hook up schema validation to xforms submission
Comment 1 User image Allan Beaufour 2006-05-16 02:07:07 PDT
I'm not 100% sure that this is a "[good first bug]". The submission handling needs to be restructured. Right now there are different code paths for turning instance data into a submit. We need to unify that to:
1) everything is serialized into an XML document
2) all non-relevant nodes are pruned
3) the XML document is fed to schema validation
4) the specific serialization method is run
Comment 2 User image Allan Beaufour 2006-05-17 07:41:25 PDT
Created attachment 222347 [details]

Have the testcase right Doron?
Comment 3 User image Allan Beaufour 2006-05-17 07:42:16 PDT
(In reply to comment #2)
> Created an attachment (id=222347) [edit]
> Testcase
> Have the testcase right Doron?

Free "I got"'s handed out at the exit.
Comment 4 User image Allan Beaufour 2006-05-17 07:45:08 PDT
I'll look at it
Comment 5 User image Allan Beaufour 2006-05-17 09:27:55 PDT
Should be fairly easy to fix bug 338314, while doing this as this should streamline the serialization process in general anyways.
Comment 6 User image Allan Beaufour 2006-05-22 03:55:32 PDT
As a result of bug 338314, it should be fairly easy to add this now. nsXFormsSubmissionElement::Submit() just needs to validate the document.

The problem is that I am not sure what to do, because I just ran through the spec., and there is not mention of what I thought we were supposed to do here.

"2. All selected instance data nodes are checked for validity according to the definition in 4.3.5 The xforms-revalidate Event (no notification events are marked for dispatching due to this operation). Any selected instance data node that is required but empty or found to be invalid stops submission processing after dispatching event xforms-submit-error."

Eh? I was expecting to see something about "perform full schema validation", because that is what I have always thought was supposed to happen.
Comment 7 User image Allan Beaufour 2006-05-24 02:26:20 PDT
Since it is not clear what this actually means, it will miss 0.6
Comment 8 User image aaronr 2006-05-24 15:50:56 PDT
Yeah, I think we need to get this all clarified.  Because we also need to know what to do if the value of a node is perfectly valid but what if the document that contains it is invalid (the instance node isn't in the right place in the document relative to another node, for example, or is missing an attribute).  The spec doesn't seem to differentiate between validating the value of an instance node and validaing the node itself.
Comment 9 User image David Bolter [:davidb] 2016-02-04 12:20:58 PST
RIP xforms

Note You need to log in before you can comment on or make changes to this bug.