[Libreoffice-commits] core.git: Branch 'libreoffice-7-2' - sw/source
Caolán McNamara (via logerrit)
logerrit at kemper.freedesktop.org
Tue Sep 21 03:48:25 UTC 2021
sw/source/uibase/dbui/mmconfigitem.cxx | 1 +
1 file changed, 1 insertion(+)
New commits:
commit b8700e9d29e0ec2b8a91dd5a360ceee57a1f3c3d
Author: Caolán McNamara <caolanm at redhat.com>
AuthorDate: Mon Sep 20 14:03:26 2021 +0100
Commit: Adolfo Jayme Barrientos <fitojb at ubuntu.com>
CommitDate: Tue Sep 21 05:47:52 2021 +0200
Resolves: tdf#144620 toolbar config item overwrites options dialog one
There are two SwMailMergeConfigItem involved. One SwMailMergeConfigItem
from the toolbar created and considered modified and with changes not
commited to the config yet.
Then the other SwMailMergeConfigItem is created by the options dialog
and commited on ok, but after that commit at
cui/source/options/treeopt.cxx at line 723 there is a
utl::ConfigManager::storeConfigItems() to flush all outstanding config
items, so the one belonging to the toolbar is now flushed after the
options dialog one was written.
The SwMailMergeConfigItem has a IsModified() of true right after ctor
which doesn't seem intentional, there is no explicit set of Modified to
true on setting the simple bool members, the Modified bit is toggled on
when using the more complicated modifier methods during ctor so very
much looks like an accidental side effect.
Change-Id: If84a6f01c7bf92704dd1e175a2bd8e2ea59e157f
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/122280
Tested-by: Jenkins
Reviewed-by: Adolfo Jayme Barrientos <fitojb at ubuntu.com>
diff --git a/sw/source/uibase/dbui/mmconfigitem.cxx b/sw/source/uibase/dbui/mmconfigitem.cxx
index 26462bccf41a..744373edefc3 100644
--- a/sw/source/uibase/dbui/mmconfigitem.cxx
+++ b/sw/source/uibase/dbui/mmconfigitem.cxx
@@ -290,6 +290,7 @@ SwMailMergeConfigItem_Impl::SwMailMergeConfigItem_Impl() :
case 39: pValues[nProp] >>= m_nCurrentAddressBlock; break;
}
}
+ ClearModified();
}
//read the list of data base assignments
Sequence<OUString> aAssignments = GetNodeNames(cAddressDataAssignments);
More information about the Libreoffice-commits
mailing list