#831

Author: sumneko

changelog.md

@@ -5,6 +5,7 @@
 * `FIX` [#825](https://github.com./sumneko/lua-language-server/issues/825)
 * `FIX` [#826](https://github.com./sumneko/lua-language-server/issues/826)
 * `FIX` [#827](https://github.com./sumneko/lua-language-server/issues/827)
+* `FIX` [#831](https://github.com./sumneko/lua-language-server/issues/831)
 * `FIX` [#837](https://github.com./sumneko/lua-language-server/issues/837)
 * `FIX` runtime errors
 

script/config/config.lua

@@ -320,13 +320,16 @@ function m.dump()
     return dump
 end
 
-function m.update(new)
+function m.update(new, null)
     local function expand(t, left)
         for key, value in pairs(t) do
             local fullKey = key
             if left then
                 fullKey = left .. '.' .. key
             end
+            if value == null then
+                value = nil
+            end
             if Template[fullKey] then
                 m.set(fullKey, value)
             elseif Template['Lua.' .. fullKey] then

script/provider/provider.lua

@@ -17,46 +17,29 @@ local converter  = require 'proto.converter'
 local filewatch  = require 'filewatch'
 local json       = require 'json'
 
-local function mergeConfig(a, b)
-    for k, v in pairs(b) do
-        if a[k] == nil then
-            a[k] = v
-        end
-    end
-end
-
 ---@async
 local function updateConfig()
-    local merged = {}
-
-    local cfg = cfgLoader.loadLocalConfig(CONFIGPATH)
-    if cfg then
-        log.debug('load config from local', CONFIGPATH)
-        -- watch directory
-        filewatch.watch(workspace.getAbsolutePath(CONFIGPATH):gsub('[^/\\]+$', ''))
-        mergeConfig(merged, cfg)
+    local clientConfig = cfgLoader.loadClientConfig()
+    if clientConfig then
+        log.debug('load config from client')
+        config.update(clientConfig, json.null)
     end
 
     local rc = cfgLoader.loadRCConfig('.luarc.json')
     if rc then
         log.debug('load config from luarc')
-        mergeConfig(merged, rc)
-    end
-
-    local clientConfig = cfgLoader.loadClientConfig()
-    if clientConfig then
-        log.debug('load config from client')
-        mergeConfig(merged, clientConfig)
+        config.update(rc, json.null)
     end
 
-    for k, v in pairs(merged) do
-        if v == json.null then
-            merged[k] = nil
-        end
+    local cfg = cfgLoader.loadLocalConfig(CONFIGPATH)
+    if cfg then
+        log.debug('load config from local', CONFIGPATH)
+        -- watch directory
+        filewatch.watch(workspace.getAbsolutePath(CONFIGPATH):gsub('[^/\\]+$', ''))
+        config.update(cfg, json.null)
     end
 
-    config.update(merged)
-    log.debug('loaded config dump:', util.dump(merged))
+    log.debug('loaded config dump:', util.dump(config.dump()))
 end
 
 ---@class provider