Closed
Bug 323809
Opened 19 years ago
Closed 19 years ago
findColumnForItem isn't defined
Categories
(Calendar :: Internal Components, defect)
Calendar
Internal Components
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: mvl, Assigned: jminta)
Details
Attachments
(1 file)
2.81 KB,
patch
|
dmosedale
:
first-review+
|
Details | Diff | Splinter Review |
findColumnForItem is used twice, but isn't defined: http://lxr.mozilla.org/seamonkey/search?string=findColumnForItem I vagualy remember the function changing names, but i can't remember where it lives, and what the new name is...
Assignee | ||
Comment 1•19 years ago
|
||
I changed it last night. It's now findColumnsForItem. http://lxr.mozilla.org/mozilla/source/calendar/base/content/calendar-multiday-view.xml#2289
Assignee | ||
Comment 2•19 years ago
|
||
Uses the new findColumnsForItem call and fixes a strict warning about not always returning a value.
Comment 3•19 years ago
|
||
Comment on attachment 208777 [details] [diff] [review] fix my mess r=dmose
Attachment #208777 -
Flags: first-review?(dmose) → first-review+
Assignee | ||
Comment 4•19 years ago
|
||
patch checked in. apologies to all for this bustage.
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•