diff options
| author | Tor Lillqvist <tml@collabora.com> | 2018-01-12 14:16:51 +0200 |
|---|---|---|
| committer | Miklos Vajna <vmiklos@collabora.co.uk> | 2018-04-03 13:06:31 +0200 |
| commit | 825ba822a9d445e81ff8b05aac69949cad40182b (patch) | |
| tree | 3f300db597c0be0435edf5cce82e74053a4a346d /scripting | |
| parent | ab4f89a3b0ece6284379b10af1e7317aa3864009 (diff) | |
Filter out "SynchronMode" too
Change-Id: I2c5111ee34929b9740796f5e1f08b3a8a58218e4
Reviewed-on: https://gerrit.libreoffice.org/51964
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Tor Lillqvist <tml@collabora.com>
(cherry picked from commit 2147cbf6204ebca8fb5a306ad5d81215b1175ac2)
Reviewed-on: https://gerrit.libreoffice.org/52089
Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk>
Tested-by: Miklos Vajna <vmiklos@collabora.co.uk>
Diffstat (limited to 'scripting')
| -rw-r--r-- | scripting/source/protocolhandler/scripthandler.cxx | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/scripting/source/protocolhandler/scripthandler.cxx b/scripting/source/protocolhandler/scripthandler.cxx index 2315fc1963fa..3f17347aa9ca 100644 --- a/scripting/source/protocolhandler/scripthandler.cxx +++ b/scripting/source/protocolhandler/scripthandler.cxx @@ -200,10 +200,12 @@ void SAL_CALL ScriptProtocolHandler::dispatchWithNotification( int argCount = 0; for ( int index = 0; index < lArgs.getLength(); index++ ) { - // Sometimes we get a propertyval with name = "Referer" - // this is not an argument to be passed to script, so - // ignore. - if ( lArgs[ index ].Name != "Referer" || + // Sometimes we get a propertyval with name = "Referer" or "SynchronMode". These + // are not actual arguments to be passed to script, but flags describing the + // call, so ignore. Who thought that passing such "meta-arguments" mixed in with + // real arguments was a good idea? + if ( (lArgs[ index ].Name != "Referer" && + lArgs[ index ].Name != "SynchronMode") || lArgs[ index ].Name.isEmpty() ) //TODO:??? { inArgs.realloc( ++argCount ); |