Merge pull request #187 from datacite/schema-4.5-post-release

Revert temporary mod to DC JSON publisher that only allowed publisher…

Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    bolognese (2.1.0)
+    bolognese (2.1.1)
       activesupport (>= 4.2.5)
       benchmark_methods (~> 0.7)
       bibtex-ruby (>= 5.1.0)

lib/bolognese/version.rb

@@ -1,3 +1,3 @@
 module Bolognese
-  VERSION = "2.1.0"
+  VERSION = "2.1.1"
 end

lib/bolognese/writers/datacite_json_writer.rb

@@ -4,11 +4,8 @@ module Bolognese
   module Writers
     module DataciteJsonWriter
       def datacite_json
-        # Remove the following change for the schema 4.5 release
         if crosscite_hsh.present?
-          datacite_json_hsh = crosscite_hsh
-          datacite_json_hsh['publisher'] = self.publisher['name'] if self.publisher&.respond_to?(:to_hash) && self.publisher.has_key?('name') && !self.publisher['name'].blank?
-          JSON.pretty_generate datacite_json_hsh.transform_keys! { |key| key.camelcase(uppercase_first_letter = :lower) }
+          JSON.pretty_generate crosscite_hsh.transform_keys! { |key| key.camelcase(uppercase_first_letter = :lower) }
         end
       end
     end

spec/writers/datacite_json_writer_spec.rb

@@ -18,9 +18,7 @@
         "rightsIdentifierScheme"=>"SPDX",
         "rightsUri"=>"https://creativecommons.org/licenses/by/3.0/legalcode",
         "schemeUri"=>"https://spdx.org/licenses/"}])
-      # REVERT the publisher line to the commented line for the schema 4.5 release
-      # expect(datacite.fetch("publisher")).to eq({"name"=>"eLife Sciences Publications, Ltd"})
-      expect(datacite.fetch("publisher")).to eq("eLife Sciences Publications, Ltd")
+      expect(datacite.fetch("publisher")).to eq({"name"=>"eLife Sciences Publications, Ltd"})
     end
 
     it "with ORCID ID" do
@@ -31,9 +29,7 @@
       expect(datacite.fetch("types")).to eq("bibtex"=>"article", "citeproc"=>"article-journal", "resourceType"=>"JournalArticle", "resourceTypeGeneral"=>"JournalArticle", "ris"=>"JOUR", "schemaOrg"=>"ScholarlyArticle")
       expect(datacite.fetch("creators").length).to eq(7)
       expect(datacite.fetch("creators").first).to eq("nameType"=>"Personal", "name"=>"Thanassi, Wendy", "givenName"=>"Wendy", "familyName"=>"Thanassi", "affiliation" => [{"name"=>"Department of Medicine, Veterans Affairs Palo Alto Health Care System, 3801 Miranda Avenue MC-, Palo Alto, CA 94304-1207, USA"}, {"name"=>"Occupational Health Strategic Health Care Group, Office of Public Health, Veterans Health Administration, Washington, DC 20006, USA"}, {"name"=>"Division of Emergency Medicine, Stanford University School of Medicine, Stanford, CA 94304, USA"}, {"name"=>"War Related Illness and Injury Study Center (WRIISC) and Mental Illness Research Education and Clinical Center (MIRECC), Department of Veterans Affairs, Palo Alto, CA 94304, USA"}])
-      # REVERT the publisher line to the commented line for the schema 4.5 release
-      # expect(datacite.fetch("publisher")).to eq({"name"=>"Hindawi Limited"})
-      expect(datacite.fetch("publisher")).to eq("Hindawi Limited")
+      expect(datacite.fetch("publisher")).to eq({"name"=>"Hindawi Limited"})
     end
 
     it "Crossref DOI" do
@@ -45,9 +41,7 @@
       expect(datacite.dig("descriptions", 0, "description")).to start_with("Among various advantages, their small size makes model organisms preferred subjects of investigation.")
       expect(datacite.fetch("creators").length).to eq(5)
       expect(datacite.fetch("creators").first).to eq("nameType"=>"Personal", "name"=>"Sankar, Martial", "givenName"=>"Martial", "familyName"=>"Sankar")
-      # REVERT the publisher line to the commented line for the schema 4.5 release
-      # expect(datacite.fetch("publisher")).to eq({"name"=>"{eLife} Sciences Organisation, Ltd."})
-      expect(datacite.fetch("publisher")).to eq("{eLife} Sciences Organisation, Ltd.")
+      expect(datacite.fetch("publisher")).to eq({"name"=>"{eLife} Sciences Organisation, Ltd."})
     end
 
     it "BlogPosting Citeproc JSON" do
@@ -58,9 +52,7 @@
       expect(datacite.fetch("titles")).to eq([{"title"=>"Eating your own Dog Food"}])
       expect(datacite.dig("descriptions", 0, "description")).to start_with("Eating your own dog food")
       expect(datacite.fetch("creators")).to eq([{"familyName"=>"Fenner", "givenName"=>"Martin", "name"=>"Fenner, Martin"}])
-      # REVERT the publisher line to the commented line for the schema 4.5 release
-      # expect(datacite.fetch("publisher")).to eq({"name"=>"DataCite"})
-      expect(datacite.fetch("publisher")).to eq("DataCite")
+      expect(datacite.fetch("publisher")).to eq({"name"=>"DataCite"})
     end
 
     it "rdataone" do
@@ -72,9 +64,7 @@
       expect(datacite.fetch("creators").first).to eq("affiliation"=>[{"name"=>"NCEAS"}], "nameType"=>"Personal", "name" => "Jones, Matt",
         "nameIdentifiers" => [{"nameIdentifier"=>"https://orcid.org/0000-0003-0077-4738", "nameIdentifierScheme"=>"ORCID", "schemeUri"=>"https://orcid.org"}], "givenName"=>"Matt", "familyName"=>"Jones")
       expect(datacite.fetch("version")).to eq("2.0.0")
-      # REVERT the publisher line to the commented line for the schema 4.5 release
-      # expect(datacite.fetch("publisher")).to eq({"name"=>"https://cran.r-project.org"})
-      expect(datacite.fetch("publisher")).to eq("https://cran.r-project.org")
+      expect(datacite.fetch("publisher")).to eq({"name"=>"https://cran.r-project.org"})
     end
 
     it "maremma" do
@@ -90,9 +80,7 @@
          [{"nameIdentifier"=>"https://orcid.org/0000-0003-0077-4738",
            "nameIdentifierScheme"=>"ORCID", "schemeUri"=>"https://orcid.org"}],
         "nameType"=>"Personal"}])
-      # REVERT the publisher line to the commented line for the schema 4.5 release
-      # expect(datacite.fetch("publisher")).to eq({"name"=>"DataCite"})
-      expect(datacite.fetch("publisher")).to eq("DataCite")
+      expect(datacite.fetch("publisher")).to eq({"name"=>"DataCite"})
     end
 
     it "with data citation schema.org" do
@@ -103,26 +91,22 @@
       expect(datacite.fetch("titles")).to eq([{"title"=>"Eating your own Dog Food"}])
       expect(datacite.fetch("relatedIdentifiers").count).to eq(3)
       expect(datacite.fetch("relatedIdentifiers").first).to eq("relatedIdentifier"=>"10.5438/0000-00ss", "relatedIdentifierType"=>"DOI", "relationType"=>"IsPartOf", "resourceTypeGeneral"=>"Text")
-      # REVERT the publisher line to the commented line for the schema 4.5 release
-      # expect(datacite.fetch("publisher")).to eq({"name"=>"DataCite"})
-      expect(datacite.fetch("publisher")).to eq("DataCite")
+      expect(datacite.fetch("publisher")).to eq({"name"=>"DataCite"})
     end
 
     it "DataCite DOI with Schema 4.5 publisher attributes" do
       input = fixture_path + "datacite-example-full-v4.5.xml"
       subject = Bolognese::Metadata.new(input: input, from: "datacite")
       datacite = JSON.parse(subject.datacite_json)
-      # REVERT the publisher line to the commented lines for the schema 4.5 release
-      # expect(datacite.fetch("publisher")).to eq(
-      #   {
-      #     "name" => "Example Publisher",
-      #     "publisherIdentifier" => "https://ror.org/04z8jg394",
-      #     "publisherIdentifierScheme" => "ROR",
-      #     "schemeUri" => "https://ror.org/",
-      #     "lang" => "en",
-      #   },
-      # )
-      expect(datacite.fetch("publisher")).to eq("Example Publisher")
+      expect(datacite.fetch("publisher")).to eq(
+        {
+          "name" => "Example Publisher",
+          "publisherIdentifier" => "https://ror.org/04z8jg394",
+          "publisherIdentifierScheme" => "ROR",
+          "schemeUri" => "https://ror.org/",
+          "lang" => "en",
+        },
+      )
     end
   end
 end