diff options
| author | Albert Thuswaldner <albert.thuswaldner@gmail.com> | 2013-05-04 01:39:24 +0200 |
|---|---|---|
| committer | Petr Mladek <pmladek@suse.cz> | 2013-05-07 13:28:34 +0200 |
| commit | e4c614bbd75f3103e1e88f9840645cc2c1f78a14 (patch) | |
| tree | 31378877386da656f636afca9743bc95ea807bef /sc | |
| parent | c0c38e90b1003d72b7ddfaddd923ee39d910c0cb (diff) | |
fdo#51828 - UI: Sort: can use more than 3 criteria once only - additional fix
Change-Id: I220b3755d3ea30bd9aa25b175f7f3eea7590ebdb
Reviewed-on: https://gerrit.libreoffice.org/3801
Reviewed-by: Miklos Vajna <vmiklos@suse.cz>
Tested-by: Miklos Vajna <vmiklos@suse.cz>
Diffstat (limited to 'sc')
| -rw-r--r-- | sc/source/ui/dbgui/tpsort.cxx | 40 | ||||
| -rw-r--r-- | sc/source/ui/inc/tpsort.hxx | 1 |
2 files changed, 25 insertions, 16 deletions
diff --git a/sc/source/ui/dbgui/tpsort.cxx b/sc/source/ui/dbgui/tpsort.cxx index 4ee9d1109307..034afec316ad 100644 --- a/sc/source/ui/dbgui/tpsort.cxx +++ b/sc/source/ui/dbgui/tpsort.cxx @@ -184,7 +184,6 @@ void ScTabPageSortFields::Reset( const SfxItemSet& /* rArgSet */ ) { maSortKeyItems[i].m_pLbSort->SelectEntryPos( GetFieldSelPos( aSortData.maKeyState[i].nField ) ); - (aSortData.maKeyState[i].bAscending) ? maSortKeyItems[i].m_pBtnUp->Check() : maSortKeyItems[i].m_pBtnDown->Check(); @@ -234,6 +233,10 @@ void ScTabPageSortFields::Reset( const SfxItemSet& /* rArgSet */ ) pDlg->SetByRows ( bSortByRows ); pDlg->SetHeaders( bHasHeader ); } + + // Make sure that there is always a last undefined sort key + if ( maSortKeyItems[nSortKeyCount - 1].m_pLbSort->GetSelectEntryPos() > 0 ) + SetLastSortKey( nSortKeyCount ); } // ----------------------------------------------------------------------- @@ -443,6 +446,25 @@ sal_uInt16 ScTabPageSortFields::GetFieldSelPos( SCCOLROW nField ) return nFieldPos; } +void ScTabPageSortFields::SetLastSortKey( sal_uInt16 nItem ) +{ + // Extend local SortParam copy + const ScSortKeyState atempKeyState = { false, 0, true }; + aSortData.maKeyState.push_back( atempKeyState ); + + // Add Sort Key Item + ++nSortKeyCount; + maSortKeyCtrl.AddSortKey( nSortKeyCount ); + maSortKeyItems[nItem].m_pLbSort->SetSelectHdl( + LINK( this, ScTabPageSortFields, SelectHdl ) ); + + FillFieldLists( nItem ); + + // Set Status + maSortKeyItems[nItem].m_pBtnUp->Check(); + maSortKeyItems[nItem].m_pLbSort->SelectEntryPos( 0 ); +} + // ----------------------------------------------------------------------- // Handler: //--------- @@ -451,26 +473,12 @@ IMPL_LINK( ScTabPageSortFields, SelectHdl, ListBox *, pLb ) { rtl::OUString aSelEntry = pLb->GetSelectEntry(); ScSortKeyItems::iterator pIter; - sal_uInt16 nSortKeyIndex = nSortKeyCount; // If last listbox is enabled add one item if ( maSortKeyItems.back().m_pLbSort == pLb ) if ( aSelEntry != aStrUndefined ) { - // Extend local SortParam copy - const ScSortKeyState atempKeyState = { false, 0, true }; - aSortData.maKeyState.push_back( atempKeyState ); - - // Add Sort Key Item - ++nSortKeyCount; - maSortKeyCtrl.AddSortKey( nSortKeyCount ); - maSortKeyItems[nSortKeyIndex].m_pLbSort->SetSelectHdl( LINK( this, ScTabPageSortFields, SelectHdl ) ); - - FillFieldLists( nSortKeyIndex ); - - // Set Status - maSortKeyItems[nSortKeyIndex].m_pBtnUp->Check(); - maSortKeyItems[nSortKeyIndex].m_pLbSort->SelectEntryPos( 0 ); + SetLastSortKey( nSortKeyCount ); return 0; } diff --git a/sc/source/ui/inc/tpsort.hxx b/sc/source/ui/inc/tpsort.hxx index 8ce4a4278a89..b23058136ba6 100644 --- a/sc/source/ui/inc/tpsort.hxx +++ b/sc/source/ui/inc/tpsort.hxx @@ -95,6 +95,7 @@ private: void Init (); void FillFieldLists ( sal_uInt16 nStartField ); sal_uInt16 GetFieldSelPos ( SCCOLROW nField ); + void SetLastSortKey( sal_uInt16 nItem ); // Handler ------------------------ DECL_LINK( SelectHdl, ListBox * ); |