forEach should await the iteratorCallback
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 3/5
- Thời gian dự kiến
- 1-2 ngày
- Mức phù hợp với người mới
- 45/100
- Loại issue
- Lỗi
- Độ rõ ràng
- Khá rõ ràng
- Mức độ hoạt động
- Đình trệ
- Công nghệ
- typescript
- Lĩnh vực
- databases
Hướng nghiên cứu
Bắt đầu từ phần triển khai storage forEach và theo dõi cách nó gọi iteratorCallback. Xác nhận rằng Promise được trả về sẽ chờ các lệnh gọi remove async hoàn tất, để một save tiếp theo không thể chạy đua với clear; payload không nêu tệp hoặc bài kiểm thử cụ thể nào.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
We have a situation where we want to clear a bunch of items from storage, and then set items immediately after. We need to await the clear step before saving to prevent the clear from clearing the newly added items.
We are using forEach to iterate through storage, check each item and remove it if need be. Since remove is async we are awaiting it in the iteratorCallback and our iteratorCallback is marked as async. From the source code for storage we can see that the iteratorCallback is not awaited. This means that the forEach Promise could resolve before all the removes have completed.
Here is a sample of what our clear function is doing:
async clear() {
await this.storage.forEach(async (v, k, i) => {
if (k.startsWith(this.barcodeStorageKey) || k.startsWith(this.principleStorageKey)) {
await this.storage.remove(k);
}
});
}
We are using "@ionic/storage": "2.1.3"
- Ngôn ngữ chính
- TypeScript
- Star
- 455
- Fork
- 97
- 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 ionic-team/ionic-storage
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 25/100
ionic-team/ionic-storage#326 · 1 bình luận ·
-
CREATE YOUR OWN LOCALSTORAGEĐang mở
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 15/100
ionic-team/ionic-storage#325 · 2 bình luận ·
-
indexeddb issue on iOS 18+Đang mở
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 25/100
ionic-team/ionic-storage#324 · 4 bình luận · 1 reaction ·
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 25/100
ionic-team/ionic-storage#323 · 2 bình luận · 1 reaction ·
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 35/100
ionic-team/ionic-storage#322 · 1 bình luận · 8 reaction ·
Tất cả issue của ionic-team/ionic-storage
Issue tương tự
-
useFragment does not require `variables` when the fragment's variables type has required fieldsĐang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
apollographql/apollo-client#13493 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Custom hosts written as trailing-dot FQDNs (`host.local.`) aren't detected as local backendsCó thể đã có người làm @roboomp đã nhận hôm nay. Đang mởbug prio:p2 providers triaged
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
can1357/oh-my-pi#15798 · 3 bình luận ·
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
code-yeongyu/oh-my-openagent#10172 ·
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 70/100
PDFCraftTool/pdfcraft#189 ·
-
Add the Yahoo chart API duplicate last-day row (yfinance#573) to incidents.md (about 1 hour)Có thể đã có người làm Có pull request liên kết đang mở hoặc đã được merge. Đang mởdata docs good first issue
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 80/100
pacocartones/market-data-sanity-checker#160 ·
Maintainer thường phản hồi trong vòng 1 ngày