Get rid of ScopedCollectionMetadata's operator bool

Since for the purposes of SERVER-32198 we are going in the direction of all collections (both sharded and unsharded) having a CollectionMetadata associated with them, the bool operator on ScopedCollectionMetadata is no longer necessary. Instead we should have a property on CollectionMetadata to indicate whether the collection is sharded or not and that should be the canonical way.