merge fix
authorAlexander Strizhakov <alex.strizhakov@gmail.com>
Mon, 20 Jan 2020 15:25:00 +0000 (18:25 +0300)
committerAlexander Strizhakov <alex.strizhakov@gmail.com>
Mon, 20 Jan 2020 15:25:00 +0000 (18:25 +0300)
lib/pleroma/config/config_db.ex
lib/pleroma/web/admin_api/admin_api_controller.ex

index 85a049f8a00d8907d20f1ffe3d00eae2a9f0b534..0091f7a1bff2abef2af459183db07bb9ec7fed59 100644 (file)
@@ -102,7 +102,7 @@ defmodule Pleroma.ConfigDB do
 
     subkeys = sub_key_full_update(group, key, intersect_keys)
 
-    merged_value = DeepMerge.deep_merge(old_value, new_value)
+    merged_value = ConfigDB.merge(old_value, new_value)
 
     Enum.reduce(subkeys, merged_value, fn subkey, acc ->
       Keyword.put(acc, subkey, new_value[subkey])
@@ -127,6 +127,24 @@ defmodule Pleroma.ConfigDB do
     |> List.flatten()
   end
 
+  def merge(config1, config2) when is_list(config1) and is_list(config2) do
+    Keyword.merge(config1, config2, fn _, app1, app2 ->
+      if Keyword.keyword?(app1) and Keyword.keyword?(app2) do
+        Keyword.merge(app1, app2, &deep_merge/3)
+      else
+        app2
+      end
+    end)
+  end
+
+  defp deep_merge(_key, value1, value2) do
+    if Keyword.keyword?(value1) and Keyword.keyword?(value2) do
+      Keyword.merge(value1, value2, &deep_merge/3)
+    else
+      value2
+    end
+  end
+
   @full_key_update [
     {:pleroma, :ecto_repos},
     {:quack, :meta},
index cc658dc0e334f83789239f48434c7a11b37f16e2..30343a9f1e9732b8b8fe0dfe3b1760ccae3fd04c 100644 (file)
@@ -826,7 +826,7 @@ defmodule Pleroma.Web.AdminAPI.AdminAPIController do
       else
         merged =
           Pleroma.Config.Holder.config()
-          |> DeepMerge.deep_merge(configs)
+          |> ConfigDB.merge(configs)
           |> Enum.map(fn {group, values} ->
             Enum.map(values, fn {key, value} ->
               db =