sfx2/qa/complex/sfx2/UndoManager.java |    4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

New commits:
commit cdf70b909910974cad49a5c4372d2682209cc831
Author:     Michael Stahl <michael.st...@allotropia.de>
AuthorDate: Tue May 10 13:16:52 2022 +0200
Commit:     Michael Stahl <michael.st...@allotropia.de>
CommitDate: Tue May 10 15:51:15 2022 +0200

    sfx2: add config warning in UndoManager complex test
    
    ... and remove obsolete comment, that problem was fixed a decade ago.
    
    Change-Id: I06e122128ff009778da87891739016ecc8975895
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/134110
    Tested-by: Jenkins
    Reviewed-by: Michael Stahl <michael.st...@allotropia.de>

diff --git a/sfx2/qa/complex/sfx2/UndoManager.java 
b/sfx2/qa/complex/sfx2/UndoManager.java
index a455a5eb4ca0..12f13ada10a1 100644
--- a/sfx2/qa/complex/sfx2/UndoManager.java
+++ b/sfx2/qa/complex/sfx2/UndoManager.java
@@ -224,7 +224,9 @@ public class UndoManager
         };
         events.replaceByName( "OnViewCreated", scriptDescriptor );
 
-        // The below doesn't work: event notification is broken in m96, see 
https://bz.apache.org/ooo/show_bug.cgi?id=116313
+        // note: this may be prevented from working by setting
+        // Office::Common::Security::Scripting::AllowedDocumentEventURLs
+        // (checked in SfxEvents_Impl::isScriptURLAllowed())
         m_callbackCalled = false;
         m_currentDocument.getCurrentView().dispatch( ".uno:NewWindow" );
         assertTrue( "triggering an event did not work as expected - basic 
script not called", m_callbackCalled );

Reply via email to