implement changes from pull request comment

Author: tobi-werner

collab_coursebook/settings.py

@@ -166,3 +166,5 @@
     "impress_text": "",
     "impress_url": ""
 }
+
+ALLOW_PUBLIC_COURSE_EDITING_BY_EVERYONE = True

frontend/locale/de_DE/LC_MESSAGES/django.po

@@ -68,6 +68,7 @@ msgstr ""
 
 #: frontend/templates/frontend/content/detail.html:83
 #: frontend/templates/frontend/course/create.html:30
+#: frontend/templates/frontend/course/edit.html:30
 #: frontend/templates/frontend/profile/profile_edit.html:19
 msgid "Cancel"
 msgstr "Abbrechen"
@@ -235,10 +236,8 @@ msgid "Course '{cleaned_data['title']}' successfully created"
 msgstr "Kurs '{cleaned_data['title']}' erfolgreich angelegt"
 
 #: frontend/views/course.py:52
-#, fuzzy
-#| msgid "Course '{cleaned_data['title']}' successfully created"
 msgid "Course '{cleaned_data['title']}' successfully edited"
-msgstr "Kurs '{cleaned_data['title']}' erfolgreich angelegt"
+msgstr "Kurs '{cleaned_data['title']}' erfolgreich bearbeitet"
 
 #: frontend/views/profile.py:25
 msgid "Profile updated"

frontend/templates/frontend/course/view.html

@@ -22,7 +22,7 @@
         {% if course in user.profile.stared_courses.all %}
             {% fa5_icon 'bookmark' 'fas' %}
         {% endif %}
-        {% if user.profile in course.owners.all %}
+        {% if user|check_edit_course_permission:course %}
             <a href="{% url 'frontend:course-edit' pk=course.pk %}" class="btn btn-warning">{% fa5_icon "pencil-alt" "fas" %} {% trans "Edit Course" %}</a>
             <a href="{% url 'frontend:course-delete' pk=course.pk %}" class="btn btn-warning">{% fa5_icon "trash" "fas" %} {% trans "Delete Course" %}</a>
         {% endif %}

frontend/templatetags/cc_frontend_tags.py

@@ -1,6 +1,7 @@
 from django import template
 from django.conf import settings
 
+from collab_coursebook.settings import ALLOW_PUBLIC_COURSE_EDITING_BY_EVERYONE
 from content.models import CONTENT_TYPES
 
 register = template.Library()
@@ -86,3 +87,10 @@ def content_card(type):
     if type in CONTENT_TYPES.keys():
         return f"content/cards/{type}.html"
     return "content/cards/blank.html"
+
+
+@register.filter
+def check_edit_course_permission(user, course):
+    # either an user is an owner or the course is public and it is allowed to edit public courses
+    return (user.profile in course.owners.all()) or (not course.restrict_changes
+                                                     and ALLOW_PUBLIC_COURSE_EDITING_BY_EVERYONE)