diff options
| author | Caolán McNamara <caolanm@redhat.com> | 2020-06-24 20:44:26 +0100 |
|---|---|---|
| committer | Caolán McNamara <caolanm@redhat.com> | 2020-06-25 20:25:44 +0200 |
| commit | 8f2e93183b22f44a9682ac6644d1626bfb89b516 (patch) | |
| tree | 36e59065e8a358411568c86fe2ca2fefd7a39f49 /svx/source | |
| parent | 00fa0d60ac20268f0a1c7cfd8a0f50b147e89169 (diff) | |
nActiveTerm is -1 in an empty filter navigator
Change-Id: Ia6eed6291375f136abbe8e50abae51e6f796911f
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/97080
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Diffstat (limited to 'svx/source')
| -rw-r--r-- | svx/source/form/filtnav.cxx | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/svx/source/form/filtnav.cxx b/svx/source/form/filtnav.cxx index 3c6c04108ea2..2b6dee83ebd8 100644 --- a/svx/source/form/filtnav.cxx +++ b/svx/source/form/filtnav.cxx @@ -635,7 +635,6 @@ FmFormItem* FmFilterModel::Find(const ::std::vector<std::unique_ptr<FmFilterData return nullptr; } - void FmFilterModel::SetCurrentController(const Reference< XFormController > & xCurrent) { if ( xCurrent == m_xController ) @@ -651,7 +650,7 @@ void FmFilterModel::SetCurrentController(const Reference< XFormController > & xC { Reference< XFilterController > xFilterController( m_xController, UNO_QUERY_THROW ); const sal_Int32 nActiveTerm( xFilterController->getActiveTerm() ); - if ( pItem->GetChildren().size() > o3tl::make_unsigned(nActiveTerm) ) + if (nActiveTerm != -1 && pItem->GetChildren().size() > o3tl::make_unsigned(nActiveTerm)) { SetCurrentItems( static_cast< FmFilterItems* >( pItem->GetChildren()[ nActiveTerm ].get() ) ); } @@ -662,7 +661,6 @@ void FmFilterModel::SetCurrentController(const Reference< XFormController > & xC } } - void FmFilterModel::AppendFilterItems( FmFormItem& _rFormItem ) { // insert the condition behind the last filter items |