linting for R0201
Signed-off-by: Corey Shields <cshields@gmail.com>merge-requests/399/head
parent
3ce647e374
commit
9989efb453
|
@ -9,7 +9,6 @@ disable=
|
||||||
C0412,
|
C0412,
|
||||||
E1101,
|
E1101,
|
||||||
E0213,
|
E0213,
|
||||||
R0201,
|
|
||||||
R0401,
|
R0401,
|
||||||
R0801,
|
R0801,
|
||||||
R0901,
|
R0901,
|
||||||
|
|
|
@ -15,7 +15,7 @@ class TransmitterViewFilter(FilterSet):
|
||||||
|
|
||||||
# see https://django-filter.readthedocs.io/en/master/ref/filters.html for
|
# see https://django-filter.readthedocs.io/en/master/ref/filters.html for
|
||||||
# unused-argument
|
# unused-argument
|
||||||
def filter_status(self, queryset, name, value): # pylint: disable=unused-argument
|
def filter_status(self, queryset, name, value): # pylint: disable=unused-argument,no-self-use
|
||||||
"""Returns Transmitters that are either functional or non-functional"""
|
"""Returns Transmitters that are either functional or non-functional"""
|
||||||
if value:
|
if value:
|
||||||
return queryset.filter(status='active')
|
return queryset.filter(status='active')
|
||||||
|
|
|
@ -1,4 +1,5 @@
|
||||||
"""SatNOGS DB API serializers, django rest framework"""
|
"""SatNOGS DB API serializers, django rest framework"""
|
||||||
|
# pylint: disable=no-self-use
|
||||||
from __future__ import absolute_import, division, print_function, \
|
from __future__ import absolute_import, division, print_function, \
|
||||||
unicode_literals
|
unicode_literals
|
||||||
|
|
||||||
|
|
|
@ -48,7 +48,7 @@ class SatelliteAdmin(admin.ModelAdmin):
|
||||||
]
|
]
|
||||||
return my_urls + urls
|
return my_urls + urls
|
||||||
|
|
||||||
def check_celery(self, request):
|
def check_celery(self, request): # pylint: disable=no-self-use
|
||||||
"""Returns status of Celery workers
|
"""Returns status of Celery workers
|
||||||
|
|
||||||
Check the delay for celery workers, return an error if a connection
|
Check the delay for celery workers, return an error if a connection
|
||||||
|
@ -72,7 +72,7 @@ class SatelliteAdmin(admin.ModelAdmin):
|
||||||
|
|
||||||
return HttpResponseRedirect(reverse('admin:index'))
|
return HttpResponseRedirect(reverse('admin:index'))
|
||||||
|
|
||||||
def decode_all_data(self, request, norad):
|
def decode_all_data(self, request, norad): # pylint: disable=no-self-use
|
||||||
"""Returns the admin home page, while triggering a Celery decode task
|
"""Returns the admin home page, while triggering a Celery decode task
|
||||||
|
|
||||||
Forces a decode of all data for a norad ID. This could be very resource
|
Forces a decode of all data for a norad ID. This could be very resource
|
||||||
|
@ -220,7 +220,7 @@ class DemodDataAdmin(admin.ModelAdmin):
|
||||||
list_display = ('id', 'satellite', 'app_source', 'observer')
|
list_display = ('id', 'satellite', 'app_source', 'observer')
|
||||||
search_fields = ('transmitter__uuid', 'satellite__norad_cat_id', 'observer')
|
search_fields = ('transmitter__uuid', 'satellite__norad_cat_id', 'observer')
|
||||||
|
|
||||||
def satellite(self, obj):
|
def satellite(self, obj): # pylint: disable=no-self-use
|
||||||
"""Returns the Satellite object associated with this DemodData
|
"""Returns the Satellite object associated with this DemodData
|
||||||
|
|
||||||
:param obj: DemodData object
|
:param obj: DemodData object
|
||||||
|
|
|
@ -210,7 +210,7 @@ class TransmitterSuggestionManager(models.Manager):
|
||||||
submitted (suggested) but not yet reviewed
|
submitted (suggested) but not yet reviewed
|
||||||
"""
|
"""
|
||||||
|
|
||||||
def get_queryset(self):
|
def get_queryset(self): # pylint: disable=no-self-use
|
||||||
"""Returns TransmitterEntries that have not been reviewed"""
|
"""Returns TransmitterEntries that have not been reviewed"""
|
||||||
return TransmitterEntry.objects.filter(reviewed=False)
|
return TransmitterEntry.objects.filter(reviewed=False)
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue