LuaLS / LuaLS/lua-language-server
Returning a value as a class doesnt set the fields
Chưa có ai nhận issue này.
- Ngôn ngữ chính
- Lua
- Star
- 4.4k
- Fork
- 442
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
Mô tả
### How are you using the lua-language-server?
Visual Studio Code Extension (sumneko.lua)
### Which OS are you using?
Windows
### What is the issue affecting?
Annotations, Type Checking
### Expected Behaviour
`value` be member of `MyClass`
### Actual Behaviour

`value` is not a member
### Reproduction steps
```lua
local function myConstructor()
---@class MyClass
return { value = true }
end
local o = myConstructor()
print(o.value)
```
### Additional Notes
This should also work with a `setmetatable` variant:
```lua
---@class MyClass
local MyClass = {}
MyClass.__index = MyClass
local function myConstructor()
---@class MyClass
return setmetatable({ value = true }, MyClass)
end
local o = myConstructor()
print(o.value)
```
It does work with this workarround:
```lua
local function myConstructor()
---@class MyClass
local o = setmetatable({ value = true }, MyClass)
return o
end
```
### Log File
_No response_
Hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Hướng nghiên cứu
Bắt đầu bằng cách tái hiện hai ví dụ về constructor từ issue và theo dõi cách language server diễn giải annotation @class trên một table được trả về. So sánh điều này với workaround dùng local-variable đang hoạt động và thêm một regression test bao quát cả các lần return trực tiếp lẫn các lần return bằng setmetatable. Hoàn thành khi value được nhận diện là member của MyClass trong quá trình kiểm tra kiểu.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Đánh giá
- Công nghệ
- lua
- Lĩnh vực
- devtools
- Loại issue
- Lỗi
- Độ khó
- 4/5
- Thời gian dự kiến
- 3-5 ngày
- Mức độ hoạt động
- Đình trệ
- Độ rõ ràng
- Khá rõ ràng
- Mức phù hợp với người mới
- 45/100