Fixes to support Django Channels 4.

fear_tracker/forms.py

@@ -6,7 +6,7 @@ from .models import Game, Player
 class PlayerForm(forms.Form):
     name = forms.CharField(max_length=80, label="Name", required=False)
     spirit = forms.TypedChoiceField(label="Spirit",
-                                    choices=enumerate(Player.SPIRIT_NAMES),
+                                    choices=Player.enumerate_spirit_names(),
                                     empty_value=None, coerce=int)
 
 

fear_tracker/models.py

@@ -247,18 +247,42 @@ class Player(models.Model):
     game = models.ForeignKey(Game, on_delete=models.CASCADE, db_index=True)
     name = models.CharField(max_length=80)
     SPIRIT_NAMES = [
-        "Lightning's Swift Strike",
-        "River Surges in Sunlight",
-        "Vital Strength of the Earth",
-        "Shadows Flicker Like Flame",
-        "Thunderspeaker",
-        "A Spread of Rampant Green",
-        "Ocean's Hungry Grasp",
-        "Bringer of Dreams and Nightmares",
-        "Keeper of the Forbidden Wilds",
-        "Sharp Fangs Behind the Leaves",
-        "Serpent Slumbering Beneath the Island",
-        "Heart of the Wildfire",
+        # From https://spiritislandwiki.com/index.php?title=List_of_Spirits
+        # Base game
+        ("Lightning's Swift Strike", "SI", "L"),
+        ("River Surges in Sunlight", "SI", "L"),
+        ("Vital Strength of the Earth", "SI", "L"),
+        ("Shadows Flicker Like Flame", "SI", "L"),
+        ("Thunderspeaker", "SI", "M"),
+        ("A Spread of Rampant Green", "SI", "M"),
+        ("Ocean's Hungry Grasp", "SI", "H"),
+        ("Bringer of Dreams and Nightmares", "SI", "H"),
+        # Branch and Claw
+        ("Keeper of the Forbidden Wilds", "BC", "M"),
+        ("Sharp Fangs Behind the Leaves", "BC", "M"),
+        # Promo Pack 1
+        ("Serpent Slumbering Beneath the Island", "P1", "H"),
+        ("Heart of the Wildfire", "P1", "H"),
+        # Jagged Earth
+        ("Stone's Unyielding Defiance", "JE", "M"),
+        ("Shifting Memory of Ages", "JE", "M"),
+        ("Grinning Trickster Stirs Up Trouble", "JE", "M"),
+        ("Lure of the Deep Wilderness", "JE", "M"),
+        ("Many Minds Move as One", "JE", "M"),
+        ("Volcano Looming High", "JE", "M"),
+        ("Shroud of Silent Mist", "JE", "H"),
+        ("Vengeance as a Burning Plague", "JE", "H"),
+        ("Starlight Seeks Its Form", "JE", "V"),
+        ("Fractured Days Split the Sky", "JE", "V"),
+        # Promo Pack 2
+        ("Downpour Drenches the World", "P2", "H"),
+        ("Finder of Paths Unseen", "P2", "V"),
+        # Horizons of Spirit Island
+        ("Devouring Teeth Lurk Underfoot", "HS", "L"),
+        ("Eyes Watch From the Trees", "HS", "L"),
+        ("Fathomless Mud of the Swamp", "HS", "L"),
+        ("Rising Heat of Stone and Sand", "HS", "L"),
+        ("Sun-Bright Whirlwind", "HS", "L"),
     ]
     spirit = models.IntegerField()
     order = models.IntegerField()
@@ -267,7 +291,12 @@ class Player(models.Model):
     unique_together = (("game", "name"), ("game", "order"))
 
     def get_spirit_name(self):
-        return Player.SPIRIT_NAMES[self.spirit]
+        return Player.SPIRIT_NAMES[self.spirit][0]
+
+    @staticmethod
+    def enumerate_spirit_names():
+        return [(i, f"[{spirit[1]},{spirit[2]}] {spirit[0]}") for (i, spirit)
+                in enumerate(Player.SPIRIT_NAMES)]
 
 
 class Phase(models.Model):

fear_tracker/routing.py

@@ -1,7 +1,12 @@
-from channels.http import AsgiHandler
 from channels.routing import ProtocolTypeRouter, URLRouter
 
+import django
 from django.conf.urls import url
+from django.core.asgi import get_asgi_application
+
+# TODO Importing views breaks if django.setup() isn't called first...
+#      ... but this isn't the way this is supposed to work.
+django.setup()
 
 from . import views
 
@@ -9,8 +14,8 @@ application = ProtocolTypeRouter({
     "http": URLRouter([
         url(r'^(?P<access_code>[a-zA-Z]{6})/', URLRouter([
             url(r"status/(?P<hashcode>[a-z0-9]{64})/",
-                views.StatusLongPollConsumer, name='status'),
+                views.StatusLongPollConsumer.as_asgi(), name='status'),
         ])),
-        url(r"", AsgiHandler),
+        url(r"", get_asgi_application()),
     ]),
 })

fear_tracker/views.py

@@ -81,7 +81,7 @@ def new_game(request):
         form = NewGameForm()
 
         initial_player_data = [{'spirit': i}
-                               for i in enumerate(Player.SPIRIT_NAMES)]
+                               for (i, _) in Player.enumerate_spirit_names()]
         formset = PlayerFormSet(initial=initial_player_data)
     for player_form in formset:
         player_form.player_id = int(player_form.prefix[5:]) + 1
@@ -262,7 +262,7 @@ def handle_game_request(request, game, update):
                                         f"#{effect_num} {effect_kind} " +
                                         "was changed by another user to " +
                                         f"{current_value}, so " +
-                                        f"your attempt to change it from " +
+                                        "your attempt to change it from " +
                                         f"{orig} to {amount} " +
                                         "was not processed.")
                 else:
@@ -377,7 +377,7 @@ class StatusLongPollConsumer(AsyncHttpConsumer):
 
     async def fear_tracker_hashcode_seen(self, event):
         if self.hashcode != event["hashcode"]:
-            if event["status_string"]:
+            if "status_string" in event and event["status_string"]:
                 body = event["status_string"].encode('utf-8')
                 await self.send_response(200, body)
                 await self.disconnect()

fear_tracker_site/asgi.py

@@ -3,10 +3,9 @@ ASGI entrypoint. Configures Django and then runs the application
 defined in the ASGI_APPLICATION setting.
 """
 
-import os
 import django
-from channels.routing import get_default_application
 
-os.environ.setdefault("DJANGO_SETTINGS_MODULE", "fear_tracker_site.settings")
+import fear_tracker.routing
+
 django.setup()
-application = get_default_application()
+application = fear_tracker.routing.application