[Libreoffice-commits] .: sc/source

[Libreoffice Gerrit user]

 sc/source/core/data/dpcachetable.cxx |    6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

New commits:
commit acfdd93f31ae59bb688c27d2fd57e1940a6b9370
Author: Kohei Yoshida <kohei.yoshida at gmail.com>
Date:   Tue Oct 30 20:38:59 2012 -0400

    fdo#56325: Don't forget to build_tree() in fillTable().
    
    Else you'll get the wrong answer in isRowActive(), which always search
    flag via tree.
    
    Change-Id: Ic764b08c3fa3e2e24ea656bb03cc9d393e06fb13

diff --git a/sc/source/core/data/dpcachetable.cxx b/sc/source/core/data/dpcachetable.cxx
index d877955..a71a8f2 100644
--- a/sc/source/core/data/dpcachetable.cxx
+++ b/sc/source/core/data/dpcachetable.cxx
@@ -143,6 +143,7 @@ void ScDPCacheTable::fillTable(
 
     maShowByFilter.clear();
     maShowByPage.clear();
+    maShowByPage.build_tree();
 
     // Process the non-empty data rows.
     for (SCROW nRow = 0; nRow < nDataSize; ++nRow)
@@ -214,9 +215,12 @@ void ScDPCacheTable::fillTable()
     if (nRowCount <= 0 || nColCount <= 0)
         return;
 
-    maShowByFilter.clear();
     maShowByPage.clear();
+    maShowByPage.build_tree();
+
+    maShowByFilter.clear();
     maShowByFilter.insert_front(0, nRowCount, true);
+    maShowByFilter.build_tree();
 
     // Initialize field entries container.
     maFieldEntries.clear();