Fix param conversion issue

Author: fbacall

app/models/concerns/has_people.rb

@@ -26,7 +26,8 @@ def set_people_for_role(value, role_name, role_key)
     current_people = send(role_name).to_a
     to_keep = []
 
-    Array(value).reject(&:blank?).map do |person_data|
+    value = value.values if value.respond_to?(:keys) && value.keys.first.match(/\A\d+\z/)
+    Array.wrap(value).reject(&:blank?).map do |person_data|
       person_data = person_data.to_h if person_data.is_a?(ActionController::Parameters)
       if person_data.is_a?(String)
         attrs = { name: person_data.strip }

test/controllers/materials_controller_test.rb

@@ -1659,4 +1659,17 @@ class MaterialsControllerTest < ActionController::TestCase
     assert_equal 'Joe', mat.authors.first[:name]
     assert_nil mat.authors.first[:orcid]
   end
+
+  test 'should update material authors using param structured authors' do
+    material = materials(:material_with_optionals)
+    sign_in material.user
+    update = { authors: { '1234' => {
+      name: 'Joe', orcid: '0000-0002-1825-0097' } } }
+    patch :update, params: { id: material, material: update }
+    mat = assigns(:material)
+    assert_redirected_to material_path(mat)
+
+    assert_equal 'Joe', mat.authors.first[:name]
+    assert_equal '0000-0002-1825-0097', mat.authors.first[:orcid]
+  end
 end