Update core module "http-client"

Bug #1189791 reported by Chris Hillery
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Zorba
Fix Committed
High
Federico Cavalieri

Bug Description

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

For this module, we are keeping the old module with the old namespace and API. However, we should still do at least the following changes:

* Ensure the documentation is accurate and up to standards
* Move the module contents out of unnecessary com/zorba-xquery/ subdirectories. The path for the module is currently modules/com/zorba-xquery/www/modules/http-client.xq .

As discussed on the merge proposal, we also need to eliminate code redundancy; the easiest (?) way to do this is probably to convert the old module to a wrapper around the new one, with only UDFs and no C++ code.

Related branches

Chris Hillery (ceejatec)
Changed in zorba:
assignee: nobody → Matthias Brantner (matthias-brantner)
importance: Undecided → High
milestone: none → 3.0
status: New → Confirmed
description: updated
tags: added: module-cleanup
summary: - This is my summary
+ Update core module "http-client"
Changed in zorba:
assignee: Matthias Brantner (matthias-brantner) → Federico Cavalieri (fcavalieri)
Chris Hillery (ceejatec)
Changed in zorba:
status: Confirmed → In Progress
Revision history for this message
Federico Cavalieri (fcavalieri) wrote : Re: [Bug 1189791] Re: Update core module "http-client"

This applies to the old http-module, correct?
Thanks
Federico

On Wed, Jul 10, 2013 at 8:54 AM, Chris Hillery
<email address hidden>wrote:

> ** Changed in: zorba
> Status: Confirmed => In Progress
>
> ** Branch linked: lp:~zorba-coders/zorba/json-http-module
>
> --
> You received this bug notification because you are a bug assignee.
> https://bugs.launchpad.net/bugs/1189791
>
> Title:
> Update core module "http-client"
>
> Status in Zorba - NoSQL Query Processor:
> In Progress
>
> Bug description:
> Update core module "http-client". 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/http-client" namespace, ensuring the
> documentation is accurate and up to standards, and moving the module
> contents out of unnecessary com/zorba-xquery/ subdirectories. The path
> for the module is currently modules/com/zorba-xquery/www/modules/http-
> client.xq .
>
> Additional tasks for this module:
> * Convert to fully JSONiq module
> * Also update implementation of non-core EXPath http-client module
> accordingly
> * Also remove http-client-error.xq module
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/zorba/+bug/1189791/+subscriptions
>

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

Yes, this bug was filed regarding the original
http://www.zorba-xquery.com/modules/http-client module. It's kind of out of
date now since we decided to keep the old module and create a new one; I'll
update the description.

Ceej
aka Chris Hillery

On Thu, Jul 11, 2013 at 1:01 PM, Federico Cavalieri <
<email address hidden>> wrote:

> This applies to the old http-module, correct?
> Thanks
> Federico
>
>
> On Wed, Jul 10, 2013 at 8:54 AM, Chris Hillery
> <email address hidden>wrote:
>
> > ** Changed in: zorba
> > Status: Confirmed => In Progress
> >
> > ** Branch linked: lp:~zorba-coders/zorba/json-http-module
> >
> > --
> > You received this bug notification because you are a bug assignee.
> > https://bugs.launchpad.net/bugs/1189791
> >
> > Title:
> > Update core module "http-client"
> >
> > Status in Zorba - NoSQL Query Processor:
> > In Progress
> >
> > Bug description:
> > Update core module "http-client". 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/http-client" namespace, ensuring the
> > documentation is accurate and up to standards, and moving the module
> > contents out of unnecessary com/zorba-xquery/ subdirectories. The path
> > for the module is currently modules/com/zorba-xquery/www/modules/http-
> > client.xq .
> >
> > Additional tasks for this module:
> > * Convert to fully JSONiq module
> > * Also update implementation of non-core EXPath http-client module
> > accordingly
> > * Also remove http-client-error.xq module
> >
> > To manage notifications about this bug go to:
> > https://bugs.launchpad.net/zorba/+bug/1189791/+subscriptions
> >
>
> --
> You received this bug notification because you are subscribed to the bug
> report.
> https://bugs.launchpad.net/bugs/1189791
>
> Title:
> Update core module "http-client"
>
> Status in Zorba - NoSQL Query Processor:
> In Progress
>
> Bug description:
> Update core module "http-client". 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/http-client" namespace, ensuring the
> documentation is accurate and up to standards, and moving the module
> contents out of unnecessary com/zorba-xquery/ subdirectories. The path
> for the module is currently modules/com/zorba-xquery/www/modules/http-
> client.xq .
>
> Additional tasks for this module:
> * Convert to fully JSONiq module
> * Also update implementation of non-core EXPath http-client module
> accordingly
> * Also remove http-client-error.xq module
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/zorba/+bug/1189791/+subscriptions
>

description: updated
Chris Hillery (ceejatec)
Changed in zorba:
status: In Progress → Fix Committed
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.