From: Scott Worley <scottworley@scottworley.com>
Date: Sat, 30 Dec 2023 12:19:00 +0000 (-0800)
Subject: Escape character names
X-Git-Url: http://git.scottworley.com/paperdoorknob/commitdiff_plain/37c47bc273c9c46d35be45ee7866ca5ca01f3159

Escape character names

This is slow. :(
---

diff --git a/glowfic.py b/glowfic.py
index 5730508..a700995 100644
--- a/glowfic.py
+++ b/glowfic.py
@@ -39,9 +39,9 @@ def flatURL(url: str) -> str:
 @dataclass(frozen=True)
 class Chunk:
     icon: str | None
-    character: str | None
-    screen_name: str | None
-    author: str | None
+    character: Tag | None
+    screen_name: Tag | None
+    author: Tag | None
     content: Tag
 
 # We avoid the name "post" because the Glowfic community uses the term
@@ -83,19 +83,18 @@ def makeChunk(chunk_dom: Tag, image_store: ImageStore) -> Chunk:
         assert isinstance(icon_img, Tag)
         return image_store.get_image(icon_img.attrs['src'])
 
-    def getTextByClass(css_class: str) -> str | None:
-        div = chunk_dom.find_next('div', class_=css_class)
-        if div is None:
-            return None
-        return div.text.strip()
+    def getByClass(css_class: str) -> Tag | None:
+        tag = chunk_dom.find_next('div', class_=css_class)
+        assert tag is None or isinstance(tag, Tag)
+        return tag
 
     content = chunk_dom.find_next('div', class_='post-content')
     assert isinstance(content, Tag)
 
     return Chunk(getIcon(),
-                 getTextByClass('post-character'),
-                 getTextByClass('post-screenname'),
-                 getTextByClass('post-author'),
+                 getByClass('post-character'),
+                 getByClass('post-screenname'),
+                 getByClass('post-author'),
                  content)
 
 
@@ -127,12 +126,11 @@ class BelowIconLayout(Layout):
         self._image_size = image_size
 
     def renderChunk(self, chunk: Chunk) -> bytes:
-        meta = [
-            renderIcon(chunk.icon, self._image_size),
-            chunk.character,
-            chunk.screen_name,
-            chunk.author,
-        ]
+        icon = renderIcon(chunk.icon, self._image_size)
+        meta = [icon.encode('UTF-8')] if icon else []
+        meta += [self._texifier.texify(x)
+                 for x in [chunk.character, chunk.screen_name, chunk.author]
+                 if x is not None]
 
         return b'''\\wrapstuffclear
 \\begin{wrapstuff}[l]
@@ -156,7 +154,7 @@ class BelowIconLayout(Layout):
 
 \\noindent %s
 ''' % (
-            b'\\\\*'.join(x.encode('UTF-8') for x in meta if x is not None),
+            b'\\\\*'.join(meta),
             self._texifier.texify(chunk.content))
 
 
@@ -191,5 +189,5 @@ class BesideIconLayout(Layout):
 
 ''' % (
             icon.encode('UTF-8') if icon else b'',
-            b'\\\\*'.join(x.encode('UTF-8') for x in meta if x is not None),
+            b'\\\\*'.join(self._texifier.texify(x) for x in meta if x is not None),
             self._texifier.texify(chunk.content))
diff --git a/glowfic_test.py b/glowfic_test.py
index d3a6d56..9b802c1 100644
--- a/glowfic_test.py
+++ b/glowfic_test.py
@@ -57,9 +57,13 @@ class TestMakeChunk(unittest.TestCase):
             c.icon,
             'stored:https://d1anwqy6ci9o1i.cloudfront.net/' +
             'users%2F366%2Ficons%2Fxqmypqvflgdy28aorw9ml_shock.png')
-        self.assertEqual(c.character, 'Keltham')
-        self.assertEqual(c.screen_name, 'lawful chaotic')
-        self.assertEqual(c.author, 'Iarwain')
+        assert c.character
+        assert c.screen_name
+        assert c.author
+        self.assertEqual(list(c.character.stripped_strings), ['Keltham'])
+        self.assertEqual(
+            list(c.screen_name.stripped_strings), ['lawful chaotic'])
+        self.assertEqual(list(c.author.stripped_strings), ['Iarwain'])
         self.assertEqual(str(c.content),
                          '<div class="post-content"><p></p></div>')