Initial Filters is not respected when array type filter
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
- 68/100
- Loại issue
- Lỗi
- Độ rõ ràng
- Đặc tả rõ ràng
- Mức độ hoạt động
- Ít trao đổi
- Công nghệ
- typescript
- Lĩnh vực
- frontend
Hướng nghiên cứu
Bắt đầu từ logic bộ lọc được thể hiện trong issue và theo dõi cách các bộ lọc array và initialFilters được tải khi khóa URL không tồn tại. Tái hiện trường hợp với một bộ lọc kiểu array và xác minh rằng các giá trị ban đầu đã cấu hình được giữ lại khi không có giá trị URL.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
When the user supplies an array type filter with initialFilters, the initialFilters are never used. The filter logic has the following code:
const urlValue = isArrayFilter ? searchParams?.getAll(key) : searchParams?.get(key);
// eslint-disable-next-line no-nested-ternary
loadedFilters[key] = urlValue
? (isArrayFilter && !Array.isArray(urlValue) ? [ urlValue ] : urlValue)
: initialFilters[key];
The issue is that searchParams.getAll returns an empty array when key doesn't exist, so the truthy check on urlValue evaluates to true even though nothing is set.
This can be resolved by checking the length of the array when isArrayFilter is true
Jira Issue: PF-4302
- Ngôn ngữ chính
- TypeScript
- Star
- 4
- Fork
- 21
- 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
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 patternfly/react-data-view
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
patternfly/react-data-view#692 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
patternfly/react-data-view#680 · 1 bình luận ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
patternfly/react-data-view#662 ·
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 55/100
patternfly/react-data-view#676 · 2 bình luận ·
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 65/100
patternfly/react-data-view#646 · 1 bình luận ·
Tất cả issue của patternfly/react-data-view
Issue tương tự
-
Browser Waiting for: Product Owner
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 85/100
getsentry/sentry-javascript#24577 · 1 bình luận ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
agilepathway/label-checker#640 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 84/100
copse-dev/agent-pane#2953 ·
-
[aw] Upgrade available Đang mởagentic-workflows
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 85/100
githubnext/rig#534 ·
-
automation missing-model model-sync provider:pioneer
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
anomalyco/models.dev#7701 ·