diff --git a/app/models/note.rb b/app/models/note.rb
index e1cd971132b8fb1d49af5c005a0ba35dd32f3fe7..f2a9680ff57165cf157156cb43fc558fced54878 100644
--- a/app/models/note.rb
+++ b/app/models/note.rb
@@ -272,7 +272,7 @@ class Note < ActiveRecord::Base
     # cross-project references.  For same-project references, return the
     # unmodified GFM reference.
     def mentioner_gfm_ref(noteable, mentioner, mentioner_project = mentioner.project)
-      if mentioner.is_a?(Commit) && project.nil?
+      if mentioner.is_a?(Commit) && mentioner_project.nil?
         return mentioner.gfm_reference.sub('commit ', 'commit %')
       end
       
@@ -283,7 +283,7 @@ class Note < ActiveRecord::Base
     # projects are not the same, add the mentioning project's path to the
     # returned value.
     def full_gfm_reference(mentioning_project, noteable_project, mentioner)
-      if mentioning_project.id == noteable_project
+      if mentioning_project == noteable_project
         mentioner.gfm_reference
       else
         if mentioner.is_a?(Commit)
diff --git a/lib/gitlab/push_data_builder.rb b/lib/gitlab/push_data_builder.rb
index f44eb872122b3da1f9d1372d700a0a309d739fb9..f97784f5abb7e88495bbe9b9c44f8eeb201e93a6 100644
--- a/lib/gitlab/push_data_builder.rb
+++ b/lib/gitlab/push_data_builder.rb
@@ -31,7 +31,6 @@ module Gitlab
         # For performance purposes maximum 20 latest commits
         # will be passed as post receive hook data.
         commit_attrs = commits_limited.map(&:hook_attrs)
-        end
 
         type = Gitlab::Git.tag_ref?(ref) ? "tag_push" : "push"
         # Hash to be passed as post_receive_data