This is just code cleanup.
Currently CollectionShardingState::_getMetadataWithVersionCheckAt returns an "UnshardedCollection" ScopedCollectionMetadata, but it should just return boost::none, since its caller (getOrphansFilter) converts a boost::none return value into an "UnshardedCollection" ScopedCollectionMetadata anyway.