Closed Bug 323809 Opened 14 years ago Closed 14 years ago

findColumnForItem isn't defined

Categories

(Calendar :: Internal Components, defect)

defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: mvl, Assigned: jminta)

Details

Attachments

(1 file)

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...
I changed it last night.  It's now findColumnsForItem. http://lxr.mozilla.org/mozilla/source/calendar/base/content/calendar-multiday-view.xml#2289
Attached patch fix my messSplinter Review
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 on attachment 208777 [details] [diff] [review]
fix my mess

r=dmose
Attachment #208777 - Flags: first-review?(dmose) → first-review+
patch checked in. apologies to all for this bustage.
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.