386481 - The view in mini calendar are not refreshed at once after subscribing to remote calendar

Status: VERIFIED FIXED

(Calendar :: Calendar Frontend, defect)

Description

[Omar Bajraszewski]

A new bold feature presented with Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1.5pre) Gecko/20070630 Calendar/0.7pre has refresing problems. To see the problem please follow this steps:

1)Start new profile
2)Subscribe to:
http://www.google.com/calendar/ical/u7pke58sc91ksbm4h9pf9ofe8c%40group.calendar.google.com/public/basic.ics
3)Switch back to date tab to see mini calendar.
4) None of the days are bold (but you can see events in calendar view
5)Clicking one of the day updates mini calendar view
6)Switch off remote calendar
7)Days are still bold.

Comment 1

[Omar Bajraszewski]

Attachment: Days are not bold

Comment 2

[Philipp Kewisch [:Fallen][📱📆]]

Attachment: Missing methods - v1

Yes, it seems I forgot some methods of calICompositeObserver. Thanks for noticing! This also fixes some error console messages about missing onStartBatch/onEndBatch/onLoad messages.

Comment 3

[Michael Büttner (no reviews TFN)]

Comment on attachment 270477 [details] [diff] [review]
Missing methods - v1

It seems that this went by unnoticed during the review of the original patch. The code looks good to me, I double checked that all interfaces are correctly implemented. Nothing I could complain about, r=mickey.

Comment 4

[Michael Büttner (no reviews TFN)]

Comment on attachment 270477 [details] [diff] [review]
Missing methods - v1

It seems that this went by unnoticed during the review of the original patch. The code looks good to me, I double checked that all interfaces are correctly implemented. Nothing I could complain about, r=mickey.

Comment 5

[Michael Büttner (no reviews TFN)]

Checked in on HEAD and MOZILLA_1_8_BRANCH

-> FIXED

Comment 6

[Andreas Treumann]

I checked this issue with the latest nightly builds, task is fixed.

Comment 7

[Andreas Treumann]

.