Coauthors actions implemented

Author: lmrodriguezr

app/controllers/registers_controller.rb

@@ -268,6 +268,32 @@ def coauthors
 
   # POST /registers/r:abc/coauthors
   def coauthors_commit
+    @register.coauthor = params.require('register').require('coauthor')
+    @coauthor = User.find_by_email_or_username(@register.coauthor)
+    if @coauthor.present?
+      rc_par = { register: @register, user: @coauthor }
+      success =
+        case params['action']
+        when 'unlink'
+          RegisterCoauthor.find_by(rc_par).destroy
+        when 'up'
+          RegisterCoauthor.find_by(rc_par).move_up
+        else
+          rc_par.merge!(order: @register.register_coauthors.size + 1)
+          RegisterCoauthor.create(rc_par)
+        end
+      
+      if success
+        flash[:notice] = 'Successfully updated coauthors'
+        redirect_back(fallback_location: @register)
+      else
+        flash.now[:alert] = 'Error updating coauthors'
+        render :coauthors
+      end
+    else
+      @register.add_error(:coauthor, 'does not exist in the system')
+      render :coauthors
+    end
   end
 
   # GET /registers/r:abc/editorial_checks

app/models/register.rb

@@ -30,7 +30,7 @@ class Register < ApplicationRecord
     :type_strains, through: :names, source: :nomenclatural_type,
     source_type: 'Strain'
   )
-  has_many(:register_coauthors)
+  has_many(:register_coauthors, -> { order(:order) }, dependent: :destroy)
   has_many(:coauthors, through: :register_coauthors, source: :user)
   alias :correspondences :register_correspondences
   alias :created_by :user
@@ -56,7 +56,7 @@ class Register < ApplicationRecord
   include Register::SampleSet
 
   attr_accessor :modal_form_id
-  attr_accessor :new_coauthor
+  attr_accessor :coauthor
 
   def to_param
     accession

app/models/register_coauthor.rb

@@ -1,5 +1,27 @@
 class RegisterCoauthor < ApplicationRecord
   belongs_to(:register)
   belongs_to(:user)
-  validates(:order, presence: true)
+  validates(
+    :order, presence: true,
+    numericality: { only_integer: true, greater_than: 0 }
+  )
+  validates(:user, uniqueness: { scope: :register })
+
+  after_destroy(:update_coauthors_order)
+
+  def move_up
+    return if order <= 1
+
+    register.register_coauthor.where(order: order - 1)
+      .update(order: order) && update(order: order - 1)
+  end
+
+  private
+
+  def update_coauthors_order
+    register.reload
+    register.register_coauthors.each_with_index do |rc, k|
+      rc.update(order: k)
+    end
+  end
 end

app/models/user.rb

@@ -77,6 +77,10 @@ def self.curator_applications
     where('curator_statement is not null').where(curator: false)
   end
 
+  def self.find_by_email_or_username(query)
+    where('LOWER(email) = ?', query.downcase).or(where(username: query)).first
+  end
+
   def roles
     o = ['User']
     o << 'Contributor' if contributor?

app/views/registers/coauthors.html.erb

@@ -22,7 +22,8 @@
         <% author = rc.user %>
         <%= display_link(author) %>
         <%= link_to(
-              coauthors_register_url(@register, unlink: author.id),
+              coauthors_register_url(
+                @register, coauthor: author.username, action: :unlink),
               class: 'badge badge-pill badge-primary',
               method: :post, title: 'Unlink coauthor',
               data: { confirm: 'Are you sure?' }
@@ -31,7 +32,8 @@
         <% end %>
         <% unless rc.order == 1 %>
           <%= link_to(
-                coauthors_register_url(@register, up: author.id),
+                coauthors_register_url(
+                  @register, coauthor: author.username, action: :up),
                 class: 'badge badge-pill badge-primary',
                 method: :post, title: 'Move coauthor up'
               ) do %>