284078 - Calendar wizard doesn't accept local calendars

Status: RESOLVED FIXED

(Calendar :: Sunbird Only, defect)

Description

[Stelian Pop]

User-Agent:       Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.7.6) Gecko/20050224 Firefox/1.0.1 Fedora/1.0.1-1.3.1
Build Identifier: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.7.6) Gecko/20050224 Firefox/1.0.1 Fedora/1.0.1-1.3.1

When trying to create a local calendar in the wizard, clicking on 'Next' says
that the calendar is created ok, but clicking on 'Finish' does not do anything,
instead an error is printed on the console:

test
file:///tmp/test.ics

************************************************************
* Call to xpconnect wrapped JSObject produced this error:  *
[Exception... "'[JavaScript Error:
"Components.classes["@mozilla.org/calendar/calendar;1?type=" + type] has no
properties" {file:
"file:///home/admin/mozilla-head/mozilla/dist/bin/components/calCalendarManager.js"
line: 149}]' when calling method: [calICalendarManager::createCalendar]" 
nsresult: "0x80570021 (NS_ERROR_XPC_JAVASCRIPT_ERROR_WITH_DETAILS)"  location:
"JS frame :: chrome://calendar/content/calendar.js :: doCreateWizardFinish ::
line 1659"  data: yes]

The only way to close the window is to "Cancel", and then many things are broken
because no calendars aredefined.


Reproducible: Always

Steps to Reproduce:
1. rm -rf ~/.mozilla
2. ./dist/bin/sunbird
3. try to create a local calendar, named 'test', located at 'file:///tmp/test.ics'

Comment 1

[Jeremy Teo]

Attachment: 1st attempt at fixing broken new calendar wizard

Ugh. For some reason, 
"var type=document.getElementById("initial-radiogroup").value;" always ends up
giving an empty value. Workaround is in patch.

Also, there is no provider called local.

Comment 2

[Stuart Parmenter]

Comment on attachment 175826 [details] [diff] [review]
1st attempt at fixing broken new calendar wizard

hm, i swore I fixed this, but I don't see it in the tree.  the first radio
should be 'storage'.  I'll land this in a few.

Comment 3

[Jeremy Teo]

Attachment: Second try at fixing the bug. One liner change.

We should use the "storage" provider for local calendars.

Comment 4

[Stuart Parmenter]

OK, I checked that in.

Comment 5

[Stelian Pop]

The patch is not in CVS yet.

Comment 6

[Stuart Parmenter]

ok, i really checked it in this time

Comment 7

[Reed Loden [:reed]]

The bugspam monkeys have been set free and are feeding on Calendar :: Sunbird Only. Be afraid for your sanity!