Remove calls to SHAMapNode::getID

This addresses RIPD-347
Calls in SHAMap::getCache and SHAMap::canonicalize are
purposefully left in place at this time (to be removed
later).
I think this is the first time I've seen std::tie() used in the wild.