tdf#110355: regression fix

The regression from 91ccb4db
"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/41328Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Michael Stahl <mstahl@redhat.com>

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 );