Add move to list option to Methodology cards

CHANGELOG

@@ -1,6 +1,7 @@
 [v#.#.#] ([month] [YYYY])
   - [entity]:
     - [future tense verb] [feature]
+  - Cards: add option to move a task to a different list
   - Upgraded gems:
     - sqlite3, websocket-driver
   - Bugs fixes:

app/assets/stylesheets/hera/modules.scss

@@ -199,10 +199,6 @@
     margin-left: auto;
 
     .action {
-      &:not(:first-child)::before {
-        content: '-';
-      }
-
       &:first-child {
         margin-left: auto;
       }

app/assets/stylesheets/hera/modules/_dropdown.scss

@@ -36,6 +36,12 @@
       font-weight: 600;
     }
 
+    &.disabled {
+      color: var(--text-muted);
+      cursor: not-allowed;
+      pointer-events: none;
+    }
+
     i {
       margin-right: 0.25rem;
     }

app/controllers/cards/position_controller.rb

@@ -0,0 +1,107 @@
+class Cards::PositionController < AuthenticatedController
+  include EventPublisher
+  include ProjectScoped
+  include ValidateMove
+
+  before_action :set_current_board_and_list
+  before_action :set_card
+  before_action :require_list_change, if: :append_to_list?
+  before_action :set_prev_item_and_next_item
+  before_action :validate_move_params
+
+  def create
+    Card.transaction do
+      List.move(@card, prev_item: @prev_item, next_item: @next_item)
+
+      if new_list
+        @card.list = new_list
+        @card.save!
+      end
+    end
+
+    publish_event('card.updated', @card.to_event_payload)
+    @card.reload
+
+    respond_to do |format|
+      format.html do
+        redirect_to project_board_list_card_path(current_project, @board, @card.list, @card),
+          notice: 'Task moved.'
+      end
+
+      format.json do
+        render json: {
+          is_card: true,
+          id: @card.id,
+          link: polymorphic_path([current_project, @board, @card.list, @card]),
+          moveLink: project_board_list_card_position_path(current_project, @board, @card.list, @card)
+        }
+      end
+    end
+  end
+
+  private
+
+  # A request with a new_list_id but no explicit position appends the card to
+  # the end of the target list (e.g. the 'move to list' dropdown).
+  def append_to_list?
+    move_params[:new_list_id].present? &&
+      move_params[:prev_id].blank? &&
+      move_params[:next_id].blank?
+  end
+
+  # Override EventPublisher#event_action_payload: the RESTful action is
+  # 'create', but moving a card is semantically an update.
+  #
+  # FIXME: Replace with ActivityService action registration once
+  # convention-over-configuration approach is implemented.
+  def event_action_payload
+    super.merge(action: 'update')
+  end
+
+  def move_params
+    params.permit(:card_id, :project_id, :board_id, :list_id, :next_id, :prev_id, :new_list_id)
+  end
+
+  def moveable_item_name
+    'card'
+  end
+
+  def moveable_items
+    @board.cards
+  end
+
+  def moveable_parent
+    new_list || @list
+  end
+
+  def new_list
+    return if move_params[:new_list_id].blank?
+
+    @new_list ||= @board.lists.find(move_params[:new_list_id])
+  end
+
+  def require_list_change
+    return if new_list.id != @card.list_id
+
+    redirect_to project_board_list_card_path(current_project, @board, @list, @card),
+      alert: 'Task is already in that list.'
+  end
+
+  def set_card
+    @card = @board.cards.find(params[:card_id])
+  end
+
+  def set_current_board_and_list
+    @board = current_project.boards.includes(:lists).find(params[:board_id])
+    @list = @board.lists.includes(:cards).find(params[:list_id])
+  end
+
+  def set_prev_item_and_next_item
+    if append_to_list?
+      @prev_item = new_list.last_card
+      @next_item = nil
+    else
+      super
+    end
+  end
+end

app/controllers/cards_controller.rb

@@ -11,12 +11,10 @@ class CardsController < AuthenticatedController
   before_action :initialize_sidebar, only: [:show, :new, :edit]
   before_action :set_auto_save_key, only: [:new, :create, :edit, :update]
 
-  # Not at top because we need board and list set first
-  include ValidateMove
-
   layout 'cards'
 
   def show
+    @lists = @board.ordered_lists
     render layout: !request.xhr?
   end
 
@@ -52,24 +50,6 @@ def update
     end
   end
 
-  def move
-    List.move(@card, prev_item: @prev_item, next_item: @next_item)
-
-    if new_list
-      @card.list = new_list
-      @card.save
-    end
-
-    track_updated(@card)
-
-    render json: {
-      is_card:  true,
-      id:       @card.id,
-      link:     polymorphic_path([current_project, @board, @card.reload.list, @card]),
-      moveLink: move_project_board_list_card_path(current_project, @board, @card.reload.list, @card)
-    }
-  end
-
   def destroy
     if @card.destroy
       track_destroyed(@card)
@@ -85,13 +65,6 @@ def card_params
     params.require(:card).permit(:name, :description, :due_date, assignee_ids: [])
   end
 
-  def move_params
-    params.
-      permit(:id, :project_id, :board_id, :list_id,
-        :next_id, :prev_id, :new_list_id
-      )
-  end
-
   def initialize_sidebar
     @sorted_cards = @list.ordered_cards.select(&:persisted?)
   end
@@ -107,7 +80,7 @@ def set_or_initialize_card
 
   def set_current_board_and_list
     @board = current_project.boards.includes(:lists).find(params[:board_id])
-    @list  = @board.lists.includes(:cards).find(params[:list_id])
+    @list = @board.lists.includes(:cards).find(params[:list_id])
   end
 
   def set_auto_save_key
@@ -119,8 +92,4 @@ def set_auto_save_key
       "#{@list.id}-card"
     end
   end
-
-  def new_list
-    @board.lists.find(move_params[:new_list_id]) if move_params[:new_list_id]
-  end
 end

app/controllers/concerns/validate_move.rb

@@ -1,15 +1,11 @@
 module ValidateMove
   extend ActiveSupport::Concern
-  included do
-    before_action :set_prev_item_and_next_item, only: :move
-    before_action :validate_move_params, only: :move
-  end
 
   protected
 
   def set_prev_item_and_next_item
-    @prev_item = @board.send(controller_name).find_by(id: move_params[:prev_id])
-    @next_item = @board.send(controller_name).find_by(id: move_params[:next_id])
+    @prev_item = moveable_items.find_by(id: move_params[:prev_id])
+    @next_item = moveable_items.find_by(id: move_params[:next_id])
   end
 
   def validate_move_params
@@ -18,27 +14,21 @@ def validate_move_params
     end
   end
 
-  def parent
-    if controller_name == 'cards'
-      new_list || @list
-    elsif controller_name == 'lists'
-      @board
-    end
-  end
+  private
 
   def valid_move_params?
     if @prev_item.present?
-      next_item_of_prev_item = @prev_item.send("next_#{controller_name.classify.downcase}")
+      next_item_of_prev_item = @prev_item.send("next_#{moveable_item_name}")
       if next_item_of_prev_item
         @next_item == next_item_of_prev_item
       else
         @next_item.nil?
       end
     else
-      if parent.items.empty?
+      if moveable_parent.items.empty?
         @next_item.nil?
       else
-        @next_item == parent.first_item
+        @next_item == moveable_parent.first_item
       end
     end
   end

app/controllers/lists_controller.rb

@@ -1,12 +1,12 @@
 class ListsController < AuthenticatedController
   include ActivityTracking
   include ProjectScoped
+  include ValidateMove
 
   before_action :set_current_board
   before_action :set_list, only: [:edit, :update, :destroy, :move]
-
-  # Not at top because we need board set first
-  include ValidateMove
+  before_action :set_prev_item_and_next_item, only: :move
+  before_action :validate_move_params, only: :move
 
   def new
     @list = @board.lists.new
@@ -63,6 +63,18 @@ def move_params
       permit(:id, :project_id, :board_id, :next_id, :prev_id)
   end
 
+  def moveable_items
+    @board.lists
+  end
+
+  def moveable_item_name
+    'list'
+  end
+
+  def moveable_parent
+    @board
+  end
+
   def set_current_board
     @board = current_project.boards.find(params[:board_id])
   end

app/models/card.rb

@@ -1,5 +1,6 @@
 class Card < ApplicationRecord
   include Commentable
+  include Eventable
   include HasFields
   include RevisionTracking
   include Subscribable
@@ -84,9 +85,19 @@ def to_xml(xml_builder, includes: [], version: 3)
     end
   end
 
+  def local_event_payload
+    {
+      name: name,
+      project: {
+        id: project.id,
+        name: project.name
+      }
+    }
+  end
+
   def local_fields
     {
-      'List'  => list.name.parameterize(preserve_case: true, separator: '_'),
+      'List' => list.name.parameterize(preserve_case: true, separator: '_'),
       'Title' => name
     }
   end

app/views/boards/_board.html.erb

@@ -35,7 +35,7 @@
           <ul class="list-cards js-sortable-cards">
             <% list.ordered_cards.each do |card| %>
               <% cache ['board-list-card', card] do %>
-                <li class="card" data-card-id="<%= card.id %>" data-move-url="<%= move_project_board_list_card_path(current_project, board, list, card) %>">
+                <li class="card" data-card-id="<%= card.id %>" data-move-url="<%= project_board_list_card_position_path(current_project, board, list, card) %>">
                   <%= link_to project_board_list_card_path(current_project, board, list, card) do %>
                     <div class="card-details">
                       <div class="card-title"><%= card.name %></div>

app/views/cards/_actions.html.erb

@@ -1,4 +1,24 @@
 <div class="actions">
+  <% if @lists.count > 1 %>
+    <span class="action dropdown">
+      <a href="#" class="dropdown-toggle" data-bs-toggle="dropdown"><i class="fa-solid fa-table-columns fa-fw"></i> <%= @list.name %></a>
+      <div class="dropdown-menu">
+        <% @lists.each do |list| %>
+          <% if list.id == @list.id %>
+            <a href="javascript:void(0)" class="dropdown-item disabled"><%= list.name %></a>
+          <% else %>
+            <%= link_to(
+              project_board_list_card_position_path(current_project, @board, @list, @card, new_list_id: list.id),
+              method: :post,
+              class: 'dropdown-item'
+            ) do %>
+              <%= list.name %>
+            <% end %>
+          <% end %>
+        <% end %>
+      </div>
+    </span>
+  <% end %>
   <span class="action">
     <%= link_to edit_project_board_list_card_path(current_project, @board, @list, @card) do %>
       <i class="fa-solid fa-pencil fa-fw"></i> Edit

app/views/cards/show.html.erb

@@ -22,7 +22,7 @@
       <div class="content-container">
         <div class="tab-content">
           <div class="tab-pane active" id="info-tab">
-            <% cache ['card-information-tab', @card] do %>
+            <% cache ['card-information-tab', @card, @card.list, @board] do %>
               <div class="note-text-inner">
                 <h4 class="mb-4 header-underline">
                   <span class="text-truncate" title="<%= @card.name %>"><%= @card.name %></span>

config/initializers/activity_service.rb

@@ -1,5 +1,6 @@
 Rails.application.reloader.to_prepare do
   ActivityService.configure do |activity_service|
+    activity_service.subscribe_namespace 'card'
     activity_service.subscribe_namespace 'inline_thread'
     activity_service.subscribe_namespace 'issue'
   end