Reorganize coauthor forms

Author: lmrodriguezr

app/controllers/registers_controller.rb

@@ -264,6 +264,9 @@ def validate
 
   # GET /registers/r:abc/coauthors
   def coauthors
+    @crumbs = [
+      ['Lists', registers_url], [@register.acc_url, @register], 'Coauthors'
+    ]
   end
 
   # POST /registers/r:abc/coauthors
@@ -273,7 +276,7 @@ def coauthors_commit
     if @coauthor.present?
       rc_par = { register: @register, user: @coauthor }
       success =
-        case params['action']
+        case params['register']['action']
         when 'unlink'
           RegisterCoauthor.find_by(rc_par).destroy
         when 'up'
@@ -288,10 +291,12 @@ def coauthors_commit
         redirect_back(fallback_location: @register)
       else
         flash.now[:alert] = 'Error updating coauthors'
+        coauthors
         render :coauthors
       end
     else
       @register.add_error(:coauthor, 'does not exist in the system')
+      coauthors
       render :coauthors
     end
   end

app/views/registers/coauthors.html.erb

@@ -23,8 +23,7 @@
         <%= display_link(author) %>
         <%= link_to(
               coauthors_register_url(@register, {
-                action: :unlink,
-                register: { coauthor: author.username }
+                register: { coauthor: author.username, action: :unlink }
               }),
               class: 'badge badge-pill badge-primary',
               method: :post, title: 'Unlink coauthor',
@@ -35,8 +34,7 @@
         <% unless rc.first? %>
           <%= link_to(
                 coauthors_register_url(@register, {
-                  action: :up,
-                  register: { coauthor: author.username }
+                  register: { coauthor: author.username, action: :up }
                 }),
                 class: 'badge badge-pill badge-primary',
                 method: :post, title: 'Move coauthor up'