Fixing BasicMapId constructor

There is a really silly bug in the BasicMapId constructor that takes a Map.   The constructor just ends up copying the map into itself due to scoping.