Ticket #1976 (closed feature: fixed)

Opened 6 years ago

Last modified 6 years ago

Format.WMSGetFeatureInfo: parse geometry if present for msGMLOutput

Reported by: bartvde Owned by: ahocevar
Priority: minor Milestone: 2.8 Release
Component: Format Version: 2.7
Keywords: Cc:
State: Complete

Description


Attachments

ticket1976.patch Download (4.1 KB) - added by bartvde 6 years ago.
patch implementing geometry node parsing for Mapserver
ticket1976-2.patch Download (4.4 KB) - added by bartvde 6 years ago.
new patch taking into account Erik's remark

Change History

Changed 6 years ago by bartvde

patch implementing geometry node parsing for Mapserver

Changed 6 years ago by bartvde

  • state set to Review

Changed 6 years ago by bartvde

  • milestone changed from 2.9 Release to 2.8 Release

Changed 6 years ago by elemoine

Bart, could we avoid creating a new Format.GML object for each feature by caching the first created format:

parseGeometry: function(node) {
    // we need to use the old Format.GML parser since we do not know the  
    // geometry name
    if(!this.gmlFormat) {
        this.gmlFormat = new OpenLayers.Format.GML();
    }
    var feature = this.gmlFormat.parseFeature(node); 
    var geometry = null; 
    if(feature && feature.geometry) { 
        geometry = feature.geometry.clone(); 
        feature.destroy(); 
    } 
    return geometry; 
},

Changed 6 years ago by bartvde

Hi Eric, sure makes sense, I'll adapt the patch.

Changed 6 years ago by bartvde

new patch taking into account Erik's remark

Changed 6 years ago by tschaub

  • owner changed from tschaub to ahocevar

Changed 6 years ago by ahocevar

  • status changed from new to assigned

Changed 6 years ago by ahocevar

  • status changed from assigned to closed
  • state changed from Review to Complete
  • resolution set to fixed

(In [9178]) Added geometry parsing for Mapserver output. Thanks bartvde for the patch. r=elemoine,me (closes #1976)

Note: See TracTickets for help on using tickets.