findColumnForItem isn't defined

RESOLVED FIXED

Status

Calendar
Internal Components
RESOLVED FIXED
12 years ago
12 years ago

People

(Reporter: Michiel van Leeuwen (email: mvl+moz@), Assigned: Joey Minta)

Tracking

Details

Attachments

(1 attachment)

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

12 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

12 years ago
Created attachment 208777 [details] [diff] [review]
fix my mess

Uses the new findColumnsForItem call and fixes a strict warning about not always returning a value.
Assignee: base → jminta
Status: NEW → ASSIGNED
Attachment #208777 - Flags: first-review?(dmose)

Comment 3

12 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

12 years ago
patch checked in. apologies to all for this bustage.
Status: ASSIGNED → RESOLVED
Last Resolved: 12 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.