1500826 - getFolderFromUri is broken

Status: RESOLVED FIXED

(MailNews Core :: General, enhancement)

Description

[Geoff Lankow (:darktrojan)]

https://searchfox.org/comm-central/source/mailnews/base/util/folderUtils.jsm#111

getFolderById isn't a member of nsIFolderLookupService. It hasn't really mattered since nothing uses it, but now I want to. Maybe I'll just make the service a member of MailServices.

Comment 1

[Geoff Lankow (:darktrojan)]

Attachment: 1500826-folder-from-uri-1.diff

Comment 2

[Jorg K (CEST = GMT+2)]

Comment on attachment 9019181 [details] [diff] [review]
1500826-folder-from-uri-1.diff

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

r+ wc (with comments, as the Calendar people say).

::: mailnews/base/util/MailServices.jsm
@@ +73,5 @@
>                                     "mozINewMailNotificationService");
> +
> +XPCOMUtils.defineLazyServiceGetter(MailServices, "folderLookup",
> +                                   "@mozilla.org/mail/folder-lookup;1",
> +                                   "nsIFolderLookupService");

Why add it in this patch? If it's a preparation for some use elsewhere, please add it in that bug. No need to add unused code here.

Or am I missing something?

::: mailnews/base/util/folderUtils.jsm
@@ -114,5 @@
> - * @param aUri  the rdf uri of the folder to return
> - */
> -function getFolderFromUri(aUri) {
> -  return Cc["@mozilla.org/mail/folder-lookup;1"].
> -         getService(Ci.nsIFolderLookupService).getFolderById(aUri);

OK, this is broken and gets removed. Fine.

Comment 3

[Geoff Lankow (:darktrojan)]

Attachment: 1500826-folder-from-uri-2.diff

Fair enough.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/95f2852bdd11
Remove broken folderUtils.getFolderFromUri. r=jorgk