[Proto] Task dialog: Cannot set Due date/time different from Start date/time

VERIFIED FIXED in 0.7

Status

Calendar
General
VERIFIED FIXED
11 years ago
11 years ago

People

(Reporter: ssitter, Unassigned)

Tracking

Bug Flags:
blocking-calendar0.7 +

Details

Attachments

(1 attachment)

(Reporter)

Description

11 years ago
Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.8.1.7pre) Gecko/20070811 Calendar/0.7pre

[Proto] Task dialog: Cannot set Due date/time different from Start date/time

Steps to Reproduce:
1. Start Sunbird with a new clean profile
2. Select menu File -> New Task
3. Enable and set Start date/time
4. Enable and set Due date/time different from Start date/time

Actual Results:
It's not possible set Due date/time different from Start date/time. You can change the Due date/time to a different value but as soon as you move the focus to another input element or save the task the Due date is set back and is the same as the Start date.

Expected Results:
It should be possible to set the Due date/time different from Start date/time.
(Reporter)

Comment 1

11 years ago
Allowing only zero-length tasks is a regression compared to our 0.5 release. Therefore I think this should be fixed before 0.7.
Flags: blocking-calendar0.7?
(Reporter)

Comment 2

11 years ago
I tracked the issue down to the following location in sun-calendar-event-dialog.js: updateEndTime() calls updateDateTime(). But if the task has both entry date and due date it will always enter the first part of the if clause and assigns the entry date to the due date.
http://bonsai.mozilla.org/cvsblame.cgi?file=mozilla/calendar/prototypes/wcap/sun-calendar-event-dialog.js&rev=1.23&mark=2267-2283,2307-2322#2264
(Reporter)

Comment 3

11 years ago
Created attachment 276698 [details] [diff] [review]
rev0 - proposed patch

Consider the case that both start and due date are set.
Attachment #276698 - Flags: review?(michael.buettner)
We decided on the QA chat this bug is a regression and the task dialog is not usable without this. -> blocks 0.7
Flags: blocking-calendar0.7? → blocking-calendar0.7+
Comment on attachment 276698 [details] [diff] [review]
rev0 - proposed patch

Interestingly enough, I already came across this issue while I wrote the patch for Bug 351084. As I thought this other patch would land much earlier I decided to add the same code snippet as is now attached to this Bug. r=mickey.
Attachment #276698 - Flags: review?(michael.buettner) → review+
patch checked in on trunk and MOZILLA_1_8_BRANCH

-> FIXED
Status: NEW → RESOLVED
Last Resolved: 11 years ago
Resolution: --- → FIXED
OS: Windows 2000 → All
Hardware: PC → All
Target Milestone: --- → 0.7

Comment 7

11 years ago
Verified with Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1.7pre) Gecko/20070829 Calendar/0.7pre
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.