Don't attempt parsing code blocks that don't look like modules (#1662)

elixir-lang · Feb 16, 2023 · d00110f · d00110f
1 parent 486dce4
commit d00110f
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/lib/ex_doc/language/elixir.ex b/lib/ex_doc/language/elixir.ex
@@ -611,10 +611,14 @@ defmodule ExDoc.Language.Elixir do
   end
 
   defp parse_module(<<first>> <> _ = string, _mode) when first in ?A..?Z do
-    do_parse_module(string)
+    if string =~ ~r/^[A-Za-z0-9_.]+$/ do
+      do_parse_module(string)
+    else
+      :error
+    end
   end
 
-  defp parse_module(<<?:>> <> _ = string, :custom_link) do
+  defp parse_module(":" <> _ = string, :custom_link) do
     do_parse_module(string)
   end
 

diff --git a/test/ex_doc/language/elixir_test.exs b/test/ex_doc/language/elixir_test.exs
@@ -496,6 +496,8 @@ defmodule ExDoc.Language.ElixirTest do
     assert_unchanged(~m"`Unknown`")
 
     assert_unchanged(~m"[Blank](about:blank)")
+
+    assert_unchanged(~m"`FOR UPDATE OF ? SKIP LOCKED`")
   end
 
   ## Helpers