diff options
| author | Noel Grandin <noelgrandin@gmail.com> | 2022-12-31 08:52:53 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2022-12-31 07:49:03 +0000 |
| commit | c6f756e968bbb72522bd01dea283288a61a286a7 (patch) | |
| tree | 5289e93f7d7c38213b3bcc323c9a5e078e7c85a7 /sw | |
| parent | 5a06b3b789063e77afe7395226359c6ce8aad156 (diff) | |
tdf#152676 Writer: Compatibility options no longer saved as default
regression from
commit 78adf246d5e99d0f5d91d2e03c1379b154289d8d
Date: Mon Jul 26 10:54:36 2021 +0200
return SvtCompatibilityEntry from SvtCompatibilityOptions::GetList
where I mistakenly assumed that the bIsDefault flag in
SvtCompatibilityEntry was being set by the SvtCompatibilityOptions
code. Rather remove this field if it is not going to be maintained
properly.
Change-Id: I23963ae885563d5abc8a99cfd144d7fe00c6f192
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/144882
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'sw')
| -rw-r--r-- | sw/source/ui/config/optcomp.cxx | 10 |
1 files changed, 8 insertions, 2 deletions
diff --git a/sw/source/ui/config/optcomp.cxx b/sw/source/ui/config/optcomp.cxx index 0ba1b147614c..d760e885a66e 100644 --- a/sw/source/ui/config/optcomp.cxx +++ b/sw/source/ui/config/optcomp.cxx @@ -205,10 +205,11 @@ void SwCompatibilityOptPage::InitControls( const SfxItemSet& rSet ) { const OUString sEntryName = rEntry.getValue<OUString>( SvtCompatibilityEntry::Index::Name ); const bool bIsUserEntry = ( sEntryName == SvtCompatibilityEntry::USER_ENTRY_NAME ); + const bool bIsDefaultEntry = ( sEntryName == SvtCompatibilityEntry::DEFAULT_ENTRY_NAME ); m_pImpl->m_aList.push_back( rEntry ); - if ( rEntry.isDefaultEntry() ) + if ( bIsDefaultEntry ) continue; OUString sNewEntry; @@ -260,7 +261,12 @@ IMPL_LINK_NOARG(SwCompatibilityOptPage, UseAsDefaultHdl, weld::Button&, void) return; auto pItem = std::find_if(m_pImpl->m_aList.begin(), m_pImpl->m_aList.end(), - [](const SvtCompatibilityEntry& rItem) { return rItem.isDefaultEntry(); }); + [](const SvtCompatibilityEntry& rItem) + { + const OUString sEntryName = rItem.getValue<OUString>( SvtCompatibilityEntry::Index::Name ); + const bool bIsDefaultEntry = ( sEntryName == SvtCompatibilityEntry::DEFAULT_ENTRY_NAME ); + return bIsDefaultEntry; + }); if (pItem != m_pImpl->m_aList.end()) { const sal_Int32 nCount = m_xOptionsLB->n_children(); |