chart2/qa/extras/chart2geometry.cxx | 7 cui/source/options/optsave.cxx | 8 dbaccess/source/core/dataaccess/databasedocument.cxx | 2 include/unotools/saveopt.hxx | 35 -- sax/source/tools/converter.cxx | 1 sc/source/filter/excel/xeroot.cxx | 1 sfx2/source/appl/appcfg.cxx | 2 sfx2/source/dialog/alienwarn.cxx | 1 sfx2/source/inc/appdata.hxx | 1 sw/source/filter/ww8/attributeoutputbase.hxx | 2 unotools/source/config/saveopt.cxx | 298 ++----------------- xmloff/source/core/xmlexp.cxx | 5 xmloff/source/style/xmlexppr.cxx | 2 13 files changed, 58 insertions(+), 307 deletions(-)
New commits: commit 134f40136a9bea265d8f2fedfdb41a1e65d81b49 Author: Noel Grandin <noel.gran...@collabora.co.uk> AuthorDate: Sat Jul 24 16:56:15 2021 +0200 Commit: Noel Grandin <noel.gran...@collabora.co.uk> CommitDate: Sun Jul 25 20:05:30 2021 +0200 use officecfg to retrieve OdfDefaultVersion Change-Id: Id54b98d978965e7ce304b83d5eff7d6c844a41d5 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/119474 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.gran...@collabora.co.uk> diff --git a/chart2/qa/extras/chart2geometry.cxx b/chart2/qa/extras/chart2geometry.cxx index c1fe65fa927d..bcfa0d7276af 100644 --- a/chart2/qa/extras/chart2geometry.cxx +++ b/chart2/qa/extras/chart2geometry.cxx @@ -440,9 +440,8 @@ void Chart2GeometryTest::testTdf135366_CustomLabelText() { // Error was, that custom text in a data label was only exported in ODF extended, // although the used <chart:data-label> element exists since ODF 1.2. - SvtSaveOptions aSaveOpt; - const SvtSaveOptions::ODFDefaultVersion nCurrentODFVersion(aSaveOpt.GetODFDefaultVersion()); - aSaveOpt.SetODFDefaultVersion(SvtSaveOptions::ODFVER_012); + const SvtSaveOptions::ODFDefaultVersion nCurrentODFVersion(GetODFDefaultVersion()); + SetODFDefaultVersion(SvtSaveOptions::ODFVER_012); load(u"/chart2/qa/extras/data/pptx/", "tdf135366_CustomLabelText.pptx"); xmlDocUniquePtr pXmlDoc = parseExport("Object 1/content.xml", "impress8"); CPPUNIT_ASSERT(pXmlDoc); @@ -457,7 +456,7 @@ void Chart2GeometryTest::testTdf135366_CustomLabelText() const OUString sOUTextContent = getXPathContent(pXmlDoc, sCustomTextPath); CPPUNIT_ASSERT_EQUAL(OUString("Custom"), sOUTextContent); - aSaveOpt.SetODFDefaultVersion(nCurrentODFVersion); + SetODFDefaultVersion(nCurrentODFVersion); } CPPUNIT_TEST_SUITE_REGISTRATION(Chart2GeometryTest); diff --git a/cui/source/options/optsave.cxx b/cui/source/options/optsave.cxx index 208bf8289f2e..14194720b7c0 100644 --- a/cui/source/options/optsave.cxx +++ b/cui/source/options/optsave.cxx @@ -227,7 +227,6 @@ bool SvxSaveTabPage::FillItemSet( SfxItemSet* rSet ) { auto xChanges = comphelper::ConfigurationChanges::create(); bool bModified = false; - SvtSaveOptions aSaveOpt; if(m_xLoadUserSettingsCB->get_state_changed_from_saved()) officecfg::Office::Common::Load::UserDefinedSettings::set(m_xLoadUserSettingsCB->get_active(), xChanges); @@ -237,7 +236,7 @@ bool SvxSaveTabPage::FillItemSet( SfxItemSet* rSet ) if ( m_xODFVersionLB->get_value_changed_from_saved() ) { sal_Int32 nVersion = m_xODFVersionLB->get_active_id().toInt32(); - aSaveOpt.SetODFDefaultVersion( SvtSaveOptions::ODFDefaultVersion( nVersion ) ); + SetODFDefaultVersion( SvtSaveOptions::ODFDefaultVersion( nVersion ), xChanges ); } if ( m_xDocInfoCB->get_state_changed_from_saved() ) @@ -364,7 +363,6 @@ static bool isODFFormat( const OUString& sFilter ) void SvxSaveTabPage::Reset( const SfxItemSet* ) { - SvtSaveOptions aSaveOpt; m_xLoadUserSettingsCB->set_active(officecfg::Office::Common::Load::UserDefinedSettings::get()); m_xLoadUserSettingsCB->save_state(); m_xLoadUserSettingsCB->set_sensitive(!officecfg::Office::Common::Load::UserDefinedSettings::isReadOnly()); @@ -456,9 +454,9 @@ void SvxSaveTabPage::Reset( const SfxItemSet* ) m_xRelativeInetCB->set_active(officecfg::Office::Common::Save::URL::Internet::get()); m_xRelativeInetCB->set_sensitive(!officecfg::Office::Common::Save::URL::Internet::isReadOnly()); - sal_Int32 nDefaultVersion = aSaveOpt.GetODFDefaultVersion(); + sal_Int32 nDefaultVersion = GetODFDefaultVersion(); m_xODFVersionLB->set_active_id(OUString::number(nDefaultVersion)); - m_xODFVersionLB->set_sensitive(!aSaveOpt.IsReadOnly(SvtSaveOptions::EOption::OdfDefaultVersion)); + m_xODFVersionLB->set_sensitive(!officecfg::Office::Common::Save::ODF::DefaultVersion::isReadOnly()); AutoClickHdl_Impl(*m_xAutoSaveCB); ODFVersionHdl_Impl(*m_xODFVersionLB); diff --git a/dbaccess/source/core/dataaccess/databasedocument.cxx b/dbaccess/source/core/dataaccess/databasedocument.cxx index c8a75c1e725d..f9f99ef8ef9a 100644 --- a/dbaccess/source/core/dataaccess/databasedocument.cxx +++ b/dbaccess/source/core/dataaccess/databasedocument.cxx @@ -1653,7 +1653,7 @@ void ODatabaseDocument::impl_writeStorage_throw( const Reference< XStorage >& _r OUString aVersion; SvtSaveOptions::ODFSaneDefaultVersion const nDefVersion = - SvtSaveOptions().GetODFSaneDefaultVersion(); + GetODFSaneDefaultVersion(); // older versions can not have this property set, // it exists only starting from ODF1.2 if (nDefVersion >= SvtSaveOptions::ODFSVER_013) diff --git a/include/unotools/saveopt.hxx b/include/unotools/saveopt.hxx index 2793187ec621..9cd6367615bd 100644 --- a/include/unotools/saveopt.hxx +++ b/include/unotools/saveopt.hxx @@ -20,20 +20,12 @@ #define INCLUDED_UNOTOOLS_SAVEOPT_HXX #include <unotools/unotoolsdllapi.h> -#include <unotools/options.hxx> +#include <memory> -struct SvtLoadSaveOptions_Impl; -class UNOTOOLS_DLLPUBLIC SvtSaveOptions final : public utl::detail::Options -{ - SvtLoadSaveOptions_Impl* pImp; - -public: - - enum class EOption - { - OdfDefaultVersion - }; +namespace comphelper { class ConfigurationChanges; } +namespace SvtSaveOptions +{ /** Keep enum values sorted that a less or greater compare maps to older and newer versions. Do not change values, they are stored in the configuration. @@ -74,21 +66,16 @@ public: ODFSVER_LATEST_EXTENDED = ODFSVER_013_EXTENDED ///< @internal DO NOT USE in comparisons }; - SvtSaveOptions(); - virtual ~SvtSaveOptions() override; - - void SetODFDefaultVersion( ODFDefaultVersion eVersion ); - ODFDefaultVersion GetODFDefaultVersion() const; - ODFSaneDefaultVersion GetODFSaneDefaultVersion() const; +}; - bool IsReadOnly( EOption eOption ) const; +UNOTOOLS_DLLPUBLIC void SetODFDefaultVersion( SvtSaveOptions::ODFDefaultVersion eVersion ); +UNOTOOLS_DLLPUBLIC void SetODFDefaultVersion( SvtSaveOptions::ODFDefaultVersion eVersion, const std::shared_ptr<comphelper::ConfigurationChanges>& ); - /** gets a sane default from the currently configured default */ - static ODFSaneDefaultVersion GetODFSaneDefaultVersion(ODFDefaultVersion eDefaultVersion); -}; +UNOTOOLS_DLLPUBLIC SvtSaveOptions::ODFDefaultVersion GetODFDefaultVersion(); +UNOTOOLS_DLLPUBLIC SvtSaveOptions::ODFSaneDefaultVersion GetODFSaneDefaultVersion(); -/** lighter-weight version of the same method in SvtSaveOptions */ -UNOTOOLS_DLLPUBLIC SvtSaveOptions::ODFSaneDefaultVersion GetODFSaneDefaultVersion(); +/** gets a sane default from the currently configured default */ +UNOTOOLS_DLLPUBLIC SvtSaveOptions::ODFSaneDefaultVersion GetODFSaneDefaultVersion(SvtSaveOptions::ODFDefaultVersion eDefaultVersion); #endif diff --git a/sax/source/tools/converter.cxx b/sax/source/tools/converter.cxx index 01657ddf30db..69fbc8104cfb 100644 --- a/sax/source/tools/converter.cxx +++ b/sax/source/tools/converter.cxx @@ -30,6 +30,7 @@ #include <rtl/math.hxx> #include <rtl/character.hxx> #include <sal/log.hxx> +#include <o3tl/typed_flags_set.hxx> #include <o3tl/unit_conversion.hxx> #include <osl/diagnose.h> #include <tools/long.hxx> diff --git a/sc/source/filter/excel/xeroot.cxx b/sc/source/filter/excel/xeroot.cxx index 88fbe1297c9f..b0ac71643674 100644 --- a/sc/source/filter/excel/xeroot.cxx +++ b/sc/source/filter/excel/xeroot.cxx @@ -58,7 +58,6 @@ XclExpRootData::XclExpRootData( XclBiff eBiff, SfxMedium& rMedium, const tools::SvRef<SotStorage>& xRootStrg, ScDocument& rDoc, rtl_TextEncoding eTextEnc ) : XclRootData( eBiff, rMedium, xRootStrg, rDoc, eTextEnc, true ) { - SvtSaveOptions aSaveOpt; mbRelUrl = mrMedium.IsRemote() ? officecfg::Office::Common::Save::URL::Internet::get() : officecfg::Office::Common::Save::URL::FileSystem::get(); diff --git a/sfx2/source/appl/appcfg.cxx b/sfx2/source/appl/appcfg.cxx index 5ed88bc71625..e2435ce2f43b 100644 --- a/sfx2/source/appl/appcfg.cxx +++ b/sfx2/source/appl/appcfg.cxx @@ -115,7 +115,6 @@ void SfxApplication::GetOptions( SfxItemSet& rSet ) SfxItemPool &rPool = GetPool(); const WhichRangesContainer& pRanges = rSet.GetRanges(); - SvtSaveOptions aSaveOptions; SvtSecurityOptions aSecurityOptions; SvtMiscOptions aMiscOptions; @@ -409,7 +408,6 @@ void SfxApplication::SetOptions_Impl( const SfxItemSet& rSet ) const SfxPoolItem *pItem = nullptr; SfxItemPool &rPool = GetPool(); - SvtSaveOptions aSaveOptions; SvtSecurityOptions aSecurityOptions; SvtMiscOptions aMiscOptions; std::shared_ptr< comphelper::ConfigurationChanges > batch( diff --git a/sfx2/source/dialog/alienwarn.cxx b/sfx2/source/dialog/alienwarn.cxx index a09469179cbc..8d27ef81d412 100644 --- a/sfx2/source/dialog/alienwarn.cxx +++ b/sfx2/source/dialog/alienwarn.cxx @@ -64,7 +64,6 @@ SfxAlienWarningDialog::~SfxAlienWarningDialog() try { // save value of "warning off" checkbox, if necessary - SvtSaveOptions aSaveOpt; bool bChecked = m_xWarningOnBox->get_active(); if (officecfg::Office::Common::Save::Document::WarnAlienFormat::get() != bChecked) { diff --git a/sfx2/source/inc/appdata.hxx b/sfx2/source/inc/appdata.hxx index 8bea6e44b6a2..3228321c66c3 100644 --- a/sfx2/source/inc/appdata.hxx +++ b/sfx2/source/inc/appdata.hxx @@ -45,7 +45,6 @@ class SfxStatusDispatcher; class SfxDdeTriggerTopic_Impl; class SfxDocumentTemplates; class SfxFrame; -class SvtSaveOptions; class SfxViewFrame; class SfxSlotPool; class SfxDispatcher; diff --git a/sw/source/filter/ww8/attributeoutputbase.hxx b/sw/source/filter/ww8/attributeoutputbase.hxx index 34920374537c..dc86307b40b5 100644 --- a/sw/source/filter/ww8/attributeoutputbase.hxx +++ b/sw/source/filter/ww8/attributeoutputbase.hxx @@ -31,7 +31,6 @@ #include <com/sun/star/drawing/TextVerticalAdjust.hpp> #include <swtypes.hxx> #include <fldbas.hxx> -#include <unotools/saveopt.hxx> class Point; class SvxCaseMapItem; @@ -143,7 +142,6 @@ enum StyleType class AttributeOutputBase { private: - SvtSaveOptions m_aSaveOpt; OUString m_sBaseURL; // To be used in ConvertURL OUString ConvertURL( const OUString& rUrl, bool bAbsoluteOut ); diff --git a/unotools/source/config/saveopt.cxx b/unotools/source/config/saveopt.cxx index aa7e875e1d97..6bac9ea28d04 100644 --- a/unotools/source/config/saveopt.cxx +++ b/unotools/source/config/saveopt.cxx @@ -21,7 +21,6 @@ #include <sal/log.hxx> #include <unotools/saveopt.hxx> -#include <rtl/instance.hxx> #include <unotools/configmgr.hxx> #include <unotools/configitem.hxx> #include <tools/debug.hxx> @@ -36,292 +35,67 @@ using namespace utl; using namespace com::sun::star::uno; -namespace { - -class SvtSaveOptions_Impl; -class SvtLoadOptions_Impl; - -} - -#define CFG_READONLY_DEFAULT false - -struct SvtLoadSaveOptions_Impl -{ - std::unique_ptr<SvtSaveOptions_Impl> pSaveOpt; -}; - -static std::unique_ptr<SvtLoadSaveOptions_Impl> pOptions; -static sal_Int32 nRefCount = 0; - -namespace { - -class SvtSaveOptions_Impl : public utl::ConfigItem -{ - sal_Int32 nAutoSaveTime; - bool bAutoSave; - - SvtSaveOptions::ODFDefaultVersion eODFDefaultVersion; - - bool bROODFDefaultVersion; - - virtual void ImplCommit() override; - -public: - SvtSaveOptions_Impl(); - - virtual void Notify( const css::uno::Sequence< OUString >& aPropertyNames ) override; - - SvtSaveOptions::ODFDefaultVersion - GetODFDefaultVersion() const { return eODFDefaultVersion; } - - void SetODFDefaultVersion( SvtSaveOptions::ODFDefaultVersion eNew ); - - bool IsReadOnly( SvtSaveOptions::EOption eOption ) const; -}; - -} - -void SvtSaveOptions_Impl::SetODFDefaultVersion( SvtSaveOptions::ODFDefaultVersion eNew ) +void SetODFDefaultVersion( SvtSaveOptions::ODFDefaultVersion eVersion, const std::shared_ptr<comphelper::ConfigurationChanges>& xChanges ) { - if ( !bROODFDefaultVersion && eODFDefaultVersion != eNew ) - { - eODFDefaultVersion = eNew; - SetModified(); - } + sal_Int16 nTmp = (eVersion == SvtSaveOptions::ODFVER_LATEST) ? sal_Int16( 3 ) : sal_Int16( eVersion ); + officecfg::Office::Common::Save::ODF::DefaultVersion::set(nTmp, xChanges); } -bool SvtSaveOptions_Impl::IsReadOnly( SvtSaveOptions::EOption eOption ) const +void SetODFDefaultVersion( SvtSaveOptions::ODFDefaultVersion eVersion ) { - bool bReadOnly = CFG_READONLY_DEFAULT; - switch(eOption) - { - case SvtSaveOptions::EOption::OdfDefaultVersion : - bReadOnly = bROODFDefaultVersion; - break; - } - return bReadOnly; + auto xChanges = comphelper::ConfigurationChanges::create(); + SetODFDefaultVersion(eVersion, xChanges); + xChanges->commit(); } -#define FORMAT 0 -#define ODFDEFAULTVERSION 1 - -static Sequence< OUString > GetPropertyNames() +SvtSaveOptions::ODFDefaultVersion GetODFDefaultVersion() { - static const char* aPropNames[] = - { - "Graphic/Format", - "ODF/DefaultVersion" - }; - - const int nCount = SAL_N_ELEMENTS( aPropNames ); - Sequence< OUString > aNames( nCount ); - OUString* pNames = aNames.getArray(); - for ( int i = 0; i < nCount; i++ ) - pNames[i] = OUString::createFromAscii( aPropNames[i] ); - - return aNames; -} - -SvtSaveOptions_Impl::SvtSaveOptions_Impl() - : ConfigItem( "Office.Common/Save" ) - , nAutoSaveTime( 0 ) - , eODFDefaultVersion( SvtSaveOptions::ODFVER_LATEST ) - , bROODFDefaultVersion( CFG_READONLY_DEFAULT ) -{ - Sequence< OUString > aNames = GetPropertyNames(); - Sequence< Any > aValues = GetProperties( aNames ); - Sequence< sal_Bool > aROStates = GetReadOnlyStates( aNames ); - EnableNotification( aNames ); - const Any* pValues = aValues.getConstArray(); - const sal_Bool* pROStates = aROStates.getConstArray(); - DBG_ASSERT( aValues.getLength() == aNames.getLength(), "GetProperties failed" ); - DBG_ASSERT( aROStates.getLength() == aNames.getLength(), "GetReadOnlyStates failed" ); - if ( aValues.getLength() == aNames.getLength() && aROStates.getLength() == aNames.getLength() ) - { - for ( int nProp = 0; nProp < aNames.getLength(); nProp++ ) - { - if ( pValues[nProp].hasValue() ) - { - switch ( nProp ) - { - case FORMAT: - // not supported anymore - break; - - case ODFDEFAULTVERSION : - { - sal_Int16 nTmp = 0; - if ( pValues[nProp] >>= nTmp ) - { - if( nTmp == 3 ) - eODFDefaultVersion = SvtSaveOptions::ODFVER_LATEST; - else - eODFDefaultVersion = SvtSaveOptions::ODFDefaultVersion( nTmp ); - } - else { - SAL_WARN( "unotools.config", "SvtSaveOptions_Impl::SvtSaveOptions_Impl(): Wrong Type!" ); - }; - bROODFDefaultVersion = pROStates[nProp]; - break; - } - - default: - OSL_FAIL( "Wrong Type!" ); - } - } - } - } - - if (!utl::ConfigManager::IsFuzzing()) - { - bAutoSave = officecfg::Office::Recovery::AutoSave::Enabled::get(); - nAutoSaveTime = officecfg::Office::Recovery::AutoSave::TimeIntervall::get(); - } + SvtSaveOptions::ODFDefaultVersion nRet; + sal_Int16 nTmp = officecfg::Office::Common::Save::ODF::DefaultVersion::get(); + if( nTmp == 3 ) + nRet = SvtSaveOptions::ODFVER_LATEST; else - { - bAutoSave = false; - nAutoSaveTime = 0; - } -} - -void SvtSaveOptions_Impl::ImplCommit() -{ - Sequence< OUString > aOrgNames = GetPropertyNames(); - OUString* pOrgNames = aOrgNames.getArray(); - sal_Int32 nOrgCount = aOrgNames.getLength(); - - Sequence< OUString > aNames( nOrgCount ); - Sequence< Any > aValues( nOrgCount ); - OUString* pNames = aNames.getArray(); - Any* pValues = aValues.getArray(); - sal_Int32 nRealCount = 0; - - for (sal_Int32 i=0; i<nOrgCount; ++i) - { - switch (i) - { - case FORMAT: - // not supported anymore - break; - case ODFDEFAULTVERSION: - if (!bROODFDefaultVersion) - { - pValues[nRealCount] <<= (eODFDefaultVersion == SvtSaveOptions::ODFVER_LATEST) ? sal_Int16( 3 ) : sal_Int16( eODFDefaultVersion ); - pNames[nRealCount] = pOrgNames[i]; - ++nRealCount; - } - break; - - default: - SAL_WARN( "unotools.config", "invalid index to save a path" ); - } - } - - aNames.realloc(nRealCount); - aValues.realloc(nRealCount); - PutProperties( aNames, aValues ); - - std::shared_ptr< comphelper::ConfigurationChanges > batch( - comphelper::ConfigurationChanges::create()); - officecfg::Office::Recovery::AutoSave::TimeIntervall::set(nAutoSaveTime, batch); - officecfg::Office::Recovery::AutoSave::Enabled::set(bAutoSave, batch); - batch->commit(); -} - -void SvtSaveOptions_Impl::Notify( const Sequence<OUString>& ) -{ -} - -namespace -{ - class LocalSingleton : public rtl::Static< osl::Mutex, LocalSingleton > - { - }; -} - -SvtSaveOptions::SvtSaveOptions() -{ - // Global access, must be guarded (multithreading) - ::osl::MutexGuard aGuard( LocalSingleton::get() ); - if ( !pOptions ) - { - pOptions.reset(new SvtLoadSaveOptions_Impl); - pOptions->pSaveOpt.reset(new SvtSaveOptions_Impl); - } - ++nRefCount; - pImp = pOptions.get(); -} - -SvtSaveOptions::~SvtSaveOptions() -{ - // Global access, must be guarded (multithreading) - ::osl::MutexGuard aGuard( LocalSingleton::get() ); - if ( !--nRefCount ) - { - if ( pOptions->pSaveOpt->IsModified() ) - pOptions->pSaveOpt->Commit(); - - pOptions.reset(); - } -} - -void SvtSaveOptions::SetODFDefaultVersion( SvtSaveOptions::ODFDefaultVersion eVersion ) -{ - pImp->pSaveOpt->SetODFDefaultVersion( eVersion ); -} - -SvtSaveOptions::ODFDefaultVersion SvtSaveOptions::GetODFDefaultVersion() const -{ - auto const nRet = pImp->pSaveOpt->GetODFDefaultVersion(); - SAL_WARN_IF(nRet == ODFVER_UNKNOWN, "unotools.config", "DefaultVersion is ODFVER_UNKNOWN?"); - return (nRet == ODFVER_UNKNOWN) ? ODFVER_LATEST : nRet; + nRet = SvtSaveOptions::ODFDefaultVersion( nTmp ); + SAL_WARN_IF(nRet == SvtSaveOptions::ODFVER_UNKNOWN, "unotools.config", "DefaultVersion is ODFVER_UNKNOWN?"); + return (nRet == SvtSaveOptions::ODFVER_UNKNOWN) ? SvtSaveOptions::ODFVER_LATEST : nRet; } SvtSaveOptions::ODFSaneDefaultVersion GetODFSaneDefaultVersion() { + SvtSaveOptions::ODFDefaultVersion nRet; sal_Int16 nTmp = officecfg::Office::Common::Save::ODF::DefaultVersion::get(); - SvtSaveOptions::ODFDefaultVersion eODFDefaultVersion; if( nTmp == 3 ) - eODFDefaultVersion = SvtSaveOptions::ODFVER_LATEST; + nRet = SvtSaveOptions::ODFVER_LATEST; else - eODFDefaultVersion = SvtSaveOptions::ODFDefaultVersion( nTmp ); - return SvtSaveOptions::GetODFSaneDefaultVersion(eODFDefaultVersion); -} + nRet = SvtSaveOptions::ODFDefaultVersion( nTmp ); -SvtSaveOptions::ODFSaneDefaultVersion SvtSaveOptions::GetODFSaneDefaultVersion() const -{ - return GetODFSaneDefaultVersion(pImp->pSaveOpt->GetODFDefaultVersion()); + return GetODFSaneDefaultVersion(nRet); } -SvtSaveOptions::ODFSaneDefaultVersion SvtSaveOptions::GetODFSaneDefaultVersion(ODFDefaultVersion eODFDefaultVersion) +SvtSaveOptions::ODFSaneDefaultVersion GetODFSaneDefaultVersion(SvtSaveOptions::ODFDefaultVersion eODFDefaultVersion) { switch (eODFDefaultVersion) { default: assert(!"map new ODFDefaultVersion to ODFSaneDefaultVersion"); break; - case ODFVER_UNKNOWN: - case ODFVER_LATEST: - return ODFSVER_LATEST_EXTENDED; - case ODFVER_010: - return ODFSVER_010; - case ODFVER_011: - return ODFSVER_011; - case ODFVER_012: - return ODFSVER_012; - case ODFVER_012_EXT_COMPAT: - return ODFSVER_012_EXT_COMPAT; - case ODFVER_012_EXTENDED: - return ODFSVER_012_EXTENDED; - case ODFVER_013: - return ODFSVER_013; + case SvtSaveOptions::ODFVER_UNKNOWN: + case SvtSaveOptions::ODFVER_LATEST: + return SvtSaveOptions::ODFSVER_LATEST_EXTENDED; + case SvtSaveOptions::ODFVER_010: + return SvtSaveOptions::ODFSVER_010; + case SvtSaveOptions::ODFVER_011: + return SvtSaveOptions::ODFSVER_011; + case SvtSaveOptions::ODFVER_012: + return SvtSaveOptions::ODFSVER_012; + case SvtSaveOptions::ODFVER_012_EXT_COMPAT: + return SvtSaveOptions::ODFSVER_012_EXT_COMPAT; + case SvtSaveOptions::ODFVER_012_EXTENDED: + return SvtSaveOptions::ODFSVER_012_EXTENDED; + case SvtSaveOptions::ODFVER_013: + return SvtSaveOptions::ODFSVER_013; } - return ODFSVER_LATEST_EXTENDED; -} - -bool SvtSaveOptions::IsReadOnly( SvtSaveOptions::EOption eOption ) const -{ - return pImp->pSaveOpt->IsReadOnly(eOption); + return SvtSaveOptions::ODFSVER_LATEST_EXTENDED; } /* vim:set shiftwidth=4 softtabstop=4 expandtab: */ diff --git a/xmloff/source/core/xmlexp.cxx b/xmloff/source/core/xmlexp.cxx index d73783994271..3159aa44e683 100644 --- a/xmloff/source/core/xmlexp.cxx +++ b/xmloff/source/core/xmlexp.cxx @@ -261,7 +261,6 @@ public: uno::Reference< embed::XStorage > mxTargetStorage; - SvtSaveOptions maSaveOptions; std::optional<SvtSaveOptions::ODFSaneDefaultVersion> m_oOverrideODFVersion; /// name of stream in package, e.g., "content.xml" @@ -430,7 +429,7 @@ void SvXMLExport::DetermineModelType_() xModule->getIdentifier() == "com.sun.star.sdb.FormDesign"); if (isBaseForm) { - switch (mpImpl->maSaveOptions.GetODFSaneDefaultVersion()) + switch (GetODFSaneDefaultVersion()) { case SvtSaveOptions::ODFSVER_013_EXTENDED: SAL_INFO("xmloff.core", "tdf#138209 force form export to ODF 1.2"); @@ -2327,7 +2326,7 @@ SvtSaveOptions::ODFSaneDefaultVersion SvXMLExport::getSaneDefaultVersion() const { return *mpImpl->m_oOverrideODFVersion; } - return mpImpl->maSaveOptions.GetODFSaneDefaultVersion(); + return GetODFSaneDefaultVersion(); } void diff --git a/xmloff/source/style/xmlexppr.cxx b/xmloff/source/style/xmlexppr.cxx index 44bab345afe2..a2d9aa546190 100644 --- a/xmloff/source/style/xmlexppr.cxx +++ b/xmloff/source/style/xmlexppr.cxx @@ -548,7 +548,7 @@ vector<XMLPropertyState> SvXMLExportPropertyMapper::Filter_( bool bDelInfo = false; if( !pFilterInfo ) { - assert(SvtSaveOptions().GetODFDefaultVersion() != SvtSaveOptions::ODFVER_UNKNOWN); + assert(GetODFDefaultVersion() != SvtSaveOptions::ODFVER_UNKNOWN); const SvtSaveOptions::ODFSaneDefaultVersion nCurrentVersion(rExport.getSaneDefaultVersion()); pFilterInfo = new FilterPropertiesInfo_Impl; for( sal_Int32 i=0; i < nProps; i++ ) _______________________________________________ Libreoffice-commits mailing list libreoffice-comm...@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits