fix #612

changelog.md

@@ -2,6 +2,7 @@
 
 ## 2.3.3
 * `NEW` config supports prop
+* `FIX` [#612](https://github.com/sumneko/lua-language-server/issues/612)
 
 ## 2.3.2
 `2021-7-21`

script/client.lua

@@ -38,6 +38,26 @@ function m.getOption(name)
     return option
 end
 
+function m.getAbility(name)
+    local current = m.info.capabilities
+    while true do
+        local parent, nextPos = name:match '^([^%.]+)()'
+        if not parent then
+            break
+        end
+        current = current[parent]
+        if not current then
+            return nil
+        end
+        if nextPos > #name then
+            break
+        else
+            name = name:sub(nextPos + 1)
+        end
+    end
+    return current
+end
+
 local function packMessage(...)
     local strs = table.pack(...)
     for i = 1, strs.n do

script/provider/provider.lua

@@ -291,11 +291,17 @@ proto.on('textDocument/definition', function (params)
         local targetUri = info.uri
         if targetUri then
             if files.exists(targetUri) then
-                response[i] = define.locationLink(targetUri
-                    , files.range(targetUri, info.target.start, info.target.finish)
-                    , files.range(targetUri, info.target.start, info.target.finish)
-                    , files.range(uri,       info.source.start, info.source.finish)
-                )
+                if client.getAbility 'textDocument.definition.linkSupport' then
+                    response[i] = define.locationLink(targetUri
+                        , files.range(targetUri, info.target.start, info.target.finish)
+                        , files.range(targetUri, info.target.start, info.target.finish)
+                        , files.range(uri,       info.source.start, info.source.finish)
+                    )
+                else
+                    response[i] = define.location(targetUri
+                        , files.range(targetUri, info.target.start, info.target.finish)
+                    )
+                end
             end
         end
     end
@@ -320,11 +326,17 @@ proto.on('textDocument/typeDefinition', function (params)
         local targetUri = info.uri
         if targetUri then
             if files.exists(targetUri) then
-                response[i] = define.locationLink(targetUri
-                    , files.range(targetUri, info.target.start, info.target.finish)
-                    , files.range(targetUri, info.target.start, info.target.finish)
-                    , files.range(uri,       info.source.start, info.source.finish)
-                )
+                if client.getAbility 'textDocument.typeDefinition.linkSupport' then
+                    response[i] = define.locationLink(targetUri
+                        , files.range(targetUri, info.target.start, info.target.finish)
+                        , files.range(targetUri, info.target.start, info.target.finish)
+                        , files.range(uri,       info.source.start, info.source.finish)
+                    )
+                else
+                    response[i] = define.location(targetUri
+                        , files.range(targetUri, info.target.start, info.target.finish)
+                    )
+                end
             end
         end
     end