X-Git-Url: https://git.mdrn.pl/edumed.git/blobdiff_plain/25da5b8159bea9655e1a1ed15cc414896a31a7eb..58a9fa0d1843e97ef7116be4670defbc555b1b2c:/wtem/admin.py
diff --git a/wtem/admin.py b/wtem/admin.py
index 47e0a77..d4b6682 100644
--- a/wtem/admin.py
+++ b/wtem/admin.py
@@ -7,7 +7,11 @@ from django import forms
from django.utils import simplejson
from django.utils.safestring import mark_safe
from django.core.urlresolvers import reverse
-
+from django.conf.urls import url, patterns
+from django.shortcuts import render
+from django.contrib.auth.models import User
+from django.http import HttpResponse
+from django.template.loader import render_to_string
from .models import Submission, Assignment, Attachment, exercises
from .middleware import get_current_request
@@ -24,12 +28,16 @@ readonly_fields = ('submitted_by', 'first_name', 'last_name', 'email', 'key', 'k
class AttachmentWidget(forms.Widget):
def render(self, name, value, *args, **kwargs):
- return mark_safe('%s' % (name, value, value, value))
+ if value:
+ a_tag = '%s' % (value, value)
+ else:
+ a_tag = 'brak'
+ return mark_safe(('' % (name, value)) + a_tag)
class SubmissionFormBase(forms.ModelForm):
class Meta:
model = Submission
- exclude = ('answers', 'marks', 'contact') + readonly_fields
+ exclude = ('answers', 'marks', 'contact', 'end_time') + readonly_fields
def get_open_answer(answers, exercise):
@@ -51,9 +59,10 @@ def get_open_answer(answers, exercise):
if exercise['type'] == 'edumed_wybor':
ok = set(map(str, exercise['answer'])) == set(map(str,answer['closed_part']))
toret = u'Czesc testowa [%s]:\n' % ('poprawna' if ok else 'niepoprawna')
- for selected in answer['closed_part']:
- option = get_option(exercise['options'], selected)
- toret += '%s: %s\n' % (selected, option['text'])
+ if len(answer['closed_part']):
+ for selected in answer['closed_part']:
+ option = get_option(exercise['options'], selected)
+ toret += '%s: %s\n' % (selected, option['text'])
else:
toret += u'\n'
toret += u'\nCzesc otwarta (%s):\n\n' % ' '.join(exercise['open_part'])
@@ -96,11 +105,18 @@ def get_form(request, submission):
initial = submission.get_mark(user_id = request.user.id, exercise_id = exercise['id']),
label = u'Twoja ocena zadania %s' % exercise['id']
)
+
+ if not request.user.is_superuser:
+ class Meta(SubmissionFormBase.Meta):
+ pass
+ Meta.exclude += ('examiners',)
+ fields['Meta'] = Meta
+
return type('SubmissionForm', (SubmissionFormBase,), fields)
class SubmissionAdmin(admin.ModelAdmin):
- list_display = ('__unicode__', 'todo',)
+ list_display = ('__unicode__', 'todo', 'examiners_repr')
readonly_fields = readonly_fields
def get_form(self, request, obj, **kwargs):
@@ -121,6 +137,11 @@ class SubmissionAdmin(admin.ModelAdmin):
user_exercises = get_user_exercises(user)
user_marks = submission.marks.get(str(user.id), {})
return ','.join([str(e['id']) for e in user_exercises if str(e['id']) not in user_marks.keys()])
+ todo.short_description = 'Twoje nieocenione zadania'
+
+ def examiners_repr(self, submission):
+ return ', '.join([u.username for u in submission.examiners.all()])
+ examiners_repr.short_description = 'Przypisani do zgÅoszenia'
def save_model(self, request, submission, form, change):
for name, value in form.cleaned_data.items():
@@ -133,16 +154,52 @@ class SubmissionAdmin(admin.ModelAdmin):
def changelist_view(self, request, extra_context=None):
context = dict(examiners = [])
- if request.user.is_superuser:
- submissions = Submission.objects.all()
- for assignment in Assignment.objects.all():
- examiner = dict(name = assignment.user.username, todo = 0)
- for submission in submissions:
- for exercise_id in assignment.exercises:
- if submission.get_mark(user_id = assignment.user.id, exercise_id = exercise_id) is None:
- examiner['todo'] += 1
- context['examiners'].append(examiner)
+ assignments = Assignment.objects.all()
+ if not request.user.is_superuser:
+ assignments = assignments.filter(user = request.user)
+ for assignment in assignments:
+ examiner = dict(name = assignment.user.username, todo = 0)
+ for submission in Submission.objects.filter(examiners = assignment.user):
+ for exercise_id in assignment.exercises:
+ if submission.get_mark(user_id = assignment.user.id, exercise_id = exercise_id) is None:
+ examiner['todo'] += 1
+ context['examiners'].append(examiner)
return super(SubmissionAdmin, self).changelist_view(request, extra_context = context)
+ def queryset(self, request):
+ qs = super(SubmissionAdmin, self).queryset(request)
+ if not request.user.is_superuser:
+ qs = qs.filter(examiners = request.user)
+ return qs
+
+ def get_urls(self):
+ urls = super(SubmissionAdmin, self).get_urls()
+ return patterns('',
+ url(r'^report/$', self.admin_site.admin_view(report_view), name='wtem_admin_report')
+ ) + super(SubmissionAdmin, self).get_urls()
+
+
+class SubmissionsSet:
+ def __init__(self, submissions):
+ self.submissions = submissions
+ self.examiners_by_exercise = dict()
+ for submission in submissions:
+ for user_id, marks in submission.marks.items():
+ user = User.objects.get(pk=user_id)
+ for exercise_id in marks.keys():
+ examiners = self.examiners_by_exercise.setdefault(exercise_id, [])
+ if not user in examiners:
+ examiners.append(user)
+
+def report_view(request):
+ submissions = sorted(Submission.objects.all(), key = lambda s: -s.final_result)
+ toret = render_to_string('wtem/admin_report.csv', dict(
+ submissionsSet = SubmissionsSet(submissions),
+ exercise_ids = map(str, range(1,len(exercises)+1))
+ ))
+ response = HttpResponse(toret, content_type = 'text/csv')
+ response['Content-Disposition'] = 'attachment; filename="wyniki.csv"'
+ return response
+
admin.site.register(Submission, SubmissionAdmin)
admin.site.register(Assignment)
\ No newline at end of file