Merge pull request #668 from avalonmediasystem/bugfix/api_logging

Add logging for when invalid fields are erased
avalonmediasystem · Jan 21, 2016 · f4fea6e · f4fea6e
2 parents eaed38d + 8989df2
commit f4fea6e
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/app/controllers/media_objects_controller.rb b/app/controllers/media_objects_controller.rb
@@ -105,8 +105,11 @@ def update_mediaobject
       invalid_fields = @mediaobject.errors.keys
       required_fields = [:title, :date_issued, :creator]
       if !required_fields.any? { |f| invalid_fields.include? f }
-        #NOTE this will erase all values for fields with multiple values
-        invalid_fields.each { |field| @mediaobject[field] = nil }
+        invalid_fields.each do |field|
+          #NOTE this will erase all values for fields with multiple values
+          logger.warn "Erasing field #{field} with bad value, bibID: #{Array(params[:fields][:bibliographic_id]).first}, avalon ID: #{@mediaobject.pid}" 
+          @mediaobject[field] = nil
+        end
       end
     end
     if !@mediaobject.save