311067 - Calendar colors not honored

Status: RESOLVED FIXED

(Calendar :: Sunbird Only, defect)

Description

[jrx]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8b5) Gecko/20051003 Firefox/1.4.1
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9a1) Gecko/20051004 Mozilla Sunbird/0.2+

In the 20051004 build, colors of calendars are not honored (events are always
the background color, rather than the one selected for the calendar).

Reproducible: Always

Steps to Reproduce:
1.Change color of calendar in "Calendars" tab
2.Create event
3.

Actual Results:  
Event is white

Expected Results:  
Event is color I chose

Started with the 1004 build (not a problem in 1003)

Comment 1

[Joey Minta]

Confirmed:
Error: uncaught exception: [Exception... "Unexpected error"  nsresult:
"0x8000ffff (NS_ERROR_UNEXPECTED)"  location: "JS frame ::
chrome://calendar/content/calendarManagement.js :: getCalendarStyleSheet :: line
260"  data: no]

For reference:
260             if (document.styleSheets[i].href.match(
261                 /chrome.*\/skin.*\/calendar.css$/ )) {

There were no checkins to this area of the calendar code in the last 24 hours.

My guess is we're getting bit by the fix for bug 309840.  Someone with more
regexp knowledge than I have ought to figure out if this is our bug or theirs.

STATUS: New

Comment 2

[Michiel van Leeuwen (email: mvl+moz@)]

in xpcshell, this works:
js> "chrome://calendar/skin/calendar.css".match(/chrome.*\/skin.*\/calendar.css$/);
chrome://calendar/skin/calendar.css
js>
so the regex does seem to work. Not sure what is going wrong here. Maybe it's
the document.styleSheets[i].href part?

Comment 3

[Michiel van Leeuwen (email: mvl+moz@)]

document.styleSheets[0] doesn't work, while document.styleSheets.item(0) does work.
A test in firefox showed that both do work in 20051003, but the [0] version
doesn't work in 20051004 (sorry, no hour information. Firefox doesn't tell me
that. ftp indicates 05)

Comment 4

[Michiel van Leeuwen (email: mvl+moz@)]

bug 310351 is a likely candidate.

Comment 5

[Blake Kaplan (:mrbkap) (inactive)]

I'm not sure how to test this, but it should be fixed by the checkin for bug 311090.

Comment 6

[Stefan Sitter [:ssitter]]

Works fine again in Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9a1)
Gecko/20051005 Mozilla Sunbird/0.2+.

I saw a similar error message in 20051004 build when trying to add/remove a
column in task/event list. This also works fine again in 20051005 build.

Comment 7

[Michiel van Leeuwen (email: mvl+moz@)]

Ok, works for me too. Marking fixed-by-bug311090.