Don't try to parse nil dates. Fixes #937

samvera · Nov 2, 2015 · 9a69ffa · 9a69ffa
1 parent 95ee653
commit 9a69ffa
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/lib/active_fedora/loadable_from_json.rb b/lib/active_fedora/loadable_from_json.rb
@@ -178,7 +178,7 @@ def date_attribute?(attribute_name)
       # @param attribute_name [String] the name of the attribute to adapt
       # @return [Object] the adapted value
       def adapt_single_attribute_value(value, attribute_name)
-        if date_attribute?(attribute_name)
+        if value && date_attribute?(attribute_name)
           DateTime.parse(value)
         else
           value

diff --git a/spec/integration/date_time_properties_spec.rb b/spec/integration/date_time_properties_spec.rb
@@ -8,6 +8,7 @@ class Foo < ActiveFedora::Base
         index.type :date
       end
       property :single_date, multiple: false, class_name: 'DateTime', predicate: ::RDF::URI.new('http://www.example.com/single_date')
+      property :missing_date, multiple: false, class_name: 'DateTime', predicate: ::RDF::URI.new('http://www.example.com/missing_date')
     end
   end