From 5ae06e138854da070122a234a0f6316d424f75b5 Mon Sep 17 00:00:00 2001
From: Piotr Witek <p.witek@visuality.pl>
Date: Wed, 17 Jul 2024 17:51:50 +0200
Subject: [PATCH] Update sortable controller for better data handling

---
 app/javascript/controllers/sortable_controller.js | 14 +++++++++++++-
 app/views/boards/show.html.erb                    |  2 +-
 config/routes.rb                                  |  6 +-----
 3 files changed, 15 insertions(+), 7 deletions(-)

diff --git a/app/javascript/controllers/sortable_controller.js b/app/javascript/controllers/sortable_controller.js
index 2b720ec..20f026a 100644
--- a/app/javascript/controllers/sortable_controller.js
+++ b/app/javascript/controllers/sortable_controller.js
@@ -58,8 +58,20 @@ export default class extends Controller {
     const newColumnId = to.closest('.board-column').dataset.sortableColumnIdValue;
     const url = this.urlValue.replace(":id", id);
 
+    const card = {
+      position: newIndex,
+      board_column_id: newColumnId,
+    };
+
+    const body = {
+      _method: 'patch',
+      authenticity_token: document.querySelector("[name='csrf-token']").content,
+      card: card,
+      commit: 'Save'
+    };
+
     put(url, {
-      body: JSON.stringify({ position: newIndex, column_id: newColumnId }),
+      body: JSON.stringify(body),
       headers: {
         'Content-Type': 'application/json',
         'X-CSRF-Token': document.querySelector("[name='csrf-token']").content
diff --git a/app/views/boards/show.html.erb b/app/views/boards/show.html.erb
index fb8e736..e3691b9 100644
--- a/app/views/boards/show.html.erb
+++ b/app/views/boards/show.html.erb
@@ -13,7 +13,7 @@
 </div>
 
 <div class="board-container">
-  <div class="board-scroll p-3" data-controller="sortable" data-sortable-group-value="<%= @board.id %>" data-sortable-url-value="<%= update_card_positions_board_column_path(":id") %>">
+  <div class="board-scroll p-3" data-controller="sortable" data-sortable-group-value="<%= @board.id %>" data-sortable-url-value="<%= card_path(":id") %>">
     <% @board_columns.each do |board_column| %>
       <%= render partial: 'board_columns/board_column', locals: { board_column: board_column } %>
     <% end %>
diff --git a/config/routes.rb b/config/routes.rb
index 0931ca1..62ce4ff 100644
--- a/config/routes.rb
+++ b/config/routes.rb
@@ -2,11 +2,7 @@
   root "boards#index"
 
   resources :boards, shallow: true do
-    resources :board_columns, except: [ :index, :show ] do
-      member do
-        put :update_card_positions
-      end
-    end
+    resources :board_columns, except: [ :index, :show ]
   end
 
   resources :cards,  except: [ :index ]