minor: Not selecting any layers -> invalid javascript

[carolinux]

My first use of the plugin and I forgot to add any layers -> the result was some javascript which was missing a '{'. I think the plugin could warn when trying to do this.

When I actually added a layer, no complaints :)

[tomchadwin]

No, it's important because some people might need simply an embedded OSM. Will investigate.

[carolinux]

I found the problem. line 1656:

f8.seek(-1, os.SEEK_END)

This deletes the last character, ie the bracket.

[tomchadwin]

So is it incorrect even if some layers are selected and included in the map? We don't want to introduce errors in other circumstances.

[carolinux]

Yes, apparently this is so to delete the last comma added when layers are added (because the layer code is added with a comma at the end everytime), so that's why it works in the case of existing layers.

[carolinux]

You can use

",".join(["foo","bar","baz"])

renders foo,bar,baz
no need to worry about the last comma