diff options
author | Tamás Zolnai <tamas.zolnai@collabora.com> | 2018-04-19 12:10:52 +0200 |
---|---|---|
committer | Tamás Zolnai <tamas.zolnai@collabora.com> | 2018-04-19 14:05:21 +0200 |
commit | 67919621cb0b95074c4401bdfced9d87b230cc2f (patch) | |
tree | b7364b38de39a3a260cbe956e8daba78fac3affd /sfx2 | |
parent | 4ec8afafea320f40806abc7bdc9149a9b26b3d4c (diff) |
sw lok: Accept / reject change is always enabled in the context menu
Use conditional fastcall for allow executing these two uno commands
even if there is no tracked changes at the cursor position instead of
enable the menu slots unconditionally.
See also:
b6011f07254f8003929320ad842d8d09daca0e09
Change-Id: Iaf8a8082961cd174c038fc021d2c41fb7cb97bff
Reviewed-on: https://gerrit.libreoffice.org/53148
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
Diffstat (limited to 'sfx2')
-rw-r--r-- | sfx2/source/control/dispatch.cxx | 18 | ||||
-rw-r--r-- | sfx2/source/control/shell.cxx | 15 |
2 files changed, 16 insertions, 17 deletions
diff --git a/sfx2/source/control/dispatch.cxx b/sfx2/source/control/dispatch.cxx index c05609f01ce7..d3e403d70e9c 100644 --- a/sfx2/source/control/dispatch.cxx +++ b/sfx2/source/control/dispatch.cxx @@ -306,22 +306,6 @@ bool SfxDispatcher::IsAppDispatcher() const return !xImp->pFrame; } -/// Decides if the request is FASTCALL or not, depending on arguments. -bool lcl_IsConditionalFastCall(SfxRequest const &rReq) -{ - sal_uInt16 nId = rReq.GetSlot(); - bool bRet = false; - - if (nId == SID_UNDO || nId == SID_REDO) - { - const SfxItemSet* pArgs = rReq.GetArgs(); - if (pArgs && pArgs->HasItem(SID_REPAIRPACKAGE)) - bRet = true; - } - - return bRet; -} - /** Helper function to check whether a slot can be executed and check the execution itself */ @@ -330,7 +314,7 @@ void SfxDispatcher::Call_Impl(SfxShell& rShell, const SfxSlot &rSlot, SfxRequest SFX_STACK(SfxDispatcher::Call_Impl); // The slot may be called (meaning enabled) - if ( rSlot.IsMode(SfxSlotMode::FASTCALL) || rShell.CanExecuteSlot_Impl(rSlot) || lcl_IsConditionalFastCall(rReq)) + if ( rSlot.IsMode(SfxSlotMode::FASTCALL) || rShell.CanExecuteSlot_Impl(rSlot) || rShell.IsConditionalFastCall(rReq)) { if ( GetFrame() ) { diff --git a/sfx2/source/control/shell.cxx b/sfx2/source/control/shell.cxx index 2f25891ce5e5..c89d162ebe9d 100644 --- a/sfx2/source/control/shell.cxx +++ b/sfx2/source/control/shell.cxx @@ -388,6 +388,21 @@ bool SfxShell::CanExecuteSlot_Impl( const SfxSlot &rSlot ) return aSet.GetItemState(nId) != SfxItemState::DISABLED; } +bool SfxShell::IsConditionalFastCall( const SfxRequest &rReq ) +{ + sal_uInt16 nId = rReq.GetSlot(); + bool bRet = false; + + if (nId == SID_UNDO || nId == SID_REDO) + { + const SfxItemSet* pArgs = rReq.GetArgs(); + if (pArgs && pArgs->HasItem(SID_REPAIRPACKAGE)) + bRet = true; + } + return bRet; +} + + void ShellCall_Impl( void* pObj, void* pArg ) { static_cast<SfxShell*>(pObj)->ExecuteSlot( *static_cast<SfxRequest*>(pArg) ); |