Opened 6 years ago

Closed 6 years ago

#441 closed enhancement (fixed)

add infoFormats to WMSCapabilitiesReader

Reported by: bartvde Owned by:
Priority: minor Milestone: 1.1
Component: GeoExt.data.WMSCapabilitiesReader Version: 1.0
Keywords: Cc:
State: Commit

Description

Currently the WMSCapabilitiesReader takes the list of formats from the GetMap Format list, this ticket proposes to add infoFormats in a similar way (taking them from the GetFeatureInfo Format list in the GetCapabilities response).

Attachments (2)

geoext-441.patch (2.5 KB) - added by bartvde 6 years ago.
geoext-441.2.patch (3.0 KB) - added by ahocevar 6 years ago.
tests also pass with OL versions that don't parse infoFormats

Download all attachments as: .zip

Change History (7)

comment:1 Changed 6 years ago by bartvde

I see now that OpenLayers already sets layer.formats, so if we would want to do the same thing, we would also need to set layer.infoFormats in OpenLayers.

comment:2 Changed 6 years ago by bartvde

See http://trac.osgeo.org/openlayers/ticket/3539 for the respective OL ticket.

Changed 6 years ago by bartvde

comment:3 Changed 6 years ago by bartvde

Tests pass in FF6.

I'll set this one to review once the OpenLayers patch gets in.

Changed 6 years ago by ahocevar

tests also pass with OL versions that don't parse infoFormats

comment:4 Changed 6 years ago by ahocevar

  • State changed from None to Commit

Bart, if you agree with my change to the tests (see geoext-441.2.patch), please commit.

comment:5 Changed 6 years ago by bartvde

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

(In [2855]) add infoFormats to WMSCapabilitiesReader, r=ahocevar (closes #441)

Note: See TracTickets for help on using tickets.