enforce doi types. #331

app/models/doi.rb

@@ -749,6 +749,18 @@ def content_url=(value)
     write_attribute(:content_url, Array.wrap(value))
   end
 
+  def container=(value)
+    write_attribute(:container, value || {})
+  end
+
+  def types=(value)
+    write_attribute(:types, value || {})
+  end
+
+  def landing_page=(value)
+    write_attribute(:landing_page, value || {})
+  end
+
   def identifier
     normalize_doi(doi, sandbox: !Rails.env.production?)
   end

app/serializers/doi_serializer.rb

@@ -29,9 +29,9 @@ class DoiSerializer
     Array.wrap(object.creators).map do |c|
       c["affiliation"] = Array.wrap(c["affiliation"]).map do |a|
         if params[:affiliation]
-          a.is_a?(Hash) ? a : { "name" => a }.compact
+          a
         else
-          a.is_a?(Hash) ? a["name"] : a
+          a["name"] 
         end
       end.compact
       c
@@ -44,9 +44,9 @@ class DoiSerializer
     Array.wrap(object.contributors).map do |c|
       c["affiliation"] = Array.wrap(c["affiliation"]).map do |a|
         if params[:affiliation]
-          a.is_a?(Hash) ? a : { "name" => a }.compact
+          a
         else
-          a.is_a?(Hash) ? a["name"] : a
+          a["name"] 
         end
       end.compact
       c
@@ -57,6 +57,14 @@ class DoiSerializer
     Array.wrap(object.rights_list)
   end
 
+  attribute :container do |object|
+    object.container || {}
+  end
+
+  attribute :types do |object|
+    object.types || {}
+  end
+
   attribute :state do |object|
     object.aasm_state
   end

spec/models/doi_spec.rb

@@ -642,7 +642,7 @@
       let(:doi) { create(:doi, container: container)}
 
       it "convert" do
-        expect(Doi.convert_container_by_id(id: doi.id)).to eq(1)
+        expect(Doi.convert_container_by_id(id: doi.id)).to eq(0)
       end
     end