X-Git-Url: https://git.openstreetmap.org./rails.git/blobdiff_plain/5ea1ba8d8427d75aef4bd91ef79891ee0b86de28..1f880528bd834bd59465b51ea8eccfe9fbd1715a:/app/controllers/reports_controller.rb diff --git a/app/controllers/reports_controller.rb b/app/controllers/reports_controller.rb index 08092c073..a04fab5b9 100644 --- a/app/controllers/reports_controller.rb +++ b/app/controllers/reports_controller.rb @@ -3,7 +3,8 @@ class ReportsController < ApplicationController before_action :authorize_web before_action :set_locale - before_action :require_user + + authorize_resource def new if required_new_report_params_present? @@ -16,11 +17,15 @@ class ReportsController < ApplicationController def create @report = current_user.reports.new(report_params) - @report.issue = Issue.find_or_initialize_by(:reportable_id => params[:report][:issue][:reportable_id], :reportable_type => params[:report][:issue][:reportable_type]) + @report.issue = Issue + .create_with(:assigned_role => default_assigned_role) + .find_or_initialize_by(issue_params) if @report.save - @report.issue.save - @report.issue.reopen! unless @report.issue.open? + @report.issue.assigned_role = "administrator" if default_assigned_role == "administrator" + @report.issue.reopen unless @report.issue.open? + @report.issue.save! + redirect_to helpers.reportable_url(@report.issue.reportable), :notice => t(".successful_report") else redirect_to new_report_path(:reportable_type => @report.issue.reportable_type, :reportable_id => @report.issue.reportable_id), :notice => t(".provide_details") @@ -38,6 +43,24 @@ class ReportsController < ApplicationController end def report_params - params[:report].permit(:details, :category) + params.require(:report).permit(:details, :category) + end + + def issue_params + params.require(:report).require(:issue).permit(:reportable_id, :reportable_type) + end + + def default_assigned_role + case issue_params[:reportable_type] + when "Note" + "moderator" + when "User" + case report_params[:category] + when "vandal" then "moderator" + else "administrator" + end + else + "administrator" + end end end