diff options
| author | Eike Rathke <erack@redhat.com> | 2019-06-01 00:18:46 +0200 |
|---|---|---|
| committer | Eike Rathke <erack@redhat.com> | 2019-06-01 02:26:40 +0200 |
| commit | 7272b9f752cb74757d6ed504202eefccc589f804 (patch) | |
| tree | 326068d0516ecbf5c4cb07989808819ffa79d891 /sc | |
| parent | e9395f87619af7c9b755ee8ff34ecfaae7df9c98 (diff) | |
Resolves: tdf#101473 handle volatile Add-In results without formula cell
... while temporarily interpreting from within the Function Wizard.
Change-Id: I88e7e062989ed395accd50ed9bfe1b76b3b297ea
Reviewed-on: https://gerrit.libreoffice.org/73290
Reviewed-by: Eike Rathke <erack@redhat.com>
Tested-by: Jenkins
Diffstat (limited to 'sc')
| -rw-r--r-- | sc/source/core/tool/interpr4.cxx | 27 |
1 files changed, 25 insertions, 2 deletions
diff --git a/sc/source/core/tool/interpr4.cxx b/sc/source/core/tool/interpr4.cxx index 8cd191c6e00b..bb6583a2c460 100644 --- a/sc/source/core/tool/interpr4.cxx +++ b/sc/source/core/tool/interpr4.cxx @@ -3023,12 +3023,21 @@ void ScInterpreter::ScExternal() rArr.AddRecalcMode( ScRecalcMode::ONLOAD_LENIENT ); uno::Reference<sheet::XVolatileResult> xRes = aCall.GetVarRes(); ScAddInListener* pLis = ScAddInListener::Get( xRes ); + // In case there is no pMyFormulaCell, i.e. while interpreting + // temporarily from within the Function Wizard, try to obtain a + // valid result from an existing listener for that volatile, or + // create a new and hope for an immediate result. If none + // available that should lead to a void result and thus #N/A. + bool bTemporaryListener = false; if ( !pLis ) { pLis = ScAddInListener::CreateListener( xRes, pDok ); - pMyFormulaCell->StartListening( *pLis ); + if (pMyFormulaCell) + pMyFormulaCell->StartListening( *pLis ); + else + bTemporaryListener = true; } - else + else if (pMyFormulaCell) { pMyFormulaCell->StartListening( *pLis ); if ( !pLis->HasDocument( pDok ) ) @@ -3038,6 +3047,20 @@ void ScInterpreter::ScExternal() } aCall.SetResult( pLis->GetResult() ); // use result from async + + if (bTemporaryListener) + { + try + { + // EventObject can be any, not evaluated by + // ScAddInListener::disposing() + css::lang::EventObject aEvent; + pLis->disposing(aEvent); // pLis is dead hereafter + } + catch (const uno::Exception&) + { + } + } } if ( aCall.GetErrCode() != FormulaError::NONE ) |