From 199ed66ddbc9e34788cdf1a9f7d06e1ad91c2da6 Mon Sep 17 00:00:00 2001
From: Gabriel Augendre <gabriel@augendre.info>
Date: Sun, 6 Nov 2022 15:19:21 +0100
Subject: [PATCH] Allow inviting characters to group

---
 .../characters_list/character_card.html       |  3 +--
 src/party/forms.py                            | 16 ++++++++++++-
 .../0002_party_invited_characters.py          | 24 +++++++++++++++++++
 src/party/migrations/max_migration.txt        |  2 +-
 src/party/models.py                           |  6 +++++
 src/party/templates/party/party_details.html  | 13 ++++++++++
 6 files changed, 60 insertions(+), 4 deletions(-)
 create mode 100644 src/party/migrations/0002_party_invited_characters.py

diff --git a/src/character/templates/character/snippets/characters_list/character_card.html b/src/character/templates/character/snippets/characters_list/character_card.html
index 73df085..6ec21bd 100644
--- a/src/character/templates/character/snippets/characters_list/character_card.html
+++ b/src/character/templates/character/snippets/characters_list/character_card.html
@@ -29,12 +29,11 @@
                             <i class="fa-solid fa-user"></i> Jouer
                         </a>
                     {% else %}
-
                         <a href="{% url "character:view" pk=character.pk %}" class="btn btn-primary">
                             <i class="fa-solid fa-cog"></i> Gérer
                         </a>
                     {% endif %}
-                    {% if party %}
+                    {% if party and character in party.characters.all %}
                         <a href="{% url "party:leave" pk=party.pk character_pk=character.pk %}" class="btn btn-warning">
                             <i class="fa-solid fa-person-walking-arrow-right"></i> Quitter le groupe
                         </a>
diff --git a/src/party/forms.py b/src/party/forms.py
index 9970d96..134fd7a 100644
--- a/src/party/forms.py
+++ b/src/party/forms.py
@@ -1,4 +1,5 @@
 from django import forms
+from django.core.exceptions import ValidationError
 
 from party.models import Party
 
@@ -6,4 +7,17 @@ from party.models import Party
 class PartyForm(forms.ModelForm):
     class Meta:
         model = Party
-        fields = ["name", "characters"]
+        fields = ["name", "invited_characters"]
+
+    def clean_invited_characters(self):
+        invited = self.cleaned_data["invited_characters"]
+        if not self.instance:
+            return invited
+        members = self.instance.characters.all()
+        for character in invited:
+            if character in members:
+                self.add_error(
+                    "invited_characters",
+                    ValidationError(f"{character} is already a group member."),
+                )
+        return invited
diff --git a/src/party/migrations/0002_party_invited_characters.py b/src/party/migrations/0002_party_invited_characters.py
new file mode 100644
index 0000000..cdd3b5b
--- /dev/null
+++ b/src/party/migrations/0002_party_invited_characters.py
@@ -0,0 +1,24 @@
+# Generated by Django 4.1.2 on 2022-11-06 14:10
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ("character", "0036_delete_party"),
+        ("party", "0001_initial"),
+    ]
+
+    operations = [
+        migrations.AddField(
+            model_name="party",
+            name="invited_characters",
+            field=models.ManyToManyField(
+                blank=True,
+                related_name="invites",
+                to="character.character",
+                verbose_name="personnages invités",
+            ),
+        ),
+    ]
diff --git a/src/party/migrations/max_migration.txt b/src/party/migrations/max_migration.txt
index cbab66d..9b5eb09 100644
--- a/src/party/migrations/max_migration.txt
+++ b/src/party/migrations/max_migration.txt
@@ -1 +1 @@
-0001_initial
+0002_party_invited_characters
diff --git a/src/party/models.py b/src/party/models.py
index ff072e2..ce6b898 100644
--- a/src/party/models.py
+++ b/src/party/models.py
@@ -31,6 +31,12 @@ class Party(UniquelyNamedModel, TimeStampedModel, models.Model):
         related_name="parties",
         verbose_name="meneur de jeu",
     )
+    invited_characters = models.ManyToManyField(
+        "character.Character",
+        blank=True,
+        related_name="invites",
+        verbose_name="personnages invités",
+    )
     characters = models.ManyToManyField(
         "character.Character",
         blank=True,
diff --git a/src/party/templates/party/party_details.html b/src/party/templates/party/party_details.html
index f734681..8ea6dad 100644
--- a/src/party/templates/party/party_details.html
+++ b/src/party/templates/party/party_details.html
@@ -7,9 +7,22 @@
 {% block content %}
     <h1>{{ party.name }}</h1>
     <p>MJ : {{ party.game_master.get_full_name|default:party.game_master.username }}</p>
+    <h2>Personnages</h2>
     <div class="row row-cols-1 row-cols-md-2 row-cols-lg-3 row-cols-xl-4 g-4">
         {% for character in party.characters.all %}
             {% include "character/snippets/characters_list/character_card.html" %}
+        {% empty %}
+            <p>Aucun...</p>
         {% endfor %}
     </div>
+    {% with party.invited_characters.all as invites %}
+        {% if invites %}
+            <h2>Invités</h2>
+            <div class="row row-cols-1 row-cols-md-2 row-cols-lg-3 row-cols-xl-4 g-4">
+                {% for character in invites %}
+                    {% include "character/snippets/characters_list/character_card.html" %}
+                {% endfor %}
+            </div>
+        {% endif %}
+    {% endwith %}
 {% endblock %}