Merge pull request #150 from projecthydra/cleaning-up-af-base-exists

Tidying up how ActiveFedora::Base.exists? behaves
samvera · Jul 11, 2013 · fa65b04 · fa65b04
2 parents 7f55fc2 + 71c96cd
commit fa65b04
Showing 1 changed file with 6 additions and 5 deletions.
diff --git a/lib/active_fedora/querying.rb b/lib/active_fedora/querying.rb
@@ -68,13 +68,14 @@ def find_each( conditions={}, opts={})
     end
 
 
-    # Returns true if the pid exists in the repository 
-    # @param[String] pid 
-    # @return[boolean] 
+    # Returns true if the pid exists in the repository
+    # @param[String] pid
+    # @return[boolean]
     def exists?(pid)
       return false if pid.nil? || pid.empty?
-      inner = DigitalObject.find_or_initialize(self, pid)
-      !inner.new?
+      !!DigitalObject.find(self, pid)
+    rescue ActiveFedora::ObjectNotFoundError
+      false
     end
 
     # Returns a solr result matching the supplied conditions