Reopen a Task that was Completed

To make the page more interactive, add a “Reopen” button to move Tasks that have been marked as “Completed” back to “Todo”.

Like the PromotionsController before it, instead of acting as a bulk action, the ReopeningsController#create operates on a single Task record.


      app/controllers/reopenings_controller.rb

diff --git a/app/controllers/reopenings_controller.rb b/app/controllers/reopenings_controller.rb
new file mode 100644
index 0000000..f292916
--- /dev/null
+++ b/app/controllers/reopenings_controller.rb
@@ -0,0 +1,9 @@
+class ReopeningsController < ApplicationController
+  def create
+    task = Task.find(params[:task_id])
+
+    task.reopen!
+
+    redirect_back fallback_location: root_url
+  end
+end


      app/models/task.rb

diff --git a/app/models/task.rb b/app/models/task.rb
index 0cd29c5..72683ef 100644
--- a/app/models/task.rb
+++ b/app/models/task.rb
@@ -14,4 +14,8 @@ class Task < ApplicationRecord
   def promote!
     update!(delayed_at: nil)
   end
+
+  def reopen!
+    update!(completed_at: nil)
+  end
 end


      app/views/tasks/index.html.erb

diff --git a/app/views/tasks/index.html.erb b/app/views/tasks/index.html.erb
index 1d027a8..8330513 100644
--- a/app/views/tasks/index.html.erb
+++ b/app/views/tasks/index.html.erb
@@ -66,6 +66,15 @@
       <tr>
         <td>&checkmark;</td>
         <td><%= task.name %></td>
+
+        <td>
+          <%= form_with(scope: :event, url: task_reopenings_path(task)) do |form| %>
+            <%= form.button do %>
+              <%= translate(:reopen, scope: [:helpers, :submit, :event]) %>
+              <%= task.name %>
+            <% end %>
+          <% end %>
+        </td>
       </tr>
     <% end %>
   </table>


      config/locales/en.yml

diff --git a/config/locales/en.yml b/config/locales/en.yml
index 401fade..3bd6a2f 100644
--- a/config/locales/en.yml
+++ b/config/locales/en.yml
@@ -36,6 +36,7 @@ en:
         complete: "Complete"
         delay: "Delay"
         promote: "Promote"
+        reopen: "Reopen"
 
   tasks:
     index:


      config/routes.rb

diff --git a/config/routes.rb b/config/routes.rb
index e003fb8..6a4097b 100644
--- a/config/routes.rb
+++ b/config/routes.rb
@@ -5,6 +5,7 @@ Rails.application.routes.draw do
 
   resources :tasks, only: [] do
     resources :promotions, only: :create
+    resources :reopenings, only: :create
   end
 
   root to: "tasks#index"


      test/controllers/reopenings_controller_test.rb

diff --git a/test/controllers/reopenings_controller_test.rb b/test/controllers/reopenings_controller_test.rb
new file mode 100644
index 0000000..807e9e5
--- /dev/null
+++ b/test/controllers/reopenings_controller_test.rb
@@ -0,0 +1,11 @@
+require "test_helper"
+
+class ReopeningsControllerTest < ActionDispatch::IntegrationTest
+  test "#create marks a Task's completed_at to nil" do
+    read_the_book = tasks(:read_the_book)
+
+    post task_reopenings_path(read_the_book)
+
+    assert_nil read_the_book.reload.completed_at
+  end
+end


      test/models/task_test.rb

diff --git a/test/models/task_test.rb b/test/models/task_test.rb
index 6cac316..ff68df8 100644
--- a/test/models/task_test.rb
+++ b/test/models/task_test.rb
@@ -69,4 +69,15 @@ class TaskTest < ActiveSupport::TestCase
       end
     end
   end
+
+  test "#reopen! marks a Task's completed_at to nil" do
+    freeze_time do
+      read_the_book = tasks(:read_the_book)
+      read_the_book.complete!
+
+      assert_changes -> { read_the_book.completed_at }, from: Time.current, to: nil do
+        read_the_book.reopen!
+      end
+    end
+  end
 end


      test/system/tasks_test.rb

diff --git a/test/system/tasks_test.rb b/test/system/tasks_test.rb
index 3acda03..28325dd 100644
--- a/test/system/tasks_test.rb
+++ b/test/system/tasks_test.rb
@@ -46,6 +46,18 @@ class TasksTest < ApplicationSystemTestCase
     end
   end
 
+  test "reopens a completed Task back to Todo" do
+    read_the_book = tasks(:read_the_book)
+
+    visit root_path
+    within_task(read_the_book.name) { click_on submit(:event, :reopen) }
+
+    assert_no_section :completed
+    within_section :todo do
+      assert_text read_the_book.name
+    end
+  end
+
   def assert_no_section(i18n_key)
     assert_no_text translate(i18n_key, scope: [:tasks, :index])
   end