From d0711892f690acd1462dcf01f0f3c50a4d5fb83c Mon Sep 17 00:00:00 2001
From: joshua stein <jcs@jcs.org>
Date: Wed, 19 Feb 2014 13:31:48 -0600
Subject: [PATCH] messages: implement batch-delete function

closes #119
---
 app/controllers/messages_controller.rb | 25 ++++++++++++
 app/views/messages/index.html.erb      | 55 +++++++++++++++++---------
 config/routes.rb                       |  2 +
 3 files changed, 63 insertions(+), 19 deletions(-)

diff --git a/app/controllers/messages_controller.rb b/app/controllers/messages_controller.rb
index 6c80d60..ffc5538 100644
--- a/app/controllers/messages_controller.rb
+++ b/app/controllers/messages_controller.rb
@@ -87,6 +87,31 @@ class MessagesController < ApplicationController
     end
   end
 
+  def batch_delete
+    deleted = 0
+
+    params.each do |k,v|
+      if v.to_s == "1" && m = k.match(/^delete_(.+)$/)
+        if (message = Message.where(:short_id => m[1]).first)
+          if message.author_user_id == @user.id
+            message.deleted_by_author = true
+          elsif message.recipient_user_id == @user.id
+            message.deleted_by_recipient = true
+          else
+            next
+          end
+
+          message.save!
+          deleted += 1
+        end
+      end
+    end
+
+    flash[:success] = "Deleted #{deleted} message#{deleted == 1 ? "" : "s"}."
+
+    return redirect_to "/messages"
+  end
+
   def keep_as_new
     @message.has_been_read = false
     @message.save
diff --git a/app/views/messages/index.html.erb b/app/views/messages/index.html.erb
index 4f97f49..d36a5b1 100644
--- a/app/views/messages/index.html.erb
+++ b/app/views/messages/index.html.erb
@@ -1,3 +1,12 @@
+<script>
+  $(document).ready(function() {
+    $("#delete_all").click(function(e) {
+      var table = $(e.target).closest("table");
+      $("td input:checkbox", table).attr("checked", e.target.checked);
+    });
+  });
+</script>
+
 <div class="box wide">
   <div class="legend" style="float: right;">
     <% if @direction == :out %>
@@ -14,27 +23,35 @@
   </div>
 
   <% if @messages.any? %>
-    <table class="data zebra" width="100%" cellspacing=0>
-    <tr>
-      <th width="15%"><%= @direction == :in ? "From" : "To" %></th>
-      <th width="20%"><%= @direction == :in ? "Received" : "Sent" %></th>
-      <th width="60%">Subject</th>
-    </tr>
-    <% @messages.includes(:author, :recipient).each do |message| %>
-      <tr class="<%= message.has_been_read? ? "" : "bold" %>">
-        <td><% if @direction == :in %>
-            <a href="/u/<%= message.author.username %>"><%=
-              message.author.username %></a>
-          <% else %>
-            <a href="/u/<%= message.recipient.username %>"><%=
-              message.recipient.username %></a>
-          <% end %></td>
-        <td><%= raw(time_ago_in_words_label(message.created_at)) %> ago</td>
-        <td><a href="/messages/<%= message.short_id %>"><%= message.subject
-          %></a></td>
+    <%= form_tag batch_delete_messages_url do %>
+      <table class="data zebra" width="100%" cellspacing=0>
+      <tr>
+        <th width="3%"><%= check_box_tag "delete_all",
+          :id => "delete_all" %></th>
+        <th width="15%"><%= @direction == :in ? "From" : "To" %></th>
+        <th width="17%"><%= @direction == :in ? "Received" : "Sent" %></th>
+        <th width="60%">Subject</th>
       </tr>
+      <% @messages.includes(:author, :recipient).each do |message| %>
+        <tr class="<%= message.has_been_read? ? "" : "bold" %>">
+          <td><%= check_box_tag "delete_#{message.short_id}" %></td>
+          <td><% if @direction == :in %>
+              <a href="/u/<%= message.author.username %>"><%=
+                message.author.username %></a>
+            <% else %>
+              <a href="/u/<%= message.recipient.username %>"><%=
+                message.recipient.username %></a>
+            <% end %></td>
+          <td><%= raw(time_ago_in_words_label(message.created_at)) %> ago</td>
+          <td><a href="/messages/<%= message.short_id %>"><%= message.subject
+            %></a></td>
+        </tr>
+      <% end %>
+      </table>
+      <p>
+      <%= submit_tag "Delete Selected" %>
+      </p>
     <% end %>
-    </table>
   <% else %>
     <p>
     You do not have any <%= @direction == :in ? "" : "sent" %> private
diff --git a/config/routes.rb b/config/routes.rb
index cb90e6d..e7f7d25 100644
--- a/config/routes.rb
+++ b/config/routes.rb
@@ -61,6 +61,8 @@ Lobsters::Application.routes.draw do
     get "/comments/page/:page" => "comments#index"
 
     get "/messages/sent" => "messages#sent"
+    post "/messages/batch_delete" => "messages#batch_delete",
+      :as => "batch_delete_messages"
     resources :messages do
       post "keep_as_new"
     end