Skip to content

Commit 5b69d15

Browse files
committed
Fix pylint gripes
1 parent 1dec775 commit 5b69d15

File tree

1 file changed

+8
-2
lines changed

1 file changed

+8
-2
lines changed

figures/serializers.py

+8-2
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,7 @@
2424
from django.contrib.sites.models import Site
2525
from django_countries import Countries
2626
from rest_framework import serializers
27+
from rest_framework.fields import empty
2728

2829
from openedx.core.djangoapps.content.course_overviews.models import CourseOverview # noqa pylint: disable=import-error
2930
from openedx.core.djangoapps.user_api.accounts.serializers import AccountLegacyProfileSerializer # noqa pylint: disable=import-error
@@ -834,6 +835,11 @@ class EnrollmentMetricsSerializerV2(serializers.ModelSerializer):
834835
progress_percent = serializers.SerializerMethodField()
835836
progress_details = serializers.SerializerMethodField()
836837

838+
def __init__(self, instance=None, data=empty, **kwargs):
839+
self._lcgm = None
840+
super(EnrollmentMetricsSerializerV2, self).__init__(
841+
instance=None, data=empty, **kwargs)
842+
837843
class Meta:
838844
model = CourseEnrollment
839845
fields = ['id', 'course_id', 'date_enrolled', 'is_enrolled',
@@ -856,11 +862,11 @@ def get_is_enrolled(self, obj):
856862
"""
857863
return CourseEnrollment.is_enrolled(obj.user, obj.course_id)
858864

859-
def get_progress_percent(self, obj):
865+
def get_progress_percent(self, obj): # pylint: disable=unused-argument
860866
value = self._lcgm.progress_percent if self._lcgm else 0
861867
return float(Decimal(value).quantize(Decimal('.00')))
862868

863-
def get_progress_details(self, obj):
869+
def get_progress_details(self, obj): # pylint: disable=unused-argument
864870
"""Get progress data for a single enrollment
865871
"""
866872
return self._lcgm.progress_details if self._lcgm else None

0 commit comments

Comments
 (0)