diff options
author | Caolán McNamara <caolanm@redhat.com> | 2020-08-31 11:47:49 +0100 |
---|---|---|
committer | Miklos Vajna <vmiklos@collabora.com> | 2020-09-01 13:40:24 +0200 |
commit | c52ac477beefc46802148033adbca9edb9970ac1 (patch) | |
tree | a40b5f7aa458c5a41762c5d6c4f5dd96595a5089 /sc | |
parent | 9ee0dbd54a60b07ff692a842e49942e56f762f92 (diff) |
Resolves: tdf#136189 don't try and sort until treeview is filled
Change-Id: Ic5ac71d88bab7627e14220912f4b9c935f2f1f26
and...
Related: tdf#136189 don't assert on unsetting non-existing previous sort column
Change-Id: If2330cc83ace9ec0133b99eec8c2f0be3919013e
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/101710
Tested-by: Jenkins
Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
Diffstat (limited to 'sc')
-rw-r--r-- | sc/source/ui/dialogs/searchresults.cxx | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/sc/source/ui/dialogs/searchresults.cxx b/sc/source/ui/dialogs/searchresults.cxx index 950726a18c4e..0f94c64f32e7 100644 --- a/sc/source/ui/dialogs/searchresults.cxx +++ b/sc/source/ui/dialogs/searchresults.cxx @@ -196,7 +196,9 @@ IMPL_LINK(SearchResultsDlg, HeaderBarClick, int, nColumn, void) } else { - mxList->set_sort_indicator(TRISTATE_INDET, mxList->get_sort_column()); + int nOldSortColumn = mxList->get_sort_column(); + if (nOldSortColumn != -1) + mxList->set_sort_indicator(TRISTATE_INDET, nOldSortColumn); mxList->set_sort_column(nColumn); } |