.editorconfig

@@ -1,5 +0,0 @@
-root = true
-
-[*.{html,js}]
-indent_size = 4
-indent_style = space

LICENSE

@@ -1,4 +1,4 @@
-Copyright 2023-2024 Luca Schmid
+Copyright 2023 Luca
 
 Licensed under the Apache License, Version 2.0 (the "License");
 you may not use this file except in compliance with the License.

README.md

@@ -40,7 +40,7 @@ black .
 
 ## License
 
-Copyright 2023-2024 Luca Schmid
+Copyright 2023 Luca Schmid
 
 Released under the terms of the Apache License 2.0
 

pretalx_musicrate/__init__.py

@@ -1 +1 @@
-__version__ = "2025.6.1"
+__version__ = "0.14.1"

pretalx_musicrate/forms.py

@@ -37,6 +37,12 @@ class MusicrateSettingsForm(I18nModelForm):
             "link_questions",
             "advance_threshold",
         )
+        widgets = {
+            "submission_types": forms.SelectMultiple(attrs={"class": "select2"}),
+            "genre_question": forms.Select(attrs={"class": "select2"}),
+            "origin_question": forms.Select(attrs={"class": "select2"}),
+            "link_questions": forms.SelectMultiple(attrs={"class": "select2"}),
+        }
         field_classes = {
             "submission_types": SafeModelMultipleChoiceField,
             "genre_question": SafeModelChoiceField,
@@ -62,9 +68,12 @@ class AssigneeForm(forms.ModelForm):
         except Assignee.DoesNotExist:
             self.instance = Assignee(submission=submission)
         super().__init__(*args, instance=self.instance, **kwargs)
-        self.fields["user"].queryset = User.objects.filter(
+        self.fields["user"].queryset = User.objects.none().union(
-            teams__in=submission.event.teams, teams__can_change_submissions=True
+            *(
-        ).distinct()
+                t.members.all()
+                for t in submission.event.teams.filter(can_change_submissions=True)
+            )
+        )
 
     class Meta:
         model = Assignee
@@ -74,10 +83,3 @@ class AssigneeForm(forms.ModelForm):
 
 class EnhancedSubmissionFilterForm(SubmissionFilterForm):
     require_all_tags = forms.BooleanField(required=False, label=_("require all"))
-
-    def filter_queryset(self, qs):
-        qs = super().filter_queryset(qs)
-        if self.cleaned_data.get("require_all_tags"):
-            for tag in self.cleaned_data.get("tags"):
-                qs = qs.filter(tags__in=[tag])
-        return qs

pretalx_musicrate/locale/de_DE/LC_MESSAGES/django.po

@@ -1,14 +1,14 @@
 # pretalx plugin for rating music.
-# Copyright (C) 2023-2024
+# Copyright (C) 2023
 # This file is distributed under the same license as the pretalx_musicrate package.
-# Luca Schmid <Luca@hackerspace-bamberg.de>, 2023-2024.
+# Luca Schmid <Luca@hackerspace-bamberg.de>, 2023.
 #
 msgid ""
 msgstr ""
 "Project-Id-Version: pretalx-musicrate 0.9.0\n"
 "Report-Msgid-Bugs-To: \n"
-"POT-Creation-Date: 2024-12-09 22:22+0100\n"
+"POT-Creation-Date: 2023-12-18 01:34+0100\n"
-"PO-Revision-Date: 2024-12-09 22:23+0100\n"
+"PO-Revision-Date: 2023-12-18 01:36+0100\n"
 "Last-Translator: Luca <Luca@hackerspace-bamberg.de>\n"
 "Language-Team: Luca <Luca@hackerspace-bamberg.de>\n"
 "Language: de_DE\n"
@@ -16,9 +16,9 @@ msgstr ""
 "Content-Type: text/plain; charset=UTF-8\n"
 "Content-Transfer-Encoding: 8bit\n"
 "Plural-Forms: nplurals=2; plural=(n != 1);\n"
-"X-Generator: Poedit 3.4.2\n"
+"X-Generator: Poedit 3.4.1\n"
 
-#: pretalx_musicrate/apps.py:12 pretalx_musicrate/signals.py:59
+#: pretalx_musicrate/apps.py:12
 msgid "pretalx-musicrate"
 msgstr "pretalx-musicrate"
 
@@ -26,10 +26,6 @@ msgstr "pretalx-musicrate"
 msgid "pretalx plugin for rating music"
 msgstr "pretalx-Plugin zur Bewertung von Musik"
 
-#: pretalx_musicrate/forms.py:79
-msgid "require all"
-msgstr "alle voraussetzen"
-
 #: pretalx_musicrate/models.py:23
 msgid ""
 "You can limit pretalx-musicrate to some session types. Leave this field "
@@ -66,103 +62,10 @@ msgstr "Weiter-Schwellenwert"
 msgid "Link Questions"
 msgstr "Fragen zu Links"
 
-#: pretalx_musicrate/signals.py:21
+#: pretalx_musicrate/signals.py:16
-msgid "The name of the submission's assignee"
-msgstr "Der Name der für die Einreichung zuständigen Person"
-
-#: pretalx_musicrate/signals.py:35
-msgid "Sessions, but better"
-msgstr "Vorträge, aber besser"
-
-#: pretalx_musicrate/signals.py:45
 msgid "Collective Rating"
 msgstr "Anhörtag"
 
-#: pretalx_musicrate/templates/pretalx_musicrate/assignee.html:5
-#, python-format
-msgid "Assignee for %(quotation_open)s%(title)s%(quotation_close)s"
-msgstr "Zuständige*r für %(quotation_open)s%(title)s%(quotation_close)s"
-
-#: pretalx_musicrate/templates/pretalx_musicrate/enhanced_list.html:43
-msgid "proposal"
-msgid_plural "proposals"
-msgstr[0] "Einreichung"
-msgstr[1] "Einreichungen"
-
-#: pretalx_musicrate/templates/pretalx_musicrate/enhanced_list.html:72
-msgid "Search"
-msgstr "Suche"
-
-#: pretalx_musicrate/templates/pretalx_musicrate/enhanced_list.html:77
-#, python-format
-msgid "List filtered by answers to question \"%(question)s\"."
-msgstr "Liste gefiltert nach Antworten zur Frage \"%(question)s\"."
-
-#: pretalx_musicrate/templates/pretalx_musicrate/enhanced_list.html:82
-msgid "Remove filter"
-msgstr "Filter entfernen"
-
-#: pretalx_musicrate/templates/pretalx_musicrate/enhanced_list.html:93
-msgid "Rating"
-msgstr "Bewertung"
-
-#: pretalx_musicrate/templates/pretalx_musicrate/enhanced_list.html:94
-msgid "Sort by rating (0-10)"
-msgstr "Nach Bewertung sortieren (0-10)"
-
-#: pretalx_musicrate/templates/pretalx_musicrate/enhanced_list.html:95
-msgid "Sort by rating (10-0)"
-msgstr "Nach Bewertung sortieren (10-0)"
-
-#: pretalx_musicrate/templates/pretalx_musicrate/enhanced_list.html:98
-msgid "Title"
-msgstr "Titel"
-
-#: pretalx_musicrate/templates/pretalx_musicrate/enhanced_list.html:99
-msgid "Sort by title (a-z)"
-msgstr "Nach Titel sortieren (a-z)"
-
-#: pretalx_musicrate/templates/pretalx_musicrate/enhanced_list.html:100
-msgid "Sort by title (z-a)"
-msgstr "Nach Titel sortieren (z-a)"
-
-#: pretalx_musicrate/templates/pretalx_musicrate/enhanced_list.html:104
-msgid "Type"
-msgstr "Typ"
-
-#: pretalx_musicrate/templates/pretalx_musicrate/enhanced_list.html:108
-msgid "State"
-msgstr "Status"
-
-#: pretalx_musicrate/templates/pretalx_musicrate/enhanced_list.html:109
-msgid "Sort by state (a-z)"
-msgstr "Nach Status sortieren (a-z)"
-
-#: pretalx_musicrate/templates/pretalx_musicrate/enhanced_list.html:110
-msgid "Sort by state (z-a)"
-msgstr "Nach Status sortieren (z-a)"
-
-#: pretalx_musicrate/templates/pretalx_musicrate/enhanced_list.html:113
-msgid "Assignee"
-msgstr "Zuständige*r"
-
-#: pretalx_musicrate/templates/pretalx_musicrate/enhanced_list.html:114
-msgid "Sort by assignee (a-z)"
-msgstr "Nach Zuständiger*m sortieren (a-z)"
-
-#: pretalx_musicrate/templates/pretalx_musicrate/enhanced_list.html:115
-msgid "Sort by assignee (z-a)"
-msgstr "Nach Zuständiger*m sortieren (z-a)"
-
-#: pretalx_musicrate/templates/pretalx_musicrate/enhanced_list.html:154
-#: pretalx_musicrate/templates/pretalx_musicrate/enhanced_list.html:162
-msgid "edit"
-msgstr "bearbeiten"
-
-#: pretalx_musicrate/templates/pretalx_musicrate/enhanced_list.html:167
-msgid "Delete"
-msgstr "Löschen"
-
 #: pretalx_musicrate/templates/pretalx_musicrate/join.html:5
 msgid "Join collective rating"
 msgstr "Beim Anhörtag mitmachen"
@@ -186,43 +89,43 @@ msgstr "Mitmach-QR-Code anzeigen"
 msgid "Go to collective rating"
 msgstr "Zum Anhörtag"
 
-#: pretalx_musicrate/templates/pretalx_musicrate/settings.html:7
+#: pretalx_musicrate/templates/pretalx_musicrate/settings.html:8
 msgid "pretalx-musicrate settings"
 msgstr "pretalx-musicrate-Einstellungen"
 
-#: pretalx_musicrate/templates/pretalx_musicrate/settings.html:12
+#: pretalx_musicrate/templates/pretalx_musicrate/settings.html:22
+msgid "Save"
+msgstr "Speichern"
+
+#: pretalx_musicrate/templates/pretalx_musicrate/settings.html:30
 msgid "Export ratings"
 msgstr "Bewertungen exportieren"
 
-#: pretalx_musicrate/templates/pretalx_musicrate/settings.html:16
+#: pretalx_musicrate/templates/pretalx_musicrate/submission_base.html:29
-msgid "Export scores"
-msgstr "Auswertung exportieren"
-
-#: pretalx_musicrate/templates/pretalx_musicrate/submission_base.html:34
 msgid "(not specified)"
 msgstr "(nicht angegeben)"
 
-#: pretalx_musicrate/templates/pretalx_musicrate/submission_base.html:47
+#: pretalx_musicrate/templates/pretalx_musicrate/submission_base.html:42
 msgid "Previous"
 msgstr "Zurück"
 
-#: pretalx_musicrate/templates/pretalx_musicrate/submission_base.html:55
+#: pretalx_musicrate/templates/pretalx_musicrate/submission_base.html:50
 msgid "Next"
 msgstr "Weiter"
 
-#: pretalx_musicrate/views.py:66
+#: pretalx_musicrate/views.py:41
 msgid "Invalid token"
 msgstr "Ungültiges Token"
 
-#: pretalx_musicrate/views.py:133
+#: pretalx_musicrate/views.py:116
 msgid "The pretalx-musicrate settings were updated."
 msgstr "Die pretalx-musicrate-Einstellungen wurden gespeichert."
 
-#: pretalx_musicrate/views.py:289 pretalx_musicrate/views.py:482
+#: pretalx_musicrate/views.py:275
 msgid "Saved!"
 msgstr "Gespeichert!"
 
-#: pretalx_musicrate/views.py:398
+#: pretalx_musicrate/views.py:377
 #, python-format
 msgid "%(num_ratings)d of %(num_jurors)d has rated this submission"
 msgid_plural "%(num_ratings)d of %(num_jurors)d have rated this submission"

pretalx_musicrate/management/commands/computescores.py

@@ -1,6 +1,6 @@
 import math
 import statistics
-from operator import attrgetter, itemgetter
+from operator import itemgetter
 
 from django.core.management.base import BaseCommand, CommandError
 from django_scopes import scope
@@ -20,9 +20,6 @@ class Command(BaseCommand):
     help = "Compute submission scores from ratings"
 
     def add_arguments(self, parser):
-        parser.add_argument(
-            "-a", "--all", action="store_true", help="include frozen jurors"
-        )
         parser.add_argument("event")
 
     def handle(self, *args, **kwargs):
@@ -34,25 +31,12 @@ class Command(BaseCommand):
         with scope(event=event):
             submissions = {}
 
-            jurors = event.jurors.prefetch_related("ratings__submission").order_by(
+            for juror in event.jurors.prefetch_related("ratings"):
-                "token"
+                ratings = {
-            )
+                    r.submission.code: int(r.rating)
-            if not kwargs["all"]:
+                    for r in juror.ratings.exclude(rating="")
-                jurors = jurors.filter(frozen=False)
+                }
-
+                values = list(ratings.values())
-            self.stderr.write(f"computing scores from {jurors.count()} jurors")
-
-            for juror in jurors:
-                ratings = list(
-                    juror.ratings.exclude(rating="").order_by("submission__created")
-                )
-                values = list(map(int, map(attrgetter("rating"), ratings)))
-                ratings = dict(
-                    zip(
-                        map(attrgetter("code"), map(attrgetter("submission"), ratings)),
-                        values,
-                    )
-                )
 
                 if len(values) < RATINGS_MIN:
                     mean = MEAN
@@ -68,7 +52,7 @@ class Command(BaseCommand):
                         submissions[code] = []
                     submissions[code].append((ratings[code] - mean) / std * STD + MEAN)
 
-            for submission in Submission.objects.select_related("score").filter(
+            for submission in Submission.objects.prefetch_related("score").filter(
                 code__in=submissions.keys()
             ):
                 try:

pretalx_musicrate/migrations/0005_juror_musicratesettings_last_submission_rating_and_more.py

@@ -1,7 +1,7 @@
 # Generated by Django 4.2.8 on 2023-12-15 18:27
 
 import django.db.models.deletion
-import pretalx.common.models.mixins
+import pretalx.common.mixins.models
 from django.db import migrations, models
 
 import pretalx_musicrate.models
@@ -51,8 +51,8 @@ class Migration(migrations.Migration):
                 "abstract": False,
             },
             bases=(
-                pretalx.common.models.mixins.LogMixin,
+                pretalx.common.mixins.models.LogMixin,
-                pretalx.common.models.mixins.FileCleanupMixin,
+                pretalx.common.mixins.models.FileCleanupMixin,
                 models.Model,
             ),
         ),

pretalx_musicrate/migrations/0010_juror_frozen.py

@@ -1,18 +0,0 @@
-# Generated by Django 5.1.5 on 2025-02-23 13:27
-
-from django.db import migrations, models
-
-
-class Migration(migrations.Migration):
-
-    dependencies = [
-        ("pretalx_musicrate", "0009_score"),
-    ]
-
-    operations = [
-        migrations.AddField(
-            model_name="juror",
-            name="frozen",
-            field=models.BooleanField(default=False),
-        ),
-    ]

pretalx_musicrate/models.py

@@ -78,7 +78,6 @@ class Juror(models.Model):
         related_name="jurors",
         null=True,
     )
-    frozen = models.BooleanField(default=False)
 
 
 class Rating(models.Model):

pretalx_musicrate/signals.py

@@ -32,7 +32,7 @@ def pretalx_musicrate_nav_event(sender, request, **kwargs):
             "active": request.resolver_match.view_name
             == "plugins:pretalx_musicrate:enhanced_list",
             "icon": "sticky-note-o",
-            "label": _("Sessions, but better"),
+            "label": _("Proposals, but better"),
             "url": reverse(
                 "plugins:pretalx_musicrate:enhanced_list",
                 kwargs={"event": request.event.slug},
@@ -56,7 +56,7 @@ def pretalx_musicrate_settings(sender, request, **kwargs):
         return []
     return [
         {
-            "label": _("pretalx-musicrate"),
+            "label": "pretalx-musicrate",
             "url": reverse(
                 "plugins:pretalx_musicrate:settings.musicrate",
                 kwargs={"event": request.event.slug},

pretalx_musicrate/static/pretalx_musicrate/submission_filter.js

@@ -1,14 +1,11 @@
-onReady(() => {
+document.addEventListener("DOMContentLoaded", function() {
-    const updateRequireAllVisibility = () => {
+  const updateRequireAllVisibility = () => {
-        if (document.querySelector("#id_tags").value) {
+    if (document.querySelector("#id_tags").value) {
-            document.querySelector("#requireAll").classList.remove("d-none")
+      document.querySelector("#requireAll").classList.remove("d-none")
-        } else {
+    } else {
-            document.querySelector("#requireAll").classList.add("d-none")
+      document.querySelector("#requireAll").classList.add("d-none")
-        }
     }
-
+  }
-    document
+  $("#id_tags").on("change", updateRequireAllVisibility)
-        .querySelector("#id_tags")
+  updateRequireAllVisibility()
-        .addEventListener("change", updateRequireAllVisibility)
-    updateRequireAllVisibility()
 })

pretalx_musicrate/templates/pretalx_musicrate/assignee.html

@@ -1,8 +1,22 @@
 {% extends "orga/base.html" %}
+{% load bootstrap4 %}
 {% load i18n %}
 
 {% block content %}
     <h2>{% blocktranslate with title=submission.title %}Assignee for {{ quotation_open }}{{ title }}{{ quotation_close }}{% endblocktranslate %}</h2>
-    {% include "orga/includes/base_form.html" %}
+    <form method="post">
+        {% csrf_token %}
+        {% bootstrap_form_errors form %}
+        {% bootstrap_field form.user layout='event' %}
+        <div class="submit-group panel">
+            <span></span>
+            <span>
+                <button type="submit" class="btn btn-success btn-lg">
+                    <i class="fa fa-check"></i>
+                    {% translate "Save" %}
+                </button>
+            </span>
+        </div>
+    </form>
 {% endblock %}
 

pretalx_musicrate/templates/pretalx_musicrate/enhanced_list.html

@@ -1,37 +1,17 @@
 {% extends "orga/base.html" %}
+{% load bootstrap4 %}
 {% load compress %}
 {% load i18n %}
 {% load rules %}
 {% load static %}
+{% load url_replace %}
 
 {% block scripts %}
     {% compress js %}
-        <script defer src="{% static "orga/js/submission_filter.js" %}"></script>
+        <script src="{% static "orga/js/submission_filter.js" %}"></script>
     {% endcompress %}
     {% compress js %}
-        <script defer src="{% static "pretalx_musicrate/submission_filter.js" %}"></script>
+        <script src="{% static "pretalx_musicrate/submission_filter.js" %}"></script>
-    {% endcompress %}
-{% endblock %}
-
-{% block stylesheets %}
-    {% compress css %}
-        <style>
-            .search-form {
-                #requireAll .form-group {
-                    margin-bottom: 0;
-                }
-
-                #requireAll label {
-                    display: inline-block;
-                    color: #6c757d;
-                }
-            }
-
-            #requireAll .form-group-inline {
-                flex-direction: row;
-                align-items: center;
-            }
-        </style>
     {% endcompress %}
 {% endblock %}
 
@@ -49,26 +29,20 @@
 
     <div class="submit-group search-submit-group">
         <form class="search-form">
-            {{ filter_form.q.as_field_group }}
+            {% bootstrap_form search_form %}
-            {% if show_submission_types and filter_form.submission_type %}{{ filter_form.submission_type.as_field_group }}{% endif %}
+            {% if show_submission_types and filter_form.submission_type %}{% bootstrap_field filter_form.submission_type %}{% endif %}
             <div class="d-flex flex-column form-group">
-                {{ filter_form.state.as_field_group }}
+                {% bootstrap_field filter_form.state layout='inline' %}
-                <div id="pending" class="ml-1 d-none">{{ filter_form.pending_state__isnull.as_field_group }}</div>
+                <div id="pending" class="d-none">{% bootstrap_field filter_form.pending_state__isnull layout='inline' %}</div>
             </div>
-            {% if filter_form.track %}{{ filter_form.track.as_field_group }}{% endif %}
+            {% if filter_form.track %}{% bootstrap_field filter_form.track %}{% endif %}
             {% if filter_form.tags %}
                 <div class="d-flex flex-column form-group">
-                    {{ filter_form.tags.as_field_group }}
+                    {% bootstrap_field filter_form.tags layout='inline' %}
-                    <div id="requireAll" class="ml-1 d-none">{{ filter_form.require_all_tags.as_field_group }}</div>
+                    <div id="requireAll" class="d-none">{% bootstrap_field filter_form.require_all_tags layout='inline' %}</div>
                 </div>
             {% endif %}
-            {% if filter_form.content_locale %}{{ filter_form.content_locale.as_field_group }}{% endif %}
+            {% if filter_form.content_locale %}{% bootstrap_field filter_form.content_locale %}{% endif %}
-            {# These fields are hidden, but included to keep question search intact #}
-            {% if request.GET.question %} <input type="hidden" name="question" value="{{ request.GET.question }}"> {% endif %}
-            {% if request.GET.answer__options %} <input type="hidden" name="answer__options" value="{{ request.GET.answer__options }}"> {% endif %}
-            {% if request.GET.answer %} <input type="hidden" name="answer" value="{{ request.GET.answer }}"> {% endif %}
-            {% if request.GET.unanswered %} <input type="hidden" name="unanswered" value="{{ request.GET.unanswered }}"> {% endif %}
-
             <button class="btn btn-success" type="submit">{% translate "Search" %}</button>
         </form>
         {% if filter_form.is_valid and filter_form.cleaned_data.question %}
@@ -77,7 +51,7 @@
                 {% blocktranslate trimmed with question=filter_form.cleaned_data.question.question %}
                     List filtered by answers to question "{{ question }}".
                 {% endblocktranslate %}
-                <a href="{% querystring question="" answer="" answer__options="" %}" class="text-muted">
+                <a href="?{% url_replace request 'question' '' 'answer' '' 'answer__options' '' %}" class="text-muted">
                     <span class="fa fa-times"></span>
                     {% translate "Remove filter" %}
                 </a>
@@ -91,13 +65,13 @@
                 <tr>
                     <th>
                         {% translate "Rating" %}
-                        <a href="{% querystring sort="score__value" %}"><i class="fa fa-caret-down" title="{% translate "Sort by rating (0-10)" %}"></i></a>
+                        <a href="?{% url_replace request 'sort' 'score__value' %}"><i class="fa fa-caret-down" title="{% translate "Sort by rating (0-10)" %}"></i></a>
-                        <a href="{% querystring sort="-score__value" %}"><i class="fa fa-caret-up" title="{% translate "Sort by rating (10-0)" %}"></i></a>
+                        <a href="?{% url_replace request 'sort' '-score__value' %}"><i class="fa fa-caret-up" title="{% translate "Sort by rating (10-0)" %}"></i></a>
                     </th>
                     <th>
                         {% translate "Title" %}
-                        <a href="?{% querystring sort="title" %}"><i class="fa fa-caret-down" title="{% translate "Sort by title (a-z)" %}"></i></a>
+                        <a href="?{% url_replace request 'sort' 'title' %}"><i class="fa fa-caret-down" title="{% translate "Sort by title (a-z)" %}"></i></a>
-                        <a href="?{% querystring sort="-title" %}"><i class="fa fa-caret-up" title="{% translate "Sort by title (z-a)" %}"></i></a>
+                        <a href="?{% url_replace request 'sort' '-title' %}"><i class="fa fa-caret-up" title="{% translate "Sort by title (z-a)" %}"></i></a>
                     </th>
                     {% if show_submission_types  %}
                         <th>
@@ -106,13 +80,13 @@
                     {% endif %}
                     <th>
                         {% translate "State" %}
-                        <a href="?{% querystring sort="state" %}"><i class="fa fa-caret-down" title="{% translate "Sort by state (a-z)" %}"></i></a>
+                        <a href="?{% url_replace request 'sort' 'state' %}"><i class="fa fa-caret-down" title="{% translate "Sort by state (a-z)" %}"></i></a>
-                        <a href="?{% querystring sort="-state" %}"><i class="fa fa-caret-up" title="{% translate "Sort by state (z-a)" %}"></i></a>
+                        <a href="?{% url_replace request 'sort' '-state' %}"><i class="fa fa-caret-up" title="{% translate "Sort by state (z-a)" %}"></i></a>
                     </th>
                     <th>
                         {% translate "Assignee" %}
-                        <a href="?{% querystring sort="assignee" %}"><i class="fa fa-caret-down" title="{% translate "Sort by assignee (a-z)" %}"></i></a>
+                        <a href="?{% url_replace request 'sort' 'assignee' %}"><i class="fa fa-caret-down" title="{% translate "Sort by assignee (a-z)" %}"></i></a>
-                        <a href="?{% querystring sort="-assignee" %}"><i class="fa fa-caret-up" title="{% translate "Sort by assignee (z-a)" %}"></i></a>
+                        <a href="?{% url_replace request 'sort' '-assignee' %}"><i class="fa fa-caret-up" title="{% translate "Sort by assignee (z-a)" %}"></i></a>
                     </th>
                     {% if can_change_submission %}
                         <th></th>
@@ -176,5 +150,5 @@
         </table>
     </div>
 
-    {% include "orga/includes/pagination.html" %}
+    {% include "orga/pagination.html" %}
 {% endblock %}

pretalx_musicrate/templates/pretalx_musicrate/settings.html

@@ -1,18 +1,32 @@
 {% extends "orga/base.html" %}
+{% load bootstrap4 %}
 {% load compress %}
 {% load i18n %}
 {% load static %}
 
 {% block content %}
     <h2>{% translate "pretalx-musicrate settings" %}</h2>
-    {% include "orga/includes/base_form.html" %}
+    <form method="post">
+        {% csrf_token %}
+        {% bootstrap_form_errors form %}
+        {% bootstrap_field form.submission_types layout='event' %}
+        {% bootstrap_field form.genre_question layout='event' %}
+        {% bootstrap_field form.origin_question layout='event' %}
+        {% bootstrap_field form.link_questions layout='event' %}
+        {% bootstrap_field form.advance_threshold layout='event' %}
+        <div class="submit-group panel">
+            <span></span>
+            <span>
+                <button type="submit" class="btn btn-success btn-lg">
+                    <i class="fa fa-check"></i>
+                    {% translate "Save" %}
+                </button>
+            </span>
+        </div>
+    </form>
     <hr>
     <a class="btn btn-success btn-lg btn-block" href="{% url "plugins:pretalx_musicrate:export" event=request.event.slug %}">
         <i class="fa fa-download"></i>
         {% translate "Export ratings" %}
     </a>
-    <a class="btn btn-success btn-lg btn-block" href="{% url "plugins:pretalx_musicrate:scores" event=request.event.slug %}">
-        <i class="fa fa-download"></i>
-        {% translate "Export scores" %}
-    </a>
 {% endblock %}

pretalx_musicrate/templates/pretalx_musicrate/submission_base.html

@@ -1,13 +1,11 @@
 {% extends "cfp/event/base.html" %}
 {% load compress %}
 {% load i18n %}
-{% load static %}
 
 {% block title %}{{ submission.title }} ::{% endblock %}
 
-{% block stylesheets %}
+{% block cfp_header %}
     {% compress css %}
-        <link rel="stylesheet" type="text/css" href="{% static "common/css/_forms.css" %}" />
         <style>
             .musicrate-pagination {
                 display: flex;
@@ -23,9 +21,6 @@
             }
         </style>
     {% endcompress %}
-{% endblock %}
-
-{% block cfp_header %}
     {% block submission_header %}
     {% endblock %}
 {% endblock %}

pretalx_musicrate/urls.py

@@ -1,5 +1,4 @@
-from django.urls import include, path, re_path
+from django.urls import include, path
-from pretalx.event.models.event import SLUG_REGEX
 
 from .views import (
     AssigneeView,
@@ -11,12 +10,11 @@ from .views import (
     PresenterView,
     QRCodeView,
     RatingView,
-    ScoreExportView,
 )
 
 urlpatterns = [
-    re_path(
+    path(
-        rf"^orga/event/(?P<event>{SLUG_REGEX})/",
+        "orga/event/<slug:event>/",
         include(
             [
                 path(
@@ -34,7 +32,6 @@ urlpatterns = [
                                 EnhancedSubmissionList.as_view(),
                                 name="enhanced_list",
                             ),
-                            path("scores/", ScoreExportView.as_view(), name="scores"),
                             path("<code>/", AssigneeView.as_view(), name="assignee"),
                         ]
                     ),
@@ -42,8 +39,8 @@ urlpatterns = [
             ]
         ),
     ),
-    re_path(
+    path(
-        rf"^(?P<event>{SLUG_REGEX})/p/pretalx_musicrate/",
+        "<slug:event>/p/pretalx_musicrate/",
         include(
             [
                 path("", QRCodeView.as_view(), name="qrcode"),

pretalx_musicrate/views.py

@@ -4,7 +4,7 @@ from hmac import compare_digest
 from urllib.parse import parse_qs, urlparse
 
 from django.contrib import messages
-from django.db.models import Case, F, FilteredRelation, Q, Value, When
+from django.db.models import Case, FilteredRelation, Q, Value, When
 from django.http import HttpResponse, JsonResponse
 from django.shortcuts import get_object_or_404, redirect
 from django.urls import reverse
@@ -14,9 +14,9 @@ from django.utils.translation import gettext_lazy as _, ngettext
 from django.views.generic import FormView, TemplateView, View
 from django.views.generic.detail import SingleObjectMixin
 from django_context_decorator import context
-from pretalx.common.views.mixins import EventPermissionRequired
+from pretalx.common.mixins.views import EventPermissionRequired
-from pretalx.orga.views.submission import BaseSubmissionList, SubmissionList
+from pretalx.orga.views.submission import SubmissionList
-from pretalx.submission.models import Submission, SubmissionStates
+from pretalx.submission.models import Submission
 
 from .forms import (
     AssigneeForm,
@@ -31,30 +31,12 @@ youtube_re = re.compile(
 )
 
 
-def get_last_submission(settings, submissions, submission=None):
-    if submission is not None and submission.state != SubmissionStates.SUBMITTED:
-        submission = None
-
-    submission = submission or settings.last_submission
-    if submission is not None and submission.state != SubmissionStates.SUBMITTED:
-        submission = None
-
-    submissions = submissions.filter(
-        submission_type__in=settings.submission_types.all(),
-        state=SubmissionStates.SUBMITTED,
-    )
-    if submission is not None and not submissions.filter(pk=submission.pk).exists():
-        submission = None
-
-    return submission or submissions.order_by("created").first()
-
-
 class JoinView(TemplateView):
     template_name = "pretalx_musicrate/join.html"
 
     def validate_token(self, token):
         try:
-            self.juror = Juror.objects.get(token=token, frozen=False)
+            self.juror = Juror.objects.get(token=token)
             return True
         except Juror.DoesNotExist:
             self.juror = None
@@ -75,10 +57,14 @@ class JoinView(TemplateView):
     def get(self, request, *args, token, **kwargs):
         token_valid = self.validate_token(token)
         if self.juror is not None:
-            submission = get_last_submission(
+            submission = (
-                self.request.event.pretalx_musicrate_settings,
+                self.juror.last_submission
-                self.request.event.submissions,
+                or self.request.event.pretalx_musicrate_settings.last_submission
-                self.juror.last_submission,
+                or self.request.event.submissions.filter(
+                    submission_type__in=self.request.event.pretalx_musicrate_settings.submission_types.all()
+                )
+                .order_by("created")
+                .first()
             )
             if submission is not None:
                 return redirect(
@@ -97,10 +83,14 @@ class JoinView(TemplateView):
                     event=self.request.event,
                     last_submission=self.request.event.pretalx_musicrate_settings.last_submission,
                 )
-            submission = get_last_submission(
+            submission = (
-                self.request.event.pretalx_musicrate_settings,
+                self.juror.last_submission
-                self.request.event.submissions,
+                or self.request.event.pretalx_musicrate_settings.last_submission
-                self.juror.last_submission,
+                or self.request.event.submissions.filter(
+                    submission_type__in=self.request.event.pretalx_musicrate_settings.submission_types.all()
+                )
+                .order_by("created")
+                .first()
             )
             if submission is not None:
                 return redirect(
@@ -150,9 +140,13 @@ class QRCodeView(EventPermissionRequired, TemplateView):
                 },
             )
         )
-        context["last_submission"] = get_last_submission(
+        context["last_submission"] = (
-            self.request.event.pretalx_musicrate_settings,
+            self.request.event.pretalx_musicrate_settings.last_submission
-            self.request.event.submissions,
+            or self.request.event.submissions.filter(
+                submission_type__in=self.request.event.pretalx_musicrate_settings.submission_types.all()
+            )
+            .order_by("created")
+            .first()
         )
         return context
 
@@ -163,8 +157,7 @@ class SubmissionMixin(SingleObjectMixin):
 
     def get_queryset(self):
         return self.request.event.submissions.prefetch_related("answers").filter(
-            submission_type__in=self.request.event.pretalx_musicrate_settings.submission_types.all(),
+            submission_type__in=self.request.event.pretalx_musicrate_settings.submission_types.all()
-            state=SubmissionStates.SUBMITTED,
         )
 
     @context
@@ -256,7 +249,7 @@ class RatingView(FormView, SubmissionMixin):
     @cached_property
     def juror(self):
         return get_object_or_404(
-            Juror, token=self.request.resolver_match.kwargs["token"], frozen=False
+            Juror, token=self.request.resolver_match.kwargs["token"]
         )
 
     @context
@@ -291,22 +284,17 @@ class RatingView(FormView, SubmissionMixin):
 
     def get(self, request, *args, **kwargs):
         self.object = self.get_object()
-
-        last_submission = get_last_submission(
-            self.request.event.pretalx_musicrate_settings,
-            self.request.event.submissions,
-        )
         if (
-            last_submission is not None
+            (settings := self.request.event.pretalx_musicrate_settings).last_submission
-            and self.object.created > last_submission.created
+            is not None
+            and self.object.created > settings.last_submission.created
         ):
             return redirect(
                 self.request.resolver_match.view_name,
                 event=self.request.event.slug,
                 token=self.juror.token,
-                code=last_submission.code,
+                code=settings.last_submission.code,
             )
-
         return super().get(request, *args, **kwargs)
 
 
@@ -365,16 +353,16 @@ class PresenterView(EventPermissionRequired, SubmissionMixin, TemplateView):
 
     def get(self, request, *args, **kwargs):
         self.object = self.get_object()
-
+        if (
-        settings = self.request.event.pretalx_musicrate_settings
+            (settings := self.request.event.pretalx_musicrate_settings).last_submission
-        last_submission = get_last_submission(settings, self.request.event.submissions)
+            is None
-        if last_submission is None or self.object.created > last_submission.created:
+            or self.object.created > settings.last_submission.created
+        ):
             try:
                 settings.last_submission = self.object
                 settings.save()
             except Exception:
                 pass
-
         response = super().get(request, *args, **kwargs)
         response._csp_update = {"frame-src": "https://www.youtube-nocookie.com"}
         return response
@@ -384,12 +372,12 @@ class MayAdvanceView(EventPermissionRequired, SubmissionMixin, View):
     permission_required = "orga.view_submissions"
 
     def get(self, request, *args, **kwargs):
-        num_ratings = self.submission.ratings.filter(juror__frozen=False).count()
+        num_ratings = self.submission.ratings.count()
-        num_jurors = self.request.event.jurors.filter(frozen=False).count()
+        num_jurors = self.request.event.jurors.count()
         return JsonResponse(
             {
                 "mayAdvance": num_ratings
-                >= round(
+                >= int(
                     num_jurors
                     * self.request.event.pretalx_musicrate_settings.advance_threshold
                 )
@@ -417,7 +405,7 @@ class ExportView(EventPermissionRequired, View):
         writer = csv.writer(response)
         genre_question = request.event.pretalx_musicrate_settings.genre_question
         origin_question = request.event.pretalx_musicrate_settings.origin_question
-        jurors = request.event.jurors.filter(frozen=False).order_by("token")
+        jurors = request.event.jurors.order_by("token")
         for submission in (
             request.event.submissions.prefetch_related("answers")
             .select_related("submission_type")
@@ -500,40 +488,11 @@ class EnhancedSubmissionList(SubmissionList):
             event=self.request.event,
             usable_states=self.usable_states,
             limit_tracks=self.limit_tracks,
-            search_fields=self.get_default_filters(),
         )
 
-    def _get_base_queryset(self, for_review=False):
+    def filter_queryset(self, qs):
-        qs = (
+        qs = super().filter_queryset(qs.prefetch_related("assignee"))
-            super(BaseSubmissionList, self)
+        if self.request.GET.get("require_all_tags", "") == "on":
-            .get_queryset(for_review=for_review)
+            for tag in self.request.GET.getlist("tags"):
-            .prefetch_related("assignee")
+                qs = qs.filter(tags__in=[tag])
-            .order_by("-id")
+        return qs
-        )
-        if not self.filter_form.is_valid():
-            return qs
-        return self.filter_form.filter_queryset(qs)
-
-
-class ScoreExportView(EventPermissionRequired, View):
-    permission_required = "orga.view_submissions"
-
-    def get(self, request, *args, **kwargs):
-        response = HttpResponse(
-            content_type="text/csv",
-            headers={
-                "Content-Disposition": f'attachment; filename="{request.event.slug}.csv"'
-            },
-        )
-
-        csv.writer(response).writerows(
-            request.event.submissions.select_related("score", "submission_type")
-            .filter(
-                submission_type__in=request.event.pretalx_musicrate_settings.submission_types.all()
-            )
-            .exclude(state__in=(SubmissionStates.CANCELED, SubmissionStates.WITHDRAWN))
-            .order_by(F("score__value").desc(nulls_last=True))
-            .values_list("score__value", "title")
-        )
-
-        return response