remove authorship
[edumed.git] / wtem / models.py
index d24430d..361bff3 100644 (file)
@@ -1,33 +1,35 @@
+# -*- coding: utf-8 -*-
 import random
 import string
 import os
+import json
 
 from django.db import models
 from django.contrib.auth.models import User
 from django.core.exceptions import ValidationError
-from django.utils import simplejson
 from django.utils.translation import ugettext as _
 from jsonfield import JSONField
 
 from contact.models import Contact
 
 f = file(os.path.dirname(__file__) + '/fixtures/exercises.json')
-exercises = simplejson.loads(f.read())
+exercises = json.loads(f.read())
 f.close()
 
 DEBUG_KEY = '12345'
 
+
 class Submission(models.Model):
-    contact = models.ForeignKey(Contact, null = True)
-    key = models.CharField(max_length = 30, unique = True)
-    first_name = models.CharField(max_length = 100)
-    last_name = models.CharField(max_length = 100)
-    email = models.EmailField(max_length = 100, unique = True)
-    answers = models.CharField(max_length = 65536, null = True, blank = True)
-    key_sent = models.BooleanField(default = False)
-    marks = JSONField()
-    examiners = models.ManyToManyField(User, null = True, blank = True)
-    end_time = models.CharField(max_length = 5, null = True, blank = True)
+    contact = models.ForeignKey(Contact, null=True)
+    key = models.CharField(max_length=30, unique=True)
+    first_name = models.CharField(max_length=100)
+    last_name = models.CharField(max_length=100)
+    email = models.EmailField(max_length=100, unique=True)
+    answers = models.CharField(max_length=65536, null=True, blank=True)
+    key_sent = models.BooleanField(default=False)
+    marks = JSONField(default={})
+    examiners = models.ManyToManyField(User, null=True, blank=True)
+    end_time = models.CharField(max_length=5, null=True, blank=True)
 
     def __unicode__(self):
         return ', '.join((self.last_name, self.first_name, self.email))
@@ -36,17 +38,18 @@ class Submission(models.Model):
     def generate_key(cls):
         key = ''
         while not key or key in [record['key'] for record in cls.objects.values('key')]:
-            key = ''.join(random.choice(string.ascii_uppercase + string.ascii_lowercase + string.digits) for i in range(30))
+            key = ''.join(random.choice(string.ascii_uppercase + string.ascii_lowercase + string.digits)
+                          for i in range(30))
         return key
 
     @classmethod
-    def create(cls, first_name, last_name, email, key = None, contact = None):
+    def create(cls, first_name, last_name, email, key=None, contact=None):
         submission = cls(
-            contact = contact,
-            key = key if key else Submission.generate_key(),
-            first_name = first_name,
-            last_name = last_name,
-            email = email
+            contact=contact,
+            key=key if key else Submission.generate_key(),
+            first_name=first_name,
+            last_name=last_name,
+            email=email
         )
 
         submission.save()
@@ -70,7 +73,6 @@ class Submission(models.Model):
         if mark == 'None':
             del self.marks[user_id][exercise_id]
 
-
     def get_exercise_marks_by_examiner(self, exercise_id):
         marks = dict()
         for examiner_id, examiner_marks in self.marks.items():
@@ -80,17 +82,18 @@ class Submission(models.Model):
         return marks
 
     def get_final_exercise_mark(self, exercise_id):
-        exercise = exercises[int(exercise_id)-1]
+        # exercise = exercises[int(exercise_id)-1]
+        exercise = [e for e in exercises if str(e['id']) == str(exercise_id)][0]
         if exercise_checked_manually(exercise):
             marks_by_examiner = self.get_exercise_marks_by_examiner(exercise_id)
             if len(marks_by_examiner):
-                return max(map(int, marks_by_examiner.values()))
+                return sum(map(float, marks_by_examiner.values())) / float(len(marks_by_examiner))
             else:
                 return None
         else:
             if not self.answers:
                 return None
-            answer = simplejson.loads(self.answers)[exercise_id]['closed_part']
+            answer = json.loads(self.answers)[exercise_id]['closed_part']
             t = exercise['type']
             if t == 'edumed_uporzadkuj':
                 return exercise['points'] if map(int, answer) == exercise['answer'] else 0
@@ -117,8 +120,8 @@ class Submission(models.Model):
                     if exercise.get('answer_mode', None) == 'all_or_nothing':
                         toret = exercise['points'] if map(int, answer) == exercise['answer'] else 0
                     else:
-                        for id in map(int, answer):
-                            if id in exercise['answer']:
+                        for answer_id in map(int, answer):
+                            if answer_id in exercise['answer']:
                                 toret += exercise['points_per_hit']
                     return toret
             if t == 'edumed_prawdafalsz':
@@ -138,7 +141,8 @@ class Submission(models.Model):
     @property
     def final_result(self):
         final = 0
-        for exercise_id in map(str,range(1, len(exercises) + 1)):
+        # for exercise_id in map(str,range(1, len(exercises) + 1)):
+        for exercise_id in [str(x['id']) for x in exercises]:
             mark = self.get_final_exercise_mark(exercise_id)
             if mark is not None:
                 final += mark
@@ -148,26 +152,28 @@ class Submission(models.Model):
     def final_result_as_string(self):
         return ('%.2f' % self.final_result).rstrip('0').rstrip('.')
 
+
 class Attachment(models.Model):
     submission = models.ForeignKey(Submission)
     exercise_id = models.IntegerField()
-    file = models.FileField(upload_to = 'wtem/attachment')
+    tag = models.CharField(max_length=128, null=True, blank=True)
+    file = models.FileField(upload_to='wtem/attachment')
 
 
 class Assignment(models.Model):
-    user = models.ForeignKey(User, unique = True)
+    user = models.ForeignKey(User, unique=True)
     exercises = JSONField()
 
     def clean(self):
         if not isinstance(self.exercises, list):
             raise ValidationError(_('Assigned exercises must be declared in a list format'))
-        for exercise in self.exercises:
-            if not isinstance(exercise, int) or exercise < 1:
-                raise ValidationError(_('Invalid exercise id: %s' % exercise))
+        for exercise in self.exercises:
+            if not isinstance(exercise, int) or exercise < 1:
+                raise ValidationError(_('Invalid exercise id: %s' % exercise))
 
     def __unicode__(self):
-        return self.user.username + ': ' + ','.join(map(str,self.exercises))
+        return self.user.username + ': ' + ','.join(map(str, self.exercises))
 
 
 def exercise_checked_manually(exercise):
-    return (exercise['type'] in ('open', 'file_upload')) or 'open_part' in exercise
\ No newline at end of file
+    return (exercise['type'] in ('open', 'file_upload')) or 'open_part' in exercise