Commit d6c58a51 authored by Alexis Reigel's avatar Alexis Reigel

only use symbols instead of enum hash accessor

parent 7ad7222a
...@@ -77,22 +77,22 @@ module Gitlab ...@@ -77,22 +77,22 @@ module Gitlab
gpg_key_primary_keyid: gpg_key&.primary_keyid || verified_signature.fingerprint, gpg_key_primary_keyid: gpg_key&.primary_keyid || verified_signature.fingerprint,
gpg_key_user_name: user_infos[:name], gpg_key_user_name: user_infos[:name],
gpg_key_user_email: user_infos[:email], gpg_key_user_email: user_infos[:email],
valid_signature: verification_status == GpgSignature.verification_statuses[:verified], valid_signature: verification_status == :verified,
verification_status: verification_status verification_status: verification_status
} }
end end
def verification_status(gpg_key) def verification_status(gpg_key)
return GpgSignature.verification_statuses[:unknown_key] unless gpg_key return :unknown_key unless gpg_key
return GpgSignature.verification_statuses[:unverified_key] unless gpg_key.verified? return :unverified_key unless gpg_key.verified?
return GpgSignature.verification_statuses[:unverified] unless verified_signature.valid? return :unverified unless verified_signature.valid?
if gpg_key.verified_and_belongs_to_email?(@commit.committer_email) if gpg_key.verified_and_belongs_to_email?(@commit.committer_email)
GpgSignature.verification_statuses[:verified] :verified
elsif gpg_key.user.all_emails.include?(@commit.committer_email) elsif gpg_key.user.all_emails.include?(@commit.committer_email)
GpgSignature.verification_statuses[:same_user_different_email] :same_user_different_email
else else
GpgSignature.verification_statuses[:other_user] :other_user
end end
end end
......
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