diff options
| author | Caolán McNamara <caolanm@redhat.com> | 2020-10-29 16:01:02 +0000 |
|---|---|---|
| committer | Caolán McNamara <caolanm@redhat.com> | 2020-10-29 21:21:41 +0100 |
| commit | bdd9ccc3f1a0db9345ad0a0109aa8b65405650ab (patch) | |
| tree | 7cae6b677102be562579d94e342e7dcc13e121fc /cui | |
| parent | 1ec39615b91b299c48be90f134840d89517ab4c3 (diff) | |
move block for RET_OK conditional return into Ok handler
making it clearer what happens on "ok", no logic change intended
Change-Id: I1d57500d2fbeded4cd7c7fd48fd1f4296b78e41d
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/105018
Tested-by: Caolán McNamara <caolanm@redhat.com>
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Diffstat (limited to 'cui')
| -rw-r--r-- | cui/source/options/treeopt.cxx | 13 |
1 files changed, 4 insertions, 9 deletions
diff --git a/cui/source/options/treeopt.cxx b/cui/source/options/treeopt.cxx index 8ec8a0b3d252..5cb3fde4677b 100644 --- a/cui/source/options/treeopt.cxx +++ b/cui/source/options/treeopt.cxx @@ -738,6 +738,9 @@ IMPL_LINK_NOARG(OfaTreeOptionsDialog, OKHdl_Impl, weld::Button&, void) ::svtools::executeRestartDialog(comphelper::getProcessComponentContext(), m_pParent, eRestartReason); } + + ApplyItemSets(); + utl::ConfigManager::storeConfigItems(); } void OfaTreeOptionsDialog::ApplyItemSets() @@ -1932,15 +1935,7 @@ short OfaTreeOptionsDialog::run() pClamp.reset( new SvxDicListChgClamp( xDictionaryList ) ); } - short nRet = SfxOkDialogController::run(); - - if( RET_OK == nRet ) - { - ApplyItemSets(); - utl::ConfigManager::storeConfigItems(); - } - - return nRet; + return SfxOkDialogController::run(); } // class ExtensionsTabPage ----------------------------------------------- |