Run black on sources

.github/workflows/mysql_settings.py

@@ -54,7 +54,7 @@ DATABASES = {
             "NAME": "evennia",
             "OPTIONS": {
                 "charset": "utf8mb4",
-                'init_command': 'set collation_connection=utf8mb4_unicode_ci'
+                "init_command": "set collation_connection=utf8mb4_unicode_ci",
             },
         },
     }

evennia/commands/default/building.py

@@ -3077,6 +3077,7 @@ class CmdScript(COMMAND_DEFAULT_CLASS):
             elif not self.switches:
                 # view all scripts
                 from evennia.commands.default.system import ScriptEvMore
+
                 ScriptEvMore(self.caller, scripts.order_by("id"), session=self.session)
                 return
             elif "start" in self.switches:

evennia/commands/default/comms.py

@@ -40,7 +40,6 @@ __all__ = (
     "CmdIRCStatus",
     "CmdRSS2Chan",
     "CmdGrapevine2Chan",
-
 )
 _DEFAULT_WIDTH = settings.CLIENT_DEFAULT_WIDTH
 

evennia/commands/default/system.py

@@ -439,7 +439,7 @@ class ScriptEvMore(EvMore):
             "|wdesc|n",
             align="r",
             border="tablecols",
-            width=self.width
+            width=self.width,
         )
 
         for script in scripts:
@@ -570,7 +570,7 @@ class CmdScripts(COMMAND_DEFAULT_CLASS):
             caller.msg(string)
         else:
             # No stopping or validation. We just want to view things.
-            ScriptEvMore(caller, scripts.order_by('id'), session=self.session)
+            ScriptEvMore(caller, scripts.order_by("id"), session=self.session)
 
 
 class CmdObjects(COMMAND_DEFAULT_CLASS):

evennia/commands/default/tests.py

@@ -1239,7 +1239,6 @@ class TestBuilding(CommandTest):
         )
 
     def test_spawn(self):
-
         def get_object(commandTest, obj_key):
             # A helper function to get a spawned object and
             # check that it exists in the process.

evennia/prototypes/prototypes.py

@@ -167,8 +167,9 @@ for mod in settings.PROTOTYPE_MODULES:
                     if "prototype_locks" in prot
                     else "use:all();edit:false()"
                 ),
-                "prototype_tags": list(set(list(
+                "prototype_tags": list(
-                    make_iter(prot.get("prototype_tags", []))) + ["module"])),
+                    set(list(make_iter(prot.get("prototype_tags", []))) + ["module"])
+                ),
             }
         )
         _MODULE_PROTOTYPES[actual_prot_key] = prot
@@ -392,35 +393,23 @@ def search_prototype(key=None, tags=None, require_single=False, return_iterators
         # exact match on tag(s)
         tags = make_iter(tags)
         tag_categories = ["db_prototype" for _ in tags]
-        db_matches = DbPrototype.objects.get_by_tag(
+        db_matches = DbPrototype.objects.get_by_tag(tags, tag_categories)
-            tags, tag_categories)
     else:
         db_matches = DbPrototype.objects.all()
 
     if key:
         # exact or partial match on key
-        exact_match = (
+        exact_match = db_matches.filter(Q(db_key__iexact=key)).order_by("db_key")
-            db_matches
-            .filter(
-                Q(db_key__iexact=key))
-            .order_by("db_key")
-        )
         if not exact_match:
             # try with partial match instead
-            db_matches = (
+            db_matches = db_matches.filter(Q(db_key__icontains=key)).order_by("db_key")
-                db_matches
-                .filter(
-                    Q(db_key__icontains=key))
-                .order_by("db_key")
-            )
         else:
             db_matches = exact_match
 
     # convert to prototype
     db_ids = db_matches.values_list("id", flat=True)
     db_matches = (
-        Attribute.objects
+        Attribute.objects.filter(scriptdb__pk__in=db_ids, db_key="prototype")
-        .filter(scriptdb__pk__in=db_ids, db_key="prototype")
         .values_list("db_value", flat=True)
         .order_by("scriptdb__db_key")
     )
@@ -517,7 +506,7 @@ class PrototypeEvMore(EvMore):
             "|wDesc|n",
             border="tablecols",
             crop=True,
-            width=self.width
+            width=self.width,
         )
 
         for prototype in page:
@@ -554,8 +543,9 @@ class PrototypeEvMore(EvMore):
         return str(table)
 
 
-def list_prototypes(caller, key=None, tags=None, show_non_use=False,
+def list_prototypes(
-                    show_non_edit=True, session=None):
+    caller, key=None, tags=None, show_non_use=False, show_non_edit=True, session=None
+):
     """
     Collate a list of found prototypes based on search criteria and access.
 
@@ -581,10 +571,14 @@ def list_prototypes(caller, key=None, tags=None, show_non_use=False,
         return None
 
     # get specific prototype (one value or exception)
-    return PrototypeEvMore(caller, (dbprot_query, modprot_list),
+    return PrototypeEvMore(
+        caller,
+        (dbprot_query, modprot_list),
         session=session,
         show_non_use=show_non_use,
-                           show_non_edit=show_non_edit)
+        show_non_edit=show_non_edit,
+    )
+
 
 def validate_prototype(
     prototype, protkey=None, protparents=None, is_prototype_base=True, strict=True, _flags=None

evennia/prototypes/tests.py

@@ -630,10 +630,8 @@ class TestPrototypeStorage(EvenniaTest):
 
         # partial match
         with mock.patch("evennia.prototypes.prototypes._MODULE_PROTOTYPES", {}):
-            self.assertCountEqual(
+            self.assertCountEqual(protlib.search_prototype("prot"), [prot1b, prot2, prot3])
-                protlib.search_prototype("prot"), [prot1b, prot2, prot3])
+            self.assertCountEqual(protlib.search_prototype(tags="foo1"), [prot1b, prot2, prot3])
-            self.assertCountEqual(
-                protlib.search_prototype(tags="foo1"), [prot1b, prot2, prot3])
 
         self.assertTrue(str(str(protlib.list_prototypes(self.char1))))
 
@@ -1078,6 +1076,7 @@ class TestOLCMenu(TestEvMenu):
         ],
     ]
 
+
 class PrototypeCrashTest(EvenniaTest):
 
     # increase this to 1000 for optimization testing
@@ -1089,9 +1088,9 @@ class PrototypeCrashTest(EvenniaTest):
         # print(f"Creating {num} additional prototypes...")
         for x in range(num):
             prot = {
-                'prototype_key': str(uuid.uuid4()),
+                "prototype_key": str(uuid.uuid4()),
-                'some_attributes': [str(uuid.uuid4()) for x in range(10)],
+                "some_attributes": [str(uuid.uuid4()) for x in range(10)],
-                'prototype_tags': list(sample(['demo', 'test', 'stuff'], 2)),
+                "prototype_tags": list(sample(["demo", "test", "stuff"], 2)),
             }
             protlib.save_prototype(prot)
 
@@ -1101,5 +1100,5 @@ class PrototypeCrashTest(EvenniaTest):
             self.create(num_prototypes)
             # print("Attempting to list prototypes...")
             # start_time = time()
-            self.char1.execute_cmd('spawn/list')
+            self.char1.execute_cmd("spawn/list")
             # print(f"Prototypes listed in {time()-start_time} seconds.")

evennia/scripts/tickerhandler.py

@@ -344,8 +344,10 @@ class TickerHandler(object):
             raise TypeError(f"{callback} is not a callable function or method.")
 
         if outobj and not inherits_from(outobj, "evennia.typeclasses.models.TypedObject"):
-            raise TypeError(f"{callback} is a method on a normal object - it must "
+            raise TypeError(
-                            "be either a method on a typeclass, or a stand-alone function.")
+                f"{callback} is a method on a normal object - it must "
+                "be either a method on a typeclass, or a stand-alone function."
+            )
 
         return outobj, outpath, outcallfunc
 

evennia/server/connection_wizard.py

@@ -176,17 +176,14 @@ def node_start(wizard):
             node_game_index_start,
             {},
         ),
-        "2": ("MSSP setup (for mud-list crawlers)",
+        "2": ("MSSP setup (for mud-list crawlers)", node_mssp_start, {}),
-              node_mssp_start, {}
-              ),
         # "3": ("Add Grapevine listing",
         #       node_grapevine_start, {}),
         # "4": ("Add IRC link",
         #      "node_irc_start", {}),
         # "5" ("Add RSS feed",
         #       "node_rss_start", {}),
-        "s": ("View and (optionally) Save created settings",
+        "s": ("View and (optionally) Save created settings", node_view_and_apply_settings, {}),
-              node_view_and_apply_settings, {}),
         "q": ("Quit", lambda *args: sys.exit(), {}),
     }
 
@@ -263,7 +260,7 @@ def node_game_index_fields(wizard, status=None):
         return True
 
     wizard.display(text)
-    wizard.game_index_listing['game_name'] = wizard.ask_input(
+    wizard.game_index_listing["game_name"] = wizard.ask_input(
         default=name_default, validator=name_validator
     )
 
@@ -494,8 +491,9 @@ def node_view_and_apply_settings(wizard):
 
     # game index
     game_index_save_text = ""
-    game_index_listing = (wizard.game_index_listing if
+    game_index_listing = (
-                          hasattr(wizard, "game_index_listing") else None)
+        wizard.game_index_listing if hasattr(wizard, "game_index_listing") else None
+    )
     if not game_index_listing and settings.GAME_INDEX_ENABLED:
         game_index_listing = settings.GAME_INDEX_LISTING
     if game_index_listing:

evennia/server/evennia_launcher.py

@@ -1278,8 +1278,11 @@ def check_main_evennia_dependencies():
         # only the main version (1.5, not 1.5.4.0)
         dversion_main = ".".join(dversion.split(".")[:2])
         if LooseVersion(dversion) < LooseVersion(DJANGO_MIN):
-            print(ERROR_DJANGO_MIN.format(dversion=dversion_main, django_min=DJANGO_MIN,
+            print(
-                                          django_lt=DJANGO_LT))
+                ERROR_DJANGO_MIN.format(
+                    dversion=dversion_main, django_min=DJANGO_MIN, django_lt=DJANGO_LT
+                )
+            )
             error = True
         elif LooseVersion(DJANGO_LT) <= LooseVersion(dversion_main):
             print(NOTE_DJANGO_NEW.format(dversion=dversion_main, django_rec=DJANGO_LT))

evennia/server/portal/telnet.py

@@ -85,6 +85,7 @@ class TelnetProtocol(Telnet, StatefulTelnetProtocol, Session):
             super().dataReceived(data)
         except ValueError as err:
             from evennia.utils import logger
+
             logger.log_err(f"Malformed telnet input: {err}")
 
     def connectionMade(self):

evennia/server/portal/webclient.py

@@ -42,6 +42,7 @@ GOING_AWAY = WebSocketServerProtocol.CLOSE_STATUS_CODE_GOING_AWAY
 
 STATE_CLOSING = WebSocketServerProtocol.STATE_CLOSING
 
+
 class WebSocketClient(WebSocketServerProtocol, Session):
     """
     Implements the server-side of the Websocket connection.

evennia/utils/evmore.py

@@ -331,7 +331,6 @@ class EvMore(object):
             # goto top of the text
             self.page_top()
 
-
     # default paginators - responsible for extracting a specific page number
 
     def paginator_index(self, pageno):

evennia/utils/utils.py

@@ -1918,9 +1918,11 @@ def at_search_result(matches, caller, query="", quiet=False, **kwargs):
             # we need to consider Commands, where .aliases is a list
             aliases = result.aliases.all() if hasattr(result.aliases, "all") else result.aliases
             # remove any pluralization aliases
-            aliases = [alias for alias in aliases if
+            aliases = [
-                       hasattr(alias, "category")
+                alias
-                       and alias.category not in ("plural_key", )]
+                for alias in aliases
+                if hasattr(alias, "category") and alias.category not in ("plural_key",)
+            ]
             error += _MULTIMATCH_TEMPLATE.format(
                 number=num + 1,
                 name=result.get_display_name(caller)