diff options
author | Alexis Reigel <mail@koffeinfrei.org> | 2017-08-24 14:22:19 +0200 |
---|---|---|
committer | Alexis Reigel <mail@koffeinfrei.org> | 2017-09-05 12:18:32 +0200 |
commit | d6c58a51e5e79071157caa801bda94b2506131d4 (patch) | |
tree | 4aa469724706ee154ae93c214d3a6dd52130ab8c /lib/gitlab/gpg | |
parent | 7ad7222a5fb2790353e772416090399b9896b129 (diff) | |
download | gitlab-ce-d6c58a51e5e79071157caa801bda94b2506131d4.tar.gz |
only use symbols instead of enum hash accessor
Diffstat (limited to 'lib/gitlab/gpg')
-rw-r--r-- | lib/gitlab/gpg/commit.rb | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/lib/gitlab/gpg/commit.rb b/lib/gitlab/gpg/commit.rb index 043045ceba7..a206249ef5a 100644 --- a/lib/gitlab/gpg/commit.rb +++ b/lib/gitlab/gpg/commit.rb @@ -77,22 +77,22 @@ module Gitlab gpg_key_primary_keyid: gpg_key&.primary_keyid || verified_signature.fingerprint, gpg_key_user_name: user_infos[:name], gpg_key_user_email: user_infos[:email], - valid_signature: verification_status == GpgSignature.verification_statuses[:verified], + valid_signature: verification_status == :verified, verification_status: verification_status } end def verification_status(gpg_key) - return GpgSignature.verification_statuses[:unknown_key] unless gpg_key - return GpgSignature.verification_statuses[:unverified_key] unless gpg_key.verified? - return GpgSignature.verification_statuses[:unverified] unless verified_signature.valid? + return :unknown_key unless gpg_key + return :unverified_key unless gpg_key.verified? + return :unverified unless verified_signature.valid? 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) - GpgSignature.verification_statuses[:same_user_different_email] + :same_user_different_email else - GpgSignature.verification_statuses[:other_user] + :other_user end end |