Clobbering `this` from `options` might be murky
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 5/5
- Thời gian dự kiến
- Hơn một tuần
- Mức phù hợp với người mới
- 25/100
- Loại issue
- Tái cấu trúc
- Độ rõ ràng
- Khá rõ ràng
- Mức độ hoạt động
- Đình trệ
- Công nghệ
- javascript
- Lĩnh vực
- developer-experience
Hướng nghiên cứu
Bắt đầu với constructor trong core-object.js ở các dòng 18–20 và xem lại cách lớp cơ sở gán các thuộc tính từ options hash. Xác định các tác động đối với tính tương thích và quá trình migration khi để constructor đó rỗng; được xem là hoàn tất khi quyết định về breaking-change và hành vi dự kiến đã được ghi lại, đồng thời các test bị ảnh hưởng đã được xác định.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
If I understand the constructor correctly, we're providing the ability to assign arbitrary properties on this through an options hash as a common base class constructor.
I used to do this in early versions of Broccoli plugin classes because it seemed convenient. I stopped doing it when I started running into cases where some library upgrade would cause property names to start colliding with option names. I simply do this.options = options in constructors ever since, and have been much happier with that.
My suggestion would be to make the base class constructor empty, and let people do their own option handling. I realize this might be a breaking change, but perhaps it's worth considering for the next major version?
- Ngôn ngữ chính
- JavaScript
- Star
- 16
- Fork
- 16
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
Hướng dẫn đóng góp
Chưa lập chỉ mục được hướng dẫn đóng góp cho kho mã nguồn này
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.
Issue khác của ember-cli/core-object
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 25/100
ember-cli/core-object#35 · 1 reaction ·
-
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 15/100
ember-cli/core-object#29 ·
Tất cả issue của ember-cli/core-object
Issue tương tự
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
-
area-deployment area-integrations triage:bot-seen
Độ khó 2/5 Nửa ngày Mức phù hợp với người mới 86/100
-
Issue-Bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
sugarlabs/musicblocks#8924 ·
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 90/100
ArduPilot/ardupilot_wiki#8088 ·
-
[BUG] createTool tools cannot be registered with Mastra when exactOptionalPropertyTypes is enabled Đang mởcustomer-eng status: needs triage
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 84/100