Merge pull request #259 from eduNEXT/jlc/fix-super-in-certificates-admin

fix: quit super with argument for certificates NELP

Author: johanseto

lms/djangoapps/certificates/admin.py

@@ -27,7 +27,7 @@ class CertificateTemplateForm(forms.ModelForm):
     Django admin form for CertificateTemplate model
     """
     def __init__(self, *args, **kwargs):
-        super(CertificateTemplateForm, self).__init__(*args, **kwargs)
+        super().__init__(*args, **kwargs)
         organizations = get_organizations()
         org_choices = [(org["id"], org["name"]) for org in organizations]
         org_choices.insert(0, ('', 'None'))
@@ -68,7 +68,7 @@ def changelist_view(self, request, extra_context=None):
             extra_context = {'title': mark_safe('Select Certificate Template Asset to change <br/><br/>'
                                                 '<div><strong style="color: red;"> Warning!</strong> Updating '
                                                 'stage asset would also update production asset</div>')}
-        return super(CertificateTemplateAssetAdmin, self).changelist_view(request, extra_context=extra_context)
+        return super().changelist_view(request, extra_context=extra_context)
 
 
 class GeneratedCertificateAdmin(admin.ModelAdmin):

lms/djangoapps/certificates/models.py

@@ -213,7 +213,7 @@ def get_queryset(self):
         Return a queryset for `GeneratedCertificate` models, filtering out
         ineligible certificates.
         """
-        return super(EligibleCertificateManager, self).get_queryset().exclude(
+        return super().get_queryset().exclude(
             status__in=(CertificateStatuses.audit_passing, CertificateStatuses.audit_notpassing)
         )
 
@@ -232,7 +232,7 @@ def get_queryset(self):
         Return a queryset for `GeneratedCertificate` models, filtering out
         ineligible certificates and any linked to nonexistent courses.
         """
-        return super(EligibleAvailableCertificateManager, self).get_queryset().extra(
+        return super().get_queryset().extra(
             tables=['course_overviews_courseoverview'],
             where=['course_id = course_overviews_courseoverview.id']
         )
@@ -409,7 +409,7 @@ def save(self, *args, **kwargs):
         signal iff we are saving a record of a learner passing the course.
         As well as the COURSE_CERT_CHANGED for any save event.
         """
-        super(GeneratedCertificate, self).save(*args, **kwargs)
+        super().save(*args, **kwargs)
         COURSE_CERT_CHANGED.send_robust(
             sender=self.__class__,
             user=self.user,
@@ -1205,10 +1205,10 @@ def save(self, *args, **kwargs):
         if self.pk is None:
             asset_image = self.asset
             self.asset = None
-            super(CertificateTemplateAsset, self).save(*args, **kwargs)
+            super().save(*args, **kwargs)
             self.asset = asset_image
 
-        super(CertificateTemplateAsset, self).save(*args, **kwargs)
+        super().save(*args, **kwargs)
 
     def __str__(self):
         return u'%s' % (self.asset.url, )

lms/djangoapps/certificates/queue.py

@@ -39,7 +39,7 @@ class XQueueAddToQueueError(Exception):
     def __init__(self, error_code, error_msg):
         self.error_code = error_code
         self.error_msg = error_msg
-        super(XQueueAddToQueueError, self).__init__(six.text_type(self))
+        super().__init__(six.text_type(self))
 
     def __str__(self):
         return (