Closed Bug 440358 Opened 16 years ago Closed 16 years ago

no WCAP calendar events visible after restart of sunbird

Categories

(Calendar :: Provider: WCAP, defect)

defect
Not set
normal

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: andreas.treumann, Assigned: dbo)

Details

Attachments

(1 file)

STEPS TO REPRODUCE:
===================

- create a new sunbird profile
- add a WCAP calendar in sunbird
- restart sunbird
 
RESULT:
=======

- no WCAP calendar events visible
- only after a reload the events are visible

EXPECTED RESULT:
================

- the WCAP calendar events should be visible without a forced reload

REPRODUCIBLE:
=============

- always

Same scenario with a Webdav or caldav calendar needs no reload. If you use a WCAP and a Webdav/caldav calendar in parallel there is no reload necessary to get the WCAP events.

This is an 'sunbird only' issue.
Flags: blocking-calendar0.9+
Hmm, reading the logs, the data is correctly provided.
Assignee: nobody → daniel.boelzle
Attached patch fix - v1Splinter Review
Comment on attachment 332737 [details] [diff] [review]
fix - v1

It's actually a bug in wcap's initial session management uncovered by a startup race:
- initial refresh of views
- another refresh caused by setting offline state

The latter causes to cancel the pending login request which has cancelled all child requests.
Attachment #332737 - Attachment is patch: true
Attachment #332737 - Attachment mime type: application/octet-stream → text/plain
Attachment #332737 - Flags: review?(philipp)
Comment on attachment 332737 [details] [diff] [review]
fix - v1

In which cases is the calendar list not loaded?

It would be nice if we could enforce loading the tree, do you have an idea how this could happen? Otherwise, there are probably lots of places where we rely on DOM nodes being there, but I'd assume that if a chrome script is loaded, we should have access to the DOM. Obviously this happens before DOMContentLoaded, but it would be nice to find out if its possible to defer the call until the content is loaded.

r=philipp for now
Attachment #332737 - Flags: review?(philipp) → review+
No idea yet, would be good to have a solution as our current code looks fragile w.r.t. that.

Checked in on HEAD and MOZILLA_1_8_BRANCH => FIXED.
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Target Milestone: --- → 0.9
Checked with sunbird 20080811 -> VERIFIED
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.