Missing nogo weight when loading nogos

[Phyks]

Hi,

It seems loading nogos from a GeoJSON while specifying a (positive) weight does not work and the URL for the query to the BRouter server is not updated accordingly.

It seems that when loading GeoJSON data here, the properties of the features in the GeoJSON object are not loaded by Leaflet (they should be loaded into options in Leaflet, for this part to work).

I am really not sure what is happening here, I'm almost sure this was fully tested and working when the PR got merged. Maybe this is due to a Leaflet update?

Best,

[nrenner]

feature.properties are not loaded into layer.options, only the options object that is passed to the GeoJSON is copied to all layers.

The commit history is lost because of squashing, but I think you first passed nogoWeight as an option to L.geoJson, and later changed it to onEachFeature to read nogoWeight from the file?

brouter-web/js/index.js

Lines 417 to 421 in 5f89287

	onEachFeature: function (feature, layer) {
	if (!feature.properties.nogoWeight) {
	feature.properties.nogoWeight = nogoWeight;
	}
	}

I would prefer to manually set the layer option here instead of PR #175, something like this:

                    onEachFeature: function (feature, layer) {
                        layer.options.nogoWeight = feature.properties.nogoWeight || nogoWeight;
                    }

[Phyks]

Indeed, this is a very nice idea to have it handled this way. I updated the PR accordingly, thanks for the feedback!