Merge lp:~zorba-coders/zorba/fix_xqdoc_schema_import_prefix into lp:zorba

[Sorin Marian Nasoi]

Fixed a namespace clash by adding a "-schema" to the import schema prefix.

Fixed a namespace clash by adding a "-schema" to the import schema prefix.

[Sorin Marian Nasoi (sorin.marian.nasoi)]

review: Approve

[Chris Hillery (ceejatec)]

The change looks good. One question: is using the same prefix that the original module used likely to cause any conflicts? Are those prefixes generally unique as well as the module prefixes?

review: Approve

[Sorin Marian Nasoi (sorin.marian.nasoi)]

> The change looks good. One question: is using the same prefix that the
> original module used likely to cause any conflicts? Are those prefixes
> generally unique as well as the module prefixes?
I think that that is what Dana meant by suggesting uniformity: as a result I have added bug lp:956318.

If you are OK with this answer please set the status of this merge to 'Approve' such that the Validation Queue is able to pick it up.
Thanks.

[Chris Hillery (ceejatec)]

Sounds reasonable.

[Matthias Brantner (matthias-brantner)]

Does this also work for schema imports without a prefix?

review: Needs Information

[Zorba Build Bot (zorba-buildbot)]

Validation queue starting for merge proposal.
Log at: http://zorbatest.lambda.nu:8080/remotequeue/fix_xqdoc_schema_import_prefix-2012-03-15T21-48-35.202Z/log.html

[Zorba Build Bot (zorba-buildbot)]

The attempt to merge lp:~zorba-coders/zorba/fix_xqdoc_schema_import_prefix into lp:zorba failed. Below is the output from the failed tests.

CMake Error at /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake:274 (message):
  Validation queue job
  fix_xqdoc_schema_import_prefix-2012-03-15T21-48-35.202Z is finished. The
  final status was:

  7 tests did not succeed - changes not commited.

Error in read script: /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake

[lp:~zorba-coders/zorba/fix_xqdoc_schema_import_prefix]

10724. By Sorin Marian Nasoi <email address hidden> on 2012-03-15

Corrected the expected result of the XQDoc tests that are different because of the change in the xqdoc printer.

[Sorin Marian Nasoi (sorin.marian.nasoi)]

> Does this also work for schema imports without a prefix?
Are you referring to:
import schema default element namespace "SOMETHING"; ?

The answer would be no, the current changes are not tested for this because we have no modules at this point using this kind of schema imports.

The xqdoc printer would have to be updated and also the xqdoc2html converter in order to support this.

IMHO this should not be a show stopper for these changes: If you agree please create a lp bug for me for this feature and approve this merge.

[Sorin Marian Nasoi (sorin.marian.nasoi)]

review: Approve

[Matthias Brantner (matthias-brantner)]

> > Does this also work for schema imports without a prefix?
> Are you referring to:
> import schema default element namespace "SOMETHING"; ?
>
> The answer would be no, the current changes are not tested for this because we
> have no modules at this point using this kind of schema imports.
>
> The xqdoc printer would have to be updated and also the xqdoc2html converter
> in order to support this.
>
> IMHO this should not be a show stopper for these changes: If you agree please
> create a lp bug for me for this feature and approve this merge.
Yes, that's what I mean. It looks like the xqdoc visitor could crash with this change. That's why I think the branch shouldn't be merged before it's tested.

[Sorin Marian Nasoi (sorin.marian.nasoi)]

> > > Does this also work for schema imports without a prefix?
> > Are you referring to:
> > import schema default element namespace "SOMETHING"; ?
> >
> > The answer would be no, the current changes are not tested for this because
> we
> > have no modules at this point using this kind of schema imports.
> >
> > The xqdoc printer would have to be updated and also the xqdoc2html converter
> > in order to support this.
> >
> > IMHO this should not be a show stopper for these changes: If you agree
> please
> > create a lp bug for me for this feature and approve this merge.
> Yes, that's what I mean. It looks like the xqdoc visitor could crash with this
> change. That's why I think the branch shouldn't be merged before it's tested.
Nope, I disagree: this merge does not introduce the problem you mentioned because it was already there :)

All this change did was to make you and then me aware that we have a problem if one uses a schema import without a prefix.

So my suggestion is: please open a bug for this issue and approve this merge.

[Dana Florescu (dflorescu)]

>>>
>> Yes, that's what I mean. It looks like the xqdoc visitor could crash with this
>> change. That's why I think the branch shouldn't be merged before it's tested.
> Nope, I disagree: this merge does not introduce the problem you mentioned because it was already there :)

I agree with Sorin. The problem was there already.

I think Sorin should go ahead and commit the change (at least the documentation will be usable..)
and then file a bug for the existing problem, and fix that after the release.

Thanks
Dana

[Zorba Build Bot (zorba-buildbot)]

Validation queue starting for merge proposal.
Log at: http://zorbatest.lambda.nu:8080/remotequeue/fix_xqdoc_schema_import_prefix-2012-03-16T02-46-08.694Z/log.html

[Zorba Build Bot (zorba-buildbot)]

Validation queue job fix_xqdoc_schema_import_prefix-2012-03-16T02-46-08.694Z is finished. The final status was:

All tests succeeded!

=== modified file 'modules/com/zorba-xquery/www/modules/xqdoc2xhtml/index.xq'
--- modules/com/zorba-xquery/www/modules/xqdoc2xhtml/index.xq	2012-03-09 06:57:41 +0000
+++ modules/com/zorba-xquery/www/modules/xqdoc2xhtml/index.xq	2012-03-15 22:38:22 +0000
@@ -1263,7 +1263,7 @@
     for $import in $xqdoc/xqdoc:imports//xqdoc:import[@type = "schema"]
       return
        concat('import schema namespace ',
-              $namespaces//xqdoc:namespace[@uri = $import/xqdoc:uri/text()][1]/@prefix,
+              $namespaces//xqdoc:namespace[(@uri = $import/xqdoc:uri/text()) and (@isSchema = "true")]/@prefix,
               ' =  "',
               string($import/xqdoc:uri/text()),'";'),"
 ")

=== modified file 'src/compiler/parsetree/parsenode_print_xqdoc_visitor.cpp'
--- src/compiler/parsetree/parsenode_print_xqdoc_visitor.cpp	2012-02-28 20:45:43 +0000
+++ src/compiler/parsetree/parsenode_print_xqdoc_visitor.cpp	2012-03-15 22:38:22 +0000
@@ -167,11 +167,18 @@
   }
 }
 
+bool is_namespace_schema(zstring aPrefix, zstring aNamespace )
+{
+  map<zstring, zstring>::iterator ite = theNamespaceSchemaMap.find(aPrefix);
+  return ((ite != theNamespaceSchemaMap.end()) ||
+          (ite->second ==  aNamespace));
+}
+
 void print_namespaces()
 {
   store::Item_t lTypeName; 
   store::Item_t lNamespaceQName, lCustomElem;
-  store::Item_t lPrefixQName, lURIQName;
+  store::Item_t lPrefixQName, lURIQName, lIsSchemaQName;
   store::Item_t lNamespace, lAttrValue;
   bool lFirst = true;
 
@@ -214,6 +221,9 @@
         lPrefixQName, "", "", "prefix");
     theFactory->createQName(
         lURIQName, "", "", "uri");
+    theFactory->createQName(
+        lIsSchemaQName, "", "", "isSchema");
+
 
     lTypeName = GENV_TYPESYSTEM.XS_UNTYPED_QNAME;
     theFactory->createElementNode(
@@ -232,6 +242,12 @@
     theFactory->createAttributeNode(
       lURIQName, lNamespace, lURIQName, lTypeName, lAttrValue);
 
+    bool lIsSchema = is_namespace_schema(lIter->first, lIter->second);
+    lTmp = lIsSchema?"true":"false";
+    theFactory->createString(lAttrValue, lTmp);
+    lTypeName = GENV_TYPESYSTEM.XS_UNTYPED_QNAME;
+    theFactory->createAttributeNode(
+      lIsSchemaQName, lNamespace, lIsSchemaQName, lTypeName, lAttrValue);
   }
 }
 
@@ -434,6 +450,9 @@
   // prefix -> uri
   map<zstring, zstring> theNamespaceMap;
 
+  // prefix -> uri for the schema imports
+  map<zstring, zstring> theNamespaceSchemaMap;
+
   const char*          theXQDocNS;
   const char*          theXQDocPrefix;
 
@@ -1034,6 +1053,7 @@
     lPrefix = n.get_prefix()->get_prefix();
   }
   theNamespaceMap[lPrefix] = n.get_uri();
+  theNamespaceSchemaMap[lPrefix] = n.get_uri();
 }
 
 XQDOC_NO_BEGIN_TAG (NamespaceDecl)

=== modified file 'test/rbkt/ExpQueryResults/zorba/xqdoc/basic.xml.res'
--- test/rbkt/ExpQueryResults/zorba/xqdoc/basic.xml.res	2012-02-28 20:45:43 +0000
+++ test/rbkt/ExpQueryResults/zorba/xqdoc/basic.xml.res	2012-03-15 22:38:22 +0000
@@ -1,7 +1,7 @@
 <xqdoc:xqdoc xmlns:xqdoc="http://www.xqdoc.org/1.0"><xqdoc:control><xqdoc:date/><xqdoc:version>1.0</xqdoc:version></xqdoc:control><xqdoc:module type="library"><xqdoc:uri>http://www.28msec.com/modules/gdata</xqdoc:uri><xqdoc:name>gdata.xqlib</xqdoc:name><xqdoc:comment><xqdoc:description> The description of this module must have the whitespaces
  normalized: 1) NL at top, middle &amp; end; 2) TABs in this
  line;			3) SPACES      here and after.
-</xqdoc:description><xqdoc:see>http://example.com</xqdoc:see></xqdoc:comment><xqdoc:custom tag="namespaces"><xqdoc:namespace prefix="atom" uri="http://www.w3.org/2005/Atom"/><xqdoc:namespace prefix="atompub" uri="http://www.28msec.com/modules/atom"/><xqdoc:namespace prefix="gdata" uri="http://www.28msec.com/modules/gdata"/><xqdoc:namespace prefix="rest" uri="http://www.zorba-xquery.com/zorba/rest-functions"/><xqdoc:namespace prefix="zorba" uri="http://www.zorba-xquery.com/zorba/internal-functions"/></xqdoc:custom></xqdoc:module><xqdoc:imports><xqdoc:import type="schema"><xqdoc:uri>http://www.w3.org/2005/Atom</xqdoc:uri></xqdoc:import><xqdoc:import type="library"><xqdoc:uri>http://www.zorba-xquery.com/zorba/internal-functions</xqdoc:uri></xqdoc:import><xqdoc:import type="library"><xqdoc:uri>http://www.28msec.com/modules/atom</xqdoc:uri></xqdoc:import><xqdoc:import type="library"><xqdoc:uri>http://www.zorba-xquery.com/zorba/rest-functions</xqdoc:uri></xqdoc:import></xqdoc:imports><xqdoc:variables><xqdoc:variable><xqdoc:uri>gdata:version</xqdoc:uri><xqdoc:comment><xqdoc:description> Import the Atom XML Schema
+</xqdoc:description><xqdoc:see>http://example.com</xqdoc:see></xqdoc:comment><xqdoc:custom tag="namespaces"><xqdoc:namespace prefix="atom" uri="http://www.w3.org/2005/Atom" isSchema="true"/><xqdoc:namespace prefix="atompub" uri="http://www.28msec.com/modules/atom" isSchema="false"/><xqdoc:namespace prefix="gdata" uri="http://www.28msec.com/modules/gdata" isSchema="false"/><xqdoc:namespace prefix="rest" uri="http://www.zorba-xquery.com/zorba/rest-functions" isSchema="false"/><xqdoc:namespace prefix="zorba" uri="http://www.zorba-xquery.com/zorba/internal-functions" isSchema="false"/></xqdoc:custom></xqdoc:module><xqdoc:imports><xqdoc:import type="schema"><xqdoc:uri>http://www.w3.org/2005/Atom</xqdoc:uri></xqdoc:import><xqdoc:import type="library"><xqdoc:uri>http://www.zorba-xquery.com/zorba/internal-functions</xqdoc:uri></xqdoc:import><xqdoc:import type="library"><xqdoc:uri>http://www.28msec.com/modules/atom</xqdoc:uri></xqdoc:import><xqdoc:import type="library"><xqdoc:uri>http://www.zorba-xquery.com/zorba/rest-functions</xqdoc:uri></xqdoc:import></xqdoc:imports><xqdoc:variables><xqdoc:variable><xqdoc:uri>gdata:version</xqdoc:uri><xqdoc:comment><xqdoc:description> Import the Atom XML Schema
 </xqdoc:description><xqdoc:see>http://www.kbcafe.com/rss/atom.xsd.xml</xqdoc:see></xqdoc:comment></xqdoc:variable><xqdoc:variable><xqdoc:uri>gdata:id</xqdoc:uri><xqdoc:comment><xqdoc:description> Google ID
 </xqdoc:description><xqdoc:see>http://www.google.com</xqdoc:see></xqdoc:comment></xqdoc:variable><xqdoc:variable><xqdoc:uri>gdata:authToken</xqdoc:uri></xqdoc:variable><xqdoc:variable><xqdoc:uri>gdata:blub</xqdoc:uri><xqdoc:comment><xqdoc:description> test xqdoc generation for a variable declaration
  whose initializer invokes a function

=== modified file 'test/rbkt/ExpQueryResults/zorba/xqdoc/content.xml.res'
--- test/rbkt/ExpQueryResults/zorba/xqdoc/content.xml.res	2012-02-28 20:45:43 +0000
+++ test/rbkt/ExpQueryResults/zorba/xqdoc/content.xml.res	2012-03-15 22:38:22 +0000
@@ -1,7 +1,7 @@
 <xqdoc:xqdoc xmlns:xqdoc="http://www.xqdoc.org/1.0"><xqdoc:control><xqdoc:date/><xqdoc:version>1.0</xqdoc:version></xqdoc:control><xqdoc:module type="library"><xqdoc:uri>http://www.28msec.com/modules/gdata</xqdoc:uri><xqdoc:name></xqdoc:name><xqdoc:comment><xqdoc:description> The description of this module must have the whitespaces
  normalized: 1) NL at top, middle &amp; end; 2) TABs in this
  line;			3) SPACES      here and after.
-</xqdoc:description><xqdoc:see>http://example.com</xqdoc:see></xqdoc:comment><xqdoc:custom tag="namespaces"><xqdoc:namespace prefix="atom" uri="http://www.w3.org/2005/Atom"/><xqdoc:namespace prefix="atompub" uri="http://www.28msec.com/modules/atom"/><xqdoc:namespace prefix="gdata" uri="http://www.28msec.com/modules/gdata"/><xqdoc:namespace prefix="rest" uri="http://www.zorba-xquery.com/zorba/rest-functions"/><xqdoc:namespace prefix="zorba" uri="http://www.zorba-xquery.com/zorba/internal-functions"/></xqdoc:custom></xqdoc:module><xqdoc:imports><xqdoc:import type="schema"><xqdoc:uri>http://www.w3.org/2005/Atom</xqdoc:uri></xqdoc:import><xqdoc:import type="library"><xqdoc:uri>http://www.zorba-xquery.com/zorba/internal-functions</xqdoc:uri></xqdoc:import><xqdoc:import type="library"><xqdoc:uri>http://www.28msec.com/modules/atom</xqdoc:uri></xqdoc:import><xqdoc:import type="library"><xqdoc:uri>http://www.zorba-xquery.com/zorba/rest-functions</xqdoc:uri></xqdoc:import></xqdoc:imports><xqdoc:variables><xqdoc:variable><xqdoc:uri>gdata:version</xqdoc:uri><xqdoc:comment><xqdoc:description> Import the Atom XML Schema
+</xqdoc:description><xqdoc:see>http://example.com</xqdoc:see></xqdoc:comment><xqdoc:custom tag="namespaces"><xqdoc:namespace prefix="atom" uri="http://www.w3.org/2005/Atom" isSchema="true"/><xqdoc:namespace prefix="atompub" uri="http://www.28msec.com/modules/atom" isSchema="false"/><xqdoc:namespace prefix="gdata" uri="http://www.28msec.com/modules/gdata" isSchema="false"/><xqdoc:namespace prefix="rest" uri="http://www.zorba-xquery.com/zorba/rest-functions" isSchema="false"/><xqdoc:namespace prefix="zorba" uri="http://www.zorba-xquery.com/zorba/internal-functions" isSchema="false"/></xqdoc:custom></xqdoc:module><xqdoc:imports><xqdoc:import type="schema"><xqdoc:uri>http://www.w3.org/2005/Atom</xqdoc:uri></xqdoc:import><xqdoc:import type="library"><xqdoc:uri>http://www.zorba-xquery.com/zorba/internal-functions</xqdoc:uri></xqdoc:import><xqdoc:import type="library"><xqdoc:uri>http://www.28msec.com/modules/atom</xqdoc:uri></xqdoc:import><xqdoc:import type="library"><xqdoc:uri>http://www.zorba-xquery.com/zorba/rest-functions</xqdoc:uri></xqdoc:import></xqdoc:imports><xqdoc:variables><xqdoc:variable><xqdoc:uri>gdata:version</xqdoc:uri><xqdoc:comment><xqdoc:description> Import the Atom XML Schema
 </xqdoc:description><xqdoc:see>http://www.kbcafe.com/rss/atom.xsd.xml</xqdoc:see></xqdoc:comment></xqdoc:variable><xqdoc:variable><xqdoc:uri>gdata:id</xqdoc:uri><xqdoc:comment><xqdoc:description> Google ID
 </xqdoc:description><xqdoc:see>http://www.google.com</xqdoc:see></xqdoc:comment></xqdoc:variable><xqdoc:variable><xqdoc:uri>gdata:authToken</xqdoc:uri></xqdoc:variable><xqdoc:variable><xqdoc:uri>gdata:blub</xqdoc:uri><xqdoc:comment><xqdoc:description> test xqdoc generation for a variable declaration
  whose initializer invokes a function

=== modified file 'test/rbkt/ExpQueryResults/zorba/xqdoc/indentation.xml.res'
--- test/rbkt/ExpQueryResults/zorba/xqdoc/indentation.xml.res	2011-08-04 02:14:56 +0000
+++ test/rbkt/ExpQueryResults/zorba/xqdoc/indentation.xml.res	2012-03-15 22:38:22 +0000
@@ -1,4 +1,3 @@
-<?xml version="1.0" encoding="UTF-8"?>
 <xqdoc:xqdoc xmlns:xqdoc="http://www.xqdoc.org/1.0">
   <xqdoc:control>
     <xqdoc:date/>
@@ -7,7 +6,7 @@
   <xqdoc:module type="main">
     <xqdoc:uri>indentation.xq</xqdoc:uri>
     <xqdoc:custom tag="namespaces">
-      <xqdoc:namespace prefix="xqd" uri="http://www.zorba-xquery.com/modules/xqdoc"/>
+      <xqdoc:namespace prefix="xqd" uri="http://www.zorba-xquery.com/modules/xqdoc" isSchema="false"/>
     </xqdoc:custom>
   </xqdoc:module>
   <xqdoc:imports>

=== modified file 'test/rbkt/ExpQueryResults/zorba/xqdoc/local.xml.res'
--- test/rbkt/ExpQueryResults/zorba/xqdoc/local.xml.res	2012-02-28 20:45:43 +0000
+++ test/rbkt/ExpQueryResults/zorba/xqdoc/local.xml.res	2012-03-15 22:38:22 +0000
@@ -1,1 +1,1 @@
-<xqdoc:xqdoc xmlns:xqdoc="http://www.xqdoc.org/1.0"><xqdoc:control><xqdoc:date/><xqdoc:version>1.0</xqdoc:version></xqdoc:control><xqdoc:module type="main"><xqdoc:uri>local.xqlib</xqdoc:uri><xqdoc:custom tag="namespaces"><xqdoc:namespace prefix="ann" uri="http://www.zorba-xquery.com/annotations"/><xqdoc:namespace prefix="xqd" uri="http://www.zorba-xquery.com/modules/xqdoc"/><xqdoc:namespace prefix="xqds" uri="http://www.xqdoc.org/1.0"/></xqdoc:custom></xqdoc:module><xqdoc:imports><xqdoc:import type="schema"><xqdoc:uri>http://www.xqdoc.org/1.0</xqdoc:uri></xqdoc:import><xqdoc:import type="library"><xqdoc:uri>http://www.zorba-xquery.com/modules/xqdoc</xqdoc:uri></xqdoc:import></xqdoc:imports><xqdoc:variables/><xqdoc:functions><xqdoc:function arity="1"><xqdoc:name>local:remove-date</xqdoc:name><xqdoc:annotations><xqdoc:annotation namespace="http://www.zorba-xquery.com/annotations" localname="sequential" value=""/></xqdoc:annotations><xqdoc:signature>declare %ann:sequential function local:remove-date($xqdoc)</xqdoc:signature><xqdoc:parameters><xqdoc:parameter><xqdoc:name>xqdoc</xqdoc:name></xqdoc:parameter></xqdoc:parameters></xqdoc:function></xqdoc:functions></xqdoc:xqdoc>
+<xqdoc:xqdoc xmlns:xqdoc="http://www.xqdoc.org/1.0"><xqdoc:control><xqdoc:date/><xqdoc:version>1.0</xqdoc:version></xqdoc:control><xqdoc:module type="main"><xqdoc:uri>local.xqlib</xqdoc:uri><xqdoc:custom tag="namespaces"><xqdoc:namespace prefix="ann" uri="http://www.zorba-xquery.com/annotations" isSchema="false"/><xqdoc:namespace prefix="xqd" uri="http://www.zorba-xquery.com/modules/xqdoc" isSchema="false"/><xqdoc:namespace prefix="xqds" uri="http://www.xqdoc.org/1.0" isSchema="true"/></xqdoc:custom></xqdoc:module><xqdoc:imports><xqdoc:import type="schema"><xqdoc:uri>http://www.xqdoc.org/1.0</xqdoc:uri></xqdoc:import><xqdoc:import type="library"><xqdoc:uri>http://www.zorba-xquery.com/modules/xqdoc</xqdoc:uri></xqdoc:import></xqdoc:imports><xqdoc:variables/><xqdoc:functions><xqdoc:function arity="1"><xqdoc:name>local:remove-date</xqdoc:name><xqdoc:annotations><xqdoc:annotation namespace="http://www.zorba-xquery.com/annotations" localname="sequential" value=""/></xqdoc:annotations><xqdoc:signature>declare %ann:sequential function local:remove-date($xqdoc)</xqdoc:signature><xqdoc:parameters><xqdoc:parameter><xqdoc:name>xqdoc</xqdoc:name></xqdoc:parameter></xqdoc:parameters></xqdoc:function></xqdoc:functions></xqdoc:xqdoc>
\ No newline at end of file

=== modified file 'test/rbkt/ExpQueryResults/zorba/xqdoc/no_global_var.xml.res'
--- test/rbkt/ExpQueryResults/zorba/xqdoc/no_global_var.xml.res	2011-08-04 02:14:56 +0000
+++ test/rbkt/ExpQueryResults/zorba/xqdoc/no_global_var.xml.res	2012-03-15 22:38:22 +0000
@@ -1,2 +1,1 @@
-<?xml version="1.0" encoding="UTF-8"?>
-<xqdoc:xqdoc xmlns:xqdoc="http://www.xqdoc.org/1.0"><xqdoc:control><xqdoc:date/><xqdoc:version>1.0</xqdoc:version></xqdoc:control><xqdoc:module type="library"><xqdoc:uri>foo:bar</xqdoc:uri><xqdoc:name>no_global_var.xqlib</xqdoc:name><xqdoc:custom tag="namespaces"><xqdoc:namespace prefix="ann" uri="http://www.zorba-xquery.com/annotations"/><xqdoc:namespace prefix="foo" uri="foo:bar"/></xqdoc:custom></xqdoc:module><xqdoc:imports/><xqdoc:variables><xqdoc:variable><xqdoc:uri>foo:foo</xqdoc:uri></xqdoc:variable></xqdoc:variables><xqdoc:functions><xqdoc:function arity="0"><xqdoc:name>foo:bar</xqdoc:name><xqdoc:annotations><xqdoc:annotation namespace="http://www.zorba-xquery.com/annotations" localname="sequential" value=""/></xqdoc:annotations><xqdoc:signature>declare %ann:sequential function foo:bar()</xqdoc:signature><xqdoc:invoked arity="2"><xqdoc:uri>http://www.w3.org/2005/xpath-functions</xqdoc:uri><xqdoc:name>concat</xqdoc:name></xqdoc:invoked></xqdoc:function></xqdoc:functions></xqdoc:xqdoc>
\ No newline at end of file
+<xqdoc:xqdoc xmlns:xqdoc="http://www.xqdoc.org/1.0"><xqdoc:control><xqdoc:date/><xqdoc:version>1.0</xqdoc:version></xqdoc:control><xqdoc:module type="library"><xqdoc:uri>foo:bar</xqdoc:uri><xqdoc:name>no_global_var.xqlib</xqdoc:name><xqdoc:custom tag="namespaces"><xqdoc:namespace prefix="ann" uri="http://www.zorba-xquery.com/annotations" isSchema="false"/><xqdoc:namespace prefix="foo" uri="foo:bar" isSchema="false"/></xqdoc:custom></xqdoc:module><xqdoc:imports/><xqdoc:variables><xqdoc:variable><xqdoc:uri>foo:foo</xqdoc:uri></xqdoc:variable></xqdoc:variables><xqdoc:functions><xqdoc:function arity="0"><xqdoc:name>foo:bar</xqdoc:name><xqdoc:annotations><xqdoc:annotation namespace="http://www.zorba-xquery.com/annotations" localname="sequential" value=""/></xqdoc:annotations><xqdoc:signature>declare %ann:sequential function foo:bar()</xqdoc:signature><xqdoc:invoked arity="2"><xqdoc:uri>http://www.w3.org/2005/xpath-functions</xqdoc:uri><xqdoc:name>concat</xqdoc:name></xqdoc:invoked></xqdoc:function></xqdoc:functions></xqdoc:xqdoc>
\ No newline at end of file

=== modified file 'test/rbkt/ExpQueryResults/zorba/xqdoc/testSchemaTypes.xml.res'
--- test/rbkt/ExpQueryResults/zorba/xqdoc/testSchemaTypes.xml.res	2012-02-28 20:45:43 +0000
+++ test/rbkt/ExpQueryResults/zorba/xqdoc/testSchemaTypes.xml.res	2012-03-15 22:38:22 +0000
@@ -1,2 +1,1 @@
-<?xml version="1.0" encoding="UTF-8"?>
-<xqdoc:xqdoc xmlns:xqdoc="http://www.xqdoc.org/1.0"><xqdoc:control><xqdoc:date/><xqdoc:version>1.0</xqdoc:version></xqdoc:control><xqdoc:module type="library"><xqdoc:uri>http://www.example.com/testModule</xqdoc:uri><xqdoc:name>image.xqlib</xqdoc:name><xqdoc:custom tag="namespaces"><xqdoc:namespace prefix="image" uri="http://www.zorba-xquery.com/modules/image/image"/><xqdoc:namespace prefix="testModule" uri="http://www.example.com/testModule"/></xqdoc:custom></xqdoc:module><xqdoc:imports><xqdoc:import type="schema"><xqdoc:uri>http://www.zorba-xquery.com/modules/image/image</xqdoc:uri></xqdoc:import></xqdoc:imports><xqdoc:variables/><xqdoc:functions><xqdoc:function arity="1"><xqdoc:name>testModule:test</xqdoc:name><xqdoc:signature>declare function testModule:test($image as image:imageType)</xqdoc:signature><xqdoc:parameters><xqdoc:parameter><xqdoc:name>image</xqdoc:name><xqdoc:type>image:imageType</xqdoc:type></xqdoc:parameter></xqdoc:parameters></xqdoc:function><xqdoc:function arity="1"><xqdoc:name>testModule:test2</xqdoc:name><xqdoc:signature>declare function testModule:test2($image as xs:string)</xqdoc:signature><xqdoc:parameters><xqdoc:parameter><xqdoc:name>image</xqdoc:name><xqdoc:type>xs:string</xqdoc:type></xqdoc:parameter></xqdoc:parameters><xqdoc:invoked arity="1"><xqdoc:uri>http://www.example.com/testModule</xqdoc:uri><xqdoc:name>test</xqdoc:name></xqdoc:invoked><xqdoc:invoked arity="1"><xqdoc:uri>http://www.zorba-xquery.com/modules/image/image</xqdoc:uri><xqdoc:name>imageType</xqdoc:name></xqdoc:invoked></xqdoc:function></xqdoc:functions></xqdoc:xqdoc>
+<xqdoc:xqdoc xmlns:xqdoc="http://www.xqdoc.org/1.0"><xqdoc:control><xqdoc:date/><xqdoc:version>1.0</xqdoc:version></xqdoc:control><xqdoc:module type="library"><xqdoc:uri>http://www.example.com/testModule</xqdoc:uri><xqdoc:name>image.xqlib</xqdoc:name><xqdoc:custom tag="namespaces"><xqdoc:namespace prefix="image" uri="http://www.zorba-xquery.com/modules/image/image" isSchema="true"/><xqdoc:namespace prefix="testModule" uri="http://www.example.com/testModule" isSchema="false"/></xqdoc:custom></xqdoc:module><xqdoc:imports><xqdoc:import type="schema"><xqdoc:uri>http://www.zorba-xquery.com/modules/image/image</xqdoc:uri></xqdoc:import></xqdoc:imports><xqdoc:variables/><xqdoc:functions><xqdoc:function arity="1"><xqdoc:name>testModule:test</xqdoc:name><xqdoc:signature>declare function testModule:test($image as image:imageType)</xqdoc:signature><xqdoc:parameters><xqdoc:parameter><xqdoc:name>image</xqdoc:name><xqdoc:type>image:imageType</xqdoc:type></xqdoc:parameter></xqdoc:parameters></xqdoc:function><xqdoc:function arity="1"><xqdoc:name>testModule:test2</xqdoc:name><xqdoc:signature>declare function testModule:test2($image as xs:string)</xqdoc:signature><xqdoc:parameters><xqdoc:parameter><xqdoc:name>image</xqdoc:name><xqdoc:type>xs:string</xqdoc:type></xqdoc:parameter></xqdoc:parameters><xqdoc:invoked arity="1"><xqdoc:uri>http://www.example.com/testModule</xqdoc:uri><xqdoc:name>test</xqdoc:name></xqdoc:invoked><xqdoc:invoked arity="1"><xqdoc:uri>http://www.zorba-xquery.com/modules/image/image</xqdoc:uri><xqdoc:name>imageType</xqdoc:name></xqdoc:invoked></xqdoc:function></xqdoc:functions></xqdoc:xqdoc>
\ No newline at end of file

=== modified file 'test/rbkt/ExpQueryResults/zorba/xqdoc/unorderedAnnotations.xml.res'
--- test/rbkt/ExpQueryResults/zorba/xqdoc/unorderedAnnotations.xml.res	2012-02-28 20:45:43 +0000
+++ test/rbkt/ExpQueryResults/zorba/xqdoc/unorderedAnnotations.xml.res	2012-03-15 22:38:22 +0000
@@ -1,6 +1,5 @@
-<?xml version="1.0" encoding="UTF-8"?>
-<xqdoc:xqdoc xmlns:xqdoc="http://www.xqdoc.org/1.0"><xqdoc:control><xqdoc:date/><xqdoc:version>1.0</xqdoc:version></xqdoc:control><xqdoc:module type="library"><xqdoc:uri>http://www.28msec.com/modules/mymodule</xqdoc:uri><xqdoc:name>unorderedAnnotations.xqlib</xqdoc:name><xqdoc:custom tag="namespaces"><xqdoc:namespace prefix="mymod" uri="http://www.28msec.com/modules/mymodule"/></xqdoc:custom></xqdoc:module><xqdoc:imports/><xqdoc:variables/><xqdoc:functions><xqdoc:function arity="2"><xqdoc:comment><xqdoc:description> This function contains some unordered annotations.
+<xqdoc:xqdoc xmlns:xqdoc="http://www.xqdoc.org/1.0"><xqdoc:control><xqdoc:date/><xqdoc:version>1.0</xqdoc:version></xqdoc:control><xqdoc:module type="library"><xqdoc:uri>http://www.28msec.com/modules/mymodule</xqdoc:uri><xqdoc:name>unorderedAnnotations.xqlib</xqdoc:name><xqdoc:custom tag="namespaces"><xqdoc:namespace prefix="mymod" uri="http://www.28msec.com/modules/mymodule" isSchema="false"/></xqdoc:custom></xqdoc:module><xqdoc:imports/><xqdoc:variables/><xqdoc:functions><xqdoc:function arity="2"><xqdoc:comment><xqdoc:description> This function contains some unordered annotations.
  They should be ordered in the resulting XQDoc XML
  according to the XQDoc schema:
  description author version param return error deprecated see since
-</xqdoc:description><xqdoc:param>$one A dummy param.</xqdoc:param><xqdoc:param>$two One param.</xqdoc:param><xqdoc:return>Place of no return</xqdoc:return><xqdoc:error>Do we throw errors?</xqdoc:error><xqdoc:deprecated/><xqdoc:see>This should be almost at the end</xqdoc:see><xqdoc:since>The beginning of the world.</xqdoc:since></xqdoc:comment><xqdoc:name>mymod:foo</xqdoc:name><xqdoc:signature>declare function mymod:foo($one, $two)</xqdoc:signature><xqdoc:parameters><xqdoc:parameter><xqdoc:name>one</xqdoc:name></xqdoc:parameter><xqdoc:parameter><xqdoc:name>two</xqdoc:name></xqdoc:parameter></xqdoc:parameters></xqdoc:function></xqdoc:functions></xqdoc:xqdoc>
+</xqdoc:description><xqdoc:param>$one A dummy param.</xqdoc:param><xqdoc:param>$two One param.</xqdoc:param><xqdoc:return>Place of no return</xqdoc:return><xqdoc:error>Do we throw errors?</xqdoc:error><xqdoc:deprecated/><xqdoc:see>This should be almost at the end</xqdoc:see><xqdoc:since>The beginning of the world.</xqdoc:since></xqdoc:comment><xqdoc:name>mymod:foo</xqdoc:name><xqdoc:signature>declare function mymod:foo($one, $two)</xqdoc:signature><xqdoc:parameters><xqdoc:parameter><xqdoc:name>one</xqdoc:name></xqdoc:parameter><xqdoc:parameter><xqdoc:name>two</xqdoc:name></xqdoc:parameter></xqdoc:parameters></xqdoc:function></xqdoc:functions></xqdoc:xqdoc>
\ No newline at end of file