Skip to content
Snippets Groups Projects
Verified Commit 3e37f4b4 authored by Tom Teichler's avatar Tom Teichler :beers:
Browse files

Fix raising error.

parent c903c34e
No related branches found
No related tags found
1 merge request!13Disallow creating lesson documentation for lessons in the future. Close #30.
......@@ -3,8 +3,9 @@ from datetime import date, datetime, timedelta
from typing import Optional
from django.contrib.auth.decorators import login_required
from django.core.exceptions import PermissionDenied
from django.db.models import Count, Exists, F, OuterRef, Q, Sum
from django.http import Http404, Http403, HttpRequest, HttpResponse
from django.http import Http404, HttpRequest, HttpResponse
from django.shortcuts import get_object_or_404, render
from django.urls import reverse
from django.utils.translation import ugettext as _
......@@ -35,8 +36,8 @@ def lesson(request: HttpRequest, year: Optional[int] = None, week: Optional[int]
if not lesson_period:
raise Http404(_('You either selected an invalid lesson or there is currently no lesson in progress.'))
if lesson_period.lesson.periods.time_start < datetime.now() or lesson_period.lesson.get_calendar_week < CalendarWeek():
raise Http403(_('You are not allowed to create a lesson documentation for a lesson in the future.'))
if datetime.combine(wanted_week[lesson_period.period.weekday - 1], lesson_period.period.time_start) < datetime.now():
raise PermissionDenied(_('You are not allowed to create a lesson documentation for a lesson in the future.'))
context['lesson_period'] = lesson_period
context['week'] = wanted_week
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment