mirrors/bookwyrm
1
0
Fork 1
mirror of https://github.com/bookwyrm-social/bookwyrm.git synced 2024-05-02 00:29:04 +00:00
Code Issues Projects Releases Wiki Activity

Merge pull request #3353 from dato/fix_quotation_str_pagenum

Browse source 
Fix creation of quotations with no end position
This commit is contained in:
Bart Schuurmans 2024-04-15 13:11:55 +02:00 committed by GitHub
parent 21a39f8170 4304cd4a79
commit 70f803a1f6
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
2 changed files with 13 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
bookwyrm/models/status.py
View file

@ -392,10 +392,10 @@ class Quotation(BookStatus):
def _format_position(self) -> Optional[str]:
"""serialize page position"""
beg = self.position
end = self.endposition or 0
end = self.endposition
if self.position_mode != "PG" or not beg:
return None
return f"pp. {beg}-{end}" if end > beg else f"p. {beg}"
return f"pp. {beg}-{end}" if end else f"p. {beg}"
@property
def pure_content(self):

17
bookwyrm/tests/models/test_status_model.py
View file

@ -340,8 +340,11 @@ class Status(TestCase):
def test_quotation_page_serialization(self, *_):
"""serialization of quotation page position"""
tests = [
("single pos", 7, None, "p. 7"),
("page range", 7, 10, "pp. 7-10"),
("single pos", "7", "", "p. 7"),
("missing beg", "", "10", None),
("page range", "7", "10", "pp. 7-10"),
("page range roman", "xv", "xvi", "pp. xv-xvi"),
("page range reverse", "14", "10", "pp. 14-10"),
]
for desc, beg, end, pages in tests:
with self.subTest(desc):
@ -355,10 +358,12 @@ class Status(TestCase):
position_mode="PG",
)
activity = status.to_activity(pure=True)
self.assertRegex(
activity["content"],
f'^<p>"my quote"</p> <p>— <a .+</a>, {pages}</p>$',
)
if pages:
pages_re = re.escape(pages)
expect_re = f'^<p>"my quote"</p> <p>— <a .+</a>, {pages_re}</p>$'
else:
expect_re = '^<p>"my quote"</p> <p>— <a .+</a></p>$'
self.assertRegex(activity["content"], expect_re)
def test_review_to_activity(self, *_):
"""subclass of the base model version with a "pure" serializer"""