Problem with multiPoly getLatLngs()

[tsurrel]

In Chrome and Firefox up-to-date, there is a javascript error saying that getLatLngs is not a function for multiPolyline and multiPolygon (it works perfectly well with Polyline or Polygon).

You can see the error here:
http://jsfiddle.net/qGM9N/2/

Regards,

Thomas

[mourner]

Will check this out, thanks.

[bobjackman]

This is also the case with MultiPolygons I've written my own for my needs... not sure why I never submitted a pull request...

Leaflet.MultiPolygon.prototype.getLatLngs = function() {
    var polygons = [];

    this.eachLayer( function( layer ) {
        polygons.push( layer._latlngs );
        if( layer._holes && layer._holes.length ) {
            polygons.addObjects( layer._holes );
        }
    }, this );

    return polygons;
};

On second thought, I think it's because we're using an Ember.js stack and I'm using their addObjects array method. Should be easy enough to work around, though... Haven't tested it, but this should work.

Instead of:

polygons.addObjects( layer._holes )

use:

polygons = polygons.concat( layer._holes )

[mourner]

Thanks, fixed! It's very weird that it was indeed documented... Don't remember if the metod was implemented in the first place.

[tsurrel]

Thanks for the quick fix !