Update core "store" modules

Bug #1189862 reported by Chris Hillery
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Zorba
Fix Released
Critical
Paul J. Lucas

Bug Description

Update core "store" modules. Refer to the Zorba 3.0 Module Update Checklist at http://my.zorba-xquery.com/dokuwiki/doku.php?id=module_update .

This minimally means renaming the module, ensuring the documentation is accurate and up to standards, and moving the module contents out of unnecessary com/zorba-xquery/ subdirectories.

The modules in question currently reside in:

com/zorba-xquery/www/modules/store/documents/dynamic.xq - rename
com/zorba-xquery/www/modules/store/dynamic/collections/ddl.xq - rename
com/zorba-xquery/www/modules/store/dynamic/collections/dml.xq - rename
com/zorba-xquery/www/modules/store/dynamic/collections/w3c/ddl.xq - rename
com/zorba-xquery/www/modules/store/dynamic/collections/w3c/dml.xq - rename
com/zorba-xquery/www/modules/store/static/collections/ddl.xq - rename
com/zorba-xquery/www/modules/store/static/collections/dml.xqy - rename
com/zorba-xquery/www/modules/store/static/indexes/ddl.xq - rename, JSONify [28msec]
com/zorba-xquery/www/modules/store/static/indexes/dml.xq - rename, JSONify [28msec]
com/zorba-xquery/www/modules/store/static/integrity_constraints/ddl.xq - rename
com/zorba-xquery/www/modules/store/static/integrity_constraints/dml.xq - rename

As noted most simply need re-naming, re-directorying, and doc cleanup. The static-indexes modules should be made into JSONiq modules.

I have filed a single bug for all of these as it makes sense that a single person handle them all in a consistent manner. In particular I am not sure whether these modules should be an exception to the "no hierarchy in the namespace URI" rule - perhaps they should be http://zorba.io/modules/store/dynamic-documents etc., or even http://zorba.io.modules/store/documents/dynamic etc.

Also, the current naming seems inconsistent, or at least confusing - why "store/documents/dynamic", but "store/dynamic/collections", for instance?

Related branches

Chris Hillery (ceejatec)
Changed in zorba:
assignee: nobody → Matthias Brantner (matthias-brantner)
tags: added: module-cleanup
removed: matthias-brantner
Revision history for this message
Chris Hillery (ceejatec) wrote :

Additional request #1: Remove all @deprecated functions through these modules.

Revision history for this message
Chris Hillery (ceejatec) wrote :

Additional request #2: At least some modules (such as dynamic/colletions/dml) accept "item()" arguments when they actually only work with structured-item(). This was done to make the module parseable in pure XQuery, but that no longer seems to be a relevant goal. Therefore we should change to use structured-item() in all places that it makes sense.

Revision history for this message
Chris Hillery (ceejatec) wrote :

Per bug 1180221 - please rename the integrity_constraints modules to integrity-constraints.

Chris Hillery (ceejatec)
Changed in zorba:
importance: High → Critical
Revision history for this message
Chris Hillery (ceejatec) wrote :

For now, just rename the two "indexes" modules, don't JSONify.

Changed in zorba:
assignee: Matthias Brantner (matthias-brantner) → Paul J. Lucas (paul-lucas)
Changed in zorba:
status: Confirmed → In Progress
Changed in zorba:
status: In Progress → Fix Committed
Changed in zorba:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.