Move processing of null parameters over to update_publisher.

Gemfile.lock

@@ -1,6 +1,6 @@
 GIT
   remote: https://github.com/datacite/bolognese
-  revision: ca90d2c7e33fdd76d0ee7ad00434740a324166df
+  revision: 714d71a4a2169d1880ccb296df3ca1331e0e1a83
   branch: schema-4.5
   specs:
     bolognese (1.11.5)

app/controllers/datacite_dois_controller.rb

@@ -797,10 +797,6 @@ def safe_params
       ParamsSanitizer.sanitize_nameIdentifiers(params[:creators])
       ParamsSanitizer.sanitize_nameIdentifiers(params[:contributors])
 
-      if params.dig(:data, :attributes, :publisher).present? && params.dig(:data, :attributes, :publisher).respond_to?(:to_hash)
-        params[:data][:attributes][:publisher].compact!
-      end
-
       p =
         params.require(:data).permit(
           :type,

app/models/doi.rb

@@ -104,11 +104,14 @@ class Doi < ApplicationRecord
   validates_presence_of :doi
   validates_presence_of :url, if: Proc.new { |doi| doi.is_registered_or_findable? }
 
-  validates :publisher_obj, if: :publisher_obj? && Proc.new { |doi| doi.validatable? },
-  json: {
-    message: ->(errors) { errors },
-    schema: PUBLISHER_JSON_SCHEMA
-  }
+validates :publisher_obj, if: :publisher_obj? && Proc.new { |doi| 
+  doi.validatable? && 
+  !(doi.publisher.blank? || doi.publisher.all?(nil))
+},
+json: {
+  message: ->(errors) { errors },
+  schema: PUBLISHER_JSON_SCHEMA
+}
 
   # from https://www.crossref.org/blog/dois-and-matching-regular-expressions/ but using uppercase
   validates_format_of :doi, with: /\A10\.\d{4,5}\/[-._;()\/:a-zA-Z0-9*~$=]+\z/, on: :create
@@ -2297,8 +2300,19 @@ def update_types
 
   def update_publisher
     if publisher_before_type_cast.respond_to?(:to_hash)
-      self.publisher_obj = publisher_before_type_cast
-      self.publisher = publisher_before_type_cast.dig(:name)
+      if !(publisher_before_type_cast.blank? || publisher_before_type_cast.values.all?(nil))
+        self.publisher_obj = {
+          :name => publisher_before_type_cast.fetch(:name, nil),
+          :lang => publisher_before_type_cast.fetch(:lang, nil), 
+          :schemeUri => publisher_before_type_cast.fetch(:schemeUri, nil),
+          :publisherIdentifier => publisher_before_type_cast.fetch(:publisherIdentifier, nil),
+          :publisherIdentifierScheme => publisher_before_type_cast.fetch(:publisherIdentifierScheme, nil)
+        }.compact
+        self.publisher = publisher_before_type_cast.dig(:name)
+      else
+        self.publisher_obj = nil
+        self.publisher = nil
+      end
     elsif publisher_before_type_cast.respond_to?(:to_str)
       self.publisher_obj = { :name => publisher_before_type_cast }
       self.publisher = publisher_before_type_cast