Merge pull request #750 from datacite/issue-749

Issue #749 - API error when calling /export/contacts. (Add array.wrap)
datacite · Jul 28, 2021 · d0da95e · d0da95e
2 parents 7c332ef + b929175
commit d0da95e
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/app/controllers/exports_controller.rb b/app/controllers/exports_controller.rb
@@ -31,7 +31,7 @@ def contacts
         "email" => contact.email,
         "firstName" => contact.given_name,
         "lastName" => contact.family_name.present? ? contact.family_name : contact.email,
-        "type" => contact.role_name ? contact.role_name.map { |r| r.camelize(:lower) }.join(";") : nil,
+        "type" => contact.role_name ? Array.wrap(contact.role_name).map { |r| r.camelize(:lower) }.join(";") : nil,
         "createdAt" => export_date_string(contact.created_at),
         "modifiedAt" => export_date_string(contact.updated_at),
         "deletedAt" => contact.deleted_at.present? ? export_date_string(contact.deleted_at) : nil,

diff --git a/app/controllers/v3/exports_controller.rb b/app/controllers/v3/exports_controller.rb
@@ -32,7 +32,7 @@ def contacts
           "email" => contact.email,
           "firstName" => contact.given_name,
           "lastName" => contact.family_name,
-          "type" => contact.role_name ? contact.role_name.join(";") : nil,
+          "type" => contact.role_name ? Array.wrap(contact.role_name).join(";") : nil,
           "createdAt" => export_date_string(contact.created_at),
           "modifiedAt" => export_date_string(contact.updated_at),
           "deletedAt" => contact.deleted_at.present? ? export_date_string(contact.deleted_at) : nil,