From acee88be992fbf79ba85e83a2657785f36dbb767 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Axel=20D=C3=B6rfler?= Date: Thu, 13 Dec 2012 22:36:09 +0100 Subject: [PATCH] E-mail settings are now stored as they should. * ProtocolSettingsView was still using Archive() instead of SaveInto(). * However, it looks like BMessage::Set*() does corrupt the message fields. --- src/preferences/mail/ConfigViews.cpp | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/src/preferences/mail/ConfigViews.cpp b/src/preferences/mail/ConfigViews.cpp index 10d3223422..7695af938e 100644 --- a/src/preferences/mail/ConfigViews.cpp +++ b/src/preferences/mail/ConfigViews.cpp @@ -158,13 +158,9 @@ ProtocolSettingsView::DetachedFromWindow() if (fSettingsView == NULL) return; - BMessage settings; - if (fSettingsView->Archive(&settings) != B_OK) + if (fSettingsView->SaveInto(fSettings) != B_OK) return; - fSettings.MakeEmpty(); - fSettings.Append(settings); - // We need to remove the settings view before unloading its add-on fSettingsView->RemoveSelf(); delete fSettingsView;