Operate on ids to avoid unnecessary loading.

2 files changed, 5 insertions, 5 deletions

diff --git a/app/models/user_interacted_project.rb b/app/models/user_interacted_project.rb
index 5e7f6d5eeda..5d661018a9d 100644
--- a/app/models/user_interacted_project.rb
+++ b/app/models/user_interacted_project.rb
@@ -2,8 +2,8 @@ class UserInteractedProject < ActiveRecord::Base
   belongs_to :user
   belongs_to :project
 
-  validates :project, presence: true
-  validates :user, presence: true
+  validates :project_id, presence: true
+  validates :user_id, presence: true
 
   CACHE_EXPIRY_TIME = 1.day
 
@@ -15,7 +15,7 @@ class UserInteractedProject < ActiveRecord::Base
       # For events without a project, we simply don't care.
       # An example of this is the creation of a snippet (which
       # is not related to any project).
-      return unless event.project
+      return unless event.project_id
 
       # This is a precaution because the cache lookup
       # will work just fine without an author.
diff --git a/spec/models/user_interacted_project_spec.rb b/spec/models/user_interacted_project_spec.rb
index 1428fc2d433..cb4bb3372d4 100644
--- a/spec/models/user_interacted_project_spec.rb
+++ b/spec/models/user_interacted_project_spec.rb
@@ -55,6 +55,6 @@ describe UserInteractedProject do
     end
   end
 
-  it { is_expected.to validate_presence_of(:project) }
-  it { is_expected.to validate_presence_of(:user) }
+  it { is_expected.to validate_presence_of(:project_id) }
+  it { is_expected.to validate_presence_of(:user_id) }
 end