add_basemap shows unusual behaviour and does not throw error on misspelling layername

[amanbagrecha]

I was trying out this example and tried adding a basemap which is not in the ee_basemaps list as shown in the image. To my surprise it did not throw any error.

Similar behavior on trying leaflet.Map(google_map="HYbRIDss"). It does not throw any error if the argument is not in the list.

Additionally, even if the argument is correct and the layer is already in self.layers it stills adds another layer on top of the existing one as shown in the image.

I think the problem stems from this line. The if statement returns false instead of throwing an error.

Was not sure if this is a bug so added in discussion. Please guide if there is something I am doing wrong. Thank you!

[giswqs]

@amanbagrecha Thanks for reporting. This bug has been fixed (44a2530). Please update the package using leafmap.update_package() and try again.

[berbagi12]

good sir