reading exercise CRUD

Author: rockstarr-programmerr

classroom/filters/__init__.py

@@ -0,0 +1,2 @@
+from .classroom import ClassroomTeacherFilter
+from .exercise import ReadingExerciseTeacherFilter

classroom/filters/teacher/__init__.py

@@ -0,0 +1,22 @@
+from django_filters import rest_framework as filters
+
+from classroom.models import ReadingExercise
+
+
+class ReadingExerciseTeacherFilter(filters.FilterSet):
+    class Meta:
+        model = ReadingExercise
+        fields = {
+            'identifier': ['exact', 'icontains'],
+            'content': ['icontains'],
+            'classrooms': ['exact'],  # TODO: how does this work?
+        }
+
+    @property
+    def qs(self):
+        parent = super().qs
+        teacher = self.request.user
+        exercises_pks = teacher.reading_exercises_created.all().values_list('pk', flat=True)
+        qs = parent.filter(pk__in=list(exercises_pks))\
+                   .select_related('creator')
+        return qs

classroom/filters/classroom_teacher.py renamed to classroom/filters/teacher/classroom.py

@@ -0,0 +1,26 @@
+# Generated by Django 3.2.7 on 2021-10-04 14:45
+
+from django.conf import settings
+from django.db import migrations, models
+import django.db.models.deletion
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        migrations.swappable_dependency(settings.AUTH_USER_MODEL),
+        ('classroom', '0002_readinganswer_readingexercise_readingquestion'),
+    ]
+
+    operations = [
+        migrations.AddField(
+            model_name='readingexercise',
+            name='classrooms',
+            field=models.ManyToManyField(related_name='reading_exercises', to='classroom.Classroom'),
+        ),
+        migrations.AddField(
+            model_name='readingexercise',
+            name='creator',
+            field=models.ForeignKey(null=True, on_delete=django.db.models.deletion.SET_NULL, related_name='reading_exercises_created', to=settings.AUTH_USER_MODEL),
+        ),
+    ]

classroom/filters/teacher/exercise.py

@@ -0,0 +1,19 @@
+# Generated by Django 3.2.7 on 2021-10-04 15:01
+
+from django.conf import settings
+from django.db import migrations
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        migrations.swappable_dependency(settings.AUTH_USER_MODEL),
+        ('classroom', '0003_auto_20211004_2145'),
+    ]
+
+    operations = [
+        migrations.AlterUniqueTogether(
+            name='readingexercise',
+            unique_together={('creator', 'identifier')},
+        ),
+    ]

classroom/migrations/0003_auto_20211004_2145.py

@@ -24,3 +24,6 @@ class Classroom(models.Model):
     class Meta:
         ordering = ['-create_datetime']
         unique_together = ['name', 'teacher']
+
+    def __str__(self):
+        return f'{self.name} - (Teacher: {self.teacher.email})'

classroom/migrations/0004_alter_readingexercise_unique_together.py

@@ -1,9 +1,17 @@
+from django.contrib.auth import get_user_model
 from django.db import models
 
+from .classroom import Classroom
+
+User = get_user_model()
+
 
 class ReadingExercise(models.Model):
+    creator = models.ForeignKey(User, on_delete=models.SET_NULL, null=True, related_name='reading_exercises_created')
     identifier = models.CharField(max_length=20)
     content = models.TextField()
+    classrooms = models.ManyToManyField(Classroom, related_name='reading_exercises')
 
     class Meta:
         ordering = ['identifier']
+        unique_together = ['creator', 'identifier']

classroom/models/classroom.py

@@ -22,6 +22,16 @@ def has_object_permission(self, request, view, classroom):
         )
 
 
+class IsTeacherOwnsExercise(IsTeacher):
+    message = _('Only teacher who owns this exercise has permission for this.')
+
+    def has_object_permission(self, request, view, exercise):
+        return (
+            super().has_object_permission(request, view, exercise) and
+            request.user == exercise.creator
+        )
+
+
 class IsStudent(IsAuthenticated):
     message = _('Only student has permission for this.')