microsoft / microsoft/TypeScript

intellisense in static method suggests TypeParameter of outer class

Đang mở
#28,472 1 bình luận 0 reaction 0 người được giao Xem trên GitHub

Chưa có ai nhận issue này.

In Discussion Suggestion
Ngôn ngữ chính
Go
Star
111k
Fork
14.3k
Merge trung bình
2 ngày 4 giờ
Pull request đã merge (30 ngày)
132

Mô tả

TypeScript Version: 3.2.0-dev.20181101

Search Terms:

Code

class Outer<T> {
    doStuff() {
        class Inner<T> {
            static doStuff(param: /**/) {}
        }
    }
}

Expected behavior:

Intellisense does not suggest T as it will result in an error.
OR
T of Inner doesn't shadow the outer T in static members, which is therefore accessible.

Actual behavior:

Suggests T from Outer.

Playground Link: https://agentcooper.github.io/typescript-play/#code/MYGwhgzhAEDyCuAXApgJwDwBUB80DeAUNMdACYD2AyovAGa0AUAlPkSe6JDAJIB2vaLLkLtRJCIjCIAlsDJUa9BgAcwqMAFsAXNAD0AKn26WeAL5sx59udNA

Related Issues:
#28471

TypeChecker.getSymbolsInScope correctly excludes TypeParameters of the containing class for its static members. Because of that an outer scope can add a symbol with the same name.

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. 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.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. 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 trường hợp generic lồng nhau với static member từ issue hoặc liên kết Playground của issue, sau đó kiểm tra TypeChecker.getSymbolsInScope và việc thu thập symbol của IntelliSense xung quanh các static member. Được xem là hoàn tất khi IntelliSense không còn đề xuất T bên ngoài không thể truy cập, đồng thời vẫn giữ nguyên cách xử lý dự kiến đối với Inner's T.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Đánh giá

Công nghệ
typescript
Lĩnh vực
compilers, developer-experience
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
35/100

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.