Fix bug for user that are not logged in

advlabdb/model_dependent_funs.py

@@ -67,3 +67,12 @@ def reportBadAttempt(message):
 
 def get_count(table):
     return db.session.scalar(select(func.count()).select_from(table))
+
+
+def active_semester_repr():
+    active_semester = current_user.active_semester
+    repr = active_semester.repr()
+    if active_semester != lastSemester():
+        flash(f"You are in the old semester {repr}!", "warning")
+
+    return repr

advlabdb/routes.py

@@ -2,17 +2,17 @@ from flask import flash, redirect, render_template, request, url_for
 from flask_security import auth_required, current_user, roles_accepted, roles_required
 
 from . import app, db
-from .model_dependent_funs import lastSemester, setUserActiveSemester
+from .model_dependent_funs import (
+    active_semester_repr,
+    lastSemester,
+    setUserActiveSemester,
+)
 from .models import Semester
 
 
 @app.context_processor
 def util_processor():
-    active_semester = current_user.active_semester
-    if active_semester != lastSemester():
-        flash(f"You are in the old semester {current_user.active_semester.repr()}!", "warning")
-
-    return dict(active_semester=active_semester, current_user=current_user)
+    return dict(active_semester_repr=active_semester_repr, current_user=current_user)
 
 
 @app.route("/")

advlabdb/templates/actions.html

@@ -2,7 +2,7 @@
 {% extends "admin/master.html" %}
 
 {% block body %}
-    {{information(current_user, active_semester, role="admin")}}
+    {{information(current_user, active_semester_repr, role="admin")}}
 
     <hr>
 

advlabdb/templates/admin_create.html

@@ -2,6 +2,6 @@
 {% extends "admin/model/create.html" %}
 
 {% block body %}
-    {{information(current_user, active_semester, role="admin")}}
+    {{information(current_user, active_semester_repr, role="admin")}}
     {{ super() }}
 {% endblock %}

advlabdb/templates/admin_details.html

@@ -2,6 +2,6 @@
 {% extends "admin/model/details.html" %}
 
 {% block body %}
-    {{information(current_user, active_semester, role="admin")}}
+    {{information(current_user, active_semester_repr, role="admin")}}
     {{ super() }}
 {% endblock %}

advlabdb/templates/admin_edit.html

@@ -2,6 +2,6 @@
 {% extends "admin/model/edit.html" %}
 
 {% block body %}
-    {{information(current_user, active_semester, role="admin")}}
+    {{information(current_user, active_semester_repr, role="admin")}}
     {{ super() }}
 {% endblock %}

advlabdb/templates/admin_index.html

@@ -2,7 +2,7 @@
 {% extends "admin/index.html" %}
 
 {% block body %}
-    {{information(current_user, active_semester, role="admin")}}
+    {{information(current_user, active_semester_repr, role="admin")}}
 
     <hr>
 

advlabdb/templates/admin_list.html

@@ -2,6 +2,6 @@
 {% extends "admin/model/list.html" %}
 
 {% block body %}
-    {{information(current_user, active_semester, role="admin")}}
+    {{information(current_user, active_semester_repr, role="admin")}}
     {{ super() }}
 {% endblock %}

advlabdb/templates/analysis/analysis.html

@@ -2,7 +2,7 @@
 {% extends "admin/master.html" %}
 
 {% block body %}
-    {{information(current_user, active_semester, role="admin")}}
+    {{information(current_user, active_semester_repr, role="admin")}}
 
     <hr>
 

advlabdb/templates/analysis/assistant_marks.html

@@ -2,7 +2,7 @@
 {% extends "admin/master.html" %}
 
 {% block body %}
-    {{information(current_user, active_semester, role="admin")}}
+    {{information(current_user, active_semester_repr, role="admin")}}
 
     <hr>
 

advlabdb/templates/analysis/final_part_marks.html

@@ -2,7 +2,7 @@
 {% extends "admin/master.html" %}
 
 {% block body %}
-    {{information(current_user, active_semester, role="admin")}}
+    {{information(current_user, active_semester_repr, role="admin")}}
 
     <hr>
 

advlabdb/templates/assistant_create.html

@@ -2,6 +2,6 @@
 {% extends "admin/model/create.html" %}
 
 {% block body %}
-    {{information(current_user, active_semester, role="assistant")}}
+    {{information(current_user, active_semester_repr, role="assistant")}}
     {{ super() }}
 {% endblock %}

advlabdb/templates/assistant_details.html

@@ -2,6 +2,6 @@
 {% extends "admin/model/details.html" %}
 
 {% block body %}
-    {{information(current_user, active_semester, role="assistant")}}
+    {{information(current_user, active_semester_repr, role="assistant")}}
     {{ super() }}
 {% endblock %}

advlabdb/templates/assistant_edit.html

@@ -2,6 +2,6 @@
 {% extends "admin/model/edit.html" %}
 
 {% block body %}
-    {{information(current_user, active_semester, role="assistant")}}
+    {{information(current_user, active_semester_repr, role="assistant")}}
     {{ super() }}
 {% endblock %}

advlabdb/templates/assistant_index.html

@@ -2,7 +2,7 @@
 {% extends "admin/index.html" %}
 
 {% block body %}
-    {{information(current_user, active_semester, role="assistant")}}
+    {{information(current_user, active_semester_repr, role="assistant")}}
 
     <hr>
 

advlabdb/templates/assistant_list.html

@@ -2,6 +2,6 @@
 {% extends "admin/model/list.html" %}
 
 {% block body %}
-    {{information(current_user, active_semester, role="assistant")}}
+    {{information(current_user, active_semester_repr, role="assistant")}}
     {{ super() }}
 {% endblock %}

advlabdb/templates/import.html

@@ -2,7 +2,7 @@
 {% extends "admin/master.html" %}
 
 {% block body %}
-    {{information(current_user, active_semester, role="admin")}}
+    {{information(current_user, active_semester_repr, role="admin")}}
 
     <hr>
 

advlabdb/templates/macros.html

@@ -1,7 +1,7 @@
-{% macro information(current_user, active_semester, role) %}
+{% macro information(current_user, active_semester_repr, role) %}
     User: <a href="{{ url_for('index') }}{{ role }}/user/details/?id={{ current_user.id }}">{{ current_user.repr() }}</a>
 
-    | Active semester: {{ active_semester.repr() }}
+    | Active semester: {{ active_semester_repr() }}
 
     {% if (role == "admin") and (current_user.has_role("assistant")) %}
         | <a href="{{ url_for('index') }}assistant">Assistant space</a>.