Note: There are a few cases of duplicates in user autocompletion which are being worked on.

Get rid of calIRecurrenceDateSet

RESOLVED FIXED in 2.0

Status

Calendar
Internal Components
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: Fallen, Assigned: Fallen)

Tracking

unspecified
Dependency tree / graph

Details

Attachments

(1 attachment)

(Assignee)

Description

5 years ago
This interface isn't used often and items are usually parsed into multiple EXDATE properties anyway. This will help me transition to using the jsical library in the future.
(Assignee)

Comment 1

5 years ago
Created attachment 655366 [details] [diff] [review]
Fix - v1
Attachment #655366 - Flags: review?(matthew.mecca)
(Assignee)

Updated

5 years ago
Blocks: 785733
Comment on attachment 655366 [details] [diff] [review]
Fix - v1

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

r=mmecca

::: calendar/providers/storage/calStorageCalendar.js
@@ +1814,4 @@
>          item.setProperty("LAST-MODIFIED", savedLastModifiedTime);
>      },
>  
> +    addRecurrenceItemsFromRow: function cSC_getRecurrenceItemFromRow(row, item) {

I'd suggest cSC_addRecurrenceItemsFromRow for consistency
Attachment #655366 - Flags: review?(matthew.mecca) → review+
(In reply to Matthew Mecca [:mmecca] from comment #2)
> > +    addRecurrenceItemsFromRow: function cSC_getRecurrenceItemFromRow(row, item) {
> 
> I'd suggest cSC_addRecurrenceItemsFromRow for consistency

Nevermind, I see the name changes in Bug 785733
(Assignee)

Comment 4

5 years ago
Pushed to comm-central changeset 0e6bba2e7c54
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → 2.0
You need to log in before you can comment on or make changes to this bug.