Closed Bug 326331 Opened 19 years ago Closed 19 years ago

New event/task dialog does not use selected calendar as default

Categories

(Calendar :: Sunbird Only, defect)

x86
All
defect
Not set
normal

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: ssitter, Assigned: jminta)

Details

(Keywords: regression)

Attachments

(1 file)

New events/tasks are not created in the selected calendar.

1. Start Sunbird with a clean profile.
2. Create a second calendar named 'test'.
3. Go to calendar tab and ensure that the new calendar 'test' is selected.
4. Create new event/task via drag'n'drop, double click, toolbar icon or menu command.

Actual results:
The new event is created in the 'Home' calendar and not in the selected calendar 'test'.

Expected results:
The new event/task is created in the selected calendar.

Tested on windows and linux with 2006-02-07 sunbird trunk build. No problems with lighting.
Ok, seems that event creation via drag'n'drop always created the entries in the first calendar and not the selected calendar. Removing that issue from this bug.

The remaining issue about creation via double click, toolbar icon or menu
command seems a regression from the new item dialog landing. 
Works in sunbird win32-2006-01-21-10-trunk build
Fails in sunbird win32-2006-01-22-07-trunk build

Summary "New events/tasks not created in selected calendar"
    --> "New event/task dialog does not use selected calendar as default"
Status: UNCONFIRMED → NEW
Ever confirmed: true
Summary: New events/tasks not created in selected calendar → New event/task dialog does not use selected calendar as default
Attached patch look at arguments too — — Splinter Review
All the support for this was already there, we just didn't look at the .calendar argument that was passed in.
Assignee: nobody → jminta
Status: NEW → ASSIGNED
Attachment #211123 - Flags: first-review?(mvl)
Comment on attachment 211123 [details] [diff] [review]
look at arguments too

>Index: calendar/base/content/calendar-event-dialog.js
>+    // If this is a new calendar, the item might not have a calendar, but a 
>+    // default option could well have been passed in.

I guess that read 'If this is a new item,...' ?

r=mvl with that fixed
Attachment #211123 - Flags: first-review?(mvl) → first-review+
patch checked in
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
verified with
Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9a1) Gecko/20060926 Calendar/0.3a2+

however (my working profile) sometimes first calendar is selected but event is assigned to second one, can not create scenario to reproduce it
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: