better handling of non-gendered source objects

evennia/contrib/game_systems/gendersub/gendersub.py

@@ -101,7 +101,7 @@ class GenderCharacter(DefaultCharacter):
         super().at_object_creation()
         self.db.gender = "ambiguous"
 
-    def _get_pronoun(self, regex_match):
+    def _get_pronoun(self, regex_match, source=None):
         """
         Get pronoun from the pronoun marker in the text. This is used as
         the callable for the re.sub function.
@@ -116,8 +116,10 @@ class GenderCharacter(DefaultCharacter):
             - `|a`, `|A`: Absolute Possessive form: his, hers, its, His, Hers, Its, Theirs
 
         """
+        if not source:
+            source = self
         typ = regex_match.group()[1]  # "s", "O" etc
-        gender = self.attributes.get("gender", default="ambiguous")
+        gender = source.attributes.get("gender", default="ambiguous")
         gender = gender if gender in ("male", "female", "neutral") else "ambiguous"
         pronoun = _GENDER_PRONOUN_MAP[gender][typ.lower()]
         return pronoun.capitalize() if typ.isupper() else pronoun
@@ -147,17 +149,19 @@ class GenderCharacter(DefaultCharacter):
             super().msg(from_obj=from_obj, session=session, **kwargs)
             return
 
-        gender_source = from_obj if from_obj else self
+        try:
-
+            if text and isinstance(text, tuple):
-        if hasattr(gender_source, "_get_pronoun"):
+                text = (
-            try:
+                    _RE_GENDER_PRONOUN.sub(
-                if text and isinstance(text, tuple):
+                        lambda x: self._get_pronoun(x, source=from_obj), text[0]
-                    text = (_RE_GENDER_PRONOUN.sub(gender_source._get_pronoun, text[0]), *text[1:])
+                    ),
-                else:
+                    *text[1:],
-                    text = _RE_GENDER_PRONOUN.sub(gender_source._get_pronoun, text)
+                )
-            except TypeError:
+            else:
-                pass
+                text = _RE_GENDER_PRONOUN.sub(lambda x: self._get_pronoun(x, source=from_obj), text)
-            except Exception as e:
+        except TypeError:
-                logger.log_trace(e)
+            pass
+        except Exception as e:
+            logger.log_trace(e)
 
         super().msg(text, from_obj=from_obj, session=session, **kwargs)

evennia/contrib/game_systems/gendersub/tests.py

@@ -54,3 +54,12 @@ class TestGenderSub(BaseEvenniaCommandTest):
             self.assertIn("Test his gender", mock_msg.call_args.args)
             masc.msg(txt, from_obj=fem)
             self.assertIn("Test her gender", mock_msg.call_args.args)
+
+    def test_ungendered_source(self):
+        char = create_object(gendersub.GenderCharacter, key="Gendered", location=self.room1)
+        txt = "Test |p gender"
+        with patch(
+            "evennia.contrib.game_systems.gendersub.gendersub.DefaultCharacter.msg"
+        ) as mock_msg:
+            char.msg(txt, from_obj=self.char1)
+            self.assertIn("Test their gender", mock_msg.call_args.args)