Closed Bug 1399998 Opened 2 years ago Closed 2 years ago

assignment to undeclared variable res in calAuthUtils.jsm

Categories

(Calendar :: General, defect)

defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: mkmelin, Assigned: mkmelin)

Details

Attachments

(1 file, 1 obsolete file)

Attached patch bugXXX_cal_res_undefined.patch (obsolete) — Splinter Review
Getting this on trunk, I guess because modules are now strict by default?

22:15:47.980 runnablePrompter:run: [Exception... "[JavaScript Error: "assignment to undeclared variable res" {file: "resource://calendar/modules/calAuthUtils.jsm" line: 304}]'[JavaScript Error: "assignment to undeclared variable res" {file: "resource://calendar/modules/calAuthUtils.jsm" line: 304}]' when calling method: [nsIMsgAsyncPromptListener::onPromptStart]"  nsresult: "0x80570021 (NS_ERROR_XPC_JAVASCRIPT_ERROR_WITH_DETAILS)"  location: "JS frame :: jar:file:///opt/nightly/2017-09-13/thunderbird/omni.ja!/components/msgAsyncPrompter.js :: runnablePrompter.prototype.run< :: line 28"  data: yes]
 1 msgAsyncPrompter.js:31
	runnablePrompter.prototype.run< jar:file:///opt/nightly/2017-09-13/thunderbird/omni.ja!/components/msgAsyncPrompter.js:31:7
	next self-hosted:1190:9
	TaskImpl_run resource://gre/modules/Task.jsm:331:42
	TaskImpl_run self-hosted:997:17
Attachment #8908310 - Flags: review?(ssitter)
Attachment #8908310 - Flags: review?(ssitter) → review+
Attachment #8908310 - Attachment is obsolete: true
Attachment #8908739 - Flags: review+
Keywords: checkin-needed
Another new Calendar contributor like myself ;-)
Pushed by mozilla@jorgk.com:
https://hg.mozilla.org/comm-central/rev/7dce3083bbf1
fix assignment to undeclared variable res in calAuthUtils.jsm. r=ssitter
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → 5.9
You need to log in before you can comment on or make changes to this bug.