Closed
Bug 241487
Opened 21 years ago
Closed 21 years ago
duetoday not working
Categories
(Calendar :: Sunbird Only, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: mvgrad78, Assigned: belhaire)
Details
Attachments
(1 file)
|
792 bytes,
patch
|
mostafah
:
first-review+
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.8a) Gecko/20040418 Firefox/0.8.0+
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.8a) Gecko/20040418 Firefox/0.8.0+
On the task list, a task that is due today (set aside with blue text) shows to
be overdue (red text) unless the task is due today at 11:59 pm.
Reproducible: Always
Steps to Reproduce:
1. create task with today's date. text is red
2. change time due to 11:59 p.m., text changes to blue
3.
Actual Results:
text was red
Expected Results:
should be blue.
In unifinderToDo.js, line 259 is:
if( tonightMidnight.getTime() > dueDate.getTime() )
it should be
if( thisMorning.getTime() > dueDate.getTime() )
| Assignee | ||
Comment 1•21 years ago
|
||
Assignee: mostafah → belhaire
Status: UNCONFIRMED → ASSIGNED
| Assignee | ||
Updated•21 years ago
|
Attachment #147232 -
Flags: first-review?(mostafah)
| Assignee | ||
Comment 2•21 years ago
|
||
Comment on attachment 147232 [details] [diff] [review]
patch as described above. Worksforme with this workaround
>--- new/content/calendar/unifinderToDo.js 2004-04-28 16:46:16.000000000 +0200
>+++ orig/content/calendar/unifinderToDo.js 2004-04-07 00:53:20.000000000 +0200
>@@ -254,13 +254,13 @@
> }
>
> var startDate = new Date( calendarToDo.start.getTime() );
> var dueDate = new Date( calendarToDo.due.getTime() );
> var tonightMidnight = new Date( now.getFullYear(), now.getMonth(), now.getDate(), 23, 59, 00 );
>
>- if( thisMorning.getTime() > dueDate.getTime() )
>+ if( tonightMidnight.getTime() > dueDate.getTime() )
> {
> return("overdue");
> } else
> if ( dueDate.getFullYear() == now.getFullYear() &&
> dueDate.getMonth() == now.getMonth() &&
> dueDate.getDate() == now.getDate() )
Attachment #147232 -
Attachment description: patch as described above → patch as described above. Worksforme with this workaround
Comment 3•21 years ago
|
||
Comment on attachment 147232 [details] [diff] [review]
patch as described above. Worksforme with this workaround
Reverse of patch checked in.
Attachment #147232 -
Flags: first-review?(mostafah) → first-review+
Comment 4•21 years ago
|
||
Fixed in CVS.
Status: ASSIGNED → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
Comment 5•19 years ago
|
||
The bugspam monkeys have been set free and are feeding on Calendar :: Sunbird Only. Be afraid for your sanity!
QA Contact: gurganbl → sunbird
You need to log in
before you can comment on or make changes to this bug.
Description
•