diff options
author | Jan Provaznik <jprovaznik@gitlab.com> | 2019-09-06 11:54:58 +0200 |
---|---|---|
committer | Jan Provaznik <jprovaznik@gitlab.com> | 2019-09-07 06:43:17 +0200 |
commit | f2754e416e16c1b56810b835505346c295578c7a (patch) | |
tree | 42867740b585563f6569e5b9a46071c67466e5bc /app/models/issue.rb | |
parent | 093858adc76f893c35b0bcf25127c3f229a7bcb6 (diff) | |
download | gitlab-ce-f2754e416e16c1b56810b835505346c295578c7a.tar.gz |
Use delete_all for deleting eventsevents-delete-all
Because we don't have any destroy callbacks (or other logic
triggered on event destroy), there is no reason for deleting events
inefficiently one by one, instead we can use :delete_all.
Diffstat (limited to 'app/models/issue.rb')
-rw-r--r-- | app/models/issue.rb | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/app/models/issue.rb b/app/models/issue.rb index 75d4fc8c1c5..7c5a139ab55 100644 --- a/app/models/issue.rb +++ b/app/models/issue.rb @@ -31,7 +31,7 @@ class Issue < ApplicationRecord has_internal_id :iid, scope: :project, init: ->(s) { s&.project&.issues&.maximum(:iid) } - has_many :events, as: :target, dependent: :destroy # rubocop:disable Cop/ActiveRecordDependent + has_many :events, as: :target, dependent: :delete_all # rubocop:disable Cop/ActiveRecordDependent has_many :merge_requests_closing_issues, class_name: 'MergeRequestsClosingIssues', |