Remove reliance on noid for find (#6924)

* Remove reliance on noid for find

Switches from using noids to using a fallback... look first by id,
and then by alternate_identifier instead.

I created a work in dassie with VALKYRIE_TRANSITION=true and then I was
unable to edit the work because I don’t have an alternative identifier.

* Rearrange code to allow fallback to alternative_id

lib/hyrax/valkyrie_can_can_adapter.rb

@@ -19,10 +19,17 @@ def self.for_class?(member_class)
     #
     # @raise Hyrax::ObjectNotFoundError
     def self.find(_model_class, id)
-      return Hyrax.query_service.find_by(id: id) unless Hyrax.config.enable_noids?
+      self.find_by(id:) ||
       Hyrax.query_service.find_by_alternate_identifier(alternate_identifier: id)
     rescue Valkyrie::Persistence::ObjectNotFoundError => err
       raise Hyrax::ObjectNotFoundError, err.message
     end
+
+    private
+
+    def self.find_by(id:)
+      Hyrax.query_service.find_by(id:)
+    rescue Valkyrie::Persistence::ObjectNotFoundError
+    end
   end
 end