Browse Source

filter: Removed unused validation calls

The parent class for filters in openshift_master defines a validate
method which does nothing. This change removes calls to this method
while leaving the validation implementations as is.
Steve Milner 8 years ago
parent
commit
a9e85db351
1 changed files with 0 additions and 3 deletions
  1. 0 3
      filter_plugins/openshift_master.py

+ 0 - 3
filter_plugins/openshift_master.py

@@ -176,7 +176,6 @@ class LDAPPasswordIdentityProvider(IdentityProviderBase):
 
     def validate(self):
         ''' validate this idp instance '''
-        super(self.__class__, self).validate()
         if not isinstance(self.provider['attributes'], dict):
             raise errors.AnsibleFilterError("|failed attributes for provider "
                                             "{0} must be a dictionary".format(self.__class__.__name__))
@@ -238,7 +237,6 @@ class RequestHeaderIdentityProvider(IdentityProviderBase):
 
     def validate(self):
         ''' validate this idp instance '''
-        super(self.__class__, self).validate()
         if not isinstance(self.provider['headers'], list):
             raise errors.AnsibleFilterError("|failed headers for provider {0} "
                                             "must be a list".format(self.__class__.__name__))
@@ -341,7 +339,6 @@ class IdentityProviderOauthBase(IdentityProviderBase):
 
     def validate(self):
         ''' validate this idp instance '''
-        super(self.__class__, self).validate()
         if self.challenge:
             raise errors.AnsibleFilterError("|failed provider {0} does not "
                                             "allow challenge authentication".format(self.__class__.__name__))