Closed
Bug 345348
Opened 19 years ago
Closed 19 years ago
Tasks appear in calendar view when progress is modified
Categories
(Calendar :: Calendar Frontend, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: george, Assigned: jminta)
Details
Attachments
(2 files)
2.62 KB,
patch
|
jminta
:
first-review+
|
Details | Diff | Splinter Review |
2.63 KB,
patch
|
jminta
:
first-review+
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.8.0.4) Gecko/20060614 Fedora/1.5.0.4-1.2.fc5 Firefox/1.5.0.4 pango-text
Build Identifier: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9a1) Gecko/20060720 Calendar/0.3a2+
"Tasks in View" is disabled.
However, when I modify the progress of a task, the task appears in the calendar view, even though I have "Tasks in View" disabled. If I exit calendar and reload, the task is no longer displayed in the calendar view.
Reproducible: Always
Steps to Reproduce:
1. Disable "View -> Tasks in View"
2. Create a task
3. Modify the progress by right-clicking on the task.
Actual Results:
The task appeared in the week view and the progress was changed in the task list.
Expected Results:
Changed the progress in the task list.
Assignee | ||
Comment 1•19 years ago
|
||
Yeah, need to add a check for this in the view's observers.
Status: UNCONFIRMED → NEW
Component: Tasks → Calendar Views
Ever confirmed: true
Assignee | ||
Comment 2•19 years ago
|
||
We should also check mTasksInView to filter out tasks in the observer when we're not showing them.
Assignee | ||
Comment 3•19 years ago
|
||
Comment on attachment 230392 [details] [diff] [review]
also check for mTasksInView
r=dmose
Attachment #230392 -
Flags: first-review?(dmose) → first-review+
Assignee | ||
Comment 4•19 years ago
|
||
patch checked in.
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
Comment 5•19 years ago
|
||
Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9a1) Gecko/20060802 Calendar/0.3a2+:
Problem still exist with Multiweek/Month view.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Assignee | ||
Comment 6•19 years ago
|
||
Same patch for the month view. Really, at this point, we should come up with a common set of add/remove item functions and share this observer between the views. But that's another bug.
Attachment #231785 -
Flags: first-review?(dmose)
Assignee | ||
Comment 7•19 years ago
|
||
Comment on attachment 231785 [details] [diff] [review]
more checks
r=dmose
Attachment #231785 -
Flags: first-review?(dmose) → first-review+
Assignee | ||
Comment 8•19 years ago
|
||
'more checks' patch checked in.
Status: REOPENED → RESOLVED
Closed: 19 years ago → 19 years ago
Resolution: --- → FIXED
Updated•19 years ago
|
QA Contact: tasks → views
You need to log in
before you can comment on or make changes to this bug.
Description
•