Duplicated function in calendar-management.js

VERIFIED FIXED in 0.7

Status

Calendar
General
--
trivial
VERIFIED FIXED
11 years ago
11 years ago

People

(Reporter: martinschroeder, Assigned: martinschroeder)

Tracking

unspecified

Details

Attachments

(1 attachment)

(Assignee)

Description

11 years ago
Created attachment 277330 [details] [diff] [review]
Patch v1

In Error Console with javascript.option.strict set:
Warning: redeclaration of property isSeparator
Source File: chrome://calendar/content/calendar-management.js
Line: 374

We have a duplicated function:
/calendar/base/content/calendar-management.js,
    * line 366 -- isSeparator: function cLTV_isSeparator(aRow) {
    * line 374 -- isSeparator: function cLTV_isSeparator(aRow) {
Attachment #277330 - Flags: review?(philipp)
Comment on attachment 277330 [details] [diff] [review]
Patch v1

Looks fine, thanks for spotting this!

Checked in on HEAD and MOZILLA_1_8_BRANCH

-> FIXED
Attachment #277330 - Flags: review?(philipp) → review+
Status: ASSIGNED → RESOLVED
Last Resolved: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → 0.7
(Assignee)

Updated

11 years ago
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.