Closed Bug 380376 Opened 16 years ago Closed 15 years ago

CalDAV provider source code has a reference to non-existent "calendarToReturn" property

Categories

(Calendar :: Provider: CalDAV, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: damiano.albani, Assigned: browning)

Details

Attachments

(1 file)

User-Agent:       Mozilla/5.0 (X11; U; Linux i686; fr-FR; rv:1.8.1.3) Gecko/20070310 Iceweasel/2.0.0.3 (Debian-2.0.0.3-1)
Build Identifier: 

See <http://mxr.mozilla.org/mozilla/source/calendar/providers/caldav/calDavCalendar.js#562>

 562                 aListener.onOperationComplete (this.calendarToReturn,
 563                                                Components.results.NS_ERROR_FAILURE,
 564                                                aListener.MODIFY,
 565                                                aNewItem.id,
 566                                                "generation mismatch in modifyItem");

Yet the JS component doesn't have any "calendarToReturn" property -- unlike ICS provider.
Don't know what errors this could trigger though...

Reproducible: Always

Steps to Reproduce:
1.
2.
3.
We may actually need a calendarToReturn prop at some point, but we don't have it now, so this reference to it has got to go.
Assignee: nobody → browning
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Attachment #264560 - Flags: review?(ctalbert)
Comment on attachment 264560 [details] [diff] [review]
return proper calendar

r=ctalbert
Attachment #264560 - Flags: review?(ctalbert) → review+
Keywords: checkin-needed
checkin-needed for attachment 264560 [details] [diff] [review] (TRUNK and MOZILLA_1_8_BRANCH)
Checked in on HEAD and MOZILLA_1_8_BRANCH.
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Target Milestone: --- → 0.7
You need to log in before you can comment on or make changes to this bug.