Last Comment Bug 377888 - Add support for submission @standalone attr for serialization
: Add support for submission @standalone attr for serialization
Status: RESOLVED FIXED
: fixed1.8.0.12, fixed1.8.1.4
Product: Core
Classification: Components
Component: XForms (show other bugs)
: unspecified
: All All
: -- enhancement (vote)
: ---
Assigned To: Steve Speicher
: Stephen Pride
Mentors:
http://www.w3.org/TR/2006/REC-xforms-...
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2007-04-18 07:44 PDT by Steve Speicher
Modified: 2007-04-27 16:16 PDT (History)
2 users (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
test case (1.79 KB, application/xhtml+xml)
2007-04-18 07:44 PDT, Steve Speicher
no flags Details
patch (1.06 KB, patch)
2007-04-19 12:07 PDT, Steve Speicher
aaronr: review+
doronr: review+
Details | Diff | Review

Description Steve Speicher 2007-04-18 07:44:01 PDT
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: 

In updated W3C test case 3.3.3.j

Reproducible: Always
Comment 1 Steve Speicher 2007-04-18 07:44:30 PDT
Created attachment 261951 [details]
test case
Comment 2 Steve Speicher 2007-04-19 12:07:36 PDT
Created attachment 262161 [details] [diff] [review]
patch

This doesn't fix the test case as-is, as I believe there is a problem with the test suite case (I'll address that with owner).

It appears that standalone="true" should include *standalone="yes"* declaration, as I read:
  "Optional attribute specifying whether to include a standalone declaration in the serialized XML."

So I guess there is no way to explicitly make *standalone="no"*.  Or what is the default if standalone is omitted?  formsPlayer uses *standalone="yes"* if true or omitted, if false uses *standalone="no"*.

I've taken the route which I believe is what was intended.  If false (the default) omit, otherwise if true add declaration *standalone="yes"*
Comment 3 aaronr 2007-04-19 14:53:35 PDT
Comment on attachment 262161 [details] [diff] [review]
patch

I interpret it the same way you do.
Comment 4 aaronr 2007-04-27 16:16:42 PDT
checked into trunk, 1.8 and 1.8.0 branches

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