Closed
Bug 413715
Opened 16 years ago
Closed 16 years ago
Unifinder search is broken [Error: event is not defined]
Categories
(Calendar :: Calendar Frontend, defect)
Calendar
Calendar Frontend
Tracking
(Not tracked)
VERIFIED
FIXED
0.8
People
(Reporter: ssitter, Assigned: Fallen)
References
Details
(Keywords: regression)
Attachments
(1 file, 1 obsolete file)
4.64 KB,
patch
|
berend.cornelius09
:
review+
chris.j.bugzilla
:
ui-review+
|
Details | Diff | Splinter Review |
Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1.12pre) Gecko/2008012309 Calendar/0.8pre Steps to Reproduce: 1. Start Sunbird with new clean profile 2. Create some new events in the near future 3. Use the search field in the unifinder to search for an event Actual Results: Unifinder is empty. Error Console shows: Error: event is not defined Source File: chrome://calendar/content/calendar-unifinder.js Line: 931 Expected Results: Unifinder shows the events that match the search filter
Assignee | ||
Comment 1•16 years ago
|
||
Quite minimal, fix error and a warning.
Reporter | ||
Comment 2•16 years ago
|
||
Comment on attachment 298797 [details] [diff] [review] Fix v1 r=ssitter by the way: I noticed that the search results are not updated after editing an event to not match the search criteria anymore, i.e. the event is still visible but should not.
Attachment #298797 -
Flags: review?(ssitter) → review+
Assignee | ||
Comment 3•16 years ago
|
||
This patch does the same as the previous patch and additionally addresses the issue you mentioned. I think we had this problem before though. Requesting ui-r since I'm not sure its wanted behavior that changed items vanish from the list immediately.
Attachment #298797 -
Attachment is obsolete: true
Attachment #299481 -
Flags: ui-review?(christian.jansen)
Attachment #299481 -
Flags: review?(ssitter)
Reporter | ||
Comment 4•16 years ago
|
||
Comment on attachment 299481 [details] [diff] [review] Fix v2 I think you can already checkin the regression fix. Unfortunately I don't have a build environment to test such a big patch compared to the two lines in the patch before.
Attachment #299481 -
Flags: review?(ssitter) → review?(michael.buettner)
Comment 5•16 years ago
|
||
Comment on attachment 299481 [details] [diff] [review] Fix v2 r=christian
Attachment #299481 -
Flags: ui-review?(christian.jansen) → ui-review+
Updated•16 years ago
|
Attachment #299481 -
Flags: review?(michael.buettner) → review?(philipp)
Assignee | ||
Updated•16 years ago
|
Attachment #299481 -
Flags: review?(philipp) → review?(Berend.Cornelius)
Comment 6•16 years ago
|
||
Comment on attachment 299481 [details] [diff] [review] Fix v2 The patch looks very good to me. Even though I examined it very closely I could not detect anything wrong about it. r=berend.
Attachment #299481 -
Flags: review?(Berend.Cornelius) → review+
Assignee | ||
Comment 7•16 years ago
|
||
Checked in on HEAD and MOZILLA_1_8_BRANCH -> FIXED
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
OS: Windows XP → All
Hardware: PC → All
Resolution: --- → FIXED
Target Milestone: --- → 0.8
Reporter | ||
Comment 8•16 years ago
|
||
Verified using Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1.12pre) Gecko/2008020220 Calendar/0.8pre.
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•