Closed Bug 311911 Opened 19 years ago Closed 19 years ago

Allday events are not showing up in multiweek view

Categories

(Calendar :: Sunbird Only, defect)

x86
Windows XP
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: sebo.moz, Assigned: mostafah)

Details

Attachments

(1 file)

User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8b5) Gecko/20051001 Firefox/1.4.1 Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9a1) Gecko/20051008 Mozilla Sunbird/0.2+ Happened during last week. Reproducible: Always Steps to Reproduce: 1.Create Allday event 2.Change to multiweek view 3. Actual Results: no allday events shown Expected Results: show allday events
Confirmed. Regression range: 20051006 works 20051007 does not. Bug 309000 was the only change in that range to touch multiweekView, cc'ing gekacheka.
Status: UNCONFIRMED → NEW
Ever confirmed: true
I think I found the solution: With bug 309000 the property 'lastDateOfView' was renamed to 'endExDateOfView' in multiweekView.js. But 'lastDateOfView' is used in calendarWindow.js (line 668) in method CalendarView.prototype.createEventBox(). After renameing all occurrences of 'endExDateOfView' back to 'lastDateOfView' the all day event is displayed again in Multiweek View. I don't know how to create a patch so please check this and create a patch.
http://lxr.mozilla.org/seamonkey/search?string=lastDateOfView still shows a few hits. I think they all need to be replaced. gekacheka, can you look at this?
global replace. Fixes things for me.
Attachment #199733 - Flags: first-review?(jminta)
Comment on attachment 199733 [details] [diff] [review] s/lastDateOfView/endExDateOfView/g r=jminta
Attachment #199733 - Flags: first-review?(jminta) → first-review+
patch checked in
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: