Closed
Bug 324286
Opened 19 years ago
Closed 19 years ago
Create task/event via double-click does not work
Categories
(Calendar :: Sunbird Only, defect)
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: ssitter, Assigned: ssitter)
References
Details
(Keywords: regression)
Attachments
(1 file)
|
2.73 KB,
patch
|
jminta
:
first-review+
|
Details | Diff | Splinter Review |
Create task/event via double-click does not work anymore.
Regression from Bug 296893.
Error: newEventCommand is not defined
Source File: chrome://calendar/content/calendarWindow.js
Line: 68
Error: newToDoCommand is not defined
Source File: chrome://calendar/content/unifinderToDo.js
Line: 220
|
Assignee | |
Comment 1•19 years ago
|
||
I had to rename variable newEvent too, because otherwise I get the error "newEvent is not a function".
Quoting from http://www.crockford.com/jslint/lint.html:
> In JavaScript, blocks do not introduce a scope. There is only function-scope.
> JavaScript's blocks confuse experienced programmers and lead to errors.
True, true.
Assignee: mostafah → ssitter
Status: NEW → ASSIGNED
Attachment #209237 -
Flags: first-review?(jminta)
|
||
Comment 2•19 years ago
|
||
Comment on attachment 209237 [details] [diff] [review]
patch
Thanks for this. r=jminta
Attachment #209237 -
Flags: first-review?(jminta) → first-review+
|
|
||
Comment 3•19 years ago
|
||
patch checked in. Glad to have this fixed so fast, stefan, thanks.
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
|
||
Comment 4•18 years ago
|
||
verified with
Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9a1) Gecko/20060909 Calendar/0.3a2+
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•