adds moderation to comments and debates [#136]

app/assets/javascripts/comments.js.coffee

@@ -1,8 +1,11 @@
 App.Comments =
 
-  add_response: (parent_id, response_html) ->
+  add_comment: (parent_id, response_html) ->
     $(response_html).insertAfter($("#js-comment-form-#{parent_id}"))
 
+  add_reply: (parent_id, response_html) ->
+    $("##{parent_id} .comment-children:first").prepend($(response_html))
+
   display_error: (field_with_errors, error_html) ->
     $(error_html).insertAfter($("#{field_with_errors}"))
 

app/assets/javascripts/moderator_comment.js.coffee

@@ -0,0 +1,7 @@
+App.ModeratorComments =
+
+  add_class_faded: (id) ->
+    $("##{id} .comment-body:first").addClass("faded")
+
+  hide_moderator_actions: (id) ->
+    $("##{id} #moderator-comment-actions:first").hide()

app/assets/javascripts/moderator_debates.js.coffee

@@ -0,0 +1,8 @@
+App.ModeratorDebates =
+
+  add_class_faded: (id) ->
+    $("##{id}").addClass("faded")
+    $("#comments").addClass("faded")
+
+  hide_moderator_actions: (id) ->
+    $("##{id} #moderator-debate-actions:first").hide()

app/assets/stylesheets/debates.scss

@@ -571,4 +571,9 @@
       }
     }
   }
+
+}
+
+.faded {
+  opacity: 0.4;
 }

app/controllers/admin/base_controller.rb

@@ -1,4 +1,5 @@
 class Admin::BaseController < ApplicationController
+  layout 'admin'
   before_action :authenticate_user!
 
   skip_authorization_check

app/controllers/admin/comments_controller.rb

@@ -0,0 +1,13 @@
+class Admin::CommentsController < Admin::BaseController
+
+  def index
+    @comments = Comment.only_hidden
+  end
+
+  def restore
+    @comment = Comment.with_hidden.find(params[:id])
+    @comment.restore
+    redirect_to admin_comments_path, notice: t('admin.comments.restore.success')
+  end
+
+end

app/controllers/admin/debates_controller.rb

@@ -0,0 +1,16 @@
+class Admin::DebatesController < Admin::BaseController
+
+  def index
+    @debates = Debate.only_hidden
+  end
+
+  def show
+    @debate = Debate.with_hidden.find(params[:id])
+  end
+
+  def restore
+    @debate = Debate.with_hidden.find(params[:id])
+    @debate.restore
+    redirect_to admin_debates_path, notice: t('admin.debates.restore.success')
+  end
+end

app/controllers/comments_controller.rb

@@ -2,6 +2,7 @@ class CommentsController < ApplicationController
   before_action :authenticate_user!
   before_action :build_comment, only: :create
   before_action :parent, only: :create
+
   load_and_authorize_resource
   respond_to :html, :js
 
@@ -50,4 +51,5 @@ class CommentsController < ApplicationController
     def email_on_comment_reply?
       reply? && parent.author.email_on_comment_reply?
     end
+
 end

app/controllers/debates_controller.rb

@@ -1,6 +1,8 @@
 class DebatesController < ApplicationController
   before_action :authenticate_user!, except: [:index, :show]
+
   load_and_authorize_resource
+  respond_to :html, :js
 
   def index
     @debates = Debate.search(params)
@@ -9,6 +11,7 @@ class DebatesController < ApplicationController
 
   def show
     set_debate_votes(@debate)
+    @comments = @debate.root_comments.with_hidden.recent
   end
 
   def new
@@ -46,7 +49,6 @@ class DebatesController < ApplicationController
     set_debate_votes(@debate)
   end
 
-
   private
 
     def debate_params

app/controllers/moderation/comments_controller.rb

@@ -0,0 +1,8 @@
+class Moderation::CommentsController < Moderation::BaseController
+
+  def hide
+    @comment = Comment.find(params[:id])
+    @comment.hide
+  end
+
+end

app/controllers/moderation/debates_controller.rb

@@ -0,0 +1,7 @@
+class Moderation::DebatesController < Moderation::BaseController
+
+  def hide
+    @debate = Debate.find(params[:id])
+    @debate.hide
+  end
+end

app/helpers/abilities_helper.rb

@@ -0,0 +1,7 @@
+module AbilitiesHelper
+
+  def moderator?
+    current_user.try(:moderator?)
+  end
+
+end

app/models/ability.rb

@@ -15,10 +15,13 @@ class Ability
 
       can [:create, :vote], Comment
 
-      if user.moderator? or user.administrator?
+      if user.moderator?
+        can [:hide], Comment
+        can [:hide], Debate
 
       elsif user.administrator?
-
+        can [:restore], Comment
+        can [:restore], Debate
       end
     end
   end

app/models/comment.rb

@@ -1,5 +1,8 @@
 class Comment < ActiveRecord::Base
+  include ActsAsParanoidAliases
   acts_as_nested_set scope: [:commentable_id, :commentable_type], counter_cache: :children_count
+
+  acts_as_paranoid column: :hidden_at
   acts_as_votable
 
   validates :body, presence: true

app/models/debate.rb

@@ -1,5 +1,6 @@
 require 'numeric'
 class Debate < ActiveRecord::Base
+  include ActsAsParanoidAliases
   default_scope { order('created_at DESC') }
 
   apply_simple_captcha
@@ -8,6 +9,7 @@ class Debate < ActiveRecord::Base
   acts_as_votable
   acts_as_commentable
   acts_as_taggable
+  acts_as_paranoid column: :hidden_at
 
   belongs_to :author, class_name: 'User', foreign_key: 'author_id'
 

app/views/admin/_menu.html.erb

@@ -1,3 +1,5 @@
-<div id="admin_menu">
-  <%= link_to t('admin.tags.index.title'), admin_tags_path %>
-</div>
+<ul id="admin_menu">
+  <li><%= link_to t('admin.menu.debate_topics'), admin_tags_path %></li>
+  <li><%= link_to t('admin.menu.hidden_debates'), admin_debates_path %></li>
+  <li><%= link_to t('admin.menu.hidden_comments'), admin_comments_path %></li>
+</ul>

app/views/admin/comments/index.html.erb

@@ -0,0 +1,14 @@
+<div class="left">
+  <h1><%= t("admin.comments.index.title") %></h1>
+
+  <ul>
+    <% @comments.each do |comment| %>
+      <li id="<%= dom_id(comment) %>">
+        <%= comment.body %>
+
+        <%= link_to t("admin.actions.restore"), restore_admin_comment_path(comment),
+                    method: :put, data: { confirm: t("admin.actions.confirm") } %>
+      </li>
+    <% end %>
+  </ul>
+</div>

app/views/admin/debates/index.html.erb

@@ -0,0 +1,13 @@
+<div class="left">
+  <h1><%= t("admin.debates.index.title") %></h1>
+
+  <ul>
+    <% @debates.each do |debate| %>
+      <%= link_to admin_debate_path(debate) do %>
+        <li id="<%= dom_id(debate) %>">
+          <%= link_to debate.title, admin_debate_path(debate) %>
+        </li>
+      <% end %>
+    <% end %>
+  </ul>
+</div>

app/views/admin/debates/show.html.erb

@@ -0,0 +1,13 @@
+<div class="left">
+  <h1><%= t("admin.debates.index.title") %></h1>
+
+  <div>
+    <div><%= @debate.title %></div>
+    <div><%= @debate.description %></div>
+
+    <div>
+      <%= link_to t('admin.actions.restore'), restore_admin_debate_path(@debate),
+                method: :put, data: { confirm: t('admin.actions.confirm') } %>
+    </div>
+  </div>
+</div>

app/views/comments/_actions.html.erb

@@ -0,0 +1,5 @@
+<span id="moderator-comment-actions">
+  &nbsp;|&nbsp;
+  <%= link_to t("admin.actions.hide").capitalize, hide_moderation_comment_path(comment),
+            method: :put, remote: true, data: { confirm: t('admin.actions.confirm') } %>
+</span>

app/views/comments/_comment.html.erb

@@ -1,5 +1,9 @@
 <div class="row">
-  <div id="comment-<%= comment.id %>" class="comment small-12 column">
+	<div id="<%= dom_id(comment) %>" class="comment small-12 column">
+
+    <% if comment.hidden? %>
+      This comment has been deleted
+    <% else %>
 
       <%= avatar_image(comment.user, size: 32, class: 'left') %>
 
@@ -15,15 +19,25 @@
 
         <p class="reply">
           <%= t("debates.comment.responses", count: comment.children_count) %>
+
           <% if user_signed_in? %>
             &nbsp;|&nbsp;
+            <%= link_to(comment_link_text(comment), "",
+                        class: "js-add-comment-link", data: {'id': dom_id(comment)}) %>
+
+            <% if moderator? %>
+               <%= render 'comments/actions', comment: comment %>
+            <% end %>
+
             <%= render 'comments/form', {parent: comment, toggeable: true} %>
           <% end %>
         </p>
       </div>
 
+    <% end %>
+
       <div class="comment-children">
-      <%= render comment.children.reorder('id DESC, lft') %>
+        <%= render comment.children.with_deleted.reorder('id DESC, lft') %>
       </div>
 
   </div>

app/views/comments/_form.html.erb

@@ -1,5 +1,3 @@
-<%= link_to(comment_link_text(parent), "", class: "js-add-comment-link", data: {'id': dom_id(parent)}) if toggeable %>
-
 <div id="js-comment-form-<%= dom_id(parent) %>" <%= "style='display:none'".html_safe if toggeable %>>
   <%= form_for [@debate, Comment.new], remote: true do |f| %>
     <%= f.text_area :body %>

app/views/comments/create.js.erb

@@ -1,9 +1,10 @@
-var parent_id = '<%= dom_id(@parent) %>';
+var parent_id = "<%= dom_id(@parent) %>";
+var comment_html = "<%= j(render @comment) %>"
 
 <% if @parent.is_a?(Debate) -%>
   App.Comments.reset_form(parent_id);
+  App.Comments.add_comment(parent_id, comment_html);
 <% else -%>
   App.Comments.reset_and_hide_form(parent_id);
+  App.Comments.add_reply(parent_id, comment_html);
 <% end -%>
-
-App.Comments.add_response(parent_id, "<%= j(render @comment) %>");

app/views/debates/_actions.html.erb

@@ -0,0 +1,2 @@
+<%= link_to t("admin.actions.hide").capitalize, hide_moderation_debate_path(debate),
+            method: :put, remote: true, data: { confirm: t('admin.actions.confirm') } %>

app/views/debates/show.html.erb

@@ -1,5 +1,5 @@
 <section class="debate-show">
-  <div id="debate-<%= @debate.id %>" class="row">
+  <div id="<%= dom_id(@debate) %>" class="row">
     <div class="small-12 medium-9 column">
       <i class="icon-angle-left left"></i>&nbsp;<%= link_to t("debates.show.back_link"), debates_path, class: 'left back' %>
 
@@ -21,12 +21,19 @@
         <span class="bullet">&nbsp;&bullet;&nbsp;</span>
         <%= l @debate.created_at.to_date %>
         <span class="bullet">&nbsp;&bullet;&nbsp;</span>
-        <i class="icon-chat-bubble-two"></i>&nbsp;<%= link_to t("debates.show.comments", count: @debate.comment_threads.count), "#comments" %>
+        <i class="icon-chat-bubble-two"></i>&nbsp;
+        <%= link_to t("debates.show.comments", count: @debate.comment_threads.count), "#comments" %>
       </div>
 
       <%= @debate.description %>
 
       <%= render 'shared/tags', debate: @debate %>
+
+      <% if moderator? %>
+        <div id='moderator-debate-actions'>
+          <%= render 'actions', debate: @debate %>
+        </div>
+      <% end %>
     </div>
 
     <aside class="small-12 medium-3 column">
@@ -40,6 +47,7 @@
       <div class="sidebar-divider"></div>
       <h3><%= t("debates.show.share") %></h3>
       <%= social_share_button_tag(@debate.title) %>
+
         <% if user_signed_in? %>
           <%= link_to t("debates.show.leave_comment"), "#comments", class: "leave-comment" %>
         <% else %>
@@ -63,7 +71,8 @@
           <%= render 'comments/form', {parent: @debate, toggeable: false} %>
         </div>
       <% end %>
-      <%= render @debate.root_comments.recent %>
+
+      <%= render @comments %>
     </div>
   </div>
 </section>

app/views/moderation/comments/hide.js.erb

@@ -0,0 +1,3 @@
+var comment_id = '<%= dom_id(@comment) %>';
+App.ModeratorComments.add_class_faded(comment_id);
+App.ModeratorComments.hide_moderator_actions(comment_id);

app/views/moderation/debates/hide.js.erb

@@ -0,0 +1,3 @@
+var debate_id = '<%= dom_id(@debate) %>';
+App.ModeratorDebates.add_class_faded(debate_id);
+App.ModeratorDebates.hide_moderator_actions(debate_id);

config/locales/admin.en.yml

@@ -3,6 +3,14 @@ en:
     dashboard:
       index:
         title: Administration
+    menu:
+      debate_topics: Debate topics
+      hidden_debates: Hidden debates
+      hidden_comments: Hidden comments
+    actions:
+      hide: Hide
+      restore: Restore
+      confirm: 'Are you sure?'
     tags:
       index:
         title: 'Debate topics'
@@ -11,3 +19,14 @@ en:
       name:
         placeholder: 'Write a topic'
       destroy: Delete Tag
+    comments:
+      index:
+        title: Hidden comments
+      restore:
+        success: The comment has been restored
+    debates:
+      index:
+        title: Hidden debates
+      restore:
+        success: The debate has been restored
+

config/locales/admin.es.yml

@@ -3,6 +3,14 @@ es:
     dashboard:
       index:
         title: Administración
+    menu:
+      debate_topics: Temas de debate
+      hidden_debates: Debates ocultos
+      hidden_comments: Comentarios ocultos
+    actions:
+      hide: Ocultar
+      restore: Permitir
+      confirm: '¿Estás seguro?'
     tags:
       index:
         title: 'Temas de debate'
@@ -11,3 +19,13 @@ es:
       name:
         placeholder: 'Escribe el nombre del tema'
       destroy: Elimina la etiqueta
+    comments:
+      index:
+        title: Comentarios ocultos
+      restore:
+        success: El comentario ha sido permitido
+    debates:
+      index:
+        title: Debates ocultos
+      restore:
+        success: El debate ha sido permitido

config/routes.rb

@@ -8,14 +8,10 @@ Rails.application.routes.draw do
   root 'welcome#index'
 
   resources :debates do
-    member do
-      post :vote
-    end
+    member { post :vote }
 
     resources :comments, only: :create, shallow: true do
-      member do
-        post :vote
-      end
+      member { post :vote }
     end
   end
 
@@ -24,11 +20,27 @@ Rails.application.routes.draw do
   namespace :admin do
     root to: "dashboard#index"
 
+    resources :debates, only: [:index, :show] do
+      member { put :restore }
+    end
+
+    resources :comments, only: :index do
+      member { put :restore }
+    end
+
     resources :tags, only: [:index, :create, :update, :destroy]
   end
 
   namespace :moderation do
     root to: "dashboard#index"
+
+    resources :debates, only: [] do
+      member { put :hide }
+    end
+
+    resources :comments, only: [:index] do
+      member { put :hide }
+    end
   end
 
   # Example of regular route:

db/migrate/20150810201448_add_hidden_at_to_comments.rb

@@ -0,0 +1,6 @@
+class AddHiddenAtToComments < ActiveRecord::Migration
+  def change
+    add_column :comments, :hidden_at, :datetime
+    add_index :comments, :hidden_at
+  end
+end

db/migrate/20150811145628_add_hidden_at_to_debates.rb

@@ -0,0 +1,6 @@
+class AddHiddenAtToDebates < ActiveRecord::Migration
+  def change
+    add_column :debates, :hidden_at, :datetime
+    add_index :debates, :hidden_at
+  end
+end

db/schema.rb

@@ -12,7 +12,6 @@
 # It's strongly recommended that you check this file into your version control system.
 
 ActiveRecord::Schema.define(version: 20150815154430) do
-
   # These are extensions that must be enabled in order to support this database
   enable_extension "plpgsql"
 
@@ -35,9 +34,11 @@ ActiveRecord::Schema.define(version: 20150815154430) do
     t.datetime "created_at"
     t.datetime "updated_at"
     t.integer  "children_count",   default: 0
+    t.datetime "hidden_at"
   end
 
   add_index "comments", ["commentable_id", "commentable_type"], name: "index_comments_on_commentable_id_and_commentable_type", using: :btree
+  add_index "comments", ["hidden_at"], name: "index_comments_on_hidden_at", using: :btree
   add_index "comments", ["user_id"], name: "index_comments_on_user_id", using: :btree
 
   create_table "debates", force: :cascade do |t|
@@ -46,8 +47,11 @@ ActiveRecord::Schema.define(version: 20150815154430) do
     t.integer  "author_id"
     t.datetime "created_at",             null: false
     t.datetime "updated_at",             null: false
+    t.datetime "hidden_at"
   end
 
+  add_index "debates", ["hidden_at"], name: "index_debates_on_hidden_at", using: :btree
+
   create_table "moderators", force: :cascade do |t|
     t.integer "user_id"
   end

lib/acts_as_paranoid_aliases.rb

@@ -0,0 +1,33 @@
+module ActsAsParanoidAliases
+
+  def self.included(base)
+    base.extend(ClassMethods)
+
+    def viewable?
+      not deleted?
+    end
+
+    def hide
+      update_attribute(:hidden_at, Time.now)
+    end
+
+    def hidden?
+      deleted?
+    end
+  end
+
+  module ClassMethods
+    def with_hidden
+      with_deleted
+    end
+
+    def only_hidden
+      only_deleted
+    end
+  end
+
+end
+
+module ActsAsParanoid
+  include ActsAsParanoidAliases
+end

spec/models/ability_spec.rb

@@ -4,6 +4,7 @@ require 'cancan/matchers'
 describe Ability do
   subject(:ability) { Ability.new(user) }
   let(:debate) { Debate.new }
+  let(:comment) { create(:comment) }
 
   describe "Non-logged in user" do
     let(:user) { nil }
@@ -53,6 +54,11 @@ describe Ability do
     it { should be_able_to(:show, debate) }
     it { should be_able_to(:vote, debate) }
 
+    it { should be_able_to(:hide, comment) }
+    it { should be_able_to(:hide, debate) }
+
+    it { should_not be_able_to(:restore, comment) }
+    it { should_not be_able_to(:restore, debate) }
   end
 
   describe "Administrator" do
@@ -63,5 +69,7 @@ describe Ability do
     it { should be_able_to(:show, debate) }
     it { should be_able_to(:vote, debate) }
 
+    it { should be_able_to(:restore, comment) }
+    it { should be_able_to(:restore, debate) }
   end
 end