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

Status: RESOLVED FIXED

(Calendar :: Provider: CalDAV, defect)

Description

[snotling]

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.

Comment 1

[Bruno Browning]

Attachment: return proper calendar

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.

Comment 2

[cmtalbert]

Comment on attachment 264560 [details] [diff] [review]
return proper calendar

r=ctalbert

Comment 3

[Stefan Sitter [:ssitter]]

checkin-needed for attachment 264560 [details] [diff] [review] (TRUNK and MOZILLA_1_8_BRANCH)

Comment 4

[Daniel Boelzle [:dbo]]

Checked in on HEAD and MOZILLA_1_8_BRANCH.