this.mCalendar undefined in calItemBase for new event

RESOLVED FIXED

Status

--
trivial
RESOLVED FIXED
13 years ago
12 years ago

People

(Reporter: gekacheka, Unassigned)

Tracking

Details

Attachments

(1 attachment)

(Reporter)

Description

13 years ago
User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.8.0.3) Gecko/20060426 Firefox/1.5.0.3
Build Identifier: trunk

No default value for this.mCalendar is defined.
A new event does not have a calendar set.

Therefore access to get calendar produce error.



Reproducible: Always

Steps to Reproduce:
0. (In sunbird with venkman installed.)
   Open venkman (javascript debugger),
   ensure it shows errors (debug > stop for errors)
1. click New Event
Actual Results:  
Venkman shows

Warning ``reference to undefined property this.mCalendar'' [xs] in
file ``file:///C:/moz/sunbird/components/calItemBase.js'', line 419,
character 0.

Expected Results:  
No warnings.
(Reporter)

Comment 1

13 years ago
Created attachment 221131 [details] [diff] [review]
provide default value for calItemBase.mCalendar: null

(patch -l -p 2 -i file.patch)
Attachment #221131 - Flags: first-review?(jminta)

Comment 2

13 years ago
Comment on attachment 221131 [details] [diff] [review]
provide default value for calItemBase.mCalendar: null

r=jminta
Attachment #221131 - Flags: first-review?(jminta) → first-review+

Updated

13 years ago
Whiteboard: [needs landing]

Comment 3

13 years ago
patch checked in.
Status: NEW → RESOLVED
Last Resolved: 13 years ago
Resolution: --- → FIXED
Whiteboard: [needs landing]
The bugspam monkeys have struck again. They are currently chewing on default assignees for Calendar. Be afraid for your sanity!
Assignee: base → nobody
You need to log in before you can comment on or make changes to this bug.