getFieldsValue(true) allows for direct mutation of the form store
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
- 45/100
- Loại issue
- Lỗi
- Độ rõ ràng
- Đặc tả rõ ràng
- Mức độ hoạt động
- Đình trệ
- Công nghệ
- react, typescript
- Lĩnh vực
- frontend
Hướng nghiên cứu
Bắt đầu tại src/useForm.ts, khoảng dòng 271-273, và tái hiện vấn đề với getFieldsValue(true) cùng việc xóa một field. Xác nhận rằng giá trị được trả về chia sẻ trạng thái có thể thay đổi với form store; được xem là hoàn tất khi việc xóa một field khỏi object được trả về không còn làm thay đổi kết quả getFieldsValue(true) ở lần gọi sau, trong khi hành vi hiện có đối với các NamePaths cụ thể vẫn được giữ nguyên.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Repro:
Set up any basic form, and give a field an onChange like this:
() => {
const formData = form.getFieldsValue(true)
delete formData.nameOfField
console.log(form.getFieldsValue(true))
}
Normally, one would expect that this should have no effect, as react/state context should be immutable, but in this case you will find that it will log an object missing the deleting field.
The problem comes from the implementation of getFieldsValue here:
https://github.com/react-component/field-form/blob/d7892873e9eee6e4c618d4289fcdbc9ee240f68f/src/useForm.ts#L271-L273
Which just returns the store object directly.
This should be following React conventions and returning a clone, as it does when a specific list of NamePaths is requested.
- Ngôn ngữ chính
- TypeScript
- Star
- 1k
- Fork
- 287
- Merge trung bình
- 4 ngày 16 giờ
- Pull request đã merge (30 ngày)
- 1
Chuẩn bị môi trường
Dự án này không cung cấp dev container, Dockerfile hay hướng dẫn đóng góp, nên bạn cần tự thiết lập môi trường: hãy bắt đầu từ README và xem hướng dẫn đóng góp lần đầu của chúng tôi để biết các bước chung.
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 react-component/field-form
-
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 25/100
react-component/field-form#768 · 5 bình luận ·
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 25/100
react-component/field-form#751 ·
-
Add getFieldsWarning APIĐang mở
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 42/100
react-component/field-form#749 ·
-
useWatch支持dynamic pathĐang mở
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 35/100
react-component/field-form#746 ·
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 35/100
react-component/field-form#733 · 1 bình luận ·
Tất cả issue của react-component/field-form
Issue tương tự
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 66/100
cockpit-project/cockpit-machines#2835 ·
Maintainer thường phản hồi trong vòng 2 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
cloudflare/kumo#866 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
area:connector bug
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 82/100
Maintainer thường phản hồi trong vòng 1 ngày
-
environment: OPENCODE_API_KEY is described as a Console service account key, but a Go subscription key worksCó thể đã có người làm Có pull request liên kết đang mở hoặc đã được merge. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
tester-army/e2e#1015 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 62/100
Maintainer thường phản hồi trong vòng 1 ngày