Closed Bug 288889 Opened 20 years ago Closed 20 years ago

createCalendar is not defined

Categories

(Calendar :: Sunbird Only, defect)

Product:
Calendar
Component:
Sunbird Only
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: gekacheka, Assigned: mostafah)

Details

Attachments

(1 file, 1 obsolete file)

unifinderTodo.js patch: getCalendar, createCalendar ==> getDisplayComposite
2.80 KB, patch
mvl
: first-review+
Details | Diff | Splinter Review 
User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.7.6) Gecko/20050317 Firefox/1.0.2
Build Identifier: trunk

Error: createCalendar is not defined
Source File: chrome://calendar/content/unifinderToDo.js
Line: 108

Looks like it should be getCalendar as in unifinder.js

Reproducible: Always

Steps to Reproduce:
(patch -l -p 2 -i file.patch)

Change createCalendar to getCalendar, as is used in the corresponding function
in unifinder.js.
Attachment #179520 - Flags: first-review?
Comment on attachment 179520 [details] [diff] [review]
unifinderToDo.js patch:  createCalendar ==> getCalendar

see bug 288880. it should be getDisplayComposite()
Attachment #179520 - Flags: first-review? → first-review-
(patch -l -p 2 -i file.patch)

Converts all calls of getCalendar and one call of createCalendar to calls of
getDisplayComposite.

Now task list can show tasks from multiple files.
Attachment #179520 - Attachment is obsolete: true
Attachment #180299 - Flags: first-review?(mvl)
Attachment #180299 - Flags: first-review?(mvl) → first-review+
checked in
Status: NEW → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
QA Contact: gurganbl → sunbird
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: