From 08ecf0e78e1f0f3073fb0d2fa3dfa8d29acbbd04 Mon Sep 17 00:00:00 2001 From: Adam Niedzielski <adamsunday@gmail.com> Date: Tue, 9 Aug 2016 17:45:22 +0200 Subject: [PATCH] Use "is_a?" because it is more readable than checking the "type" field. --- app/views/shared/members/_member.html.haml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/app/views/shared/members/_member.html.haml b/app/views/shared/members/_member.html.haml index 0db7ecb61bb..7dde9442e24 100644 --- a/app/views/shared/members/_member.html.haml +++ b/app/views/shared/members/_member.html.haml @@ -80,7 +80,7 @@ = form_for member, remote: true do |f| .prepend-top-10 = f.select :access_level, options_for_select(member.class.access_level_roles, member.access_level), {}, class: 'form-control' - - if member.type == 'ProjectMember' + - if member.is_a?(ProjectMember) .prepend-top-10 = f.text_field :expires_at, class: 'form-control js-access-expiration-date', placeholder: 'Access expiration date', id: "member_expires_at_#{member.id}" .prepend-top-10 -- 2.30.9