fix for non existing atom
authorAlexander Strizhakov <alex.strizhakov@gmail.com>
Fri, 24 Jan 2020 09:22:26 +0000 (12:22 +0300)
committerAlexander Strizhakov <alex.strizhakov@gmail.com>
Fri, 24 Jan 2020 09:22:26 +0000 (12:22 +0300)
lib/pleroma/config/config_db.ex
test/config/transfer_task_test.exs

index be66880951b57cefcfed87564f5c4b6f9abf5923..119251bee9b5f9eb83f100f8cddf17db09976621 100644 (file)
@@ -236,15 +236,7 @@ defmodule Pleroma.ConfigDB do
   end
 
   @spec from_string(String.t()) :: atom() | no_return()
-  def from_string(":" <> entity), do: String.to_existing_atom(entity)
-
-  def from_string(entity) when is_binary(entity) do
-    if is_module_name?(entity) do
-      String.to_existing_atom("Elixir.#{entity}")
-    else
-      entity
-    end
-  end
+  def from_string(string), do: do_transform_string(string)
 
   @spec convert(any()) :: any()
   def convert(entity), do: do_convert(entity)
index b9072e0fcaf2a33c5507986217d02c4d7508cc01..53e8703fd6f05193208192fdeba5ed8a899cbde5 100644 (file)
@@ -105,17 +105,4 @@ defmodule Pleroma.Config.TransferTaskTest do
       Application.put_env(:pleroma, :assets, assets)
     end)
   end
-
-  test "non existing atom" do
-    ConfigDB.create(%{
-      group: ":pleroma",
-      key: ":undefined_atom_key",
-      value: [live: 2, com: 3]
-    })
-
-    assert ExUnit.CaptureLog.capture_log(fn ->
-             TransferTask.start_link([])
-           end) =~
-             "updating env causes error, group: \":pleroma\" key: \":undefined_atom_key\" value: [live: 2, com: 3] error: %ArgumentError{message: \"argument error\"}"
-  end
 end