Ticket #2132 (closed feature: fixed)

Opened 4 years ago

Last modified 4 years ago

Add CSW support as an OpenLayers Format

Reported by: bbinet Owned by: bbinet
Priority: minor Milestone: 2.9 Release
Component: Format Version:
Keywords: Cc:
State: Complete

Description

The following OpenLayers Formats will be added to support read/write CSW requests/responses:

  • GetRecords
  • GetDomain

Attachments

CSW_GetRecords_GetDomain.diff Download (37.9 KB) - added by bbinet 4 years ago.
Add support for CSW GetRecords and GetDomain requests
patch_2132-r9482.diff Download (36.9 KB) - added by bbinet 4 years ago.
add parsing of dc elements attributes in csw GetRecordsResponse
patch_2132-r9544.diff Download (37.0 KB) - added by bbinet 4 years ago.
add missing @requires
patch-2132-r9573-A0.diff Download (37.0 KB) - added by elemoine 4 years ago.
remove a IE-killer comma
patch-2132-r9689-A1.diff Download (38.9 KB) - added by elemoine 4 years ago.
patch-2132-r9695-A2.diff Download (38.9 KB) - added by bbinet 4 years ago.

Change History

Changed 4 years ago by bbinet

Add support for CSW GetRecords and GetDomain requests

Changed 4 years ago by bbinet

add parsing of dc elements attributes in csw GetRecordsResponse

Changed 4 years ago by bbinet

Patch patch_2132-r9482.diff Download adds parsing of dc elements attributes in csw GetRecordsResponse to the previous patch.

Changed 4 years ago by bbinet

add missing @requires

Changed 4 years ago by bbinet

Patch patch_2132-r9544.diff Download adds missing @requires on OpenLayers/Format/Filter/<version>.js.

Changed 4 years ago by elemoine

remove a IE-killer comma

Changed 4 years ago by elemoine

Changed 4 years ago by elemoine

  • state changed from Needs More Work to Commit

Bruno, your patch looks really good. patch-2132-r9689-A1.diff Download brings minor modifications here and there:

CSWGetDomain/v2_0_2.js

  • add copyright
  • add PropertyName and ParameterName to prototype
  • make read and write API methods
  • replace new Array() by [] (OL convention)

CSWGetRecords/v2_0_2.js

  • same kinds of changes as for CSWGetDomain
  • change from "if (ResponseHandler && ResponseHandle instanceof Array)" to "if (ResponseHandler instanceof Array)"

Tests pass in FF3. I think this is good to go. I'll wait a couple of days before commit, in case other developers want to have a look..

Changed 4 years ago by bbinet

Changed 4 years ago by bbinet

Thanks for the review Eric. patch-2132-r9695-A2.diff Download simply revert defaultPrefix to "csw" in CSWGetDomain format.

Changed 4 years ago by elemoine

  • keywords CSW GetRecords GetDomain removed
  • status changed from new to closed
  • state changed from Commit to Complete
  • resolution set to fixed

(In [9699]) add formats for CSW GetRecords and GetDomain requests, p=bbinet, r=me (closes #2132)

Note: See TracTickets for help on using tickets.