filter/source/xsltfilter/XSLTFilter.cxx |   12 +++++++++++-
 1 file changed, 11 insertions(+), 1 deletion(-)

New commits:
commit f4238ed900cdfa65d447fbef81b2e8ae95095a4e
Author:     Stephan Bergmann <sberg...@redhat.com>
AuthorDate: Tue Feb 14 09:46:43 2023 +0000
Commit:     Stephan Bergmann <sberg...@redhat.com>
CommitDate: Tue Feb 14 13:21:30 2023 +0000

    Revert "XSLT2Transformer::create should always succeed"
    
    This reverts commit 304ebfad5193c857e4edf18cb05d61b6ccc1f7d0.  As mst 
pointed
    out on IRC, "LO does not ship this service, it's only provided by Saxon
    extension".
    
    Change-Id: Iafb416f3a3385345fd74cf76af4e9f600ff8eb79
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/146975
    Tested-by: Jenkins
    Reviewed-by: Stephan Bergmann <sberg...@redhat.com>

diff --git a/filter/source/xsltfilter/XSLTFilter.cxx 
b/filter/source/xsltfilter/XSLTFilter.cxx
index a48edd6d30cb..e3be62221468 100644
--- a/filter/source/xsltfilter/XSLTFilter.cxx
+++ b/filter/source/xsltfilter/XSLTFilter.cxx
@@ -232,7 +232,17 @@ namespace XSLT
         // service name there)
         if (rTransformer.toBoolean() || rTransformer.startsWith("com.sun."))
         {
-            xTransformer = xslt::XSLT2Transformer::create(m_xContext, rArgs);
+            try
+            {
+                xTransformer = xslt::XSLT2Transformer::create(m_xContext, 
rArgs);
+            }
+            catch (const Exception&)
+            {
+                // TODO: put a dialog telling about the need to install
+                // xslt2-transformer extension here
+                SAL_WARN("filter.xslt", "could not create XSLT 2.0 
transformer");
+                throw;
+            }
         }
         else
         {

Reply via email to