Update non-core module "stack"

Bug #1188061 reported by Chris Hillery
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Zorba
Fix Released
Critical
Luis Rodriguez Gonzalez

Bug Description

Update non-core module "stack". 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 to the "http://zorba.io/modules/stack" namespace, ensuring the documentation is accurate and up to standards, and moving the module contents out of unnecessary com/zorba-xquery/ subdirectories.

Related branches

Chris Hillery (ceejatec)
Changed in zorba:
assignee: nobody → Luis Rodriguez Gonzalez (kuraru)
importance: Undecided → High
milestone: none → 3.0
status: New → Confirmed
description: updated
tags: added: module-cleanup
summary: - This is my summary
+ Update non-core module "stack"
Chris Hillery (ceejatec)
Changed in zorba:
importance: High → Critical
Changed in zorba:
status: Confirmed → In Progress
Revision history for this message
Chris Hillery (ceejatec) wrote :

Per Matthias, it makes sense to update this module to be JSONiq-friendly as well. Currently it only supports node items (which are only XML), but there's no real reason to limit it this way.

All the functions that accept or return "node()" should be changed to accept/return "structured-item()". (structured-item() means "node, JSON object, or JSON array".)

Also, currently the stack module implementation is using node-specific functions in the DML libraries. Those functions are being deprecated and replaced with more general item-based functions. For instance, instead of dml:delete-nodes-first(), use the dml:delete() function which takes an item() rather than a node() argument.

Chris Hillery (ceejatec)
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.