(In reply to Khushil Mistry [:khushil324] from comment #18) > When will CalDavRequestBase.calendar be null? Should we open exception dialog when calendar is null? Good questions. One case when it's null is during calendar detection for the new calendar creation dialog (that I'm working on now), where the calendar doesn't exist yet (locally) but we are making requests to see if there's a calendar on the server to subscribe to. Also the docstring for the CalDavRequestBase constructor says it can be null, so there must be other cases too. I don't think we need to do anything different for the case where calendar is null (except not refresh it).
Bug 1590472 Comment 19 Edit History
Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.
(In reply to Khushil Mistry [:khushil324] from comment #18) > When will CalDavRequestBase.calendar be null? Should we open exception dialog when calendar is null? Good questions. One case when it's null is during calendar detection for the new calendar creation dialog (that I'm working on now), where the calendar doesn't exist yet (locally) but we are making requests to see if there's a calendar on the server to subscribe to. Also the docstring for the CalDavRequestBase constructor says it can be null, so there must be other cases too. I don't think we need to do anything different for the case where calendar is null (except not try to refresh it).