Configure: crash when id in patch.js is wrong
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 2/5
- Thời gian dự kiến
- 1-3 giờ
- Mức phù hợp với người mới
- 35/100
- Loại issue
- Lỗi
- Độ rõ ràng
- Đặc tả rõ ràng
- Mức độ hoạt động
- Đình trệ
- Công nghệ
- csharp, javascript
- Lĩnh vực
- desktop
Hướng nghiên cứu
Bắt đầu trong thcrap_configure_v3/MainWindow.xaml.cs tại dòng được tham chiếu và theo dõi cách id từ patch_bootstrap được tra cứu trong knownPatches. Xác nhận hành vi bằng một id không hợp lệ từ patch.js; hoàn tất khi công cụ cấu hình báo lỗi dễ đọc thay vì bị crash khi không có patch nào khớp.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
When the id in patch.js is wrong, thcrap_configure_v3 crashes at this line: https://github.com/thpatch/thcrap/blob/ffc4277150946c8a33695b44cd272855e47dd9b6/thcrap_configure_v3/MainWindow.xaml.cs#LL105C15-L105C15
knownPatches contains the patch names from the various repo.js files, and the id variable comes from patch_bootstrap and contains the id from patch.js. That id isn't in knownPatches, so the function knownPatches.First() throws an exception because it didn't find anything.
Something is wrong and we don't need to make things work in this case, but a readable error message would be better than a crash.
- Ngôn ngữ chính
- C++
- Star
- 772
- Fork
- 51
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
Chuẩn bị môi trường
- Không có Dockerfile hay tệp Docker Compose
- Không có mẫu pull request
- Đọc 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.
Issue khác của thpatch/thcrap
-
Ending display bugs in 15.5Đang mở
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 55/100
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 55/100
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 42/100
-
Broken custom.exeĐang mở
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 45/100
-
Going to the 4th row of column in the achievements menu of touhou 19 UDoALG crashes the gameĐang mở
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 35/100
Tất cả issue của thpatch/thcrap
Issue tương tự
-
agent:WSL bug linux LOW
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Build failure: limoĐang mở0.kind: build failure
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 Dưới một giờ Mức phù hợp với người mới 72/100
finalburnneo/FBNeo#2803 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
ai_p1 comp-message-queues
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
ClickHouse/ClickHouse#125890 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 78/100
carla-simulator/carla#9973 ·
Maintainer thường phản hồi trong vòng 1 ngày