[Libreoffice-commits] core.git: 3 commits - filter/Configuration_filter.mk filter/source writerperfect/Library_wpftcalc.mk writerperfect/Library_wpftimpress.mk writerperfect/qa writerperfect/source
osnola
alonso at loria.fr
Sun Apr 2 11:43:07 UTC 2017
filter/Configuration_filter.mk | 2
filter/source/config/fragments/filters/PowerPoint3.xcu | 2
filter/source/config/fragments/filters/StarOffice_Presentation.xcu | 29 +++
filter/source/config/fragments/types/StarOffice_Presentation.xcu | 27 ++
filter/source/config/fragments/types/calc_WPS_Lotus_Document.xcu | 2
filter/source/config/fragments/types/impress_PowerPoint3.xcu | 2
writerperfect/Library_wpftcalc.mk | 1
writerperfect/Library_wpftimpress.mk | 2
writerperfect/qa/unit/DrawingImportTest.cxx | 2
writerperfect/qa/unit/PresentationImportTest.cxx | 2
writerperfect/qa/unit/SpreadsheetImportTest.cxx | 2
writerperfect/qa/unit/TextImportTest.cxx | 2
writerperfect/source/calc/MSWorksCalcImportFilter.cxx | 41 +++-
writerperfect/source/common/DocumentHandler.cxx | 6
writerperfect/source/common/WPXSvInputStream.cxx | 24 +-
writerperfect/source/impress/MWAWPresentationImportFilter.cxx | 2
writerperfect/source/impress/StarOfficePresentationImportFilter.cxx | 95 ++++++++++
writerperfect/source/impress/StarOfficePresentationImportFilter.hxx | 42 ++++
writerperfect/source/impress/wpftimpress.component | 5
19 files changed, 263 insertions(+), 27 deletions(-)
New commits:
commit 0e261a6908c2521a9967069095239eeb02810b8c
Author: osnola <alonso at loria.fr>
Date: Sun Apr 2 09:56:42 2017 +0200
libmwaw import filter improvements
+ correct a small mistake
+ add pot the file extension (the file extension for PowerPoint 95
template)
Change-Id: I4ad18088ccb9777e1b34f393effbe2d24468804e
diff --git a/filter/source/config/fragments/filters/PowerPoint3.xcu b/filter/source/config/fragments/filters/PowerPoint3.xcu
index d61a360d4eb0..d1d19e6d2845 100644
--- a/filter/source/config/fragments/filters/PowerPoint3.xcu
+++ b/filter/source/config/fragments/filters/PowerPoint3.xcu
@@ -14,7 +14,7 @@
<value>com.sun.star.comp.Impress.MWAWPresentationImportFilter</value>
</prop>
<prop oor:name="UIName">
- <value xml:lang="en-US">Microsoft PowerPoint 1-4</value>
+ <value xml:lang="en-US">Microsoft PowerPoint 1-4 and 95's</value>
</prop>
<prop oor:name="FileFormatVersion">
<value>0</value>
diff --git a/filter/source/config/fragments/types/impress_PowerPoint3.xcu b/filter/source/config/fragments/types/impress_PowerPoint3.xcu
index 45cbb2855c2f..4c6885943b76 100644
--- a/filter/source/config/fragments/types/impress_PowerPoint3.xcu
+++ b/filter/source/config/fragments/types/impress_PowerPoint3.xcu
@@ -11,7 +11,7 @@
<value>com.sun.star.comp.Impress.MWAWPresentationImportFilter</value>
</prop>
<prop oor:name="Extensions">
- <value>ppt</value>
+ <value>ppt pot</value>
</prop>
<prop oor:name="PreferredFilter">
<value>PowerPoint 3</value>
diff --git a/writerperfect/source/impress/MWAWPresentationImportFilter.cxx b/writerperfect/source/impress/MWAWPresentationImportFilter.cxx
index 3b77605a2c6e..a1b2d7f06145 100644
--- a/writerperfect/source/impress/MWAWPresentationImportFilter.cxx
+++ b/writerperfect/source/impress/MWAWPresentationImportFilter.cxx
@@ -58,7 +58,7 @@ bool MWAWPresentationImportFilter::doDetectFormat(librevenge::RVNGInputStream &r
case MWAWDocument::MWAW_T_CLARISWORKS:
rTypeName = "impress_ClarisWorks";
break;
- case MWAWDocument::MWAW_T_RESERVED9:
+ case MWAWDocument::MWAW_T_RESERVED8:
rTypeName = "impress_PowerPoint3";
break;
default:
commit 8988c925ce07395478fbbb10d25fd6c0a10fead5
Author: osnola <alonso at loria.fr>
Date: Sun Apr 2 09:53:44 2017 +0200
libstaroffice import filter improvements
+ add support for presentation file (i.e. the file created will StarImpress
are now open as presentation).
+ modify the OLE parser to be similar to the librevenge OLE parser,
ie. the librevenge parser ignores the first character of a filename if
this is a control character...
Change-Id: I913a70cba29839d43dac58d5e00dbebfc4d28abc
diff --git a/filter/Configuration_filter.mk b/filter/Configuration_filter.mk
index cbb951f3fe28..52483dc58ccf 100644
--- a/filter/Configuration_filter.mk
+++ b/filter/Configuration_filter.mk
@@ -600,6 +600,7 @@ $(eval $(call filter_Configuration_add_types,fcfg_langpack,fcfg_impress_types.xc
impress_OOXML_Presentation_Template \
impress_OOXML_Presentation_AutoPlay \
impress_ClarisWorks \
+ StarOffice_Presentation \
MWAW_Presentation \
impress_PowerPoint3 \
))
@@ -625,6 +626,7 @@ $(eval $(call filter_Configuration_add_filters,fcfg_langpack,fcfg_impress_filter
impress_OOXML_Template \
impress_OOXML_AutoPlay \
ClarisWorks_Impress \
+ StarOffice_Presentation \
MWAW_Presentation \
PowerPoint3 \
))
diff --git a/filter/source/config/fragments/filters/StarOffice_Presentation.xcu b/filter/source/config/fragments/filters/StarOffice_Presentation.xcu
new file mode 100644
index 000000000000..7179379d74d6
--- /dev/null
+++ b/filter/source/config/fragments/filters/StarOffice_Presentation.xcu
@@ -0,0 +1,29 @@
+<!--
+ * This file is part of the LibreOffice project.
+ *
+ * This Source Code Form is subject to the terms of the Mozilla Public
+ * License, v. 2.0. If a copy of the MPL was not distributed with this
+ * file, You can obtain one at http://mozilla.org/MPL/2.0/.
+ *
+-->
+
+<node oor:name="StarOffice_Presentation" oor:op="replace">
+ <prop oor:name="Flags">
+ <value>IMPORT ALIEN 3RDPARTYFILTER PREFERRED</value>
+ </prop>
+ <prop oor:name="FilterService">
+ <value>org.libreoffice.comp.Impress.StarOfficePresentationImportFilter</value>
+ </prop>
+ <prop oor:name="UIName">
+ <value xml:lang="en-US">Legacy StarOffice Presentation</value>
+ </prop>
+ <prop oor:name="FileFormatVersion">
+ <value>0</value>
+ </prop>
+ <prop oor:name="Type">
+ <value>StarOffice_Presentation</value>
+ </prop>
+ <prop oor:name="DocumentService">
+ <value>com.sun.star.presentation.PresentationDocument</value>
+ </prop>
+</node>
diff --git a/filter/source/config/fragments/types/StarOffice_Presentation.xcu b/filter/source/config/fragments/types/StarOffice_Presentation.xcu
new file mode 100644
index 000000000000..02c0f3af67a8
--- /dev/null
+++ b/filter/source/config/fragments/types/StarOffice_Presentation.xcu
@@ -0,0 +1,27 @@
+<!--
+ * This file is part of the LibreOffice project.
+ *
+ * This Source Code Form is subject to the terms of the Mozilla Public
+ * License, v. 2.0. If a copy of the MPL was not distributed with this
+ * file, You can obtain one at http://mozilla.org/MPL/2.0/.
+ *
+-->
+
+<node oor:name="StarOffice_Presentation" oor:op="replace">
+ <prop oor:name="DetectService">
+ <value>org.libreoffice.comp.Impress.StarOfficePresentationImportFilter</value>
+ </prop>
+ <prop oor:name="Extensions">
+ <value>sdd</value>
+ </prop>
+ <prop oor:name="MediaType"/>
+ <prop oor:name="Preferred">
+ <value>true</value>
+ </prop>
+ <prop oor:name="PreferredFilter">
+ <value>StarOffice_Presentation</value>
+ </prop>
+ <prop oor:name="UIName">
+ <value>Legacy StarOffice Presentation</value>
+ </prop>
+</node>
diff --git a/writerperfect/Library_wpftimpress.mk b/writerperfect/Library_wpftimpress.mk
index 116b72a1235a..18c8afa110bc 100644
--- a/writerperfect/Library_wpftimpress.mk
+++ b/writerperfect/Library_wpftimpress.mk
@@ -49,6 +49,7 @@ $(eval $(call gb_Library_use_externals,wpftimpress,\
etonyek \
mwaw \
odfgen \
+ staroffice \
revenge \
zlib \
libxml2 \
@@ -57,6 +58,7 @@ $(eval $(call gb_Library_use_externals,wpftimpress,\
$(eval $(call gb_Library_add_exception_objects,wpftimpress,\
writerperfect/source/impress/KeynoteImportFilter \
writerperfect/source/impress/MWAWPresentationImportFilter \
+ writerperfect/source/impress/StarOfficePresentationImportFilter \
))
# vim: set noet sw=4 ts=4:
diff --git a/writerperfect/source/common/WPXSvInputStream.cxx b/writerperfect/source/common/WPXSvInputStream.cxx
index 4c4998cbaca7..4137cd6f5425 100644
--- a/writerperfect/source/common/WPXSvInputStream.cxx
+++ b/writerperfect/source/common/WPXSvInputStream.cxx
@@ -105,7 +105,7 @@ const rtl::OUString concatPath(const rtl::OUString &lhs, const rtl::OUString &rh
struct OLEStreamData
{
- explicit OLEStreamData(const rtl::OString &rName);
+ OLEStreamData(const rtl::OString &rName, const rtl::OString &rvngName);
SotStorageStreamRefWrapper stream;
@@ -115,6 +115,12 @@ struct OLEStreamData
* produce const char* from it.
*/
rtl::OString name;
+ /** librevenge name of the stream.
+ *
+ * This is not @c rtl::OUString, because we need to be able to
+ * produce const char* from it.
+ */
+ rtl::OString RVNGname;
};
typedef std::unordered_map<rtl::OUString, std::size_t, rtl::OUStringHash> NameMap_t;
@@ -158,9 +164,10 @@ public:
bool mbInitialized;
};
-OLEStreamData::OLEStreamData(const rtl::OString &rName)
+OLEStreamData::OLEStreamData(const rtl::OString &rName, const rtl::OString &rvngName)
: stream()
, name(rName)
+ , RVNGname(rvngName)
{
}
@@ -197,7 +204,7 @@ tools::SvRef<SotStorageStream> OLEStorageImpl::getStream(const rtl::OUString &rP
return tools::SvRef<SotStorageStream>();
if (!maStreams[aIt->second].stream.ref.is())
- maStreams[aIt->second].stream.ref = createStream(aPath);
+ maStreams[aIt->second].stream.ref = createStream(rtl::OStringToOUString(maStreams[aIt->second].name, RTL_TEXTENCODING_UTF8));
return maStreams[aIt->second].stream.ref;
}
@@ -220,8 +227,13 @@ void OLEStorageImpl::traverse(const tools::SvRef<SotStorage> &rStorage, const rt
{
if (aIt->IsStream())
{
- maStreams.push_back(OLEStreamData(rtl::OUStringToOString(concatPath(rPath, aIt->GetName()), RTL_TEXTENCODING_UTF8)));
- maNameMap[concatPath(rPath, aIt->GetName())] = maStreams.size() - 1;
+ rtl::OUString baseName=aIt->GetName(), rvngName=baseName;
+ // librevenge::RVNGOLEStream ignores the first character when is a control code, so ...
+ if (!rvngName.isEmpty() && rvngName.toChar()<32)
+ rvngName=rvngName.copy(1);
+ maStreams.push_back(OLEStreamData(rtl::OUStringToOString(concatPath(rPath, baseName), RTL_TEXTENCODING_UTF8),
+ rtl::OUStringToOString(concatPath(rPath, rvngName), RTL_TEXTENCODING_UTF8)));
+ maNameMap[concatPath(rPath, rvngName)] = maStreams.size() - 1;
}
else if (aIt->IsStorage())
{
@@ -590,7 +602,7 @@ const char *WPXSvInputStreamImpl::subStreamName(const unsigned id)
if (mpOLEStorage->maStreams.size() <= id)
return nullptr;
- return mpOLEStorage->maStreams[id].name.getStr();
+ return mpOLEStorage->maStreams[id].RVNGname.getStr();
}
mxSeekable->seek(0);
diff --git a/writerperfect/source/impress/StarOfficePresentationImportFilter.cxx b/writerperfect/source/impress/StarOfficePresentationImportFilter.cxx
new file mode 100644
index 000000000000..00fdd5b22933
--- /dev/null
+++ b/writerperfect/source/impress/StarOfficePresentationImportFilter.cxx
@@ -0,0 +1,95 @@
+/* -*- Mode: C++; tab-width: 4; indent-tabs-mode: nil; c-basic-offset: 4 -*- */
+/*
+ * This file is part of the LibreOffice project.
+ *
+ * This Source Code Form is subject to the terms of the Mozilla Public
+ * License, v. 2.0. If a copy of the MPL was not distributed with this
+ * file, You can obtain one at http://mozilla.org/MPL/2.0/.
+ */
+
+#include <cppuhelper/supportsservice.hxx>
+
+#include <libstaroffice/libstaroffice.hxx>
+#include <libodfgen/libodfgen.hxx>
+
+#include "StarOfficePresentationImportFilter.hxx"
+
+using com::sun::star::uno::Sequence;
+using com::sun::star::uno::XInterface;
+using com::sun::star::uno::RuntimeException;
+using com::sun::star::uno::XComponentContext;
+
+static bool handleEmbeddedSTOFFGraphicObject(const librevenge::RVNGBinaryData &data, OdfDocumentHandler *pHandler, const OdfStreamType streamType)
+{
+ OdgGenerator exporter;
+ exporter.addDocumentHandler(pHandler, streamType);
+ return STOFFDocument::decodeGraphic(data, &exporter);
+}
+
+static bool handleEmbeddedSTOFFSpreadsheetObject(const librevenge::RVNGBinaryData &data, OdfDocumentHandler *pHandler, const OdfStreamType streamType)
+{
+ OdsGenerator exporter;
+ exporter.registerEmbeddedObjectHandler("image/stoff-odg", &handleEmbeddedSTOFFGraphicObject);
+ exporter.addDocumentHandler(pHandler, streamType);
+ return STOFFDocument::decodeSpreadsheet(data, &exporter);
+}
+
+bool StarOfficePresentationImportFilter::doImportDocument(librevenge::RVNGInputStream &rInput, OdpGenerator &rGenerator, utl::MediaDescriptor &)
+{
+ return STOFFDocument::STOFF_R_OK == STOFFDocument::parse(&rInput, &rGenerator);
+}
+
+bool StarOfficePresentationImportFilter::doDetectFormat(librevenge::RVNGInputStream &rInput, OUString &rTypeName)
+{
+ rTypeName.clear();
+
+ STOFFDocument::Kind docKind = STOFFDocument::STOFF_K_UNKNOWN;
+ const STOFFDocument::Confidence confidence = STOFFDocument::isFileFormatSupported(&rInput, docKind);
+
+ if (confidence == STOFFDocument::STOFF_C_EXCELLENT || confidence == STOFFDocument::STOFF_C_SUPPORTED_ENCRYPTION)
+ {
+ switch (docKind)
+ {
+ case STOFFDocument::STOFF_K_PRESENTATION:
+ rTypeName = "StarOffice_Presentation";
+ break;
+ default:
+ break;
+ }
+ }
+
+ return !rTypeName.isEmpty();
+}
+
+void StarOfficePresentationImportFilter::doRegisterHandlers(OdpGenerator &rGenerator)
+{
+ rGenerator.registerEmbeddedObjectHandler("image/stoff-odg", &handleEmbeddedSTOFFGraphicObject);
+ rGenerator.registerEmbeddedObjectHandler("image/stoff-ods", &handleEmbeddedSTOFFSpreadsheetObject);
+}
+
+// XServiceInfo
+OUString SAL_CALL StarOfficePresentationImportFilter::getImplementationName()
+{
+ return OUString("org.libreoffice.comp.Impress.StarOfficePresentationImportFilter");
+}
+
+sal_Bool SAL_CALL StarOfficePresentationImportFilter::supportsService(const OUString &rServiceName)
+{
+ return cppu::supportsService(this, rServiceName);
+}
+
+Sequence< OUString > SAL_CALL StarOfficePresentationImportFilter::getSupportedServiceNames()
+{
+ return Sequence< OUString > {"com.sun.star.document.ImportFilter", "com.sun.star.document.ExtendedTypeDetection"};
+}
+
+extern "C"
+SAL_DLLPUBLIC_EXPORT css::uno::XInterface *SAL_CALL
+org_libreoffice_comp_Presentation_StarOfficePresentationImportFilter_get_implementation(
+ css::uno::XComponentContext *const context,
+ const css::uno::Sequence<css::uno::Any> &)
+{
+ return cppu::acquire(new StarOfficePresentationImportFilter(context));
+}
+
+/* vim:set shiftwidth=4 softtabstop=4 expandtab: */
diff --git a/writerperfect/source/impress/StarOfficePresentationImportFilter.hxx b/writerperfect/source/impress/StarOfficePresentationImportFilter.hxx
new file mode 100644
index 000000000000..3651315b0e3a
--- /dev/null
+++ b/writerperfect/source/impress/StarOfficePresentationImportFilter.hxx
@@ -0,0 +1,42 @@
+/* -*- Mode: C++; tab-width: 4; indent-tabs-mode: nil; c-basic-offset: 4 -*- */
+/*
+ * This file is part of the LibreOffice project.
+ *
+ * This Source Code Form is subject to the terms of the Mozilla Public
+ * License, v. 2.0. If a copy of the MPL was not distributed with this
+ * file, You can obtain one at http://mozilla.org/MPL/2.0/.
+ */
+
+#ifndef INCLUDED_WRITERPERFECT_SOURCE_PRESENTATION_STAROFFICEPRESENTATIONIMPORTFILTER_HXX
+#define INCLUDED_WRITERPERFECT_SOURCE_PRESENTATION_STAROFFICEPRESENTATIONIMPORTFILTER_HXX
+
+#include <com/sun/star/lang/XServiceInfo.hpp>
+#include <com/sun/star/uno/XComponentContext.hpp>
+
+#include "ImportFilter.hxx"
+
+#include "DocumentHandlerForOdp.hxx"
+
+/* This component will be instantiated for both import or export. Whether it calls
+ * setSourceDocument or setTargetDocument determines which Impl function the filter
+ * member calls */
+class StarOfficePresentationImportFilter : public writerperfect::ImportFilter<OdpGenerator>
+{
+public:
+ explicit StarOfficePresentationImportFilter(const css::uno::Reference< css::uno::XComponentContext > &rxContext)
+ : writerperfect::ImportFilter<OdpGenerator>(rxContext) {}
+
+ // XServiceInfo
+ virtual OUString SAL_CALL getImplementationName() override;
+ virtual sal_Bool SAL_CALL supportsService(const OUString &ServiceName) override;
+ virtual css::uno::Sequence< OUString > SAL_CALL getSupportedServiceNames() override;
+
+private:
+ virtual bool doDetectFormat(librevenge::RVNGInputStream &rInput, OUString &rTypeName) override;
+ virtual bool doImportDocument(librevenge::RVNGInputStream &rInput, OdpGenerator &rGenerator, utl::MediaDescriptor &) override;
+ virtual void doRegisterHandlers(OdpGenerator &rGenerator) override;
+};
+
+#endif
+
+/* vim:set shiftwidth=4 softtabstop=4 expandtab: */
diff --git a/writerperfect/source/impress/wpftimpress.component b/writerperfect/source/impress/wpftimpress.component
index 5413c9a7b4bd..0f7df39fa94e 100644
--- a/writerperfect/source/impress/wpftimpress.component
+++ b/writerperfect/source/impress/wpftimpress.component
@@ -19,4 +19,9 @@
<service name="com.sun.star.document.ImportFilter"/>
<service name="com.sun.star.document.ExtendedTypeDetection"/>
</implementation>
+ <implementation name="org.libreoffice.comp.Impress.StarOfficePresentationImportFilter"
+ constructor="org_libreoffice_comp_Presentation_StarOfficePresentationImportFilter_get_implementation">
+ <service name="com.sun.star.document.ImportFilter"/>
+ <service name="com.sun.star.document.ExtendedTypeDetection"/>
+ </implementation>
</component>
commit b0067c89e6b2a4e29465d9da9a731ae30a66dce6
Author: osnola <alonso at loria.fr>
Date: Sun Apr 2 09:42:59 2017 +0200
libwps import filter improvements
+ some astyle modifications,
+ add .wk4 and .123 to the list of file extensions,
+ add support to open Lotus files protected by a password.
Change-Id: I94d4afffd73f0999ff2b1958704cb3985fcd0cc9
diff --git a/filter/source/config/fragments/types/calc_WPS_Lotus_Document.xcu b/filter/source/config/fragments/types/calc_WPS_Lotus_Document.xcu
index 0ecc7114c3ed..abea24321e6b 100644
--- a/filter/source/config/fragments/types/calc_WPS_Lotus_Document.xcu
+++ b/filter/source/config/fragments/types/calc_WPS_Lotus_Document.xcu
@@ -18,7 +18,7 @@
<node oor:name="calc_WPS_Lotus_Document" oor:op="replace" >
<prop oor:name="DetectService"><value>com.sun.star.comp.Calc.MSWorksCalcImportFilter</value></prop>
<prop oor:name="URLPattern"/>
- <prop oor:name="Extensions"><value>wk1 wk3</value></prop>
+ <prop oor:name="Extensions"><value>wk1 wk3 wk4 123</value></prop>
<prop oor:name="MediaType"></prop>
<prop oor:name="Preferred"><value>true</value></prop>
<prop oor:name="PreferredFilter"><value>WPS_Lotus_Calc</value></prop>
diff --git a/writerperfect/Library_wpftcalc.mk b/writerperfect/Library_wpftcalc.mk
index 311741d26f5d..c4aecf391d44 100644
--- a/writerperfect/Library_wpftcalc.mk
+++ b/writerperfect/Library_wpftcalc.mk
@@ -33,6 +33,7 @@ $(eval $(call gb_Library_use_libraries,wpftcalc,\
cppu \
cppuhelper \
sal \
+ sfx \
sot \
svx \
tl \
diff --git a/writerperfect/qa/unit/DrawingImportTest.cxx b/writerperfect/qa/unit/DrawingImportTest.cxx
index 433c9a4458c4..661f851d3a0f 100644
--- a/writerperfect/qa/unit/DrawingImportTest.cxx
+++ b/writerperfect/qa/unit/DrawingImportTest.cxx
@@ -120,7 +120,7 @@ void DrawingImportTest::test()
{
using namespace css;
- rtl::Reference<DrawingImportFilter> xFilter{new DrawingImportFilter(m_xContext)};
+ rtl::Reference<DrawingImportFilter> xFilter {new DrawingImportFilter(m_xContext)};
writerperfect::test::WpftLoader aLoader(createDummyInput(), xFilter.get(), "private:factory/sdraw", m_xDesktop, m_xContext);
uno::Reference<drawing::XDrawPagesSupplier> xDoc(aLoader.getDocument(), uno::UNO_QUERY);
diff --git a/writerperfect/qa/unit/PresentationImportTest.cxx b/writerperfect/qa/unit/PresentationImportTest.cxx
index 7c1154ba6cfa..4181d55be85b 100644
--- a/writerperfect/qa/unit/PresentationImportTest.cxx
+++ b/writerperfect/qa/unit/PresentationImportTest.cxx
@@ -120,7 +120,7 @@ void PresentationImportTest::test()
{
using namespace css;
- rtl::Reference<PresentationImportFilter> xFilter{new PresentationImportFilter(m_xContext)};
+ rtl::Reference<PresentationImportFilter> xFilter {new PresentationImportFilter(m_xContext)};
writerperfect::test::WpftLoader aLoader(createDummyInput(), xFilter.get(), "private:factory/simpress", m_xDesktop, m_xContext);
uno::Reference<drawing::XDrawPagesSupplier> xDoc(aLoader.getDocument(), uno::UNO_QUERY);
diff --git a/writerperfect/qa/unit/SpreadsheetImportTest.cxx b/writerperfect/qa/unit/SpreadsheetImportTest.cxx
index 379f57ae1fd5..55d03151924b 100644
--- a/writerperfect/qa/unit/SpreadsheetImportTest.cxx
+++ b/writerperfect/qa/unit/SpreadsheetImportTest.cxx
@@ -115,7 +115,7 @@ void SpreadsheetImportTest::test()
{
using namespace css;
- rtl::Reference<SpreadsheetImportFilter> xFilter{new SpreadsheetImportFilter(m_xContext)};
+ rtl::Reference<SpreadsheetImportFilter> xFilter {new SpreadsheetImportFilter(m_xContext)};
writerperfect::test::WpftLoader aLoader(createDummyInput(), xFilter.get(), "private:factory/scalc", m_xDesktop, m_xContext);
uno::Reference<sheet::XSpreadsheetDocument> xDoc(aLoader.getDocument(), uno::UNO_QUERY);
diff --git a/writerperfect/qa/unit/TextImportTest.cxx b/writerperfect/qa/unit/TextImportTest.cxx
index ba464c6c0ac4..4789d003abec 100644
--- a/writerperfect/qa/unit/TextImportTest.cxx
+++ b/writerperfect/qa/unit/TextImportTest.cxx
@@ -108,7 +108,7 @@ void TextImportTest::test()
{
using namespace css;
- rtl::Reference<TextImportFilter> xFilter{new TextImportFilter(m_xContext)};
+ rtl::Reference<TextImportFilter> xFilter {new TextImportFilter(m_xContext)};
writerperfect::test::WpftLoader aLoader(createDummyInput(), xFilter.get(), "private:factory/swriter", m_xDesktop, m_xContext);
uno::Reference<text::XTextDocument> xDoc(aLoader.getDocument(), uno::UNO_QUERY);
diff --git a/writerperfect/source/calc/MSWorksCalcImportFilter.cxx b/writerperfect/source/calc/MSWorksCalcImportFilter.cxx
index 94a111ec7127..11881202ad28 100644
--- a/writerperfect/source/calc/MSWorksCalcImportFilter.cxx
+++ b/writerperfect/source/calc/MSWorksCalcImportFilter.cxx
@@ -16,6 +16,7 @@
#include <com/sun/star/ucb/XContentAccess.hpp>
#include <comphelper/processfactory.hxx>
#include <cppuhelper/supportsservice.hxx>
+#include <sfx2/passwd.hxx>
#include <tools/urlobj.hxx>
#include <ucbhelper/content.hxx>
@@ -90,7 +91,7 @@ public:
/*! \brief seeks to a offset position, from actual, beginning or ending position
* \return 0 if ok
*/
- int seek(long , librevenge::RVNG_SEEK_TYPE) override
+ int seek(long, librevenge::RVNG_SEEK_TYPE) override
{
return 1;
}
@@ -191,8 +192,10 @@ bool MSWorksCalcImportFilter::doImportDocument(librevenge::RVNGInputStream &rInp
bool needEncoding;
const libwps::WPSConfidence confidence = libwps::WPSDocument::isFileFormatSupported(&rInput, kind, creator, needEncoding);
+ if ((kind != libwps::WPS_SPREADSHEET && kind != libwps::WPS_DATABASE) || (confidence == libwps::WPS_CONFIDENCE_NONE))
+ return false;
std::string fileEncoding("");
- if ((kind == libwps::WPS_SPREADSHEET || kind == libwps::WPS_DATABASE) && (confidence == libwps::WPS_CONFIDENCE_EXCELLENT) && needEncoding)
+ if (needEncoding)
{
OUString title, encoding;
if (creator == libwps::WPS_MSWORKS)
@@ -232,7 +235,25 @@ bool MSWorksCalcImportFilter::doImportDocument(librevenge::RVNGInputStream &rInp
SAL_WARN("writerperfect", "ignoring Exception in MSWorksCalcImportFilter::doImportDocument");
}
}
- return libwps::WPS_OK == libwps::WPSDocument::parse(&rInput, &rGenerator, "", fileEncoding.c_str());
+ OString aUtf8Passwd;
+ if (confidence==libwps::WPS_CONFIDENCE_SUPPORTED_ENCRYPTION)
+ {
+ // try to ask for a password
+ try
+ {
+ ScopedVclPtrInstance< SfxPasswordDialog > aPasswdDlg(nullptr);
+ aPasswdDlg->SetMinLen(1);
+ if (!aPasswdDlg->Execute())
+ return false;
+ OUString aPasswd = aPasswdDlg->GetPassword();
+ aUtf8Passwd = OUStringToOString(aPasswd, RTL_TEXTENCODING_UTF8);
+ }
+ catch (...)
+ {
+ return false;
+ }
+ }
+ return libwps::WPS_OK == libwps::WPSDocument::parse(&rInput, &rGenerator, confidence==libwps::WPS_CONFIDENCE_SUPPORTED_ENCRYPTION ? aUtf8Passwd.getStr() : nullptr, fileEncoding.c_str());
}
//XExtendedFilterDetection
@@ -326,12 +347,12 @@ sal_Bool MSWorksCalcImportFilter::filter(const css::uno::Sequence< css::beans::P
structuredInput.addFile(sWM3Name,"WK3");
structuredInput.addFile(sFM3Name,"FM3");
- // If the file is valid and libwps is at least 0.4.4, doImportDocument will convert it.
- // If libwps is at most 0.4.3, doImportDocument will fail when checking if the file is supported
- // and it is ok to call again doImportDocument with the main input.
- // If the file is corrupted beyond all retrieval, doImportDocument will fail two times :-~
- if (this->doImportDocument(structuredInput, exporter, aDescriptor))
- return true;
+ libwps::WPSKind kind = libwps::WPS_TEXT;
+ libwps::WPSCreator creator;
+ bool needEncoding;
+ const libwps::WPSConfidence confidence = libwps::WPSDocument::isFileFormatSupported(&structuredInput, kind, creator, needEncoding);
+ if (confidence!=libwps::WPS_CONFIDENCE_NONE)
+ return this->doImportDocument(structuredInput, exporter, aDescriptor);
}
}
}
@@ -350,7 +371,7 @@ bool MSWorksCalcImportFilter::doDetectFormat(librevenge::RVNGInputStream &rInput
bool needEncoding;
const libwps::WPSConfidence confidence = libwps::WPSDocument::isFileFormatSupported(&rInput, kind, creator, needEncoding);
- if ((kind == libwps::WPS_SPREADSHEET || kind == libwps::WPS_DATABASE) && confidence == libwps::WPS_CONFIDENCE_EXCELLENT)
+ if ((kind == libwps::WPS_SPREADSHEET || kind == libwps::WPS_DATABASE) && confidence != libwps::WPS_CONFIDENCE_NONE)
{
if (creator == libwps::WPS_MSWORKS)
{
diff --git a/writerperfect/source/common/DocumentHandler.cxx b/writerperfect/source/common/DocumentHandler.cxx
index 29788200b6de..a38f8fc0fcf8 100644
--- a/writerperfect/source/common/DocumentHandler.cxx
+++ b/writerperfect/source/common/DocumentHandler.cxx
@@ -111,10 +111,10 @@ using com::sun::star::xml::sax::XAttributeList;
using com::sun::star::xml::sax::XDocumentHandler;
DocumentHandler::DocumentHandler(Reference < XDocumentHandler > &xHandler) :
- mxHandler( xHandler )
+ mxHandler(xHandler)
{
- if (SvXMLImport *pFastHandler = dynamic_cast<SvXMLImport*>(mxHandler.get()))
- mxHandler.set( new SvXMLLegacyToFastDocHandler( pFastHandler ) );
+ if (SvXMLImport *pFastHandler = dynamic_cast<SvXMLImport *>(mxHandler.get()))
+ mxHandler.set(new SvXMLLegacyToFastDocHandler(pFastHandler));
}
void DocumentHandler::startDocument()
More information about the Libreoffice-commits
mailing list