diff --git a/app/assets/stylesheets/admin.scss b/app/assets/stylesheets/admin.scss index 5afa088d1..08f95cd41 100644 --- a/app/assets/stylesheets/admin.scss +++ b/app/assets/stylesheets/admin.scss @@ -164,6 +164,11 @@ body.admin { font-size: rem-calc(12); } +.ignored { + color: $text-medium; + font-size: rem-calc(12); +} + .rejected { color: $delete; } diff --git a/app/controllers/admin/comments_controller.rb b/app/controllers/admin/comments_controller.rb index 49dbc2889..40f9b31dc 100644 --- a/app/controllers/admin/comments_controller.rb +++ b/app/controllers/admin/comments_controller.rb @@ -1,13 +1,35 @@ class Admin::CommentsController < Admin::BaseController + before_filter :set_valid_filters, only: :index + before_filter :parse_filter, only: :index + + before_filter :load_comment, only: [:confirm_hide, :restore] def index - @comments = Comment.only_hidden.page(params[:page]) + @comments = Comment.only_hidden.send(@filter).page(params[:page]) + end + + def confirm_hide + @comment.confirm_hide + redirect_to request.query_parameters.merge(action: :index) end def restore - @comment = Comment.with_hidden.find(params[:id]) @comment.restore - redirect_to admin_comments_path, notice: t('admin.comments.restore.success') + redirect_to request.query_parameters.merge(action: :index) end -end \ No newline at end of file + private + def load_comment + @comment = Comment.with_hidden.find(params[:id]) + end + + def set_valid_filters + @valid_filters = %w{all with_confirmed_hide} + end + + def parse_filter + @filter = params[:filter] + @filter = 'all' unless @valid_filters.include?(@filter) + end + +end diff --git a/app/controllers/admin/debates_controller.rb b/app/controllers/admin/debates_controller.rb index 9f2f21f62..4487fcf81 100644 --- a/app/controllers/admin/debates_controller.rb +++ b/app/controllers/admin/debates_controller.rb @@ -1,16 +1,36 @@ class Admin::DebatesController < Admin::BaseController + before_filter :set_valid_filters, only: :index + before_filter :parse_filter, only: :index + + before_filter :load_debate, only: [:confirm_hide, :restore] def index - @debates = Debate.only_hidden.page(params[:page]) + @debates = Debate.only_hidden.send(@filter).page(params[:page]) end - def show - @debate = Debate.with_hidden.find(params[:id]) + def confirm_hide + @debate.confirm_hide + redirect_to request.query_parameters.merge(action: :index) end def restore - @debate = Debate.with_hidden.find(params[:id]) @debate.restore - redirect_to admin_debates_path, notice: t('admin.debates.restore.success') + redirect_to request.query_parameters.merge(action: :index) end -end \ No newline at end of file + + private + + def load_debate + @debate = Debate.with_hidden.find(params[:id]) + end + + def set_valid_filters + @valid_filters = %w{all with_confirmed_hide} + end + + def parse_filter + @filter = params[:filter] + @filter = 'all' unless @valid_filters.include?(@filter) + end + +end diff --git a/app/controllers/admin/users_controller.rb b/app/controllers/admin/users_controller.rb index 9b23b7927..61c0f5650 100644 --- a/app/controllers/admin/users_controller.rb +++ b/app/controllers/admin/users_controller.rb @@ -1,25 +1,42 @@ class Admin::UsersController < Admin::BaseController + before_filter :set_valid_filters, only: :index + before_filter :parse_filter, only: :index + + before_filter :load_user, only: [:confirm_hide, :restore] def index - @users = User.only_hidden.page(params[:page]) + @users = User.only_hidden.send(@filter).page(params[:page]) end def show @user = User.with_hidden.find(params[:id]) - @debates = Debate.where(author_id: @user.id).with_hidden.page(params[:page]) - @comments = Comment.where(user_id: @user.id).with_hidden.page(params[:page]) + @debates = @user.debates.with_hidden.page(params[:page]) + @comments = @user.comments.with_hidden.page(params[:page]) + end + + def confirm_hide + @user.confirm_hide + redirect_to request.query_parameters.merge(action: :index) end def restore - user = User.with_hidden.find(params[:id]) - if hidden_at = user.hidden_at - debates_ids = Debate.only_hidden.where(author_id: user.id).where("debates.hidden_at > ?", hidden_at).pluck(:id) - comments_ids = Comment.only_hidden.where(user_id: user.id).where("comments.hidden_at > ?", hidden_at).pluck(:id) - - user.restore - Debate.restore_all debates_ids - Comment.restore_all comments_ids - end - redirect_to admin_users_path, notice: t('admin.users.restore.success') + @user.restore + redirect_to request.query_parameters.merge(action: :index) end -end \ No newline at end of file + + private + + def load_user + @user = User.with_hidden.find(params[:id]) + end + + def set_valid_filters + @valid_filters = %w{all with_confirmed_hide} + end + + def parse_filter + @filter = params[:filter] + @filter = 'all' unless @valid_filters.include?(@filter) + end + +end diff --git a/app/controllers/comments_controller.rb b/app/controllers/comments_controller.rb index 798196ca0..c342b8d10 100644 --- a/app/controllers/comments_controller.rb +++ b/app/controllers/comments_controller.rb @@ -22,14 +22,14 @@ class CommentsController < ApplicationController respond_with @comment end - def flag_as_inappropiate - InappropiateFlag.flag!(current_user, @comment) - respond_with @comment, template: 'comments/_refresh_flag_as_inappropiate_actions' + def flag + Flag.flag!(current_user, @comment) + respond_with @comment, template: 'comments/_refresh_flag_actions' end - def undo_flag_as_inappropiate - InappropiateFlag.unflag!(current_user, @comment) - respond_with @comment, template: 'comments/_refresh_flag_as_inappropiate_actions' + def unflag + Flag.unflag!(current_user, @comment) + respond_with @comment, template: 'comments/_refresh_flag_actions' end private diff --git a/app/controllers/debates_controller.rb b/app/controllers/debates_controller.rb index efa441d81..60819fcf6 100644 --- a/app/controllers/debates_controller.rb +++ b/app/controllers/debates_controller.rb @@ -52,14 +52,14 @@ class DebatesController < ApplicationController set_debate_votes(@debate) end - def flag_as_inappropiate - InappropiateFlag.flag!(current_user, @debate) - respond_with @debate, template: 'debates/_refresh_flag_as_inappropiate_actions' + def flag + Flag.flag!(current_user, @debate) + respond_with @debate, template: 'debates/_refresh_flag_actions' end - def undo_flag_as_inappropiate - InappropiateFlag.unflag!(current_user, @debate) - respond_with @debate, template: 'debates/_refresh_flag_as_inappropiate_actions' + def unflag + Flag.unflag!(current_user, @debate) + respond_with @debate, template: 'debates/_refresh_flag_actions' end private diff --git a/app/controllers/moderation/comments_controller.rb b/app/controllers/moderation/comments_controller.rb index 8a4d3768e..3622d02ac 100644 --- a/app/controllers/moderation/comments_controller.rb +++ b/app/controllers/moderation/comments_controller.rb @@ -19,19 +19,19 @@ class Moderation::CommentsController < Moderation::BaseController redirect_to request.query_parameters.merge(action: :index) end - def archive - @comment.archive + def ignore_flag + @comment.ignore_flag redirect_to request.query_parameters.merge(action: :index) end private def load_comments - @comments = Comment.accessible_by(current_ability, :hide).flagged_as_inappropiate.sorted_for_moderation.includes(:commentable) + @comments = Comment.accessible_by(current_ability, :hide).flagged.sorted_for_moderation.includes(:commentable) end def set_valid_filters - @valid_filters = %w{all pending archived} + @valid_filters = %w{all pending_flag_review with_ignored_flag} end def parse_filter diff --git a/app/controllers/moderation/debates_controller.rb b/app/controllers/moderation/debates_controller.rb index 3492ee423..abb8e964b 100644 --- a/app/controllers/moderation/debates_controller.rb +++ b/app/controllers/moderation/debates_controller.rb @@ -19,19 +19,19 @@ class Moderation::DebatesController < Moderation::BaseController redirect_to request.query_parameters.merge(action: :index) end - def archive - @debate.archive + def ignore_flag + @debate.ignore_flag redirect_to request.query_parameters.merge(action: :index) end private def load_debates - @debates = Debate.accessible_by(current_ability, :hide).flagged_as_inappropiate.sorted_for_moderation + @debates = Debate.accessible_by(current_ability, :hide).flagged.sorted_for_moderation end def set_valid_filters - @valid_filters = %w{all pending archived} + @valid_filters = %w{all pending_flag_review with_ignored_flag} end def parse_filter diff --git a/app/models/ability.rb b/app/models/ability.rb index c8ed63d22..e931d037f 100644 --- a/app/models/ability.rb +++ b/app/models/ability.rb @@ -22,20 +22,20 @@ class Ability can :create, Comment can :create, Debate - can :flag_as_inappropiate, Comment do |comment| - comment.author_id != user.id && !InappropiateFlag.flagged?(user, comment) + can :flag, Comment do |comment| + comment.author_id != user.id && !Flag.flagged?(user, comment) end - can :undo_flag_as_inappropiate, Comment do |comment| - comment.author_id != user.id && InappropiateFlag.flagged?(user, comment) + can :unflag, Comment do |comment| + comment.author_id != user.id && Flag.flagged?(user, comment) end - can :flag_as_inappropiate, Debate do |debate| - debate.author_id != user.id && !InappropiateFlag.flagged?(user, debate) + can :flag, Debate do |debate| + debate.author_id != user.id && !Flag.flagged?(user, debate) end - can :undo_flag_as_inappropiate, Debate do |debate| - debate.author_id != user.id && InappropiateFlag.flagged?(user, debate) + can :unflag, Debate do |debate| + debate.author_id != user.id && Flag.flagged?(user, debate) end unless user.organization? @@ -53,14 +53,14 @@ class Ability can :hide, Comment, hidden_at: nil cannot :hide, Comment, user_id: user.id - can :archive, Comment, archived_at: nil, hidden_at: nil - cannot :archive, Comment, user_id: user.id + can :ignore_flag, Comment, ignored_flag_at: nil, hidden_at: nil + cannot :ignore_flag, Comment, user_id: user.id can :hide, Debate, hidden_at: nil cannot :hide, Debate, author_id: user.id - can :archive, Debate, archived_at: nil, hidden_at: nil - cannot :archive, Debate, author_id: user.id + can :ignore_flag, Debate, ignored_flag_at: nil, hidden_at: nil + cannot :ignore_flag, Debate, author_id: user.id can :hide, User cannot :hide, User, id: user.id @@ -72,8 +72,23 @@ class Ability if user.administrator? can :restore, Comment + cannot :restore, Comment, hidden_at: nil + can :restore, Debate + cannot :restore, Debate, hidden_at: nil + can :restore, User + cannot :restore, User, hidden_at: nil + + can :confirm_hide, Comment + cannot :confirm_hide, Comment, hidden_at: nil + + can :confirm_hide, Debate + cannot :confirm_hide, Debate, hidden_at: nil + + can :confirm_hide, User + cannot :confirm_hide, User, hidden_at: nil + can :comment_as_administrator, [Debate, Comment] end end diff --git a/app/models/comment.rb b/app/models/comment.rb index bbfeca4f5..fa5f72164 100644 --- a/app/models/comment.rb +++ b/app/models/comment.rb @@ -1,7 +1,7 @@ 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 + include ActsAsParanoidAliases acts_as_votable attr_accessor :as_moderator, :as_administrator @@ -12,14 +12,14 @@ class Comment < ActiveRecord::Base belongs_to :commentable, polymorphic: true, counter_cache: true belongs_to :user, -> { with_hidden } - has_many :inappropiate_flags, :as => :flaggable + has_many :flags, :as => :flaggable scope :recent, -> { order(id: :desc) } - scope :sorted_for_moderation, -> { order(inappropiate_flags_count: :desc, updated_at: :desc) } - scope :pending, -> { where(archived_at: nil, hidden_at: nil) } - scope :archived, -> { where("archived_at IS NOT NULL AND hidden_at IS NULL") } - scope :flagged_as_inappropiate, -> { where("inappropiate_flags_count > 0") } + scope :sorted_for_moderation, -> { order(flags_count: :desc, updated_at: :desc) } + scope :pending_flag_review, -> { where(ignored_flag_at: nil, hidden_at: nil) } + scope :with_ignored_flag, -> { where("ignored_flag_at IS NOT NULL AND hidden_at IS NULL") } + scope :flagged, -> { where("flags_count > 0") } scope :for_render, -> { with_hidden.includes(user: :organization) } @@ -65,8 +65,12 @@ class Comment < ActiveRecord::Base hidden? || user.hidden? end - def archived? - archived_at.present? + def ignored_flag? + ignored_flag_at.present? + end + + def ignore_flag + update(ignored_flag_at: Time.now) end def as_administrator? @@ -77,10 +81,6 @@ class Comment < ActiveRecord::Base moderator_id.present? end - def archive - update(archived_at: Time.now) - end - # TODO: faking counter cache since there is a bug with acts_as_nested_set :counter_cache # Remove when https://github.com/collectiveidea/awesome_nested_set/issues/294 is fixed # and reset counters using diff --git a/app/models/debate.rb b/app/models/debate.rb index 779cca33a..fdf939b2f 100644 --- a/app/models/debate.rb +++ b/app/models/debate.rb @@ -1,6 +1,5 @@ require 'numeric' class Debate < ActiveRecord::Base - include ActsAsParanoidAliases default_scope { order(created_at: :desc) } apply_simple_captcha @@ -10,9 +9,10 @@ class Debate < ActiveRecord::Base acts_as_commentable acts_as_taggable acts_as_paranoid column: :hidden_at + include ActsAsParanoidAliases belongs_to :author, -> { with_hidden }, class_name: 'User', foreign_key: 'author_id' - has_many :inappropiate_flags, :as => :flaggable + has_many :flags, :as => :flaggable validates :title, presence: true validates :description, presence: true @@ -23,10 +23,10 @@ class Debate < ActiveRecord::Base before_validation :sanitize_description before_validation :sanitize_tag_list - scope :sorted_for_moderation, -> { order(inappropiate_flags_count: :desc, updated_at: :desc) } - scope :pending, -> { where(archived_at: nil, hidden_at: nil) } - scope :archived, -> { where("archived_at IS NOT NULL AND hidden_at IS NULL") } - scope :flagged_as_inappropiate, -> { where("inappropiate_flags_count > 0") } + scope :sorted_for_moderation, -> { order(flags_count: :desc, updated_at: :desc) } + scope :pending_flag_review, -> { where(ignored_flag_at: nil, hidden_at: nil) } + scope :with_ignored_flag, -> { where("ignored_flag_at IS NOT NULL AND hidden_at IS NULL") } + scope :flagged, -> { where("flags_count > 0") } scope :for_render, -> { includes(:tags) } scope :sort_by_total_votes, -> { reorder(cached_votes_total: :desc) } scope :sort_by_likes , -> { reorder(cached_votes_up: :desc) } @@ -79,12 +79,12 @@ class Debate < ActiveRecord::Base count < 0 ? 0 : count end - def archived? - archived_at.present? + def ignored_flag? + ignored_flag_at.present? end - def archive - update(archived_at: Time.now) + def ignore_flag + update(ignored_flag_at: Time.now) end protected diff --git a/app/models/inappropiate_flag.rb b/app/models/flag.rb similarity index 72% rename from app/models/inappropiate_flag.rb rename to app/models/flag.rb index b60b98618..d2f30d284 100644 --- a/app/models/inappropiate_flag.rb +++ b/app/models/flag.rb @@ -1,7 +1,7 @@ -class InappropiateFlag < ActiveRecord::Base +class Flag < ActiveRecord::Base belongs_to :user - belongs_to :flaggable, polymorphic: true, counter_cache: true, touch: :flagged_as_inappropiate_at + belongs_to :flaggable, polymorphic: true, counter_cache: true scope(:by_user_and_flaggable, lambda do |user, flaggable| where(user_id: user.id, @@ -12,13 +12,13 @@ class InappropiateFlag < ActiveRecord::Base class AlreadyFlaggedError < StandardError def initialize - super "The flaggable was already flagged as inappropiate by this user" + super "The flaggable was already flagged by this user" end end class NotFlaggedError < StandardError def initialize - super "The flaggable was not flagged as inappropiate by this user" + super "The flaggable was not flagged by this user" end end diff --git a/app/models/user.rb b/app/models/user.rb index d7b77a7e0..6941e6d17 100644 --- a/app/models/user.rb +++ b/app/models/user.rb @@ -1,5 +1,4 @@ class User < ActiveRecord::Base - include ActsAsParanoidAliases OMNIAUTH_EMAIL_PREFIX = 'omniauth@participacion' OMNIAUTH_EMAIL_REGEX = /\A#{OMNIAUTH_EMAIL_PREFIX}/ @@ -10,12 +9,15 @@ class User < ActiveRecord::Base acts_as_voter acts_as_paranoid column: :hidden_at + include ActsAsParanoidAliases has_one :administrator has_one :moderator has_one :organization - has_many :inappropiate_flags + has_many :flags has_many :identities, dependent: :destroy + has_many :debates, -> { with_hidden }, foreign_key: :author_id + has_many :comments, -> { with_hidden } validates :username, presence: true, unless: :organization? validates :official_level, inclusion: {in: 0..5} @@ -114,4 +116,5 @@ class User < ActiveRecord::Base !!(email && email !~ OMNIAUTH_EMAIL_REGEX) || !!(unconfirmed_email && unconfirmed_email !~ OMNIAUTH_EMAIL_REGEX) end + end diff --git a/app/views/admin/comments/index.html.erb b/app/views/admin/comments/index.html.erb index 87b25680d..941e1e9ed 100644 --- a/app/views/admin/comments/index.html.erb +++ b/app/views/admin/comments/index.html.erb @@ -1,13 +1,17 @@

<%= t("admin.comments.index.title") %>

-
<%= t("admin.comments.index.filter") %>:
-
<%= t("admin.comments.filters.all") %>
-
<%= t("admin.comments.filters.pending") %>
-
<%= t("admin.comments.filters.archived") %>
+ + <% @valid_filters.each do |filter| %> + <% if @filter == filter %> +
<%= t("admin.comments.index.filters.#{filter}") %>
+ <% else %> +
<%= link_to t("admin.comments.index.filters.#{filter}"), + admin_comments_path(filter: filter) %>
+ <% end %> + <% end %>
-

<%= page_entries_info @comments %>

@@ -17,17 +21,19 @@
<%= comment.body %> - - <%= link_to t("admin.comments.index.show_debate"), "#" %> - + <%= link_to comment.commentable.title, comment.commentable %>
- - <%= link_to t("admin.actions.archive"), "#", class: "button radius tiny warning" %> - - <%= link_to t("admin.actions.restore"), restore_admin_comment_path(comment), - method: :put, data: { confirm: t("admin.actions.confirm") }, - class: "button radius tiny success" %> + <%= link_to t("admin.actions.restore"), + restore_admin_comment_path(comment, request.query_parameters), + method: :put, + data: { confirm: t("admin.actions.confirm") }, + class: "button radius tiny success right" %> + + <%= link_to t("admin.actions.confirm_hide"), + confirm_hide_admin_comment_path(comment, request.query_parameters), + method: :put, + class: "button radius tiny warning right" %>
diff --git a/app/views/admin/debates/index.html.erb b/app/views/admin/debates/index.html.erb index dbd85a46f..cdeb7b016 100644 --- a/app/views/admin/debates/index.html.erb +++ b/app/views/admin/debates/index.html.erb @@ -1,30 +1,37 @@

<%= t("admin.debates.index.title") %>

-
<%= t("admin.debates.index.filter") %>:
-
<%= t("admin.debates.filters.all") %>
-
<%= t("admin.debates.filters.pending") %>
-
<%= t("admin.debates.filters.archived") %>
+ + <% @valid_filters.each do |filter| %> + <% if @filter == filter %> +
<%= t("admin.debates.index.filters.#{filter}") %>
+ <% else %> +
<%= link_to t("admin.debates.index.filters.#{filter}"), + admin_debates_path(filter: filter) %>
+ <% end %> + <% end %>
-

<%= page_entries_info @debates %>

-<%= paginate @debates %> \ No newline at end of file +<%= paginate @debates %> diff --git a/app/views/admin/debates/show.html.erb b/app/views/admin/debates/show.html.erb deleted file mode 100644 index e740baf47..000000000 --- a/app/views/admin/debates/show.html.erb +++ /dev/null @@ -1,12 +0,0 @@ -

<%= t("admin.debates.index.title") %>

- -

<%= @debate.title %>

- -
<%= @debate.description %>
- -<%= link_to t("admin.debates.show.back"), admin_debates_path, - class: "button radius small secondary" %> - -<%= link_to t("admin.actions.restore"), restore_admin_debate_path(@debate), - method: :put, data: { confirm: t("admin.actions.confirm") }, - class: "button radius small success" %> diff --git a/app/views/admin/users/index.html.erb b/app/views/admin/users/index.html.erb index 5c84468f8..8f728295d 100644 --- a/app/views/admin/users/index.html.erb +++ b/app/views/admin/users/index.html.erb @@ -1,5 +1,18 @@

<%= t("admin.users.index.title") %>

+
+
<%= t("admin.users.index.filter") %>:
+ + <% @valid_filters.each do |filter| %> + <% if @filter == filter %> +
<%= t("admin.users.index.filters.#{filter}") %>
+ <% else %> +
<%= link_to t("admin.users.index.filters.#{filter}"), + admin_users_path(filter: filter) %>
+ <% end %> + <% end %> +
+

<%= page_entries_info @users %>

diff --git a/app/views/admin/users/show.html.erb b/app/views/admin/users/show.html.erb index 375d9110e..79a4b223e 100644 --- a/app/views/admin/users/show.html.erb +++ b/app/views/admin/users/show.html.erb @@ -6,8 +6,6 @@ <%= t("admin.users.show.hidden_at") %> <%= @user.hidden_at %>

- <%= link_to t("admin.users.show.restore"), restore_admin_user_path(@user), - method: :put, data: { confirm: t('admin.actions.confirm') }, class: "button radius tiny" %> <%= link_to t("admin.users.show.back"), admin_users_path, class: "button radius tiny secondary" %>

@@ -19,7 +17,7 @@ diff --git a/app/views/comments/_actions.html.erb b/app/views/comments/_actions.html.erb index 51a6743e0..c4bb0d072 100644 --- a/app/views/comments/_actions.html.erb +++ b/app/views/comments/_actions.html.erb @@ -1,5 +1,5 @@ - - <%= render 'comments/flag_as_inappropiate_actions', comment: comment %> + + <%= render 'comments/flag_actions', comment: comment %> diff --git a/app/views/comments/_flag_as_inappropiate_actions.html.erb b/app/views/comments/_flag_actions.html.erb similarity index 65% rename from app/views/comments/_flag_as_inappropiate_actions.html.erb rename to app/views/comments/_flag_actions.html.erb index 9fba8b5b2..f1557668c 100644 --- a/app/views/comments/_flag_as_inappropiate_actions.html.erb +++ b/app/views/comments/_flag_actions.html.erb @@ -1,23 +1,23 @@ -<% if can? :flag_as_inappropiate, comment %> +<% if can? :flag, comment %>  |  - <% end %> -<% if can? :undo_flag_as_inappropiate, comment %> +<% if can? :unflag, comment %>  |  - <% end %> diff --git a/app/views/comments/_refresh_flag_actions.js.erb b/app/views/comments/_refresh_flag_actions.js.erb new file mode 100644 index 000000000..01c1dcf76 --- /dev/null +++ b/app/views/comments/_refresh_flag_actions.js.erb @@ -0,0 +1 @@ +$("#<%= dom_id(@comment) %> .js-flag-actions").html('<%= j render("comments/flag_actions", comment: @comment) %>'); diff --git a/app/views/comments/_refresh_flag_as_inappropiate_actions.js.erb b/app/views/comments/_refresh_flag_as_inappropiate_actions.js.erb deleted file mode 100644 index 58f356f2b..000000000 --- a/app/views/comments/_refresh_flag_as_inappropiate_actions.js.erb +++ /dev/null @@ -1 +0,0 @@ -$("#<%= dom_id(@comment) %> .js-flag-as-inappropiate-actions").html('<%= j render("comments/flag_as_inappropiate_actions", comment: @comment) %>'); diff --git a/app/views/debates/_flag_as_inappropiate_actions.html.erb b/app/views/debates/_flag_actions.html.erb similarity index 63% rename from app/views/debates/_flag_as_inappropiate_actions.html.erb rename to app/views/debates/_flag_actions.html.erb index 6c3d34236..07fff9329 100644 --- a/app/views/debates/_flag_as_inappropiate_actions.html.erb +++ b/app/views/debates/_flag_actions.html.erb @@ -1,21 +1,21 @@ -<% if can? :flag_as_inappropiate, debate %> - <% end %> -<% if can? :undo_flag_as_inappropiate, debate %> - <% end %> diff --git a/app/views/debates/_refresh_flag_actions.js.erb b/app/views/debates/_refresh_flag_actions.js.erb new file mode 100644 index 000000000..f511f347f --- /dev/null +++ b/app/views/debates/_refresh_flag_actions.js.erb @@ -0,0 +1 @@ +$("#<%= dom_id(@debate) %> .js-flag-actions").html('<%= j render("debates/flag_actions", debate: @debate) %>'); diff --git a/app/views/debates/_refresh_flag_as_inappropiate_actions.js.erb b/app/views/debates/_refresh_flag_as_inappropiate_actions.js.erb deleted file mode 100644 index 7be008e35..000000000 --- a/app/views/debates/_refresh_flag_as_inappropiate_actions.js.erb +++ /dev/null @@ -1 +0,0 @@ -$("#<%= dom_id(@debate) %> .js-flag-as-inappropiate-actions").html('<%= j render("debates/flag_as_inappropiate_actions", debate: @debate) %>'); diff --git a/app/views/debates/show.html.erb b/app/views/debates/show.html.erb index af25f28b1..8fab71128 100644 --- a/app/views/debates/show.html.erb +++ b/app/views/debates/show.html.erb @@ -42,8 +42,8 @@   <%= link_to t("debates.show.comments", count: @debate.comments_count), "#comments" %>  •  - - <%= render 'debates/flag_as_inappropiate_actions', debate: @debate %> + + <%= render 'debates/flag_actions', debate: @debate %> diff --git a/app/views/moderation/comments/index.html.erb b/app/views/moderation/comments/index.html.erb index f4ce8ec67..f9c95e9be 100644 --- a/app/views/moderation/comments/index.html.erb +++ b/app/views/moderation/comments/index.html.erb @@ -35,18 +35,18 @@ <%= l comment.updated_at.to_date %> <%= comment.body %> - <%= comment.inappropiate_flags_count %> + <%= comment.flags_count %> <%= link_to t("moderation.comments.index.hide"), hide_in_moderation_screen_moderation_comment_path(comment, request.query_parameters), method: :put, class: "delete" %> - <% if can? :archive, comment %> + <% if can? :ignore_flag, comment %> - <%= link_to t("moderation.comments.index.archive"), archive_moderation_comment_path(comment, request.query_parameters), method: :put, class: "button radius tiny warning" %> + <%= link_to t("moderation.comments.index.ignore_flag"), ignore_flag_moderation_comment_path(comment, request.query_parameters), method: :put, class: "button radius tiny warning" %> <% end %> - <% if comment.archived? %> - - <%= t("moderation.comments.index.archived") %> + <% if comment.ignored_flag? %> + + <%= t("moderation.comments.index.ignored_flag") %> <% end %> diff --git a/app/views/moderation/debates/index.html.erb b/app/views/moderation/debates/index.html.erb index ac11d2159..cbd6baa5d 100644 --- a/app/views/moderation/debates/index.html.erb +++ b/app/views/moderation/debates/index.html.erb @@ -34,18 +34,18 @@
<%= debate.description %> - <%= debate.inappropiate_flags_count %> + <%= debate.flags_count %> <%= link_to t("moderation.debates.index.hide"), hide_in_moderation_screen_moderation_debate_path(debate, request.query_parameters), method: :put, class: "delete" %> - <% if can? :archive, debate %> + <% if can? :ignore_flag, debate %> - <%= link_to t("moderation.debates.index.archive"), archive_moderation_debate_path(debate, request.query_parameters), method: :put, class: "button radius tiny warning" %> + <%= link_to t("moderation.debates.index.ignore_flag"), ignore_flag_moderation_debate_path(debate, request.query_parameters), method: :put, class: "button radius tiny warning" %> <% end %> - <% if debate.archived? %> - - <%= t("moderation.debates.index.archived") %> + <% if debate.ignored_flag? %> + + <%= t("moderation.debates.index.ignored_flag") %> <% end %> diff --git a/config/locales/admin.en.yml b/config/locales/admin.en.yml index 971699c0b..7a7f8f342 100644 --- a/config/locales/admin.en.yml +++ b/config/locales/admin.en.yml @@ -36,7 +36,7 @@ en: hide_author: Ban author restore: Restore confirm: 'Are you sure?' - archive: Archive + confirm_hide: Confirm tags: index: title: 'Debate topics' @@ -48,39 +48,30 @@ en: comments: index: title: Hidden comments - show_debate: Show debate filter: Filter - restore: - success: The comment has been restored - filters: - all: All - pending: Pending - archived: Archived + filters: + all: All + with_confirmed_hide: Confirmed debates: index: title: Hidden debates filter: Filter - show: - back: Back - restore: - success: The debate has been restored - filters: - all: All - pending: Pending - archived: Archived + filters: + all: All + with_confirmed_hide: Confirmed users: index: title: Banned users - restore: Restore user + filter: Filter + filters: + all: All + with_confirmed_hide: Confirmed show: title: "User activity from %{user}" - restore: Restore user back: Back email: "Email:" registered_at: "Registered at:" hidden_at: "Hidden at:" - restore: - success: The user has been restored officials: level_0: Level 0 level_1: Level 1 diff --git a/config/locales/admin.es.yml b/config/locales/admin.es.yml index 3c264447f..2e9dffe03 100644 --- a/config/locales/admin.es.yml +++ b/config/locales/admin.es.yml @@ -36,7 +36,7 @@ es: hide_author: Bloquear al autor restore: Volver a mostrar confirm: '¿Estás seguro?' - archive: Archivar + confirm_hide: Confirmar tags: index: title: 'Temas de debate' @@ -48,39 +48,30 @@ es: comments: index: title: Comentarios ocultos - show_debate: Ver debate - filter: Filtro - restore: - success: El comentario ha sido permitido - filters: - all: Todos - pending: Pendientes - archived: Archivados + filter: Firar + filters: + all: Todos + with_confirmed_hide: Confirmados debates: index: title: Debates ocultos filter: Filtro - show: - back: Volver - restore: - success: El debate ha sido permitido - filters: - all: Todos - pending: Pendientes - archived: Archivados + filters: + all: Todos + with_confirmed_hide: Confirmados users: index: title: Usuarios bloqueados - restore: Restaurar usuario + filter: Filro + filters: + all: Todos + with_confirmed_hide: Confirmados show: title: "Actividad del usuario %{user}" - restore: Restaurar usuario back: Volver email: "Email:" registered_at: "Fecha de alta:" hidden_at: "Bloqueado:" - restore: - success: El usuario y sus contenidos han sido restaurados officials: level_0: Nivel 0 level_1: Nivel 1 diff --git a/config/locales/en.yml b/config/locales/en.yml index f0d78bc23..48cb717b6 100644 --- a/config/locales/en.yml +++ b/config/locales/en.yml @@ -159,8 +159,8 @@ en: shared: tags_cloud: tags: Topics - flag_as_inappropiate: Flag as inappropriate - undo_flag_as_inappropiate: Undo flag + flag: Flag as inappropriate + unflag: Undo flag collective: Collective mailer: comment: diff --git a/config/locales/es.yml b/config/locales/es.yml index 02a2f0e27..76015f4c8 100644 --- a/config/locales/es.yml +++ b/config/locales/es.yml @@ -159,8 +159,8 @@ es: shared: tags_cloud: tags: Temas - flag_as_inappropiate: Denunciar como inapropiado - undo_flag_as_inappropiate: Deshacer denuncia + flag: Denunciar como inapropiado + unflag: Deshacer denuncia collective: Colectivo mailer: comment: diff --git a/config/locales/moderation.en.yml b/config/locales/moderation.en.yml index 3d61c93a2..6e04e8765 100644 --- a/config/locales/moderation.en.yml +++ b/config/locales/moderation.en.yml @@ -16,13 +16,13 @@ en: commentable: Root comment: Comment hide: Hide - archive: Archive - archived: Archived + ignore_flag: Ignore + ignored_flag: Ignored filter: Filter filters: all: All - pending: Pending - archived: Archived + pending_flag_review: Pending + with_ignored_flag: Ignored debates: index: title: Debates flagged as inappropriate @@ -33,10 +33,10 @@ en: description: Description actions: Actions hide: Hide - archive: Archive - archived: Archived + ignore_flag: Ignore + ignored_flag: Ignored filter: Filter filters: all: All - pending: Pending - archived: Archived + pending_flag_review: Pending + with_ignored_flag: Ignored diff --git a/config/locales/moderation.es.yml b/config/locales/moderation.es.yml index 73886ddb0..04889d214 100644 --- a/config/locales/moderation.es.yml +++ b/config/locales/moderation.es.yml @@ -16,13 +16,13 @@ es: commentable: Raíz comment: Comentario hide: Ocultar - archive: Archivar - archived: Archivado + ignore_flag: Ignorar + ignored_flag: Ignorado filter: Filtrar filters: all: Todos - pending: Pendientes - archived: Archivados + pending_flag_review: Pendientes + with_ignored_flag: Ignorados debates: index: title: Debates denunciados como inapropiados @@ -33,10 +33,10 @@ es: description: Descripción actions: Acciones hide: Ocultar - archive: Archivar - archived: Archivado + ignore_flag: Ignorar + ignored_flag: Ignorado filter: Filtrar filters: all: Todos - pending: Pendientes - archived: Archivados + pending_flag_review: Pendientes + with_ignored_flag: Ignorados diff --git a/config/routes.rb b/config/routes.rb index 3a75c8634..eb3953ea9 100644 --- a/config/routes.rb +++ b/config/routes.rb @@ -28,15 +28,15 @@ Rails.application.routes.draw do resources :debates do member do post :vote - put :flag_as_inappropiate - put :undo_flag_as_inappropiate + put :flag + put :unflag end resources :comments, only: :create, shallow: true do member do post :vote - put :flag_as_inappropiate - put :undo_flag_as_inappropiate + put :flag + put :unflag end end end @@ -58,15 +58,24 @@ Rails.application.routes.draw do end resources :users, only: [:index, :show] do - member { put :restore } + member do + put :restore + put :confirm_hide + end end - resources :debates, only: [:index, :show] do - member { put :restore } + resources :debates, only: :index do + member do + put :restore + put :confirm_hide + end end resources :comments, only: :index do - member { put :restore } + member do + put :restore + put :confirm_hide + end end resources :tags, only: [:index, :create, :update, :destroy] @@ -88,7 +97,7 @@ Rails.application.routes.draw do member do put :hide put :hide_in_moderation_screen - put :archive + put :ignore_flag end end @@ -96,7 +105,7 @@ Rails.application.routes.draw do member do put :hide put :hide_in_moderation_screen - put :archive + put :ignore_flag end end end diff --git a/db/migrate/20150827080624_rename_inappropiate_flags_as_flags.rb b/db/migrate/20150827080624_rename_inappropiate_flags_as_flags.rb new file mode 100644 index 000000000..26058bde0 --- /dev/null +++ b/db/migrate/20150827080624_rename_inappropiate_flags_as_flags.rb @@ -0,0 +1,5 @@ +class RenameInappropiateFlagsAsFlags < ActiveRecord::Migration + def change + rename_table :inappropiate_flags, :flags + end +end diff --git a/db/migrate/20150827080641_rename_archived_at_to_ignored_flag_at_in_comments_and_debates.rb b/db/migrate/20150827080641_rename_archived_at_to_ignored_flag_at_in_comments_and_debates.rb new file mode 100644 index 000000000..5a0c90085 --- /dev/null +++ b/db/migrate/20150827080641_rename_archived_at_to_ignored_flag_at_in_comments_and_debates.rb @@ -0,0 +1,6 @@ +class RenameArchivedAtToIgnoredFlagAtInCommentsAndDebates < ActiveRecord::Migration + def change + rename_column :comments, :archived_at, :ignored_flag_at + rename_column :debates, :archived_at, :ignored_flag_at + end +end diff --git a/db/migrate/20150827080701_add_confirmed_hide_at_to_comments_and_debates.rb b/db/migrate/20150827080701_add_confirmed_hide_at_to_comments_and_debates.rb new file mode 100644 index 000000000..20d2872ac --- /dev/null +++ b/db/migrate/20150827080701_add_confirmed_hide_at_to_comments_and_debates.rb @@ -0,0 +1,6 @@ +class AddConfirmedHideAtToCommentsAndDebates < ActiveRecord::Migration + def change + add_column :debates, :confirmed_hide_at, :datetime + add_column :comments, :confirmed_hide_at, :datetime + end +end diff --git a/db/migrate/20150827081657_rename_inappropiate_flags_count_to_flags_count_in_debates_and_comments.rb b/db/migrate/20150827081657_rename_inappropiate_flags_count_to_flags_count_in_debates_and_comments.rb new file mode 100644 index 000000000..2f2a9285d --- /dev/null +++ b/db/migrate/20150827081657_rename_inappropiate_flags_count_to_flags_count_in_debates_and_comments.rb @@ -0,0 +1,6 @@ +class RenameInappropiateFlagsCountToFlagsCountInDebatesAndComments < ActiveRecord::Migration + def change + rename_column :debates, :inappropiate_flags_count, :flags_count + rename_column :comments, :inappropiate_flags_count, :flags_count + end +end diff --git a/db/migrate/20150827083232_remove_flagged_as_inappropiate_at_from_comments_and_debates.rb b/db/migrate/20150827083232_remove_flagged_as_inappropiate_at_from_comments_and_debates.rb new file mode 100644 index 000000000..c226ecb3f --- /dev/null +++ b/db/migrate/20150827083232_remove_flagged_as_inappropiate_at_from_comments_and_debates.rb @@ -0,0 +1,6 @@ +class RemoveFlaggedAsInappropiateAtFromCommentsAndDebates < ActiveRecord::Migration + def change + remove_column :debates, :flagged_as_inappropiate_at + remove_column :comments, :flagged_as_inappropiate_at + end +end diff --git a/db/migrate/20150828085718_add_confirmed_hide_at_to_users.rb b/db/migrate/20150828085718_add_confirmed_hide_at_to_users.rb new file mode 100644 index 000000000..ca299aa5e --- /dev/null +++ b/db/migrate/20150828085718_add_confirmed_hide_at_to_users.rb @@ -0,0 +1,5 @@ +class AddConfirmedHideAtToUsers < ActiveRecord::Migration + def change + add_column :users, :confirmed_hide_at, :datetime + end +end diff --git a/db/schema.rb b/db/schema.rb index ad2a412ff..0da2f7c6a 100644 --- a/db/schema.rb +++ b/db/schema.rb @@ -11,7 +11,7 @@ # # It's strongly recommended that you check this file into your version control system. -ActiveRecord::Schema.define(version: 20150826112500) do +ActiveRecord::Schema.define(version: 20150828085718) do # These are extensions that must be enabled in order to support this database enable_extension "plpgsql" @@ -41,22 +41,22 @@ ActiveRecord::Schema.define(version: 20150826112500) do t.string "title" t.text "body" t.string "subject" - t.integer "user_id", null: false + t.integer "user_id", null: false t.integer "parent_id" t.integer "lft" t.integer "rgt" t.datetime "created_at" t.datetime "updated_at" - t.integer "children_count", default: 0 + t.integer "children_count", default: 0 t.datetime "hidden_at" - t.datetime "flagged_as_inappropiate_at" - t.integer "inappropiate_flags_count", default: 0 - t.datetime "archived_at" + t.integer "flags_count", default: 0 + t.datetime "ignored_flag_at" t.integer "moderator_id" t.integer "administrator_id" - t.integer "cached_votes_total", default: 0 - t.integer "cached_votes_up", default: 0 - t.integer "cached_votes_down", default: 0 + t.integer "cached_votes_total", default: 0 + t.integer "cached_votes_up", default: 0 + t.integer "cached_votes_down", default: 0 + t.datetime "confirmed_hide_at" end add_index "comments", ["cached_votes_down"], name: "index_comments_on_cached_votes_down", using: :btree @@ -67,20 +67,20 @@ ActiveRecord::Schema.define(version: 20150826112500) do add_index "comments", ["user_id"], name: "index_comments_on_user_id", using: :btree create_table "debates", force: :cascade do |t| - t.string "title", limit: 80 + t.string "title", limit: 80 t.text "description" t.integer "author_id" - t.datetime "created_at", null: false - t.datetime "updated_at", null: false + t.datetime "created_at", null: false + t.datetime "updated_at", null: false t.datetime "hidden_at" t.string "visit_id" - t.datetime "flagged_as_inappropiate_at" - t.integer "inappropiate_flags_count", default: 0 - t.integer "cached_votes_total", default: 0 - t.integer "cached_votes_up", default: 0 - t.integer "cached_votes_down", default: 0 - t.datetime "archived_at" - t.integer "comments_count", default: 0 + t.integer "flags_count", default: 0 + t.integer "cached_votes_total", default: 0 + t.integer "cached_votes_up", default: 0 + t.integer "cached_votes_down", default: 0 + t.datetime "ignored_flag_at" + t.integer "comments_count", default: 0 + t.datetime "confirmed_hide_at" end add_index "debates", ["cached_votes_down"], name: "index_debates_on_cached_votes_down", using: :btree @@ -88,6 +88,18 @@ ActiveRecord::Schema.define(version: 20150826112500) do add_index "debates", ["cached_votes_up"], name: "index_debates_on_cached_votes_up", using: :btree add_index "debates", ["hidden_at"], name: "index_debates_on_hidden_at", using: :btree + create_table "flags", force: :cascade do |t| + t.integer "user_id" + t.string "flaggable_type" + t.integer "flaggable_id" + t.datetime "created_at" + t.datetime "updated_at" + end + + add_index "flags", ["flaggable_type", "flaggable_id"], name: "index_flags_on_flaggable_type_and_flaggable_id", using: :btree + add_index "flags", ["user_id", "flaggable_type", "flaggable_id"], name: "access_inappropiate_flags", using: :btree + add_index "flags", ["user_id"], name: "index_flags_on_user_id", using: :btree + create_table "identities", force: :cascade do |t| t.integer "user_id" t.string "provider" @@ -98,18 +110,6 @@ ActiveRecord::Schema.define(version: 20150826112500) do add_index "identities", ["user_id"], name: "index_identities_on_user_id", using: :btree - create_table "inappropiate_flags", force: :cascade do |t| - t.integer "user_id" - t.string "flaggable_type" - t.integer "flaggable_id" - t.datetime "created_at" - t.datetime "updated_at" - end - - add_index "inappropiate_flags", ["flaggable_type", "flaggable_id"], name: "index_inappropiate_flags_on_flaggable_type_and_flaggable_id", using: :btree - add_index "inappropiate_flags", ["user_id", "flaggable_type", "flaggable_id"], name: "access_inappropiate_flags", using: :btree - add_index "inappropiate_flags", ["user_id"], name: "index_inappropiate_flags_on_user_id", using: :btree - create_table "moderators", force: :cascade do |t| t.integer "user_id" end @@ -184,6 +184,7 @@ ActiveRecord::Schema.define(version: 20150826112500) do t.datetime "hidden_at" t.string "phone_number", limit: 30 t.string "username" + t.datetime "confirmed_hide_at" end add_index "users", ["confirmation_token"], name: "index_users_on_confirmation_token", unique: true, using: :btree @@ -237,8 +238,8 @@ ActiveRecord::Schema.define(version: 20150826112500) do add_index "votes", ["voter_id", "voter_type", "vote_scope"], name: "index_votes_on_voter_id_and_voter_type_and_vote_scope", using: :btree add_foreign_key "administrators", "users" + add_foreign_key "flags", "users" add_foreign_key "identities", "users" - add_foreign_key "inappropiate_flags", "users" add_foreign_key "moderators", "users" add_foreign_key "organizations", "users" -end \ No newline at end of file +end diff --git a/lib/acts_as_paranoid_aliases.rb b/lib/acts_as_paranoid_aliases.rb index b4faa19b9..1d7dbf6d9 100644 --- a/lib/acts_as_paranoid_aliases.rb +++ b/lib/acts_as_paranoid_aliases.rb @@ -14,9 +14,26 @@ module ActsAsParanoidAliases def after_hide end + + def confirmed_hide? + confirmed_hide_at.present? + end + + def confirm_hide + update_attribute(:confirmed_hide_at, Time.now) + end + + def restore(opts={}) + super(opts) + update_attribute(:confirmed_hide_at, nil) + end end module ClassMethods + def with_confirmed_hide + where("confirmed_hide_at IS NOT NULL") + end + def with_hidden with_deleted end @@ -35,9 +52,5 @@ module ActsAsParanoidAliases only_hidden.where(id: ids).update_all(hidden_at: nil) end end - end -module ActsAsParanoid - include ActsAsParanoidAliases -end \ No newline at end of file diff --git a/spec/factories.rb b/spec/factories.rb index ded2e20a0..3e355499a 100644 --- a/spec/factories.rb +++ b/spec/factories.rb @@ -4,6 +4,14 @@ FactoryGirl.define do sequence(:email) { |n| "manuela#{n}@madrid.es" } password 'judgmentday' confirmed_at { Time.now } + + trait :hidden do + hidden_at Time.now + end + + trait :with_confirmed_hide do + confirmed_hide_at Time.now + end end factory :identity do @@ -22,13 +30,17 @@ FactoryGirl.define do hidden_at Time.now end - trait :archived do - archived_at Time.now + trait :with_ignored_flag do + ignored_flag_at Time.now end - trait :flagged_as_inappropiate do + trait :with_confirmed_hide do + confirmed_hide_at Time.now + end + + trait :flagged do after :create do |debate| - InappropiateFlag.flag!(FactoryGirl.create(:user), debate) + Flag.flag!(FactoryGirl.create(:user), debate) end end end @@ -51,13 +63,17 @@ FactoryGirl.define do hidden_at Time.now end - trait :archived do - archived_at Time.now + trait :with_ignored_flag do + ignored_flag_at Time.now end - trait :flagged_as_inappropiate do + trait :with_confirmed_hide do + confirmed_hide_at Time.now + end + + trait :flagged do after :create do |debate| - InappropiateFlag.flag!(FactoryGirl.create(:user), debate) + Flag.flag!(FactoryGirl.create(:user), debate) end end end diff --git a/spec/features/admin/comments_spec.rb b/spec/features/admin/comments_spec.rb index 6e4a0d53d..92d010cac 100644 --- a/spec/features/admin/comments_spec.rb +++ b/spec/features/admin/comments_spec.rb @@ -2,27 +2,71 @@ require 'rails_helper' feature 'Admin comments' do - scenario 'Restore', :js do - citizen = create(:user) + background do admin = create(:administrator) - - debate = create(:debate) - comment = create(:comment, :hidden, commentable: debate, body: 'Not really SPAM') - login_as(admin.user) - visit admin_comments_path - - within("#comment_#{comment.id}") do - first(:link, "Restore").click - end - - expect(page).to have_content 'The comment has been restored' - - login_as(citizen) - visit debate_path(debate) - - expect(page).to have_css('.comment', count: 1) - expect(page).to have_content('Not really SPAM') end -end \ No newline at end of file + scenario 'Restore', :js do + comment = create(:comment, :hidden, body: 'Not really SPAM') + visit admin_comments_path + + click_link 'Restore' + + expect(page).to_not have_content(comment.body) + + expect(comment.reload).to_not be_hidden + end + + scenario 'Confirm hide' do + comment = create(:comment, :hidden, body: 'SPAM') + visit admin_comments_path + + click_link 'Confirm' + + expect(page).to have_content(comment.body) + expect(page).to have_content('Confirmed') + + expect(comment.reload).to be_confirmed_hide + end + + scenario "Current filter is properly highlighted" do + visit admin_comments_path + expect(page).to_not have_link('All') + expect(page).to have_link('Confirmed') + + visit admin_comments_path(filter: 'all') + expect(page).to_not have_link('All') + expect(page).to have_link('Confirmed') + + visit admin_comments_path(filter: 'with_confirmed_hide') + expect(page).to have_link('All') + expect(page).to_not have_link('Confirmed') + end + + scenario "Filtering comments" do + create(:comment, :hidden, body: "Unconfirmed comment") + create(:comment, :hidden, :with_confirmed_hide, body: "Confirmed comment") + + visit admin_comments_path(filter: 'all') + expect(page).to have_content('Unconfirmed comment') + expect(page).to have_content('Confirmed comment') + + visit admin_comments_path(filter: 'with_confirmed_hide') + expect(page).to_not have_content('Unconfirmed comment') + expect(page).to have_content('Confirmed comment') + end + + scenario "Action links remember the pagination setting and the filter" do + per_page = Kaminari.config.default_per_page + (per_page + 2).times { create(:comment, :hidden, :with_confirmed_hide) } + + visit admin_comments_path(filter: 'with_confirmed_hide', page: 2) + + click_on('Restore', match: :first, exact: true) + + expect(current_url).to include('filter=with_confirmed_hide') + expect(current_url).to include('page=2') + end + +end diff --git a/spec/features/admin/debates_spec.rb b/spec/features/admin/debates_spec.rb index 50859816e..9d956582e 100644 --- a/spec/features/admin/debates_spec.rb +++ b/spec/features/admin/debates_spec.rb @@ -2,22 +2,71 @@ require 'rails_helper' feature 'Admin debates' do - scenario 'Restore', :js do - citizen = create(:user) + background do admin = create(:administrator) - - debate = create(:debate, :hidden) - login_as(admin.user) - visit admin_debate_path(debate) + end + + scenario 'Restore' do + debate = create(:debate, :hidden) + visit admin_debates_path click_link 'Restore' - expect(page).to have_content 'The debate has been restored' + expect(page).to_not have_content(debate.title) - login_as(citizen) - visit debates_path - - expect(page).to have_css('.debate', count: 1) + expect(debate.reload).to_not be_hidden end + + scenario 'Confirm hide' do + debate = create(:debate, :hidden) + visit admin_debates_path + + click_link 'Confirm' + + expect(page).to have_content(debate.title) + expect(page).to have_content('Confirmed') + + expect(debate.reload).to be_confirmed_hide + end + + scenario "Current filter is properly highlighted" do + visit admin_debates_path + expect(page).to_not have_link('All') + expect(page).to have_link('Confirmed') + + visit admin_debates_path(filter: 'all') + expect(page).to_not have_link('All') + expect(page).to have_link('Confirmed') + + visit admin_debates_path(filter: 'with_confirmed_hide') + expect(page).to have_link('All') + expect(page).to_not have_link('Confirmed') + end + + scenario "Filtering debates" do + create(:debate, :hidden, title: "Unconfirmed debate") + create(:debate, :hidden, :with_confirmed_hide, title: "Confirmed debate") + + visit admin_debates_path(filter: 'all') + expect(page).to have_content('Unconfirmed debate') + expect(page).to have_content('Confirmed debate') + + visit admin_debates_path(filter: 'with_confirmed_hide') + expect(page).to_not have_content('Unconfirmed debate') + expect(page).to have_content('Confirmed debate') + end + + scenario "Action links remember the pagination setting and the filter" do + per_page = Kaminari.config.default_per_page + (per_page + 2).times { create(:debate, :hidden, :with_confirmed_hide) } + + visit admin_debates_path(filter: 'with_confirmed_hide', page: 2) + + click_on('Restore', match: :first, exact: true) + + expect(current_url).to include('filter=with_confirmed_hide') + expect(current_url).to include('page=2') + end + end diff --git a/spec/features/admin/organizations_spec.rb b/spec/features/admin/organizations_spec.rb index d13758117..7c1756603 100644 --- a/spec/features/admin/organizations_spec.rb +++ b/spec/features/admin/organizations_spec.rb @@ -119,11 +119,8 @@ feature 'Admin::Organizations' do click_on('Verify', match: :first) - uri = URI.parse(current_url) - query_params = Rack::Utils.parse_nested_query(uri.query).symbolize_keys - - expect(query_params[:filter]).to eq('pending') - expect(query_params[:page]).to eq('2') + expect(current_url).to include('filter=pending') + expect(current_url).to include('page=2') end end diff --git a/spec/features/admin/users_spec.rb b/spec/features/admin/users_spec.rb index ed5c80f34..84dbccf50 100644 --- a/spec/features/admin/users_spec.rb +++ b/spec/features/admin/users_spec.rb @@ -2,60 +2,20 @@ require 'rails_helper' feature 'Admin users' do - scenario 'Restore hidden user' do - citizen = create(:user) + background do admin = create(:administrator) - create(:moderator, user: admin.user) - - debate_previously_hidden = create(:debate, :hidden, author: citizen) - debate = create(:debate, author: citizen) - comment_previously_hidden = create(:comment, :hidden, user: citizen, commentable: debate, body: "You have the manners of a beggar") - comment = create(:comment, user: citizen, commentable: debate, body: 'Not Spam') - login_as(admin.user) - visit debate_path(debate) - - within("#debate_#{debate.id}") do - click_link 'Ban author' - end - - visit debates_path - expect(page).to_not have_content(debate.title) - expect(page).to_not have_content(debate_previously_hidden) - - click_link "Administration" - click_link "Hidden users" - click_link "Restore user" - - visit debates_path - expect(page).to have_content(debate.title) - expect(page).to_not have_content(debate_previously_hidden) - - visit debate_path(debate) - expect(page).to have_content(comment.body) - expect(page).to_not have_content(comment_previously_hidden.body) end scenario 'Show user activity' do - citizen = create(:user) - admin = create(:administrator) - create(:moderator, user: admin.user) + user = create(:user, :hidden) - debate1 = create(:debate, :hidden, author: citizen) - debate2 = create(:debate, author: citizen) - comment1 = create(:comment, :hidden, user: citizen, commentable: debate2, body: "You have the manners of a beggar") - comment2 = create(:comment, user: citizen, commentable: debate2, body: 'Not Spam') + debate1 = create(:debate, :hidden, author: user) + debate2 = create(:debate, author: user) + comment1 = create(:comment, :hidden, user: user, commentable: debate2, body: "You have the manners of a beggar") + comment2 = create(:comment, user: user, commentable: debate2, body: 'Not Spam') - login_as(admin.user) - visit debate_path(debate2) - - within("#debate_#{debate2.id}") do - click_link 'Ban author' - end - - click_link "Administration" - click_link "Hidden users" - click_link citizen.name + visit admin_user_path(user) expect(page).to have_content(debate1.title) expect(page).to have_content(debate2.title) @@ -63,4 +23,66 @@ feature 'Admin users' do expect(page).to have_content(comment2.body) end -end \ No newline at end of file + scenario 'Restore' do + user = create(:user, :hidden) + visit admin_users_path + + click_link 'Restore' + + expect(page).to_not have_content(user.username) + + expect(user.reload).to_not be_hidden + end + + scenario 'Confirm hide' do + user = create(:user, :hidden) + visit admin_users_path + + click_link 'Confirm' + + expect(page).to have_content(user.username) + expect(page).to have_content('Confirmed') + + expect(user.reload).to be_confirmed_hide + end + + scenario "Current filter is properly highlighted" do + visit admin_users_path + expect(page).to_not have_link('All') + expect(page).to have_link('Confirmed') + + visit admin_users_path(filter: 'all') + expect(page).to_not have_link('All') + expect(page).to have_link('Confirmed') + + visit admin_users_path(filter: 'with_confirmed_hide') + expect(page).to have_link('All') + expect(page).to_not have_link('Confirmed') + end + + scenario "Filtering users" do + create(:user, :hidden, username: "Unconfirmed") + create(:user, :hidden, :with_confirmed_hide, username: "Confirmed user") + + visit admin_users_path(filter: 'all') + expect(page).to have_content('Unconfirmed') + expect(page).to have_content('Confirmed user') + + visit admin_users_path(filter: 'with_confirmed_hide') + expect(page).to_not have_content('Unconfirmed') + expect(page).to have_content('Confirmed user') + end + + scenario "Action links remember the pagination setting and the filter" do + per_page = Kaminari.config.default_per_page + (per_page + 2).times { create(:user, :hidden, :with_confirmed_hide) } + + visit admin_users_path(filter: 'with_confirmed_hide', page: 2) + + click_on('Restore', match: :first, exact: true) + + expect(current_url).to include('filter=with_confirmed_hide') + expect(current_url).to include('page=2') + end + +end diff --git a/spec/features/comments_spec.rb b/spec/features/comments_spec.rb index 58626b02b..0986412bc 100644 --- a/spec/features/comments_spec.rb +++ b/spec/features/comments_spec.rb @@ -148,14 +148,14 @@ feature 'Comments' do expect(page).to have_css("#unflag-expand-comment-#{comment.id}") end - expect(InappropiateFlag.flagged?(user, comment)).to be + expect(Flag.flagged?(user, comment)).to be end scenario "Undoing flagging as inappropriate", :js do user = create(:user) debate = create(:debate) comment = create(:comment, commentable: debate) - InappropiateFlag.flag!(user, comment) + Flag.flag!(user, comment) login_as(user) visit debate_path(debate) @@ -167,7 +167,7 @@ feature 'Comments' do expect(page).to have_css("#flag-expand-comment-#{comment.id}") end - expect(InappropiateFlag.flagged?(user, comment)).to_not be + expect(Flag.flagged?(user, comment)).to_not be end feature "Moderators" do diff --git a/spec/features/debates_spec.rb b/spec/features/debates_spec.rb index b9e174915..3f01bd312 100644 --- a/spec/features/debates_spec.rb +++ b/spec/features/debates_spec.rb @@ -315,7 +315,7 @@ feature 'Debates' do end end - scenario "Flagging as inappropiate", :js do + scenario "Flagging", :js do user = create(:user) debate = create(:debate) @@ -329,13 +329,13 @@ feature 'Debates' do expect(page).to have_css("#unflag-expand-debate-#{debate.id}") end - expect(InappropiateFlag.flagged?(user, debate)).to be + expect(Flag.flagged?(user, debate)).to be end - scenario "Undoing flagging as inappropiate", :js do + scenario "Unflagging", :js do user = create(:user) debate = create(:debate) - InappropiateFlag.flag!(user, debate) + Flag.flag!(user, debate) login_as(user) visit debate_path(debate) @@ -347,7 +347,7 @@ feature 'Debates' do expect(page).to have_css("#flag-expand-debate-#{debate.id}") end - expect(InappropiateFlag.flagged?(user, debate)).to_not be + expect(Flag.flagged?(user, debate)).to_not be end feature 'Debate index order filters', :js do diff --git a/spec/features/moderation/comments_spec.rb b/spec/features/moderation/comments_spec.rb index 2e3d1196e..6c2689f0e 100644 --- a/spec/features/moderation/comments_spec.rb +++ b/spec/features/moderation/comments_spec.rb @@ -104,65 +104,62 @@ feature 'Moderate Comments' do visit moderation_comments_path expect(page).to_not have_link('All') expect(page).to have_link('Pending') - expect(page).to have_link('Archived') + expect(page).to have_link('Ignored') visit moderation_comments_path(filter: 'all') expect(page).to_not have_link('All') expect(page).to have_link('Pending') - expect(page).to have_link('Archived') + expect(page).to have_link('Ignored') - visit moderation_comments_path(filter: 'pending') + visit moderation_comments_path(filter: 'pending_flag_review') expect(page).to have_link('All') expect(page).to_not have_link('Pending') - expect(page).to have_link('Archived') + expect(page).to have_link('Ignored') - visit moderation_comments_path(filter: 'archived') + visit moderation_comments_path(filter: 'with_ignored_flag') expect(page).to have_link('All') expect(page).to have_link('Pending') - expect(page).to_not have_link('Archived') + expect(page).to_not have_link('Ignored') end scenario "Filtering comments" do - create(:comment, :flagged_as_inappropiate, body: "Pending comment") - create(:comment, :flagged_as_inappropiate, :hidden, body: "Hidden comment") - create(:comment, :flagged_as_inappropiate, :archived, body: "Archived comment") + create(:comment, :flagged, body: "Pending comment") + create(:comment, :flagged, :hidden, body: "Hidden comment") + create(:comment, :flagged, :with_ignored_flag, body: "Ignored comment") visit moderation_comments_path(filter: 'all') expect(page).to have_content('Pending comment') expect(page).to_not have_content('Hidden comment') - expect(page).to have_content('Archived comment') + expect(page).to have_content('Ignored comment') - visit moderation_comments_path(filter: 'pending') + visit moderation_comments_path(filter: 'pending_flag_review') expect(page).to have_content('Pending comment') expect(page).to_not have_content('Hidden comment') - expect(page).to_not have_content('Archived comment') + expect(page).to_not have_content('Ignored comment') - visit moderation_comments_path(filter: 'archived') + visit moderation_comments_path(filter: 'with_ignored_flag') expect(page).to_not have_content('Pending comment') expect(page).to_not have_content('Hidden comment') - expect(page).to have_content('Archived comment') + expect(page).to have_content('Ignored comment') end scenario "Reviewing links remember the pagination setting and the filter" do per_page = Kaminari.config.default_per_page - (per_page + 2).times { create(:comment, :flagged_as_inappropiate) } + (per_page + 2).times { create(:comment, :flagged) } - visit moderation_comments_path(filter: 'pending', page: 2) + visit moderation_comments_path(filter: 'pending_flag_review', page: 2) - click_link('Archive', match: :first, exact: true) + click_link('Ignore', match: :first, exact: true) - uri = URI.parse(current_url) - query_params = Rack::Utils.parse_nested_query(uri.query).symbolize_keys - - expect(query_params[:filter]).to eq('pending') - expect(query_params[:page]).to eq('2') + expect(current_url).to include('filter=pending_flag_review') + expect(current_url).to include('page=2') end feature 'A flagged comment exists' do background do debate = create(:debate, title: 'Democracy') - @comment = create(:comment, :flagged_as_inappropiate, commentable: debate, body: 'spammy spam') + @comment = create(:comment, :flagged, commentable: debate, body: 'spammy spam') visit moderation_comments_path end @@ -172,7 +169,7 @@ feature 'Moderate Comments' do expect(page).to have_content('spammy spam') expect(page).to have_content('1') expect(page).to have_link('Hide') - expect(page).to have_link('Archive') + expect(page).to have_link('Ignore') end end @@ -187,18 +184,18 @@ feature 'Moderate Comments' do expect(@comment.reload).to be_hidden end - scenario 'Marking the comment as archived' do + scenario 'Marking the comment as ignored' do within("#comment_#{@comment.id}") do - click_link('Archive') + click_link('Ignore') end expect(current_path).to eq(moderation_comments_path) within("#comment_#{@comment.id}") do - expect(page).to have_content('Archived') + expect(page).to have_content('Ignored') end - expect(@comment.reload).to be_archived + expect(@comment.reload).to be_ignored_flag end end end diff --git a/spec/features/moderation/debates_spec.rb b/spec/features/moderation/debates_spec.rb index b1234bb1b..60ec2c0cf 100644 --- a/spec/features/moderation/debates_spec.rb +++ b/spec/features/moderation/debates_spec.rb @@ -47,64 +47,61 @@ feature 'Moderate debates' do visit moderation_debates_path expect(page).to_not have_link('All') expect(page).to have_link('Pending') - expect(page).to have_link('Archived') + expect(page).to have_link('Ignored') visit moderation_debates_path(filter: 'all') expect(page).to_not have_link('All') expect(page).to have_link('Pending') - expect(page).to have_link('Archived') + expect(page).to have_link('Ignored') - visit moderation_debates_path(filter: 'pending') + visit moderation_debates_path(filter: 'pending_flag_review') expect(page).to have_link('All') expect(page).to_not have_link('Pending') - expect(page).to have_link('Archived') + expect(page).to have_link('Ignored') - visit moderation_debates_path(filter: 'archived') + visit moderation_debates_path(filter: 'with_ignored_flag') expect(page).to have_link('All') expect(page).to have_link('Pending') - expect(page).to_not have_link('Archived') + expect(page).to_not have_link('Ignored') end scenario "Filtering debates" do - create(:debate, :flagged_as_inappropiate, title: "Pending debate") - create(:debate, :flagged_as_inappropiate, :hidden, title: "Hidden debate") - create(:debate, :flagged_as_inappropiate, :archived, title: "Archived debate") + create(:debate, :flagged, title: "Pending debate") + create(:debate, :flagged, :hidden, title: "Hidden debate") + create(:debate, :flagged, :with_ignored_flag, title: "Ignored debate") visit moderation_debates_path(filter: 'all') expect(page).to have_content('Pending debate') expect(page).to_not have_content('Hidden debate') - expect(page).to have_content('Archived debate') + expect(page).to have_content('Ignored debate') - visit moderation_debates_path(filter: 'pending') + visit moderation_debates_path(filter: 'pending_flag_review') expect(page).to have_content('Pending debate') expect(page).to_not have_content('Hidden debate') - expect(page).to_not have_content('Archived debate') + expect(page).to_not have_content('Ignored debate') - visit moderation_debates_path(filter: 'archived') + visit moderation_debates_path(filter: 'with_ignored_flag') expect(page).to_not have_content('Pending debate') expect(page).to_not have_content('Hidden debate') - expect(page).to have_content('Archived debate') + expect(page).to have_content('Ignored debate') end scenario "Reviewing links remember the pagination setting and the filter" do per_page = Kaminari.config.default_per_page - (per_page + 2).times { create(:debate, :flagged_as_inappropiate) } + (per_page + 2).times { create(:debate, :flagged) } - visit moderation_debates_path(filter: 'pending', page: 2) + visit moderation_debates_path(filter: 'pending_flag_review', page: 2) - click_link('Archive', match: :first, exact: true) + click_link('Ignore', match: :first, exact: true) - uri = URI.parse(current_url) - query_params = Rack::Utils.parse_nested_query(uri.query).symbolize_keys - - expect(query_params[:filter]).to eq('pending') - expect(query_params[:page]).to eq('2') + expect(current_url).to include('filter=pending_flag_review') + expect(current_url).to include('page=2') end feature 'A flagged debate exists' do background do - @debate = create(:debate, :flagged_as_inappropiate, title: 'spammy spam', description: 'buy buy buy') + @debate = create(:debate, :flagged, title: 'spammy spam', description: 'buy buy buy') visit moderation_debates_path end @@ -114,7 +111,7 @@ feature 'Moderate debates' do expect(page).to have_content('buy buy buy') expect(page).to have_content('1') expect(page).to have_link('Hide') - expect(page).to have_link('Archive') + expect(page).to have_link('Ignore') end end @@ -129,18 +126,18 @@ feature 'Moderate debates' do expect(@debate.reload).to be_hidden end - scenario 'Marking the debate as archived' do + scenario 'Marking the debate as ignored' do within("#debate_#{@debate.id}") do - click_link('Archive') + click_link('Ignore') end expect(current_path).to eq(moderation_debates_path) within("#debate_#{@debate.id}") do - expect(page).to have_content('Archived') + expect(page).to have_content('Ignored') end - expect(@debate.reload).to be_archived + expect(@debate.reload).to be_ignored_flag end end end diff --git a/spec/lib/acts_as_paranoid_aliases_spec.rb b/spec/lib/acts_as_paranoid_aliases_spec.rb index 1c99884f3..b47be8cea 100644 --- a/spec/lib/acts_as_paranoid_aliases_spec.rb +++ b/spec/lib/acts_as_paranoid_aliases_spec.rb @@ -2,7 +2,7 @@ require 'rails_helper' describe 'Paranoid methods' do - describe '#hide_all' do + describe '.hide_all' do it 'hides all instances in the id list' do debate1 = create(:debate) debate2 = create(:debate) @@ -17,7 +17,7 @@ describe 'Paranoid methods' do end end - describe '#restore_all' do + describe '.restore_all' do it 'restores all instances in the id list' do debate1 = create(:debate) debate2 = create(:debate) @@ -34,4 +34,14 @@ describe 'Paranoid methods' do end end + describe '#restore' do + it 'resets the confirmed_hide_at attribute' do + debate = create(:debate, :hidden, :with_confirmed_hide) + + debate.restore + + expect(debate.reload.confirmed_hide?).to_not be + end + end + end diff --git a/spec/models/ability_spec.rb b/spec/models/ability_spec.rb index a7cfd195f..06e2a9a62 100644 --- a/spec/models/ability_spec.rb +++ b/spec/models/ability_spec.rb @@ -31,38 +31,38 @@ describe Ability do it { should_not be_able_to(:comment_as_administrator, debate) } it { should_not be_able_to(:comment_as_moderator, debate) } - describe 'flagging content as inappropiate' do - it { should be_able_to(:flag_as_inappropiate, debate) } - it { should_not be_able_to(:undo_flag_as_inappropiate, debate) } - it { should be_able_to(:flag_as_inappropiate, comment) } - it { should_not be_able_to(:undo_flag_as_inappropiate, comment) } + describe 'flagging content' do + it { should be_able_to(:flag, debate) } + it { should_not be_able_to(:unflag, debate) } + it { should be_able_to(:flag, comment) } + it { should_not be_able_to(:unflag, comment) } describe "own comments" do let(:own_comment) { create(:comment, author: user) } - it { should_not be_able_to(:flag_as_inappropiate, own_comment) } - it { should_not be_able_to(:undo_flag_as_inappropiate, own_comment) } + it { should_not be_able_to(:flag, own_comment) } + it { should_not be_able_to(:unflag, own_comment) } end describe "own debates" do let(:own_debate) { create(:debate, author: user) } - it { should_not be_able_to(:flag_as_inappropiate, own_debate) } - it { should_not be_able_to(:undo_flag_as_inappropiate, own_debate) } + it { should_not be_able_to(:flag, own_debate) } + it { should_not be_able_to(:unflag, own_debate) } end describe "already-flagged comments" do - before(:each) { InappropiateFlag.flag!(user, comment) } + before(:each) { Flag.flag!(user, comment) } - it { should_not be_able_to(:flag_as_inappropiate, comment) } - it { should be_able_to(:undo_flag_as_inappropiate, comment) } + it { should_not be_able_to(:flag, comment) } + it { should be_able_to(:unflag, comment) } end describe "already-flagged debates" do - before(:each) { InappropiateFlag.flag!(user, debate) } + before(:each) { Flag.flag!(user, debate) } - it { should_not be_able_to(:flag_as_inappropiate, debate) } - it { should be_able_to(:undo_flag_as_inappropiate, debate) } + it { should_not be_able_to(:flag, debate) } + it { should be_able_to(:unflag, debate) } end end @@ -131,8 +131,8 @@ describe Ability do let(:own_debate) { create(:debate, author: user) } let(:hidden_comment) { create(:comment, :hidden) } let(:hidden_debate) { create(:debate, :hidden) } - let(:archived_comment) { create(:comment, :archived) } - let(:archived_debate) { create(:debate, :archived) } + let(:ignored_comment) { create(:comment, :with_ignored_flag) } + let(:ignored_debate) { create(:debate, :with_ignored_flag) } it { should be_able_to(:hide, comment) } it { should be_able_to(:hide_in_moderation_screen, comment) } @@ -144,15 +144,15 @@ describe Ability do it { should_not be_able_to(:hide, hidden_debate) } it { should_not be_able_to(:hide, own_debate) } - it { should be_able_to(:archive, comment) } - it { should_not be_able_to(:archive, hidden_comment) } - it { should_not be_able_to(:archive, archived_comment) } - it { should_not be_able_to(:archive, own_comment) } + it { should be_able_to(:ignore_flag, comment) } + it { should_not be_able_to(:ignore_flag, hidden_comment) } + it { should_not be_able_to(:ignore_flag, ignored_comment) } + it { should_not be_able_to(:ignore_flag, own_comment) } - it { should be_able_to(:archive, debate) } - it { should_not be_able_to(:archive, hidden_debate) } - it { should_not be_able_to(:archive, archived_debate) } - it { should_not be_able_to(:archive, own_debate) } + it { should be_able_to(:ignore_flag, debate) } + it { should_not be_able_to(:ignore_flag, hidden_debate) } + it { should_not be_able_to(:ignore_flag, ignored_debate) } + it { should_not be_able_to(:ignore_flag, own_debate) } it { should_not be_able_to(:hide, user) } it { should be_able_to(:hide, other_user) } @@ -169,15 +169,34 @@ describe Ability do describe "Administrator" do let(:user) { create(:user) } before { create(:administrator, user: user) } - let(:other_user) { create(:user) } + + let(:other_user) { create(:user) } + let(:hidden_user) { create(:user, :hidden) } + + let(:hidden_debate) { create(:debate, :hidden) } + let(:hidden_comment) { create(:comment, :hidden) } + let(:own_debate) { create(:debate, author: user)} + let(:own_comment) { create(:comment, author: user)} it { should be_able_to(:index, Debate) } 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) } - it { should be_able_to(:restore, other_user) } + it { should_not be_able_to(:restore, comment) } + it { should_not be_able_to(:restore, debate) } + it { should_not be_able_to(:restore, other_user) } + + it { should be_able_to(:restore, hidden_comment) } + it { should be_able_to(:restore, hidden_debate) } + it { should be_able_to(:restore, hidden_user) } + + it { should_not be_able_to(:confirm_hide, comment) } + it { should_not be_able_to(:confirm_hide, debate) } + it { should_not be_able_to(:confirm_hide, other_user) } + + it { should be_able_to(:confirm_hide, hidden_comment) } + it { should be_able_to(:confirm_hide, hidden_debate) } + it { should be_able_to(:confirm_hide, hidden_user) } it { should be_able_to(:comment_as_administrator, debate) } it { should_not be_able_to(:comment_as_moderator, debate) } diff --git a/spec/models/inappropiate_flag_spec.rb b/spec/models/flag_spec.rb similarity index 65% rename from spec/models/inappropiate_flag_spec.rb rename to spec/models/flag_spec.rb index 0ae831ee0..a5725e7c8 100644 --- a/spec/models/inappropiate_flag_spec.rb +++ b/spec/models/flag_spec.rb @@ -1,6 +1,6 @@ require 'rails_helper' -describe InappropiateFlag do +describe Flag do let(:user) { create(:user) } let(:comment) { create(:comment) } @@ -8,43 +8,35 @@ describe InappropiateFlag do describe '.flag!' do it 'creates a flag when there is none' do - expect { described_class.flag!(user, comment) }.to change{ InappropiateFlag.count }.by(1) - expect(InappropiateFlag.last.user).to eq(user) - expect(InappropiateFlag.last.flaggable).to eq(comment) + expect { described_class.flag!(user, comment) }.to change{ Flag.count }.by(1) + expect(Flag.last.user).to eq(user) + expect(Flag.last.flaggable).to eq(comment) end it 'raises an error if the flag has already been created' do described_class.flag!(user, comment) - expect { described_class.flag!(user, comment) }.to raise_error(InappropiateFlag::AlreadyFlaggedError) + expect { described_class.flag!(user, comment) }.to raise_error(Flag::AlreadyFlaggedError) end it 'increases the flag count' do - expect { described_class.flag!(user, comment) }.to change{ comment.reload.inappropiate_flags_count }.by(1) - end - - it 'updates the flagged_as date' do - expect { described_class.flag!(user, comment) }.to change{ comment.reload.flagged_as_inappropiate_at } + expect { described_class.flag!(user, comment) }.to change{ comment.reload.flags_count }.by(1) end end describe '.unflag!' do it 'raises an error if the flag does not exist' do - expect { described_class.unflag!(user, comment) }.to raise_error(InappropiateFlag::NotFlaggedError) + expect { described_class.unflag!(user, comment) }.to raise_error(Flag::NotFlaggedError) end describe 'when the flag already exists' do before(:each) { described_class.flag!(user, comment) } it 'removes an existing flag' do - expect { described_class.unflag!(user, comment) }.to change{ InappropiateFlag.count }.by(-1) + expect { described_class.unflag!(user, comment) }.to change{ Flag.count }.by(-1) end it 'decreases the flag count' do - expect { described_class.unflag!(user, comment) }.to change{ comment.reload.inappropiate_flags_count }.by(-1) - end - - it 'does not update the flagged_as date' do - expect { described_class.unflag!(user, comment) }.to_not change{ comment.flagged_as_inappropiate_at } + expect { described_class.unflag!(user, comment) }.to change{ comment.reload.flags_count }.by(-1) end end