diff options
author | Douwe Maan <douwe@gitlab.com> | 2015-10-01 16:17:18 +0300 |
---|---|---|
committer | Douwe Maan <douwe@gitlab.com> | 2015-10-01 16:17:18 +0300 |
commit | 96c2e22549917a32bae03c3c1e850a3225049239 (patch) | |
tree | 869c8d9fb86beb3306a9cad7fa1fdc0345dd934c /app/models | |
parent | 114853063b3a1747acfe96fa8c8159f4779e291c (diff) | |
parent | ea72d53ec083676ee1171e97c0869132f360d0c9 (diff) |
Merge branch 'disable-report-button-if-already-reported' into 'master'
Disable the "Report abuse" button if a user has already been reported
Hello,
I've implemented the feature request #2330. Here is what it looks like:
![report-abuse-button-disabled](https://gitlab.com/rymai/gitlab-ce/uploads/420d51906eac1c40c50701a0a340474f/report-abuse-button-disabled.png)
I hope that's an acceptable solution.
cc @DouweM
See merge request !1456
Diffstat (limited to 'app/models')
-rw-r--r-- | app/models/abuse_report.rb | 4 | ||||
-rw-r--r-- | app/models/user.rb | 1 |
2 files changed, 3 insertions, 2 deletions
diff --git a/app/models/abuse_report.rb b/app/models/abuse_report.rb index 07c87a7fe87..89b3116b9f2 100644 --- a/app/models/abuse_report.rb +++ b/app/models/abuse_report.rb @@ -11,11 +11,11 @@ # class AbuseReport < ActiveRecord::Base - belongs_to :reporter, class_name: "User" + belongs_to :reporter, class_name: 'User' belongs_to :user validates :reporter, presence: true validates :user, presence: true validates :message, presence: true - validates :user_id, uniqueness: { scope: :reporter_id } + validates :user_id, uniqueness: true end diff --git a/app/models/user.rb b/app/models/user.rb index 3879f3fd381..9ea7cabff15 100644 --- a/app/models/user.rb +++ b/app/models/user.rb @@ -129,6 +129,7 @@ class User < ActiveRecord::Base has_many :assigned_issues, dependent: :destroy, foreign_key: :assignee_id, class_name: "Issue" has_many :assigned_merge_requests, dependent: :destroy, foreign_key: :assignee_id, class_name: "MergeRequest" has_many :oauth_applications, class_name: 'Doorkeeper::Application', as: :owner, dependent: :destroy + has_one :abuse_report, dependent: :destroy # |