Fix handling of links with []
characters around them.
This commit is contained in:
parent
bd4e4f8cb4
commit
efadb7814a
@ -1306,10 +1306,12 @@ def tokenize_contents(contents: str):
|
|||||||
len(contents) > i + 3
|
len(contents) > i + 3
|
||||||
# At least 3 characters more to open and close a link
|
# At least 3 characters more to open and close a link
|
||||||
and contents[i + 1] == "["
|
and contents[i + 1] == "["
|
||||||
|
# TODO: Generalize this to a backtracking, don't just fix the test case...
|
||||||
|
and contents[i + 2] != "["
|
||||||
):
|
):
|
||||||
close = contents.find("]", i)
|
close = contents.find("]]", i)
|
||||||
|
|
||||||
if close != -1 and contents[close + 1] == "]":
|
if close != -1:
|
||||||
# Link with no description
|
# Link with no description
|
||||||
cut_string()
|
cut_string()
|
||||||
|
|
||||||
@ -1333,7 +1335,7 @@ def tokenize_contents(contents: str):
|
|||||||
continue
|
continue
|
||||||
|
|
||||||
# Possible link close or open of description
|
# Possible link close or open of description
|
||||||
if char == "]" and in_link:
|
if char == "]" and len(contents) > i + 1 and in_link:
|
||||||
if contents[i + 1] == "]":
|
if contents[i + 1] == "]":
|
||||||
cut_string()
|
cut_string()
|
||||||
|
|
||||||
@ -1343,19 +1345,13 @@ def tokenize_contents(contents: str):
|
|||||||
in_link_description = False
|
in_link_description = False
|
||||||
continue
|
continue
|
||||||
|
|
||||||
if contents[i + 1] == "[" and not in_link_description:
|
elif contents[i + 1] == "[":
|
||||||
cut_string()
|
cut_string()
|
||||||
|
|
||||||
tokens.append((TOKEN_TYPE_OPEN_DESCRIPTION, None))
|
tokens.append((TOKEN_TYPE_OPEN_DESCRIPTION, None))
|
||||||
assert "[" == (next(cursor)[1])
|
assert "[" == (next(cursor)[1])
|
||||||
continue
|
continue
|
||||||
|
|
||||||
raise Exception(
|
|
||||||
"Link cannot contain ']' not followed by '[' or ']'. Starting with {}".format(
|
|
||||||
contents[last_link_start : i + 10]
|
|
||||||
)
|
|
||||||
)
|
|
||||||
|
|
||||||
if in_link and not in_link_description:
|
if in_link and not in_link_description:
|
||||||
# Link's pointer have no formatting
|
# Link's pointer have no formatting
|
||||||
pass
|
pass
|
||||||
|
@ -218,7 +218,7 @@ class TestSerde(unittest.TestCase):
|
|||||||
self.assertEqual(links[4].description, "web link")
|
self.assertEqual(links[4].description, "web link")
|
||||||
|
|
||||||
self.assertEqual(links[5].value, "https://codigoparallevar.com/4")
|
self.assertEqual(links[5].value, "https://codigoparallevar.com/4")
|
||||||
self.assertEqual(links[5].description, "[tricky web link]")
|
self.assertEqual(links[5].description, "[tricky web link]\u200b")
|
||||||
|
|
||||||
self.assertEqual(links[6].value, "https://codigoparallevar.com/5")
|
self.assertEqual(links[6].value, "https://codigoparallevar.com/5")
|
||||||
self.assertEqual(links[6].description, "another tricky web link")
|
self.assertEqual(links[6].description, "another tricky web link")
|
||||||
@ -279,7 +279,7 @@ class TestSerde(unittest.TestCase):
|
|||||||
SPAN("\n"),
|
SPAN("\n"),
|
||||||
SPAN(
|
SPAN(
|
||||||
" This is a ",
|
" This is a ",
|
||||||
WEB_LINK("[tricky web link]", "https://codigoparallevar.com/4"),
|
WEB_LINK("[tricky web link]\u200b", "https://codigoparallevar.com/4"),
|
||||||
" followed up with some text.\n",
|
" followed up with some text.\n",
|
||||||
),
|
),
|
||||||
SPAN("\n"),
|
SPAN("\n"),
|
||||||
@ -327,7 +327,7 @@ class TestSerde(unittest.TestCase):
|
|||||||
links[4].description = "web link #3 with update"
|
links[4].description = "web link #3 with update"
|
||||||
|
|
||||||
self.assertEqual(links[5].value, "https://codigoparallevar.com/4")
|
self.assertEqual(links[5].value, "https://codigoparallevar.com/4")
|
||||||
self.assertEqual(links[5].description, "[tricky web link]")
|
self.assertEqual(links[5].description, "[tricky web link]\u200b")
|
||||||
links[5].value = "https://codigoparallevar.com/4-updated"
|
links[5].value = "https://codigoparallevar.com/4-updated"
|
||||||
links[5].description = "[tricky web link #4 with update]"
|
links[5].description = "[tricky web link #4 with update]"
|
||||||
|
|
||||||
|
Loading…
Reference in New Issue
Block a user