Conditionally assign ref_name for more efficiency

44374434 · Matija Čupić · 86c0558c · 44374434
Commit 44374434 authored Nov 30, 2018 by Matija Čupić
Hide whitespace changes
Inline Side-by-side

Showing with 5 additions and 1 deletion

app/models/project.rb app/models/project.rb +5 -1

No files found.
--- a/app/models/project.rb
+++ b/app/models/project.rb
@@ -1741,7 +1741,11 @@ class Project < ActiveRecord::Base
    raise Repository::AmbiguousRefError if repository.ambiguous_ref?(ref)

    resolved_ref = repository.expand_ref(ref) || ref
-    ref_name = Gitlab::Git.ref_name(resolved_ref)
+    ref_name = if resolved_ref == ref
+                 Gitlab::Git.ref_name(resolved_ref)
+               else
+                 ref
+               end

    if Gitlab::Git.branch_ref?(resolved_ref)
      ProtectedBranch.protected?(self, ref_name)