pleroma

commit: c3b8c77967b0c42f93286f864236b7d6f1471c13
parent 861f1928526930eeb78f79c4840c69cee5c2f215
Author: Mark Felder <feld@feld.me>
Date:   Tue, 13 Apr 2021 14:25:15 -0500

Improve string_to_elixir_types/1 with guards

Diffstat:
M
lib/pleroma/config_db.ex
8
+++++---

1 file changed, 5 insertions(+), 3 deletions(-)
diff --git a/lib/pleroma/config_db.ex b/lib/pleroma/config_db.ex
@@ -327,7 +327,7 @@ defmodule Pleroma.ConfigDB do
 
   @spec string_to_elixir_types(String.t()) ::
           atom() | Regex.t() | module() | String.t() | no_return()
-  def string_to_elixir_types("~r" <> _pattern = regex) do
+  def string_to_elixir_types("~r" <> _pattern = regex) when is_binary(regex) do
     pattern =
       ~r/^~r(?'delimiter'[\/|"'([{<]{1})(?'pattern'.+)[\/|"')\]}>]{1}(?'modifier'[uismxfU]*)/u
 
@@ -341,9 +341,9 @@ defmodule Pleroma.ConfigDB do
     end
   end
 
-  def string_to_elixir_types(":" <> atom), do: String.to_atom(atom)
+  def string_to_elixir_types(":" <> atom) when is_binary(atom), do: String.to_atom(atom)
 
-  def string_to_elixir_types(value) do
+  def string_to_elixir_types(value) when is_binary(value) do
     if module_name?(value) do
       String.to_existing_atom("Elixir." <> value)
     else
@@ -351,6 +351,8 @@ defmodule Pleroma.ConfigDB do
     end
   end
 
+  def string_to_elixir_types(value) when is_atom(value), do: value
+
   defp parse_host("localhost"), do: :localhost
 
   defp parse_host(host) do