2019-05-08 22:20:51 +08:00
|
|
|
# frozen_string_literal: true
|
|
|
|
|
2019-01-04 01:03:01 +08:00
|
|
|
class ReviewablesController < ApplicationController
|
|
|
|
requires_login
|
|
|
|
|
|
|
|
PER_PAGE = 10
|
|
|
|
|
|
|
|
before_action :version_required, only: [:update, :perform]
|
|
|
|
|
|
|
|
def index
|
|
|
|
offset = params[:offset].to_i
|
|
|
|
|
|
|
|
if params[:type].present?
|
|
|
|
raise Discourse::InvalidParameter.new(:type) unless Reviewable.valid_type?(params[:type])
|
|
|
|
end
|
|
|
|
|
|
|
|
status = (params[:status] || 'pending').to_sym
|
|
|
|
raise Discourse::InvalidParameter.new(:status) unless allowed_statuses.include?(status)
|
|
|
|
|
|
|
|
topic_id = params[:topic_id] ? params[:topic_id].to_i : nil
|
|
|
|
category_id = params[:category_id] ? params[:category_id].to_i : nil
|
|
|
|
|
|
|
|
filters = {
|
|
|
|
status: status,
|
|
|
|
category_id: category_id,
|
|
|
|
topic_id: topic_id,
|
2019-05-08 01:25:11 +08:00
|
|
|
priority: params[:priority],
|
2019-01-04 01:03:01 +08:00
|
|
|
username: params[:username],
|
2019-06-06 01:19:57 +08:00
|
|
|
type: params[:type],
|
|
|
|
sort_order: params[:sort_order]
|
2019-01-04 01:03:01 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
total_rows = Reviewable.list_for(current_user, filters).count
|
|
|
|
reviewables = Reviewable.list_for(current_user, filters.merge(limit: PER_PAGE, offset: offset)).to_a
|
|
|
|
|
2019-05-08 22:20:51 +08:00
|
|
|
claimed_topics = ReviewableClaimedTopic.claimed_hash(reviewables.map { |r| r.topic_id }.uniq)
|
|
|
|
|
2019-01-04 01:03:01 +08:00
|
|
|
# This is a bit awkward, but ActiveModel serializers doesn't seem to serialize STI. Note `hash`
|
|
|
|
# is mutated by the serializer and contains the side loaded records which must be merged in the end.
|
|
|
|
hash = {}
|
|
|
|
json = {
|
|
|
|
reviewables: reviewables.map! do |r|
|
2019-05-08 22:20:51 +08:00
|
|
|
result = r.serializer.new(
|
|
|
|
r,
|
|
|
|
root: nil,
|
|
|
|
hash: hash,
|
|
|
|
scope: guardian,
|
|
|
|
claimed_topics: claimed_topics
|
|
|
|
).as_json
|
2019-01-04 01:03:01 +08:00
|
|
|
hash[:bundled_actions].uniq!
|
|
|
|
(hash['actions'] || []).uniq!
|
|
|
|
result
|
|
|
|
end,
|
|
|
|
meta: filters.merge(
|
2019-04-05 22:34:02 +08:00
|
|
|
total_rows_reviewables: total_rows, types: meta_types, reviewable_types: Reviewable.types,
|
|
|
|
reviewable_count: Reviewable.list_for(current_user).count
|
2019-01-04 01:03:01 +08:00
|
|
|
)
|
|
|
|
}
|
|
|
|
if (offset + PER_PAGE) < total_rows
|
|
|
|
json[:meta][:load_more_reviewables] = review_path(filters.merge(offset: offset + PER_PAGE))
|
|
|
|
end
|
|
|
|
json.merge!(hash)
|
|
|
|
|
|
|
|
render_json_dump(json, rest_serializer: true)
|
|
|
|
end
|
|
|
|
|
|
|
|
def topics
|
|
|
|
topic_ids = Set.new
|
|
|
|
|
|
|
|
stats = {}
|
|
|
|
unique_users = {}
|
|
|
|
|
|
|
|
# topics isn't indexed on `reviewable_score` and doesn't know what the current user can see,
|
|
|
|
# so let's query from the inside out.
|
2019-03-30 03:34:43 +08:00
|
|
|
pending = Reviewable.viewable_by(current_user).pending
|
2019-05-08 01:25:11 +08:00
|
|
|
pending = pending.where("score >= ?", Reviewable.min_score_for_priority)
|
2019-03-30 03:34:43 +08:00
|
|
|
|
|
|
|
pending.each do |r|
|
2019-01-04 01:03:01 +08:00
|
|
|
topic_ids << r.topic_id
|
|
|
|
|
|
|
|
meta = stats[r.topic_id] ||= { count: 0, unique_users: 0 }
|
|
|
|
users = unique_users[r.topic_id] ||= Set.new
|
|
|
|
|
|
|
|
r.reviewable_scores.each do |rs|
|
|
|
|
users << rs.user_id
|
|
|
|
meta[:count] += 1
|
|
|
|
end
|
|
|
|
meta[:unique_users] = users.size
|
|
|
|
end
|
|
|
|
|
|
|
|
topics = Topic.where(id: topic_ids).order('reviewable_score DESC')
|
2019-05-08 22:20:51 +08:00
|
|
|
render_serialized(
|
|
|
|
topics,
|
|
|
|
ReviewableTopicSerializer,
|
|
|
|
root: 'reviewable_topics',
|
|
|
|
stats: stats,
|
|
|
|
claimed_topics: ReviewableClaimedTopic.claimed_hash(topic_ids),
|
|
|
|
rest_serializer: true,
|
|
|
|
meta: {
|
|
|
|
types: meta_types
|
|
|
|
}
|
|
|
|
)
|
2019-01-04 01:03:01 +08:00
|
|
|
end
|
|
|
|
|
2019-09-04 23:56:25 +08:00
|
|
|
def explain
|
|
|
|
reviewable = find_reviewable
|
|
|
|
|
|
|
|
render_serialized(
|
|
|
|
{ reviewable: reviewable, scores: reviewable.explain_score },
|
|
|
|
ReviewableExplanationSerializer,
|
|
|
|
rest_serializer: true,
|
|
|
|
root: 'reviewable_explanation'
|
|
|
|
)
|
|
|
|
end
|
|
|
|
|
2019-01-04 01:03:01 +08:00
|
|
|
def show
|
|
|
|
reviewable = find_reviewable
|
|
|
|
|
|
|
|
render_serialized(
|
|
|
|
reviewable,
|
|
|
|
reviewable.serializer,
|
|
|
|
rest_serializer: true,
|
2019-05-08 22:20:51 +08:00
|
|
|
claimed_topics: ReviewableClaimedTopic.claimed_hash([reviewable.topic_id]),
|
2019-01-04 01:03:01 +08:00
|
|
|
root: 'reviewable',
|
|
|
|
meta: {
|
|
|
|
types: meta_types
|
|
|
|
}
|
|
|
|
)
|
|
|
|
end
|
|
|
|
|
2019-04-12 21:55:27 +08:00
|
|
|
def destroy
|
|
|
|
reviewable = Reviewable.find_by(id: params[:reviewable_id], created_by: current_user)
|
|
|
|
raise Discourse::NotFound.new if reviewable.blank?
|
|
|
|
|
|
|
|
reviewable.perform(current_user, :delete)
|
|
|
|
|
|
|
|
render json: success_json
|
|
|
|
end
|
|
|
|
|
2019-01-04 01:03:01 +08:00
|
|
|
def update
|
|
|
|
reviewable = find_reviewable
|
2019-05-08 22:20:51 +08:00
|
|
|
if error = claim_error?(reviewable)
|
|
|
|
return render_json_error(error)
|
|
|
|
end
|
|
|
|
|
2019-01-04 01:03:01 +08:00
|
|
|
editable = reviewable.editable_for(guardian)
|
|
|
|
raise Discourse::InvalidAccess.new unless editable.present?
|
|
|
|
|
|
|
|
# Validate parameters are all editable
|
|
|
|
edit_params = params[:reviewable] || {}
|
|
|
|
edit_params.each do |name, value|
|
|
|
|
if value.is_a?(ActionController::Parameters)
|
|
|
|
value.each do |pay_name, pay_value|
|
|
|
|
raise Discourse::InvalidAccess.new unless editable.has?("#{name}.#{pay_name}")
|
|
|
|
end
|
|
|
|
else
|
|
|
|
raise Discourse::InvalidAccess.new unless editable.has?(name)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
begin
|
|
|
|
if reviewable.update_fields(edit_params, current_user, version: params[:version].to_i)
|
|
|
|
result = edit_params.merge(version: reviewable.version)
|
|
|
|
render json: result
|
|
|
|
else
|
|
|
|
render_json_error(reviewable.errors)
|
|
|
|
end
|
|
|
|
rescue Reviewable::UpdateConflict
|
|
|
|
return render_json_error(I18n.t('reviewables.conflict'), status: 409)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
def perform
|
|
|
|
args = { version: params[:version].to_i }
|
|
|
|
|
2019-05-08 22:20:51 +08:00
|
|
|
result = nil
|
2019-01-04 01:03:01 +08:00
|
|
|
begin
|
2019-05-08 22:20:51 +08:00
|
|
|
reviewable = find_reviewable
|
|
|
|
|
|
|
|
if error = claim_error?(reviewable)
|
|
|
|
return render_json_error(error)
|
|
|
|
end
|
|
|
|
|
|
|
|
result = reviewable.perform(current_user, params[:action_id].to_sym, args)
|
2019-01-04 01:03:01 +08:00
|
|
|
rescue Reviewable::InvalidAction => e
|
|
|
|
# Consider InvalidAction an InvalidAccess
|
|
|
|
raise Discourse::InvalidAccess.new(e.message)
|
|
|
|
rescue Reviewable::UpdateConflict
|
|
|
|
return render_json_error(I18n.t('reviewables.conflict'), status: 409)
|
|
|
|
end
|
|
|
|
|
|
|
|
if result.success?
|
|
|
|
render_serialized(result, ReviewablePerformResultSerializer)
|
|
|
|
else
|
|
|
|
render_json_error(result)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2019-04-03 05:00:15 +08:00
|
|
|
def settings
|
|
|
|
raise Discourse::InvalidAccess.new unless current_user.admin?
|
|
|
|
|
|
|
|
post_action_types = PostActionType.where(id: PostActionType.flag_types.values).order('id')
|
|
|
|
|
|
|
|
if request.put?
|
2019-05-23 05:23:45 +08:00
|
|
|
params[:reviewable_priorities].each do |id, priority|
|
|
|
|
if !priority.nil? && Reviewable.priorities.has_value?(priority.to_i)
|
|
|
|
# For now, the score bonus is equal to the priority. In the future we might want
|
|
|
|
# to calculate it a different way.
|
|
|
|
PostActionType.where(id: id).update_all(
|
|
|
|
reviewable_priority: priority.to_i,
|
|
|
|
score_bonus: priority.to_f
|
|
|
|
)
|
|
|
|
end
|
2019-04-03 05:00:15 +08:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2019-05-23 05:23:45 +08:00
|
|
|
data = { reviewable_score_types: post_action_types }
|
2019-04-03 05:00:15 +08:00
|
|
|
render_serialized(data, ReviewableSettingsSerializer, rest_serializer: true)
|
|
|
|
end
|
|
|
|
|
2019-01-04 01:03:01 +08:00
|
|
|
protected
|
|
|
|
|
2019-05-08 22:20:51 +08:00
|
|
|
def claim_error?(reviewable)
|
|
|
|
return if SiteSetting.reviewable_claiming == "disabled" || reviewable.topic_id.blank?
|
|
|
|
|
|
|
|
claimed_by_id = ReviewableClaimedTopic.where(topic_id: reviewable.topic_id).pluck(:user_id)[0]
|
|
|
|
if SiteSetting.reviewable_claiming == "required" && claimed_by_id.blank?
|
|
|
|
return I18n.t('reviewables.must_claim')
|
|
|
|
end
|
|
|
|
|
|
|
|
claimed_by_id.present? && claimed_by_id != current_user.id
|
|
|
|
end
|
|
|
|
|
2019-01-04 01:03:01 +08:00
|
|
|
def find_reviewable
|
|
|
|
reviewable = Reviewable.viewable_by(current_user).where(id: params[:reviewable_id]).first
|
|
|
|
raise Discourse::NotFound.new if reviewable.blank?
|
|
|
|
reviewable
|
|
|
|
end
|
|
|
|
|
|
|
|
def allowed_statuses
|
|
|
|
@allowed_statuses ||= (%i[reviewed all] + Reviewable.statuses.keys)
|
|
|
|
end
|
|
|
|
|
|
|
|
def version_required
|
|
|
|
if params[:version].blank?
|
|
|
|
render_json_error(I18n.t('reviewables.missing_version'), status: 422)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
def meta_types
|
|
|
|
{
|
|
|
|
created_by: 'user',
|
2019-04-10 04:43:08 +08:00
|
|
|
target_created_by: 'user',
|
2019-05-08 22:20:51 +08:00
|
|
|
reviewed_by: 'user',
|
|
|
|
claimed_by: 'user'
|
2019-01-04 01:03:01 +08:00
|
|
|
}
|
|
|
|
end
|
|
|
|
|
|
|
|
end
|