diff --git a/lib/puppet/type/keycloak_protocol_mapper.rb b/lib/puppet/type/keycloak_protocol_mapper.rb
index 27ef4b1..543b2bb 100644
--- a/lib/puppet/type/keycloak_protocol_mapper.rb
+++ b/lib/puppet/type/keycloak_protocol_mapper.rb
@@ -1,268 +1,275 @@
 require_relative '../provider/keycloak_api'
 require_relative '../../puppet_x/keycloak/type'
 require_relative '../../puppet_x/keycloak/array_property'
 
 Puppet::Type.newtype(:keycloak_protocol_mapper) do
   desc <<-DESC
 Manage Keycloak client scope protocol mappers
 @example Add email protocol mapper to oidc-client client scope in realm test
   keycloak_protocol_mapper { "email for oidc-clients on test":
     claim_name     => 'email',
     user_attribute => 'email',
   }
   DESC
 
   extend PuppetX::Keycloak::Type
   add_autorequires
 
   ensurable
 
   newparam(:name, namevar: true) do
     desc 'The protocol mapper name'
   end
 
   newparam(:id) do
     desc 'Id.'
   end
 
   newparam(:resource_name, namevar: true) do
     desc 'The protocol mapper name. Defaults to `name`.'
     defaultto do
       @resource[:name]
     end
   end
 
   newparam(:client_scope, namevar: true) do
     desc 'client scope'
   end
 
   newparam(:realm, namevar: true) do
     desc 'realm'
   end
 
   newproperty(:protocol) do
     desc 'protocol'
     defaultto('openid-connect')
     newvalues('openid-connect', 'saml')
     munge { |v| v }
   end
 
   newparam(:type) do
     desc <<-DESC
     protocolMapper.
 
     Default is `oidc-usermodel-property-mapper` for `protocol` `openid-connect` and
     `saml-user-property-mapper` for `protocol` `saml`.
     DESC
     newvalues(
       'oidc-usermodel-property-mapper',
       'oidc-usermodel-attribute-mapper',
       'oidc-full-name-mapper',
       'oidc-group-membership-mapper',
       'oidc-audience-mapper',
       'saml-user-property-mapper',
       'saml-role-list-mapper',
       'saml-javascript-mapper',
     )
     defaultto do
       if @resource[:protocol] == 'openid-connect'
         'oidc-usermodel-property-mapper'
       elsif @resource[:protocol] == 'saml'
         'saml-user-property-mapper'
       else
         nil
       end
     end
     munge { |v| v }
   end
 
   newproperty(:user_attribute) do
     desc 'user.attribute. Default to `resource_name` for `type` `oidc-usermodel-property-mapper` or `saml-user-property-mapper`'
     defaultto do
       if ['oidc-usermodel-property-mapper', 'saml-user-property-mapper', 'oidc-usermodel-attribute-mapper'].include?(@resource[:type])
         @resource[:resource_name]
       else
         nil
       end
     end
   end
 
   newproperty(:json_type_label) do
     desc 'json.type.label. Default to `String` for `type` `oidc-usermodel-property-mapper` and `oidc-group-membership-mapper`.'
     defaultto do
       if ['oidc-usermodel-property-mapper', 'oidc-group-membership-mapper', 'oidc-usermodel-attribute-mapper'].include?(@resource[:type])
         'String'
       else
         nil
       end
     end
   end
 
   newproperty(:full_path, boolean: true) do
     desc 'full.path. Default to `false` for `type` `oidc-group-membership-mapper`.'
     newvalues(:true, :false)
     defaultto do
       if @resource[:type] == 'oidc-group-membership-mapper'
         :false
       else
         nil
       end
     end
   end
 
   newproperty(:friendly_name) do
     desc 'friendly.name. Default to `resource_name` for `type` `saml-user-property-mapper`.'
     defaultto do
       if @resource[:type] == 'saml-user-property-mapper'
         @resource[:resource_name]
       else
         nil
       end
     end
   end
 
   newproperty(:attribute_name) do
     desc 'attribute.name Default to `resource_name` for `type` `saml-user-property-mapper`.'
     defaultto do
       if @resource[:type] == 'saml-user-property-mapper'
         @resource[:resource_name]
       else
         nil
       end
     end
   end
 
   newproperty(:claim_name) do
     desc 'claim.name'
   end
 
   newproperty(:id_token_claim, boolean: true) do
     desc 'id.token.claim. Default to `true` for `protocol` `openid-connect`.'
     newvalues(:true, :false)
     defaultto do
       if @resource['protocol'] == 'openid-connect'
         :true
       else
         nil
       end
     end
   end
 
   newproperty(:access_token_claim, boolean: true) do
     desc 'access.token.claim. Default to `true` for `protocol` `openid-connect`.'
     newvalues(:true, :false)
     defaultto do
       if @resource['protocol'] == 'openid-connect'
         :true
       else
         nil
       end
     end
   end
 
   newproperty(:userinfo_token_claim, boolean: true) do
     desc 'userinfo.token.claim. Default to `true` for `protocol` `openid-connect` except `type` of `oidc-audience-mapper`.'
     newvalues(:true, :false)
     defaultto do
       if @resource['protocol'] == 'openid-connect' && @resource['type'] != 'oidc-audience-mapper'
         :true
       else
         nil
       end
     end
   end
 
   newproperty(:attribute_nameformat) do
     desc 'attribute.nameformat'
     validate do |v|
       unless [:basic, :uri, :unspecified].include?(v.downcase.to_sym)
         raise ArgumentError, 'attribute_nameformat must be one of basic, uri, or unspecified'
       end
     end
     munge do |v|
       v.downcase.to_sym
     end
   end
 
   newproperty(:single, boolean: true) do
     desc 'single. Default to `false` for `type` `saml-role-list-mapper` or `saml-javascript-mapper`.'
     newvalues(:true, :false)
     defaultto do
       if ['saml-role-list-mapper', 'saml-javascript-mapper'].include?(@resource['type'])
         :false
       else
         nil
       end
     end
   end
 
   newproperty(:script) do
     desc <<-EOS
     Script, only valid for `type` of `saml-javascript-mapper`'
 
     Array values will be joined with newlines. Strings will be kept unchanged.
     EOS
   end
 
   newproperty(:included_client_audience) do
     desc 'included.client.audience Required for `type` of `oidc-audience-mapper`'
   end
 
   autorequire(:keycloak_client_scope) do
     requires = []
     catalog.resources.each do |resource|
       next unless resource.class.to_s == 'Puppet::Type::Keycloak_client_scope'
       if resource[:resource_name] == self[:client_scope]
         requires << resource.name
       end
     end
     requires
   end
 
   def self.title_patterns
     [
       [
         %r{^((.+) for (\S+) on (\S+))$},
         [
           [:name],
           [:resource_name],
           [:client_scope],
           [:realm],
         ],
       ],
       [
         %r{(.*)},
         [
           [:name],
         ],
       ],
     ]
   end
 
   validate do
-    if self[:protocol] == 'openid-connect' && !['oidc-usermodel-property-mapper', 'oidc-full-name-mapper', 'oidc-group-membership-mapper', 'oidc-audience-mapper', 'oidc-usermodel-attribute-mapper'].include?(self[:type])
+    openid_connect_types = [
+      'oidc-usermodel-property-mapper',
+      'oidc-full-name-mapper',
+      'oidc-group-membership-mapper',
+      'oidc-audience-mapper',
+      'oidc-usermodel-attribute-mapper'
+    ]
+    if self[:protocol] == 'openid-connect' && !openid_connect_types.include?(self[:type])
       raise Puppet::Error, "type #{self[:type]} is not valid for protocol openid-connect"
     end
     if self[:protocol] == 'saml' && !['saml-user-property-mapper', 'saml-role-list-mapper', 'saml-javascript-mapper'].include?(self[:type])
       raise Puppet::Error, "type #{self[:type]} is not valid for protocol saml"
     end
     if self[:friendly_name] && !['saml-user-property-mapper', 'saml-javascript-mapper'].include?(self[:type])
       raise Puppet::Error, "friendly_name is not valid for type #{self[:type]}"
     end
     if self[:attribute_name] && self[:protocol] != 'saml'
       raise Puppet::Error, "attribute_name is not valid for type #{self[:type]}"
     end
     if self[:attribute_nameformat] && self[:protocol] != 'saml'
       raise Puppet::Error, "attribute_nameformat is not valid for protocol #{self[:protocol]}"
     end
     if self[:single] && !['saml-role-list-mapper', 'saml-javascript-mapper'].include?(self[:type])
       raise Puppet::Error, "single is not valid for type #{self[:type]}"
     end
     if self[:type] == 'saml-javascript-mapper' && self[:script].nil?
       raise Puppet::Error, 'script is required for saml-javascript-mapper'
     end
     if self[:type] == 'oidc-audience-mapper' && self[:included_client_audience].nil?
       raise Puppet::Error, 'included_client_audience is required for oidc-audience-mapper'
     end
   end
 end