Move getCalendarDirectory into calProviderUtils.jsm

RESOLVED FIXED in 6.2

Status

enhancement
RESOLVED FIXED
Last year
Last year

People

(Reporter: Fallen, Assigned: Fallen)

Tracking

Lightning 6.2
Dependency tree / graph

Details

Attachments

(1 attachment)

This function is only used by the providers right now. I'm fine with moving it somewhere else instead, but this seemed acceptable.
Posted patch Fix - v1 β€” β€” Splinter Review
In patch order, this is after bug 1436557.
Attachment #8949202 - Flags: review?(makemyday)

Comment 3

Last year
Comment on attachment 8949202 [details] [diff] [review]
Fix - v1

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

looks good, r=me
Attachment #8949202 - Flags: review?(makemyday) → review+
Please push both the automatic and manual patches, after the bug mentioned in comment 1.
Keywords: checkin-needed

Comment 5

Last year
Pushed by mozilla@jorgk.com:
https://hg.mozilla.org/comm-central/rev/f1a768d1bbed
Move getCalendarDirectory into calProviderUtils.jsm. r=MakeMyDay
Status: ASSIGNED → RESOLVED
Closed: Last year
Keywords: checkin-needed
Resolution: --- → FIXED

Updated

Last year
Target Milestone: --- → 6.2
You need to log in before you can comment on or make changes to this bug.