[Libreoffice-commits] core.git: sc/source

[Noel Grandin (via logerrit)]

 sc/source/filter/oox/workbookfragment.cxx |   12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

New commits:
commit c9b81503a85a2763e1bfd1b6722fcf6d96a983e7
Author:     Noel Grandin <noelgrandin at gmail.com>
AuthorDate: Thu Aug 27 19:02:27 2020 +0200
Commit:     Noel Grandin <noel.grandin at collabora.co.uk>
CommitDate: Fri Aug 28 20:23:35 2020 +0200

    tdf#136113 shape editing does not change modified status
    
    regression from
        commit edf13fe1247e7ef411a9ff5435385573fad01f56
        tdf#93831 xlsx file full of pictures of numbers slow to open
    
    No real idea why this fixes it - I suspect that the
    referenced commit was forcing the initialisation of the
    drawing layer too early.
    
    This commit does not making the related xlsx file any slower
    to load.
    
    Change-Id: I1d8fdc5134a1b70bf214df642a9e7a386b853895
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/101491
    Tested-by: Jenkins
    Reviewed-by: Noel Grandin <noel.grandin at collabora.co.uk>

diff --git a/sc/source/filter/oox/workbookfragment.cxx b/sc/source/filter/oox/workbookfragment.cxx
index e6a531d88e1e..ac92e96d6e41 100644
--- a/sc/source/filter/oox/workbookfragment.cxx
+++ b/sc/source/filter/oox/workbookfragment.cxx
@@ -344,12 +344,6 @@ void importSheetFragments( WorkbookFragment& rWorkbookHandler, SheetFragmentVect
 
 void WorkbookFragment::finalizeImport()
 {
-    // lock the model to prevent broadcasting, speeds up load a lot
-    getScDocument().InitDrawLayer();
-    auto pModel = getScDocument().GetDrawLayer();
-    bool bWasLocked = pModel->isLocked();
-    pModel->setLock(true);
-
     ISegmentProgressBarRef xGlobalSegment = getProgressBar().createSegment( PROGRESS_LENGTH_GLOBALS );
 
     // read the theme substream
@@ -473,6 +467,12 @@ void WorkbookFragment::finalizeImport()
         }
     }
 
+    // lock the model to prevent broadcasting, speeds up load a lot
+    getScDocument().InitDrawLayer();
+    auto pModel = getScDocument().GetDrawLayer();
+    bool bWasLocked = pModel->isLocked();
+    pModel->setLock(true);
+
     // load all worksheets
     importSheetFragments(*this, aSheetFragments);