Commit 666ddec9 authored by Mark Chao's avatar Mark Chao

Combine two updates in one query

parent c4bcb3a9
...@@ -95,8 +95,7 @@ class Project < ActiveRecord::Base ...@@ -95,8 +95,7 @@ class Project < ActiveRecord::Base
unless: :ci_cd_settings, unless: :ci_cd_settings,
if: proc { ProjectCiCdSetting.available? } if: proc { ProjectCiCdSetting.available? }
after_create :set_last_activity_at after_create :set_timestamps_for_create
after_create :set_last_repository_updated_at
after_update :update_forks_visibility_level after_update :update_forks_visibility_level
before_destroy :remove_private_deploy_keys before_destroy :remove_private_deploy_keys
...@@ -2102,13 +2101,8 @@ class Project < ActiveRecord::Base ...@@ -2102,13 +2101,8 @@ class Project < ActiveRecord::Base
gitlab_shell.exists?(repository_storage, "#{disk_path}.git") gitlab_shell.exists?(repository_storage, "#{disk_path}.git")
end end
# set last_activity_at to the same as created_at def set_timestamps_for_create
def set_last_activity_at update_columns(last_activity_at: self.created_at, last_repository_updated_at: self.created_at)
update_column(:last_activity_at, self.created_at)
end
def set_last_repository_updated_at
update_column(:last_repository_updated_at, self.created_at)
end end
def cross_namespace_reference?(from) def cross_namespace_reference?(from)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment