diff options
| author | Kohei Yoshida <kohei.yoshida@gmail.com> | 2012-10-30 20:38:59 -0400 |
|---|---|---|
| committer | Kohei Yoshida <kohei.yoshida@gmail.com> | 2012-10-30 20:38:59 -0400 |
| commit | acfdd93f31ae59bb688c27d2fd57e1940a6b9370 (patch) | |
| tree | 484571da375ab3b2d07e58740ec9dab1626aa062 | |
| parent | Somehow that comment makes no sense. (diff) | |
| download | core-acfdd93f31ae59bb688c27d2fd57e1940a6b9370.tar.gz core-acfdd93f31ae59bb688c27d2fd57e1940a6b9370.zip | |
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
| -rw-r--r-- | sc/source/core/data/dpcachetable.cxx | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/sc/source/core/data/dpcachetable.cxx b/sc/source/core/data/dpcachetable.cxx index d8779554d6ab..a71a8f299c41 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(); |