for listeners the range needs to be set before the formula, tdf#115530

Reviewed-on: https://gerrit.libreoffice.org/49957Reviewed-by: Aron Budea <aron.budea@collabora.com>
Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
Tested-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
(cherry picked from commit 26b51c95)
Reviewed-on: https://gerrit.libreoffice.org/50016Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Eike Rathke <erack@redhat.com>
(cherry picked from commit 56e08957)

Change-Id: I001795fd456375c4babab2c2e505bedff03e991f

sc/source/filter/oox/condformatbuffer.cxx

@@ -1060,10 +1060,6 @@ void CondFormat::finalizeImport()
     // probably some error in the xml if we are not ready
     if ( !mbReadyForFinalize )
         return;
-    ScDocument& rDoc = getScDocument();
-    maRules.forEachMem( &CondFormatRule::finalizeImport );
-    SCTAB nTab = maModel.maRanges.getBaseAddress().Tab();
-    sal_Int32 nIndex = getScDocument().AddCondFormat(mpFormat, nTab);
 
     ScRangeList aList;
     for( ::std::vector< CellRangeAddress >::const_iterator itr = maModel.maRanges.begin(); itr != maModel.maRanges.end(); ++itr)
@@ -1072,8 +1068,14 @@ void CondFormat::finalizeImport()
         ScUnoConversion::FillScRange(aRange, *itr);
         aList.Append(aRange);
     }
-    rDoc.AddCondFormatData( aList, nTab, nIndex );
     mpFormat->SetRange(aList);
+
+    ScDocument& rDoc = getScDocument();
+    maRules.forEachMem( &CondFormatRule::finalizeImport );
+    SCTAB nTab = maModel.maRanges.getBaseAddress().Tab();
+    sal_Int32 nIndex = getScDocument().AddCondFormat(mpFormat, nTab);
+
+    rDoc.AddCondFormatData( aList, nTab, nIndex );
 }
 
 CondFormatRuleRef CondFormat::createRule()