Items not added to the offline cache when network fails (TypeError: calendar is null)
Categories
(Calendar :: Internal Components, defect)
Tracking
(thunderbird_esr115 affected, thunderbird_esr128? fixed, thunderbird130 fixed)
People
(Reporter: Fallen, Assigned: mkmelin)
References
(Regression)
Details
(Keywords: regression)
Attachments
(1 file)
48 bytes,
text/x-phabricator-request
|
corey
:
approval-comm-beta+
corey
:
approval-comm-esr128+
|
Details | Review |
If the network fails (isUnavailableCode() is true), then a message is shown that includes calendar.name
. However, in line 734, the callback is always called without a calendar. Hence items are never actually added to the offline cache on failure.
The same happens for other operations.
I'd recommend uplifting this once done as it causes events not to be changed while failing the network.
Assignee | ||
Comment 1•2 months ago
|
||
Maybe this is the root cause of various bugs we've got for not-properly-syncing nextcloud calendars...
Reporter | ||
Comment 2•2 months ago
|
||
This seems to go back to bug 1769200 based on the code. I'm requesting tracking this for 128, as it inhibits adding items while offline or during a network outage. You might also consider backporting this to 115, I expect the fix to be low risk.
Assignee | ||
Comment 3•2 months ago
|
||
Updated•2 months ago
|
Assignee | ||
Updated•2 months ago
|
Pushed by brendan@thunderbird.net:
https://hg.mozilla.org/comm-central/rev/247126682247
Fix "calendar is null" error in adoptItemCallback. r=Fallen
Assignee | ||
Comment 5•1 month ago
|
||
Comment on attachment 9417954 [details]
Bug 1908530 - Fix "calendar is null" error in adoptItemCallback. r=Fallen
[Approval Request Comment]
Regression caused by (bug #): bug 1769200
User impact if declined: offline cache inconsistency after network errors
Testing completed (on c-c, etc.): c-c
Risk to taking this patch (and alternatives if risky): not too risky
Comment 6•27 days ago
|
||
Comment on attachment 9417954 [details]
Bug 1908530 - Fix "calendar is null" error in adoptItemCallback. r=Fallen
[Triage Comment]
Approved for beta
Comment 7•27 days ago
|
||
bugherder uplift |
Thunderbird 130.0b3:
https://hg.mozilla.org/releases/comm-beta/rev/7fc04b6c039b
Comment 8•12 days ago
|
||
Comment on attachment 9417954 [details]
Bug 1908530 - Fix "calendar is null" error in adoptItemCallback. r=Fallen
[Triage Comment]
Approved for esr128
Comment 9•11 days ago
|
||
bugherder uplift |
Thunderbird 128.2.1esr:
https://hg.mozilla.org/releases/comm-esr128/rev/78c69d57bfff
Description
•