463470 - Changing radio button for "Place a copy in" doesn't work (sent mail folder)

Status: RESOLVED FIXED

(Thunderbird :: Account Manager, defect)

Description

[Mark Banner (:standard8)]

Go into account manager, select Copies & Folders, under "Place a copy in" select a different option (either other or sent folder on). Select OK, and then go back into account settings.

Result: the radio button is back in its original setting.

Comment 1

[Magnus Melin [:mkmelin]]

... though if you really select a folder on another server it does work.

I think there's another bug also about restoring it to "<folder> on <server>" like it used to be.

Comment 2

[David :Bienvenu]

this is pretty bad...I'll double check that it still happens, and see if there's anything on the error console.

Comment 3

[David :Bienvenu]

the problem appears to be in setPickersState, we don't know what the currentAccount is - it's undefined, so we bail out early. But we seem to be setting it in other places...perhaps we're clearing it, or somehow don't have access to it.

Comment 4

[David :Bienvenu]

Attachment: proposed fix

this fixes it for me, but I'll do some more testing.

Comment 5

[David :Bienvenu]

I believe this was a regression from getting rid of rdf in the account setting dialog but I don't know that for sure.

Comment 6

[David :Bienvenu]

I'm going to put this in the b1 bucket, because if we do a respin, I'd like to take this fix.

Comment 7

[David :Bienvenu]

I'd be surprised if SM didn't have the same problem

Comment 8

[David :Bienvenu]

jminta might have some thoughts on this.

Comment 9

[Mark Banner (:standard8)]

Comment on attachment 351068 [details] [diff] [review]
proposed fix

This looks good to me.

Comment 10

[David :Bienvenu]

I've landed this on the trunk. I would want this for a respin, so I'm going to leave this open for now.

Comment 11

[David :Bienvenu]

marking fixed - we're tracking changes for the respin with a tracking bug.