Added name symbol validation.

cogs/slash_commands.py

@@ -50,7 +50,7 @@ class SlashCommands(commands.Cog):
         print("a")
 
     @app_commands.command(
-        name="domt5-pingme",
+        name="dom5-pingme",
         description="Signs you up to be pinged for a game. Run the command again to not get pinged anymore.",
     )
     async def pingme(self, interaction: discord.Interaction, name: str):

servermanager.py

@@ -58,7 +58,7 @@ def server_command_builder(
         tuple[int, int]
     ] = [],  # list of nations that will be ai tuple format: NationID, AI Level (1-6), 0 being closed
     client_start: bool = True,
-    teams: list[tuple[int, int, int]] = [],
+    teams: list[tuple[int, int, int]] = [],  # list of teams with format nation id,
     clustered_start: bool = False,
     team_game: bool = False,
     mapfile: str = "",
@@ -100,7 +100,7 @@ def server_command_builder(
     if not (0 < era < 4):
         return "ERROR_ERA"
 
-    ai_indexes = [t[1] for t in ai_slots]
+    ai_indexes = [t[0] for t in ai_slots]
     if not (len(ai_indexes) == len(set(ai_indexes))):
         return "ERROR_AIS"
 

webui.py

@@ -1,6 +1,6 @@
 from nicegui import ui
-from pandas.core.internals.blocks import external_values
 from bot_instance import bot
+import re
 from dom5game import Dom5game
 import servermanager
 import os
@@ -16,8 +16,8 @@ def create_ui():
 
     pages.add("/", lambda: main_page(rows))
     pages.add("/create", lambda: creator_page())
-
+    # ui.dark_mode().enable()
-    ui.run(reload=False)
+    ui.run(reload=False, show=False)
 
 
 def main_page(rows):
@@ -44,7 +44,10 @@ def creator_page():
         validation={
             "Game with that name already exists": lambda value: not (
                 os.path.isdir("games/" + value)
-            )
+            ),
+            "Name contains an invalid symbol": lambda value: bool(
+                re.match("^[A-Za-z0-9_-]*$", value)
+            ),
         },
     )
     # Port
@@ -90,6 +93,7 @@ def creator_page():
         hwrap = ui.checkbox("East West Wrapping")
     description_container = ui.row()
     toggle_container = ui.row()
+    # TODO: Are these containers still needed?
     with toggle_container:
         event_rarity = ui.toggle({1: "Common", 2: "Rare"}, value=1)
         random_map = ui.toggle([10, 15, 20], value=15)
@@ -97,6 +101,42 @@ def creator_page():
         ui.label("Event Rarity")
         # TODO: Fix Padding
         ui.label("Provinces per player:")
+    with ui.row():
+        ui.label("Gold Multiplier:").classes("w-36")
+        richness = ui.input(
+            validation={
+                "Value must be between 50 and 300": lambda value: value.isdigit()
+                and 50 <= int(value) <= 300,
+            },
+            value="100",
+        )
+    with ui.row():
+        ui.label("Resource Multiplier:").classes("w-36")
+        resources = ui.input(
+            validation={
+                "Value must be between 50 and 300": lambda value: value.isdigit()
+                and 50 <= int(value) <= 300,
+            },
+            value="100",
+        )
+    with ui.row():
+        ui.label("Recruitment Multiplier:").classes("w-36")
+        recruitment = ui.input(
+            validation={
+                "Value must be between 50 and 300": lambda value: value.isdigit()
+                and 50 <= int(value) <= 300,
+            },
+            value="100",
+        )
+    with ui.row():
+        ui.label("Supplies Multiplier:").classes("w-36")
+        supplies = ui.input(
+            validation={
+                "Value must be between 50 and 300": lambda value: value.isdigit()
+                and 50 <= int(value) <= 300,
+            },
+            value="100",
+        )
 
     # TODO: REMOVE THIS it is just for testing
     ui.button(
@@ -107,7 +147,23 @@ def creator_page():
                 port=int(port.value),
                 era=int(era.value),
                 ai_slots=ai_slots,
-                random_map=15,
+                random_map=int(random_map.value),
+                event_rarity=int(event_rarity.value),
+                hwrap=hwrap.value,
+                vwrap=vwrap.value,
+                no_new_ai=no_new_ai.value,
+                no_artifact_rest=no_artifact_rest.value,
+                no_cheat_det=no_cheat_det.value,
+                renaming=renaming.value,
+                score_graphs=score_graphs.value,
+                random_start_research=random_start_research.value,
+                clustered_start=clustered_start.value,
+                team_game=team_game.value,
+                no_nation_info=no_nation_info.value,
+                recruitment=int(recruitment.value),
+                resources=int(resources.value),
+                richness=int(richness.value),
+                supplies=int(supplies.value),
             )
         ),
     )
@@ -131,29 +187,34 @@ def create_ai_table(era, table_container, ai_slots):
         with ui.card().classes("w-full"):
             with ui.row().classes("font-bold w-full"):
                 ui.label("ID").classes("w-16")
-                ui.label("Nation").classes("w-40")
+                ui.label("Nation").classes("w-104")
-                ui.label("Title").classes("w-64")
                 ui.label("AI Level")
         ui.separator()
+        with ui.row().classes("items-center, w-full"):
+            ui.label("All").classes("w-124")
+            master_select = ui.select(
+                AI_LEVELS,
+                value="Human",
+            )
+
         for nation_id, nation in nations[str(era)].items():
             nation_name, title = nation["name"]
 
             with ui.row().classes("items-center w-full"):
                 ui.label(str(nation_id)).classes("w-16")
-                ui.label(nation_name).classes("w-40")
+                ui.label(nation_name + ", " + title).classes("w-104")
-                ui.label(title).classes("w-64")
-                # TODO: Set all button
                 # TODO: Random nations?
+                # Although that logic should probably go into servermanager.py
                 ui.select(
                     AI_LEVELS,
                     value="Human",
                     on_change=lambda e, nid=nation_id: set_ai_level(
                         e.value, nid, ai_slots
                     ),
-                )
+                ).bind_value_from(master_select)
 
 
-def set_ai_level(value, nation_id, ai_slots):
+def set_ai_level(value, nid, ai_slots):
     level_map = {
         "Closed": 0,
         "Easy": 1,
@@ -163,12 +224,10 @@ def set_ai_level(value, nation_id, ai_slots):
         "Master": 5,
         "Impossible": 6,
     }
-    ai_slots[:] = [
+    for ai in ai_slots:
-        (nation_id, ai_level)
+        if ai[0] == nid:
-        for nation_id, ai_level in ai_slots
+            ai_slots.remove(ai)
-        if nation_id != nation_id
-    ]
     if value == "Human":
         return
 
-    ai_slots.append((nation_id, level_map[value]))
+    ai_slots.append((nid, level_map[value]))