diff options
| author | Luboš Luňák <l.lunak@collabora.com> | 2021-11-12 10:56:54 +0100 |
|---|---|---|
| committer | Luboš Luňák <l.lunak@collabora.com> | 2021-11-12 13:02:08 +0100 |
| commit | 309bffba603ae65e76148dff3add1a541a2b5d81 (patch) | |
| tree | 42d839b28398ed3397ef20038d624f69db69e3a4 /desktop | |
| parent | sw, out of order undo: allow multiple actions from other views (diff) | |
| download | core-309bffba603ae65e76148dff3add1a541a2b5d81.tar.gz core-309bffba603ae65e76148dff3add1a541a2b5d81.zip | |
make it explicit whether to ignore the result of getLOKPayload()
Returning an empty string to signify 'ignore' was a poor design,
as some messages types actually may have valid empty messages.
Change-Id: Ia82d3d97d150bc5ef412a1bd4b1091d9b2d84385
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/124979
Tested-by: Jenkins
Reviewed-by: Luboš Luňák <l.lunak@collabora.com>
Diffstat (limited to 'desktop')
| -rw-r--r-- | desktop/source/lib/init.cxx | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/desktop/source/lib/init.cxx b/desktop/source/lib/init.cxx index 6a1e0871fceb..2191a87f5b87 100644 --- a/desktop/source/lib/init.cxx +++ b/desktop/source/lib/init.cxx @@ -2191,8 +2191,9 @@ void CallbackFlushHandler::enqueueUpdatedTypes() void CallbackFlushHandler::enqueueUpdatedType( int type, const SfxViewShell* viewShell, int viewId ) { - OString payload = viewShell->getLOKPayload( type, viewId ); - if(payload.isEmpty()) + bool ignore = false; + OString payload = viewShell->getLOKPayload( type, viewId, &ignore ); + if(ignore) return; // No actual payload to send. CallbackData callbackData(payload.getStr(), viewId); m_queue1.emplace_back(type); |