Format/GeoJSON does not parse features with null geometries

[bartvde]

When there is no geometry, Geoserver outputs "geometry": null in the GeoJSON output. However OpenLayers cannot parse this and will end up with 0 features.

An easy way to fix this is to add a null check for obj.geometry in parseFeature:

        if (obj.geometry) {
          try {
              geometry = this.parseGeometry(obj.geometry);
          } catch(err) {
              // deal with bad geometries
              throw err;
          }
        }

But there can be more code which assumes that a geometry object exists (like the bounds code probably), so this needs to be discussed more.

[elemoine]

I've looked at  http://wiki.geojson.org/GeoJSON_draft_version_5 and null geometries are indeed something that needs to be expected.

[euzuro]

this seems like a genuine concern.

[elemoine]

In parseGeometry you could just define the geometry local variable a bit later:

parseGeometry: function(obj) { 
    if (obj == null) { 
        return null; 
    }
    var geometry;

no big deal.

Please commit.

[crschmidt]

(In [7689]) "Format.GeoJSON does not parse features with null geometries", r=elemoine (Closes #1145)