[Libreoffice-commits] .: Branch 'libreoffice-3-6' - sc/source

Libreoffice Gerrit user logerrit at kemper.freedesktop.org
Mon Dec 17 21:16:04 PST 2012


 sc/source/filter/oox/workbookhelper.cxx |    7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

New commits:
commit f81cae7a1e9d85ad3e17ced53bac40bfd1690a16
Author: Noel Power <noel.power at suse.com>
Date:   Thu Dec 13 16:52:50 2012 +0000

    fix sometimes data corruption with xlsx import ( with scenario sheets )
    
    formula import is buffered 'till the end of import. The processing of the
    formula data need to happen before the scenario import happens. This is
    necessary because sheet numbers stored in the formula addresses can become
    invalid as scenario import insertes new hidden sheets upsetting the previous
    table order
    
    Change-Id: I9357f028f31bec1b1504ca991f5534f80d79c9bc
    Reviewed-on: https://gerrit.libreoffice.org/1338
    Reviewed-by: Kohei Yoshida <kohei.yoshida at gmail.com>
    Tested-by: Kohei Yoshida <kohei.yoshida at gmail.com>

diff --git a/sc/source/filter/oox/workbookhelper.cxx b/sc/source/filter/oox/workbookhelper.cxx
index 071b9a6..dce942c 100644
--- a/sc/source/filter/oox/workbookhelper.cxx
+++ b/sc/source/filter/oox/workbookhelper.cxx
@@ -591,11 +591,6 @@ void WorkbookGlobals::finalize()
         // #i79826# enable updating automatic row height after loading the document
         aPropSet.setProperty( PROP_IsAdjustHeightEnabled, true );
 
-        getFormulaBuffer().finalizeImport();
-
-        // hack, setting it true the second time will delete the cache
-        aPropSet.setProperty( PROP_IsAdjustHeightEnabled, true );
-
         // Insert all pivot tables. Must be done after loading all sheets and
         // formulas, because data pilots expect existing source data on
         // creation.
@@ -667,6 +662,8 @@ void WorkbookHelper::finalizeWorkbookImport()
     mrBookGlob.getWorkbookSettings().finalizeImport();
     mrBookGlob.getViewSettings().finalizeImport();
 
+    // need to import formulas before scenarios
+    mrBookGlob.getFormulaBuffer().finalizeImport();
     /*  Insert scenarios after all sheet processing is done, because new hidden
         sheets are created for scenarios which would confuse code that relies
         on certain sheet indexes. Must be done after pivot tables too. */


More information about the Libreoffice-commits mailing list