SubmissionFileAdmin.get_queryset() - Code Metrics - troeger/opensubmit - Measure and Improve Code Quality continuously with Scrutinizer

SubmissionFileAdmin.get_queryset() A
last analyzed 2018-05-24 09:45 UTC

↳ Parent: SubmissionFileAdmin

Complexity

Conditions

Size

Total Lines

Duplication

Lines	7
Ratio	100 %

Importance

Changes

Metric	Value
dl	7
loc	7
rs	9.4285
c	0
b	0
f	0
cc	2

from django.contrib.admin import ModelAdmin, StackedInline
from django.db.models import Q
from opensubmit.models import Submission

def submissions(submfile):
    while submfile.replaced_by is not None:
        submfile = submfile.replaced_by
    subms = submfile.submissions.all()
    return ','.join([str(sub) for sub in subms])


def not_withdrawn(submfile):
    return submfile.replaced_by is None
not_withdrawn.boolean = True

class SubmissionFileAdmin(ModelAdmin):

    list_display = ['__str__', 'fetched', submissions, not_withdrawn]

    class Media:
        css = {'all': ('css/teacher.css',)}

    def get_queryset(self, request):
        ''' Restrict the listed submission files for the current user.'''
        qs = super(SubmissionFileAdmin, self).get_queryset(request)
        if request.user.is_superuser:
            return qs
        else:
            return qs.filter(Q(submissions__assignment__course__tutors__pk=request.user.pk) | Q(submissions__assignment__course__owner=request.user)).distinct()


1		from django.contrib.admin import ModelAdmin, StackedInline
2		from django.db.models import Q
3		from opensubmit.models import Submission
4
5		def submissions(submfile):
6		while submfile.replaced_by is not None:
7		submfile = submfile.replaced_by
8		subms = submfile.submissions.all()
9		return ','.join([str(sub) for sub in subms])
10
11
12		def not_withdrawn(submfile):
13		return submfile.replaced_by is None
14		not_withdrawn.boolean = True
15
16	View Code Duplication	class SubmissionFileAdmin(ModelAdmin):
		0 ignored issues – show Duplication introduced 2017-11-30 10:01 UTC by Report Bug Copy Issue Report This code seems to be duplicated in your project. Loading history...
17		list_display = ['__str__', 'fetched', submissions, not_withdrawn]
18
19		class Media:
20		css = {'all': ('css/teacher.css',)}
21
22		def get_queryset(self, request):
23		''' Restrict the listed submission files for the current user.'''
24		qs = super(SubmissionFileAdmin, self).get_queryset(request)
25		if request.user.is_superuser:
26		return qs
27		else:
28		return qs.filter(Q(submissions__assignment__course__tutors__pk=request.user.pk) \| Q(submissions__assignment__course__owner=request.user)).distinct()
29

troeger / opensubmit

SubmissionFileAdmin.get_queryset() A last analyzed 2018-05-24 09:45 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

SubmissionFileAdmin.get_queryset() A
last analyzed 2018-05-24 09:45 UTC