397742 - clashing calendarManagerObserver definition for calendar.prototypes.wcap

Status: VERIFIED FIXED

(Calendar :: General, defect)

Description

[Daniel Boelzle [:dbo]]

definitions:
<http://mxr.mozilla.org/seamonkey/source/calendar/base/content/calendar-management.js#490>
<http://mxr.mozilla.org/seamonkey/source/calendar/prototypes/wcap/sun-messenger-overlay-sidebar.js#71>

the latter is bound in via:
<http://mxr.mozilla.org/seamonkey/source/calendar/lightning/content/messenger-overlay-sidebar.js#406>

Comment 1

[Daniel Boelzle [:dbo]]

Attachment: fix

Comment 2

[Michael Büttner (no reviews TFN)]

Comment on attachment 282522 [details] [diff] [review]
fix

>+var sideBarCalendarManagerObserver = {
I'd rather name this observer invitationManagerCalendarManagerObserver as it is the instance from the invitation manager that observes the calendar manager. But I leave the choice up to you. r=mickey.

Comment 3

[Daniel Boelzle [:dbo]]

I picked that name because the other listener in that file is prefixed with sideBar, too.

Checked in on HEAD and MOZILLA_1_8_BRANCH.