Add support for raid groups.

drakul/bank/models.py

@@ -13,50 +13,33 @@ class Item(models.Model):
         max_length=50
     )
 
-    QUALITY_POOR = 0
+    class Quality(models.IntegerChoices):
-    QUALITY_COMMON = 1
+        POOR = 0
-    QUALITY_UNCOMMON = 2
+        COMMON = 1
-    QUALITY_RARE = 3
+        UNCOMMON = 2
-    QUALITY_EPIC = 4
+        RARE = 3
-    QUALITY_LEGENDARY = 5
+        EPIC = 4
-    QUALITY_ARTIFACT = 6
+        LEGENDARY = 5
-    QUALITY_CHOICES = [
+        ARTIFACT = 6
-        (QUALITY_POOR, "Poor"),
+
-        (QUALITY_COMMON, "Common"),
-        (QUALITY_UNCOMMON, "Uncommon"),
-        (QUALITY_RARE, "Rare"),
-        (QUALITY_EPIC, "Epic"),
-        (QUALITY_LEGENDARY, "Legendary"),
-        (QUALITY_ARTIFACT, "Artifact"),
-    ]
     quality = models.PositiveSmallIntegerField(
-        choices=QUALITY_CHOICES
+        choices=Quality.choices
     )
 
-    KLASS_CONSUMABLES = 0
+    class Klass(models.IntegerChoices):
-    KLASS_CONTAINERS = 1
+        CONSUMABLES = 0
-    KLASS_WEAPONS = 2
+        CONTAINERS = 1
-    KLASS_ARMOR = 4
+        WEAPONS = 2
-    KLASS_PROJECTILES = 6
+        ARMOR = 4
-    KLASS_TRADE_GOODS = 7
+        PROJECTILES = 6
-    KLASS_RECIPES = 9
+        TRADE_GOODS = 7
-    KLASS_QUIVER = 11
+        RECIPES = 9
-    KLASS_QUEST_ITEMS = 12
+        QUIVER = 11
-    KLASS_MISCELLANEOUS = 15
+        QUEST_ITEMS = 12
-    KLASS_CHOICES = [
+        MISCELLANEOUS = 15
-        (KLASS_CONSUMABLES, "Consumables"),
+
-        (KLASS_CONTAINERS, "Containers"),
-        (KLASS_WEAPONS, "Weapons"),
-        (KLASS_ARMOR, "Armor"),
-        (KLASS_PROJECTILES, "Projectiles"),
-        (KLASS_TRADE_GOODS, "Trade Goods"),
-        (KLASS_RECIPES, "Recipes"),
-        (KLASS_QUIVER, "Quiver"),
-        (KLASS_QUEST_ITEMS, "Quest Items"),
-        (KLASS_MISCELLANEOUS, "Miscellaneous"),
-    ]
     klass = models.PositiveSmallIntegerField(
-        choices=KLASS_CHOICES
+        choices=Klass.choices
     )
 
     sub_klass = models.SmallIntegerField(

drakul/base/settings/common.py

@@ -160,11 +160,11 @@ CRISPY_TEMPLATE_PACK = "bootstrap4"
 
 # Drakul
 DEFAULT_ATTENDANCE = {
-    "No Response": 0.0,
+    "NO_RESPONSE": 0.0,
-    "Signed Off": 0.0,
+    "SIGNED_OFF": 0.0,
-    "Signed Up": 1.0,
+    "SIGNED_UP": 1.0,
-    "Stand By": 1.0,
+    "STANDBY": 1.0,
-    "Confirmed": 1.0,
+    "CONFIRMED": 1.0,
 }
 
 ATTENDANCE_COLORS = [

drakul/raids/admin.py

@@ -5,7 +5,7 @@ from .models import Raid, RaidResponse, RaidComment, InstanceReset
 
 class RaidResponseInline(admin.TabularInline):
     model = RaidResponse
-    fields = ["character", "role", "status", "attendance", "note"]
+    fields = ["character", "role", "status", "group", "attendance", "note"]
     extra = 0
 
 

drakul/raids/forms.py

@@ -16,16 +16,16 @@ class RaidResponseForm(ModelForm):
         self.fields["character"].queryset = user.characters
         self.fields["character"].initial = user.main
         self.fields["role"].initial = user.main.role
-        self.fields["status"].choices = RaidResponse.USER_STATUS_CHOICES
+        self.fields["status"].choices = RaidResponse.UserStatus.choices
 
         self.helper = FormHelper()
 
-        if self.instance.status <= RaidResponse.SIGNED_OFF:
+        if self.instance.status <= RaidResponse.Status.SIGNED_OFF:
             signup_button = StrictButton(
                 "Sign Up",
                 type="submit",
                 name="status",
-                value=RaidResponse.SIGNED_UP,
+                value=RaidResponse.Status.SIGNED_UP,
                 css_class="btn-success btn-block"
             )
         else:
@@ -33,7 +33,7 @@ class RaidResponseForm(ModelForm):
                 "Change",
                 type="submit",
                 name="status",
-                value=RaidResponse.SIGNED_UP,
+                value=RaidResponse.Status.SIGNED_UP,
                 css_class="btn-primary btn-block"
             )
         self.helper.layout = Layout(
@@ -46,9 +46,9 @@ class RaidResponseForm(ModelForm):
                         "Sign Off",
                         type="submit",
                         name="status",
-                        value=RaidResponse.SIGNED_OFF,
+                        value=RaidResponse.Status.SIGNED_OFF,
                         css_class="btn-danger btn-block",
-                        disabled=self.instance.pk and self.instance.status == RaidResponse.SIGNED_OFF
+                        disabled=self.instance.pk and self.instance.status == RaidResponse.Status.SIGNED_OFF
                     ),
                     css_class="col-md-3"
                 ),
@@ -109,7 +109,7 @@ class RaidCommentForm(ModelForm):
 RaidResponseFormSet = inlineformset_factory(
     Raid,  # parent model
     RaidResponse,
-    fields=["character", "role", "status", "note", "attendance"],
+    fields=["character", "role", "status", "group", "note", "attendance"],
     can_delete=False,
     extra=1
 )
@@ -124,6 +124,7 @@ class RaidResponseFormSetHelper(FormHelper):
             Field("character", css_class="character-select"),
             Field("role", css_class="role-select"),
             Field("status", css_class="status-select"),
+            Field("group", css_class="group-select"),
             Field("note"),
             Field("attendance", css_class="attendance-input"),
         )

drakul/raids/migrations/0007_auto_20200421_0256.py

@@ -0,0 +1,18 @@
+# Generated by Django 3.0.2 on 2020-04-21 02:56
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('raids', '0006_auto_20200417_0854'),
+    ]
+
+    operations = [
+        migrations.AlterField(
+            model_name='raidresponse',
+            name='status',
+            field=models.PositiveSmallIntegerField(choices=[(0, 'No Response'), (1, 'Signed Off'), (2, 'Signed Up'), (3, 'Standby'), (4, 'Confirmed')]),
+        ),
+    ]

drakul/raids/migrations/0008_auto_20200421_1753.py

@@ -0,0 +1,22 @@
+# Generated by Django 3.0.2 on 2020-04-21 17:53
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('raids', '0007_auto_20200421_0256'),
+    ]
+
+    operations = [
+        migrations.AlterModelOptions(
+            name='raidresponse',
+            options={'ordering': ['-status', 'group', 'role', 'character__klass', 'character__user__rank', 'character__name']},
+        ),
+        migrations.AddField(
+            model_name='raidresponse',
+            name='group',
+            field=models.PositiveSmallIntegerField(choices=[(1, 'Group 1'), (2, 'Group 2'), (3, 'Group 3'), (4, 'Group 4')], default=1),
+        ),
+    ]

drakul/raids/models.py

@@ -57,38 +57,40 @@ class RaidResponse(models.Model):
     )
 
     role = models.PositiveSmallIntegerField(
-        choices=Character.ROLE_CHOICES,
+        choices=Character.Roles.choices,
         blank=True,
         null=True
     )
 
-    NO_RESPONSE = 0
+    class Status(models.IntegerChoices):
-    SIGNED_OFF = 1
+        NO_RESPONSE = 0
-    SIGNED_UP = 2
+        SIGNED_OFF = 1
-    STANDBY = 3
+        SIGNED_UP = 2
-    CONFIRMED = 4
+        STANDBY = 3
-    STATUS_CHOICES = [
+        CONFIRMED = 4
-        (NO_RESPONSE, "No Response"),
+
-        (SIGNED_OFF, "Signed Off"),
+        @property
-        (SIGNED_UP, "Signed Up"),
+        def default_attendance(self):
-        (STANDBY, "Stand By"),
+            return settings.DEFAULT_ATTENDANCE[self.name]
-        (CONFIRMED, "Confirmed"),
+
-    ]
+    class UserStatus(models.IntegerChoices):
-    USER_STATUS_CHOICES = [
+        SIGNED_OFF = 1
-        (SIGNED_OFF, "Signed Off"),
+        SIGNED_UP = 2
-        (SIGNED_UP, "Signed Up"),
+
-    ]
     status = models.PositiveSmallIntegerField(
-        choices=STATUS_CHOICES
+        choices=Status.choices
     )
 
-    STATUS_DEFAULT_ATTENDANCE = {
+    class Groups(models.IntegerChoices):
-        NO_RESPONSE: settings.DEFAULT_ATTENDANCE["No Response"],
+        GROUP_1 = 1
-        SIGNED_OFF: settings.DEFAULT_ATTENDANCE["Signed Off"],
+        GROUP_2 = 2
-        SIGNED_UP: settings.DEFAULT_ATTENDANCE["Signed Up"],
+        GROUP_3 = 3
-        STANDBY: settings.DEFAULT_ATTENDANCE["Stand By"],
+        GROUP_4 = 4
-        CONFIRMED: settings.DEFAULT_ATTENDANCE["Confirmed"],
+
-    }
+    group = models.PositiveSmallIntegerField(
+        choices=Groups.choices,
+        default=Groups.GROUP_1
+    )
 
     attendance = models.DecimalField(
         max_digits=3,
@@ -102,7 +104,7 @@ class RaidResponse(models.Model):
     )
 
     class Meta:
-        ordering = ["-status", "role", "character__klass", "character__user__rank", "character__name"]
+        ordering = ["-status", "group", "role", "character__klass", "character__user__rank", "character__name"]
         constraints = [
             models.UniqueConstraint(fields=["raid", "character"], name="unique_character_raid_signup")
         ]
@@ -113,14 +115,14 @@ class RaidResponse(models.Model):
 
     def clean(self):
         # Make sure no-responses and sign-offs are role-agnostic, but all other responses are not
-        if self.status <= RaidResponse.SIGNED_OFF:
+        if self.status <= RaidResponse.Status.SIGNED_OFF:
             self.role = None
         elif self.role is None:
             raise ValidationError({"role": "This field is required."})
 
         # Set attendance to one of the default values if status was changed
         if self.status != self._original_status:
-            self.attendance = RaidResponse.STATUS_DEFAULT_ATTENDANCE[self.status]
+            self.attendance = RaidResponse.Status(self.status).default_attendance
 
     def save(self, *args, **kwargs):
         super().save(*args, **kwargs)

drakul/raids/signals.py

@@ -11,7 +11,7 @@ User = get_user_model()
 @receiver(post_save, sender=Raid)
 def create_raid_no_responses(instance: Raid, **kwargs):
     # Delete all pre-existing no-responses, in case the deadline was changed
-    instance.responses.filter(status=RaidResponse.NO_RESPONSE).delete()
+    instance.responses.filter(status=RaidResponse.Status.NO_RESPONSE).delete()
     # Then create them (again)
     users = User.objects \
         .exclude(Q(date_joined__gt=instance.date) | Q(characters__raid_responses__raid=instance) | Q(is_active=False)) \
@@ -20,8 +20,8 @@ def create_raid_no_responses(instance: Raid, **kwargs):
         RaidResponse(
             raid=instance,
             character=user.main,
-            status=RaidResponse.NO_RESPONSE,
+            status=RaidResponse.Status.NO_RESPONSE,
-            attendance=RaidResponse.STATUS_DEFAULT_ATTENDANCE[RaidResponse.NO_RESPONSE]
+            attendance=RaidResponse.Status.NO_RESPONSE.default_attendance
         )
         for user in users
     )
@@ -30,7 +30,7 @@ def create_raid_no_responses(instance: Raid, **kwargs):
 @receiver(post_save, sender=User)
 def create_user_no_responses(instance: User, **kwargs):
     # Delete all pre-existing no-responses for this user, in case date_joined or main was changed
-    RaidResponse.objects.filter(character__user=instance, status=RaidResponse.NO_RESPONSE).delete()
+    RaidResponse.objects.filter(character__user=instance, status=RaidResponse.Status.NO_RESPONSE).delete()
     # Then create them (again)
     if not instance.is_active:
         return
@@ -38,8 +38,8 @@ def create_user_no_responses(instance: User, **kwargs):
         RaidResponse(
             raid=raid,
             character=instance.main,
-            status=RaidResponse.NO_RESPONSE,
+            status=RaidResponse.Status.NO_RESPONSE,
-            attendance=RaidResponse.STATUS_DEFAULT_ATTENDANCE[RaidResponse.NO_RESPONSE]
+            attendance=RaidResponse.Status.NO_RESPONSE.default_attendance
         )
         for raid in Raid.objects.exclude(
             Q(date__lt=instance.date_joined) | Q(responses__character__user=instance)

drakul/raids/templates/raids/raid_detail.html

@@ -54,44 +54,49 @@
 
 {% regroup raid.responses.all by get_status_display as status_responses_list %}
 {% for status, status_responses in status_responses_list %}
-<div class="card mb-2">
+    {% regroup status_responses by get_group_display as group_responses_list %}
-    <h6 class="card-header d-flex response-status-{{ status | slugify }}-bg">
+    {% for group, group_responses in group_responses_list %}
-        <span class="mr-auto">{{ status }} ({{ status_responses | length }})</span>
+    <div class="card mb-2">
-        <span type="button" class="badge badge-dark" data-toggle="modal" data-target="#exportModal">Export</span>
+        <h6 class="card-header d-flex response-status-{{ status | slugify }}-bg">
-    </h6>
+            <span class="mr-auto">
-    <div class="card-body mb-n4">
+                {{ status }}{% if group_responses_list|length > 1 %}: {{ group }}{% endif %} ({{ status_responses | length }})
-        {% regroup status_responses by get_role_display as role_responses_list %}
+            </span>
-        {% for role, role_responses in role_responses_list %}
+            <span type="button" class="badge badge-dark" data-toggle="modal" data-target="#exportModal">Export</span>
-        {% if role is not None %}
+        </h6>
-            <h6 class="card-title border-bottom pb-2">{{ role }} ({{ role_responses | length }})</h6>
+        <div class="card-body mb-n4">
-        {% endif %}
+            {% regroup group_responses by get_role_display as role_responses_list %}
-        <div class="d-flex flex-wrap">
+            {% for role, role_responses in role_responses_list %}
-            {% regroup role_responses by character.klass as class_responses_list %}
+            {% if role is not None %}
-            {% for class, class_responses in class_responses_list %}
+                <h6 class="card-title border-bottom pb-2">{{ role }} ({{ role_responses | length }})</h6>
-            <div class="d-flex flex-column mr-2 mb-4">
+            {% endif %}
-                {% regroup class_responses by character.user.rank as rank_responses_list %}
+            <div class="d-flex flex-wrap">
-                {% for rank, rank_responses in rank_responses_list %}
+                {% regroup role_responses by character.klass as class_responses_list %}
-                <div class="d-flex justify-content-center align-items-center">
+                {% for class, class_responses in class_responses_list %}
-                    <hr class="flex-grow-1 my-0">
+                <div class="d-flex flex-column mr-2 mb-4">
-                    <div class="mx-2 my-0 font-weight-light text-muted small">{{ rank }}</div>
+                    {% regroup class_responses by character.user.rank as rank_responses_list %}
-                    <hr class="flex-grow-1 my-0">
+                    {% for rank, rank_responses in rank_responses_list %}
-                </div>
+                    <div class="d-flex justify-content-center align-items-center">
-                    {% for response in rank_responses %}
+                        <hr class="flex-grow-1 my-0">
-                        <a class="btn response-character-button mb-1 btn-class-{{ response.character.klass }}" role="button" href="#">
+                        <div class="mx-2 my-0 font-weight-light text-muted small">{{ rank }}</div>
-                            {% if response.character != response.character.user.main %}
+                        <hr class="flex-grow-1 my-0">
-                                <abbr title="{{ response.character.user.main }}">{{ response.character.name }}</abbr>
+                    </div>
-                            {% else %}
+                        {% for response in rank_responses %}
-                                {{ response.character.name }}
+                            <a class="btn response-character-button mb-1 btn-class-{{ response.character.klass }}" role="button" href="#">
-                            {% endif %}
+                                {% if response.character != response.character.user.main %}
-                        </a>
+                                    <abbr title="{{ response.character.user.main }}">{{ response.character.name }}</abbr>
+                                {% else %}
+                                    {{ response.character.name }}
+                                {% endif %}
+                            </a>
+                        {% endfor %}
                     {% endfor %}
+                </div>
                 {% endfor %}
             </div>
             {% endfor %}
         </div>
-        {% endfor %}
     </div>
-</div>
+    {% endfor %}
 {% endfor %}
 
 <div id="comments" class="card mt-4 mb-4">
@@ -104,7 +109,7 @@
                         <strong>{{ comment.user.main.name }}</strong> <small class="text-muted">&middot; {{ comment.date_created }}</small>
                     </p>
                     <p class="card-text">
-                        {{ comment.body }}
+                        {{ comment.body | linebreaksbr }}
                     </p>
                 </div>
             </div>

drakul/raids/templates/raids/raid_response_table_inline_formset.html

@@ -44,17 +44,23 @@
 
             {% regroup formset by status.value as status_forms_list %}
             {% for status, status_forms in status_forms_list %}
-                {% regroup status_forms by role.value as role_forms_list %}
+                {% regroup status_forms by group.value as group_forms_list %}
-                {% for role, role_forms in role_forms_list %}
+                {% for group, group_forms in group_forms_list %}
-                    {% for form in role_forms %}
+                    {% regroup group_forms by role.value as role_forms_list %}
-                        {% if form_show_errors and not form.is_extra %}
+                    {% for role, role_forms in role_forms_list %}
-                            {% include "bootstrap4/errors.html" %}
+                        {% for form in role_forms %}
+                            {% if form_show_errors and not form.is_extra %}
+                                {% include "bootstrap4/errors.html" %}
+                            {% endif %}
+                            <tr>
+                                {% for field in form %}
+                                    {% include 'bootstrap4/field.html' with tag="td" form_show_labels=False %}
+                                {% endfor %}
+                            </tr>
+                        {% endfor %}
+                        {% if not forloop.last %}
+                            <tr><td class="p-1"></td></tr>
                         {% endif %}
-                        <tr>
-                            {% for field in form %}
-                                {% include 'bootstrap4/field.html' with tag="td" form_show_labels=False %}
-                            {% endfor %}
-                        </tr>
                     {% endfor %}
                     {% if not forloop.last %}
                         <tr><td class="p-2"></td></tr>

drakul/raids/views.py

@@ -4,7 +4,7 @@ from datetime import timedelta
 
 from django.contrib.auth import get_user_model
 from django.contrib.auth.mixins import PermissionRequiredMixin
-from django.db.models import Q, Max, prefetch_related_objects
+from django.db.models import Q, Max
 from django.http import Http404
 from django.urls import reverse, reverse_lazy
 from django.utils import timezone

drakul/users/models.py

@@ -93,41 +93,29 @@ class Character(models.Model):
         unique=True
     )
 
-    DRUID = 1
+    class Klasses(models.IntegerChoices):
-    HUNTER = 2
+        DRUID = 1
-    MAGE = 3
+        HUNTER = 2
-    PALADIN = 4
+        MAGE = 3
-    PRIEST = 5
+        PALADIN = 4
-    ROGUE = 6
+        PRIEST = 5
-    SHAMAN = 7
+        ROGUE = 6
-    WARLOCK = 8
+        SHAMAN = 7
-    WARRIOR = 9
+        WARLOCK = 8
-    CLASS_CHOICES = [
+        WARRIOR = 9
-        (DRUID, "Druid"),
+
-        (HUNTER, "Hunter"),
-        (MAGE, "Mage"),
-        (PALADIN, "Paladin"),
-        (PRIEST, "Priest"),
-        (ROGUE, "Rogue"),
-        (SHAMAN, "Shaman"),
-        (WARLOCK, "Warlock"),
-        (WARRIOR, "Warrior"),
-    ]
     klass = models.PositiveSmallIntegerField(
         "class",
-        choices=CLASS_CHOICES
+        choices=Klasses.choices
     )
 
-    TANK = 1
+    class Roles(models.IntegerChoices):
-    HEALER = 2
+        TANK = 1
-    DAMAGE = 3
+        HEALER = 2
-    ROLE_CHOICES = [
+        DAMAGE = 3
-        (TANK, "Tank"),
+
-        (HEALER, "Healer"),
-        (DAMAGE, "Damage"),
-    ]
     role = models.PositiveSmallIntegerField(
-        choices=ROLE_CHOICES
+        choices=Roles.choices
     )
 
     class Meta:

drakul/users/templatetags/users_extras.py

@@ -14,7 +14,7 @@ def attendance_cell(response: RaidResponse):
     if response is None:
         return ""
 
-    if response.attendance == RaidResponse.STATUS_DEFAULT_ATTENDANCE[response.status]:
+    if response.attendance == RaidResponse.Status(response.status).default_attendance:
         cell = ""
     else:
         cell = response.attendance