Opened 9 years ago

Closed 9 years ago

#9 closed task (fixed)

WMS Capabilities Store

Reported by: sbenthall Owned by:
Priority: major Milestone: 0.5
Component: GeoExt.data Version:
Keywords: Cc:
State: Review

Description

A Store for WMSCapabilities, where records represent layers that are not necessarily loaded onto any map.

This requires the creation of a WMS Capabilities Reader. This should probably use the WMSCapabilities format from, e.g., here: http://svn.openlayers.org/sandbox/topp/geoext/lib/OpenLayers/Format/

Attachments (5)

9.patch (2.5 KB) - added by tschaub 9 years ago.
add a wms capabilities store
wmscaps-lib.patch (6.8 KB) - added by dwins 9 years ago.
patch with examples and tests
wmscaps-examples.patch (160.4 KB) - added by dwins 9 years ago.
wmscaps-tests.patch (192.2 KB) - added by dwins 9 years ago.
wmscaps-examples.2.patch (152.3 KB) - added by sbenthall 9 years ago.
examples patch without second example

Download all attachments as: .zip

Change History (14)

comment:1 Changed 9 years ago by tschaub

  • Resolution set to wontfix
  • Status changed from new to closed

I think we can use just a plain old layer store instead of a specific WMS capabilities store. The wms capabilities reader is addressed in #26.

comment:2 Changed 9 years ago by tschaub

  • Resolution wontfix deleted
  • Status changed from closed to reopened

Ok, I take that back. It is nice to have helper stores that configure the reader and the proxy.

comment:3 Changed 9 years ago by tschaub

(In [257]) adding a wms capabilities store (see #9)

Changed 9 years ago by tschaub

add a wms capabilities store

comment:4 Changed 9 years ago by tschaub

Patch is here for safe keeping only. Still needs work.

Changed 9 years ago by dwins

patch with examples and tests

Changed 9 years ago by dwins

Changed 9 years ago by dwins

comment:5 Changed 9 years ago by dwins

  • State changed from Needs more work to Review

i added a set of patches (with the sample capabilities documents, a single patch was too large to attach) for this.

comment:6 Changed 9 years ago by sbenthall

Looks great! My one concern is that I think there is some Ext licensed code in wmscaps-examples.patch for the second example. That Row Expander comes right from their example. (I know because I pulled it from there :-/ ).

Rather than blocking the patch on an impossible licensing question, maybe it would be best to just go with the first of the two examples for now?

comment:7 Changed 9 years ago by dwins

+1 on removing the example; it's more a demo of Ext functionality than the GeoExt class anyway.

Changed 9 years ago by sbenthall

examples patch without second example

comment:8 Changed 9 years ago by sbenthall

I see now that the WMSCapabilitiesReader, included in these patches, is listed at #26.

comment:9 Changed 9 years ago by sbenthall

  • Resolution set to fixed
  • Status changed from reopened to closed

(In [453]) WMSCapabilitiesReader and WMSCapabilitiesStore classes, example, and tests. Thanks to tschaub for the code and dwins for the patch. (Closes #9) (Closes #26)

Note: See TracTickets for help on using tickets.