353066 - problem with refreshing tasks in main view when end due starts is set

Status: VERIFIED FIXED

(Calendar :: Tasks, defect)

Description

[Damian Szczepanik]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; pl; rv:1.8.0.7) Gecko/20060909 Firefox/1.5.0.7
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9a1) Gecko/20060917 Calendar/0.3a2+

new task is not visible in week view when due time is set

Reproducible: Always

Steps to Reproduce:
1. switch to week view and enable "Tasks in view"
2. Go to today
3. create new task start date = today, due date is disabled
  -> task is not visible in main view - ok
4. edit created task, enable due date = start date + 1 hour
Actual Results:  
task is not visible in main view

Expected Results:  
task should be visible because if I switch to day view and go back to week task is visible so this is problem with refresing

Comment 1

[Dan Mosedale (:dmosedale, :dmose)]

This is unfortunate, but since it gets fixed by a refresh, relnoting should be good enough.

Comment 4

[cmtalbert]

This bug continues to come up.  Can we fix it for 0.5?

Comment 5

[Bas van den Bosch]

I knew I had seen it before: bug 278302 which is a dupe of bug 257632 which was fixed by bug 297934. This ended in a regression-bug: bug 321164

Comment 6

[Stefan Sitter [:ssitter]]

Attachment: rev0 - use correct scope

this.mTasksInView is undefined. This means we will return to early from onModifyItem() for tasks. Some lines above/below we already use the correct scoped version this.calView.mTasksInView.

Comment 7

[Joey Minta]

Comment on attachment 257533 [details] [diff] [review]
rev0 - use correct scope

yep yep.

Comment 8

[Matthew (lilmatt) Willis]

Patch checked in on MOZILLA_1_8_BRANCH and trunk.

-> FIXED

Comment 9

[Stefan Sitter [:ssitter]]

Note: There is still an open issue when editing a task with only start or due date set to a task with both dates set. I filed Bug 372855 for this issue.

Comment 10

[Damian Szczepanik]

task is not visible, I have to refresh to see it
-> Reopen

Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9a3pre) Gecko/20070308 Calendar/0.6a1

Comment 11

[Bas van den Bosch]

confirmed in Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1.3pre) Gecko/20070310 Calendar/0.5pre

Error: aDate has no properties
Source File: file:///C:/Program%20Files/Calendar/js/calUtils.js
Line: 567

Comment 12

[Stefan Sitter [:ssitter]]

I already filed Bug 372855 for the open issue as written in Comment #9, there was no need to reopen the bug.

Comment 13

[Matthew (lilmatt) Willis]

Marking FIXED per comment #12.

Comment 14

[Martin Schröder [:mschroeder]]

VERIFIED with Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1.4pre) Gecko/20070522 Calendar/0.5pre.