diff options
| author | Vitaliy Anderson <vanderson@smartru.com> | 2017-08-19 17:18:19 +0000 |
|---|---|---|
| committer | Michael Stahl <mstahl@redhat.com> | 2017-08-22 15:46:38 +0200 |
| commit | 52275c689bcdd46f23ff5ebd5cfcc94614bfbffe (patch) | |
| tree | 0c888b27856f35a398f28cb2f0bf3deb4d1df1a0 /unotools/source | |
| parent | 33b094a8949c34756c593bfad52450ec2b7daa54 (diff) | |
tdf#110355: regression fix
The regression from 91ccb4dbf7cbe7e684c7a8183863e597d7205e57
"Compatibility
options refactoring. Part 1" patch.
SvtCompatibilityOptions_Impl::SvtCompatibilityOptions_Impl() solve
problem with wrong filling SvtCompatibilityEntry item fields from
lValues sequence
Change-Id: I695ad78bacbcce41b19b5fb90b86ff08fc041971
Reviewed-on: https://gerrit.libreoffice.org/41328
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Michael Stahl <mstahl@redhat.com>
Diffstat (limited to 'unotools/source')
| -rw-r--r-- | unotools/source/config/compatibility.cxx | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/unotools/source/config/compatibility.cxx b/unotools/source/config/compatibility.cxx index bea5cb841a3b..aa10fbc18601 100644 --- a/unotools/source/config/compatibility.cxx +++ b/unotools/source/config/compatibility.cxx @@ -169,6 +169,7 @@ SvtCompatibilityOptions_Impl::SvtCompatibilityOptions_Impl() : ConfigItem( ROOTN // Get names/values for new menu. // 4 subkeys for every item! bool bDefaultFound = false; + sal_Int32 nDestStep = 0; for ( sal_uInt32 nItem = 0; nItem < nCount; ++nItem ) { SvtCompatibilityEntry aItem; @@ -176,7 +177,10 @@ SvtCompatibilityOptions_Impl::SvtCompatibilityOptions_Impl() : ConfigItem( ROOTN aItem.setValue<OUString>( SvtCompatibilityEntry::Index::Name, lNodes[ nItem ] ); for ( int i = static_cast<int>(SvtCompatibilityEntry::Index::Module); i < static_cast<int>(SvtCompatibilityEntry::Index::INVALID); ++i ) - aItem.setValue( SvtCompatibilityEntry::Index(i), lValues[ i - 1 ] ); + { + aItem.setValue( SvtCompatibilityEntry::Index(i), lValues[ nDestStep ] ); + nDestStep++; + } m_aOptions.push_back( aItem ); |