*: remove some now useless pylint:disable=super-on-old-class

This is no longer needed now, maybe since Zope4 or maybe since a patch
in slapos. This option does not exist on python3 and using pylint on
python3 complains about these disable, so we have to remove them on
python2 as well.

bt5/erp5_base/DocumentTemplateItem/portal_components/document.erp5.Person.py

@@ -95,7 +95,7 @@ class Person(EncryptedPasswordMixin, Node, LoginAccountProviderMixin, ERP5UserMi
 
   security.declareProtected(Permissions.AccessContentsInformation,
                             'getTitle')
-  def getTitle(self, **kw):  # pylint: disable=super-on-old-class
+  def getTitle(self, **kw):
     """
     Returns the title if it exists or a combination of
     first name, middle name and last name
@@ -109,7 +109,7 @@ class Person(EncryptedPasswordMixin, Node, LoginAccountProviderMixin, ERP5UserMi
 
   security.declareProtected(Permissions.AccessContentsInformation,
                             'getTranslatedTitle')
-  def getTranslatedTitle(self, **kw):  # pylint: disable=super-on-old-class
+  def getTranslatedTitle(self, **kw):
     """
     Returns the title if it exists or a combination of
     first name, middle name and last name

bt5/erp5_simulation/DocumentTemplateItem/portal_components/document.erp5.SpecialiseEquivalenceTester.py

@@ -43,7 +43,7 @@ class SpecialiseEquivalenceTester(CategoryMembershipEquivalenceTester):
   movement_specialise_type = ()
   movement_exclude_specialise_type = ()
 
-  def _compare(self, prevision_movement, decision_movement, sorted=lambda x:x): # pylint: disable=super-on-old-class,redefined-builtin
+  def _compare(self, prevision_movement, decision_movement, sorted=lambda x:x): # pylint: disable=redefined-builtin
     return super(SpecialiseEquivalenceTester, self)._compare(
       prevision_movement, decision_movement, sorted)
 

bt5/erp5_test_result/DocumentTemplateItem/portal_components/document.erp5.CloudPerformanceUnitTestDistributor.py

@@ -67,7 +67,7 @@ class CloudPerformanceUnitTestDistributor(ERP5ProjectUnitTestDistributor):
       test_node.setAggregateList(test_suite_list)
 
   security.declarePublic("startTestSuite")
-  def startTestSuite(self,title, computer_guid=None, **kw): # pylint: disable=super-on-old-class
+  def startTestSuite(self,title, computer_guid=None, **kw):
     """
     give the list of test suite to start. We will take all test suites
     associated to the testnode. Then we add the test node title to the
@@ -82,7 +82,7 @@ class CloudPerformanceUnitTestDistributor(ERP5ProjectUnitTestDistributor):
     return json.dumps(config_list)
 
   security.declarePublic("generateConfiguration")
-  def generateConfiguration(self, test_suite_title, batch_mode=0): # pylint: disable=super-on-old-class
+  def generateConfiguration(self, test_suite_title, batch_mode=0):
     """
     return the list of configuration to create instances, in the case of ERP5 unit tests,
     we will have only one configuration (unlike scalability tests). But for API consistency,
@@ -94,6 +94,6 @@ class CloudPerformanceUnitTestDistributor(ERP5ProjectUnitTestDistributor):
     return super(CloudPerformanceUnitTestDistributor, self) \
       .generateConfiguration("ERP5-Cloud-Reliability", batch_mode)
 
-  def _getTestSuiteFromTitle(self, suite_title): # pylint: disable=super-on-old-class
+  def _getTestSuiteFromTitle(self, suite_title):
     return super(CloudPerformanceUnitTestDistributor,
                  self)._getTestSuiteFromTitle(suite_title.split("|")[0])