Fixing pylint

Author: snyaggarwal

.pylintrc

@@ -1,5 +1,5 @@
 [MESSAGES CONTROL]
-disable=missing-docstring,too-few-public-methods,no-member,too-many-ancestors,attribute-defined-outside-init,fixme,broad-except,import-outside-toplevel,too-many-instance-attributes,unsupported-membership-test,too-many-public-methods,too-many-lines,unused-private-member,consider-using-with,duplicate-code,unspecified-encoding,invalid-name,cyclic-import,missing-timeout
+disable=missing-docstring,too-few-public-methods,no-member,too-many-ancestors,attribute-defined-outside-init,fixme,broad-except,import-outside-toplevel,too-many-instance-attributes,unsupported-membership-test,too-many-public-methods,too-many-lines,unused-private-member,consider-using-with,duplicate-code,unspecified-encoding,invalid-name,cyclic-import,missing-timeout,too-many-positional-arguments
 [MASTER]
 ignore=migrations,fixtures,scripts,commands,documents.py,password_validation.py,v1_dump
 [FORMAT]

core/common/mixins.py

@@ -51,8 +51,7 @@ def __init__(  # pylint: disable=too-many-arguments
         if not is_sliced:
             bottom = (self.page_number - 1) * self.page_size
             top = bottom + self.page_size
-            if top >= self.total:
-                top = self.total
+            top = min(top, self.total)
             self.queryset = self.queryset[bottom:top]
         if isinstance(self.queryset, QuerySet):
             self.queryset.count = None

core/concept_maps/views.py

@@ -138,6 +138,7 @@ def get_serializer(self, *args, **kwargs):
                     if mapping.map_type == SAME_AS:
                         equivalence = "equivalent"
 
+                    to_url = None
                     if mapping.to_source and mapping.to_source.canonical_url:
                         to_url = mapping.to_source.canonical_url
                     elif mapping.to_source_url:

core/services/storages/cloud/aws.py

@@ -117,8 +117,7 @@ def get_object(self, key):
 
     @staticmethod
     def file_iterator(response):
-        for chunk in response['Body'].iter_chunks():
-            yield chunk
+        yield from response['Body'].iter_chunks()
 
     def get_streaming_response(self, key):
         s3_response = self.get_object(key)