Closed Bug 1376123 Opened 7 years ago Closed 7 years ago

Warning: "ReferenceError: assignment to undeclared variable ritem" {file: "file:///[...]/components/calStorageCalendar.js" line: 1893}

Categories

(Calendar :: Provider: Local Storage, defect)

Lightning 5.8
defect
Not set
trivial

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: mschroeder, Assigned: mschroeder)

Details

Attachments

(1 file)

      No description provided.
Attached patch Patch v1 — — Splinter Review
Attachment #8881063 - Flags: review?(philipp)
Comment on attachment 8881063 [details] [diff] [review]
Patch v1

Review of attachment 8881063 [details] [diff] [review]:
-----------------------------------------------------------------

LGTM
Attachment #8881063 - Flags: review+
Attachment #8881063 - Flags: review?(philipp)
Keywords: checkin-needed
Is this an actual error or just a strict warning? In the former case, what regressed it and what branches are affected, i.e. do we need to port it back to beta or release? Do you have a test case that could be used for unit test to prevent similar regression?
Flags: needinfo?(mschroeder)
It is just a JavaScript warning, so I do not intend to request a backport or write an unit test as this is a pure programming oversight.
Flags: needinfo?(mschroeder)
Summary: "ReferenceError: assignment to undeclared variable ritem" {file: "file:///[...]/components/calStorageCalendar.js" line: 1893} → Warning: "ReferenceError: assignment to undeclared variable ritem" {file: "file:///[...]/components/calStorageCalendar.js" line: 1893}
Version: Trunk → Lightning 5.8
https://hg.mozilla.org/comm-central/rev/5f8fb8c490aa4ecbf5236a84ab6f069b85235dd7
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → 5.8
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: