1134944 - Move getSummaryFile to folder so that maildir can be selected in imap mail setup

Status: RESOLVED FIXED

(MailNews Core :: Backend, defect)

Description

[Kent James (:rkent)]

This bug is a followup to bug 1096006.

Although UI was added to account setup in that bug, that UI is already disabled when you reach advanced account setup. By moving getSummaryFile to the folder, when we make maildir setup work for imap setup.

Comment 1

[Kent James (:rkent)]

Sorry, this is a followup to bug 1124015.

Comment 2

[Kent James (:rkent)]

Attachment: move getSummaryFile to folder

This is the same patch as bug 1124015 part 1

Comment 3

[Joshua Cranmer [:jcranmer]]

Comment on attachment 8566923 [details] [diff] [review]
move getSummaryFile to folder

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

::: mailnews/base/public/nsIMsgFolder.idl
@@ +454,5 @@
>     */
>    attribute nsIFile filePath;
>  
> +  /// an nsIFile corresponding to the .msf file.
> +  readonly attribute nsIFile summaryFile;

I wonder if this wouldn't be better named "databaseFile" ?

Comment 4

[Kent James (:rkent)]

(In reply to Joshua Cranmer [:jcranmer] from comment #3)
> Comment on attachment 8566923 [details] [diff] [review]
> move getSummaryFile to folder
> 
> Review of attachment 8566923 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> ::: mailnews/base/public/nsIMsgFolder.idl
> @@ +454,5 @@
> >     */
> >    attribute nsIFile filePath;
> >  
> > +  /// an nsIFile corresponding to the .msf file.
> > +  readonly attribute nsIFile summaryFile;
> 
> I wonder if this wouldn't be better named "databaseFile" ?

I'm afraid that ship has sailed, and summaryFile is now in fairly common usage in mailnews code. Renaming it now would only create confusion.

Comment 5

[Kent James (:rkent)]

https://hg.mozilla.org/comm-central/rev/18c31acd69e5

Comment 6

[Kent James (:rkent)]

Comment on attachment 8566923 [details] [diff] [review]
move getSummaryFile to folder

Approval Request Comment
[Feature/regressing bug #]: none
[User impact if declined]: maildir cannot be chosen during new account setup
[Describe test coverage new/current, TreeHerder]: general tests cover this
[Risks and why]: Low, merely moves functionality up one level in the hierarchy
[String/UUID change made/needed]: UUID changed, we have agreed that UUID is not yet frozen in aurora.

I'll approve and push this after watchin the c-c landing.

Comment 7

[Lukas Blakk [:lsblakk] use ?needinfo]

Comment on attachment 8566923 [details] [diff] [review]
move getSummaryFile to folder

since this landed to comm-central, you probably meant to set the approval-comm-aurora flag instead.

Comment 8

[Kent James (:rkent)]

https://hg.mozilla.org/releases/comm-aurora/rev/90580bc940ab