[Libreoffice-commits] core.git: cui/source framework/source officecfg/registry unotools/source
Krisztian Pinter
pin.terminator at gmail.com
Mon Mar 11 07:37:30 PDT 2013
cui/source/options/optsave.src | 2 -
framework/source/services/autorecovery.cxx | 17 +++++++++--
officecfg/registry/schema/org/openoffice/Office/Recovery.xcs | 2 -
unotools/source/config/saveopt.cxx | 16 ----------
4 files changed, 17 insertions(+), 20 deletions(-)
New commits:
commit 8933f19100adbf9f2601db0f3ac67dd3b1887ef7
Author: Krisztian Pinter <pin.terminator at gmail.com>
Date: Mon Mar 11 12:46:18 2013 +0100
fdo#47011 autosave feature
removed unnecessary prop. name "Document/UserAutoSave" from saveopt.cxx
fixed autosave feature setting "losing its value"
set autosave feature default value to False
Change-Id: I473154b21bab53bf595a5a59e87dc16e472dcbf9
Reviewed-on: https://gerrit.libreoffice.org/2663
Reviewed-by: Bosdonnat Cedric <cedric.bosdonnat at free.fr>
Tested-by: Bosdonnat Cedric <cedric.bosdonnat at free.fr>
diff --git a/cui/source/options/optsave.src b/cui/source/options/optsave.src
index 68f0b54..71b29d6 100644
--- a/cui/source/options/optsave.src
+++ b/cui/source/options/optsave.src
@@ -113,7 +113,7 @@ TabPage RID_SFXPAGE_SAVE
};
CheckBox BTN_USERAUTOSAVE
{
- HelpID = "cui:CheckBox:RID_SFXPAGE_SAVE:BTN_USERAUTOSAVE"; //? FIX ME
+ HelpID = "cui:CheckBox:RID_SFXPAGE_SAVE:BTN_USERAUTOSAVE";
Pos = MAP_APPFONT ( 21 , 74 ) ;
Size = MAP_APPFONT ( 163 , 10 ) ;
Text [ en-US ] = "Automatically save the document too" ;
diff --git a/framework/source/services/autorecovery.cxx b/framework/source/services/autorecovery.cxx
index d0f1fab..431b5a4 100644
--- a/framework/source/services/autorecovery.cxx
+++ b/framework/source/services/autorecovery.cxx
@@ -21,8 +21,6 @@
#include "services/autorecovery.hxx"
#include <loadenv/loadenv.hxx>
-#include <sfx2/sfxbasemodel.hxx> //?
-
#include <loadenv/targethelper.hxx>
#include <pattern/frame.hxx>
#include <threadhelp/readguard.hxx>
@@ -90,6 +88,8 @@
#include <fwkdllapi.h>
+#include <sfx2/objsh.hxx>
+
//_______________________________________________
// namespaces
@@ -583,6 +583,8 @@ void AutoRecovery::implts_dispatch(const DispatchParams& aParams)
// in case a new dispatch overwrites a may ba active AutoSave session
// we must restore this session later. see below ...
sal_Bool bWasAutoSaveActive = ((eJob & AutoRecovery::E_AUTO_SAVE) == AutoRecovery::E_AUTO_SAVE);
+ sal_Bool bWasUserAutoSaveActive =
+ ((eJob & AutoRecovery::E_USER_AUTO_SAVE) == AutoRecovery::E_USER_AUTO_SAVE);
// On the other side it make no sense to reactivate the AutoSave operation
// if the new dispatch indicates a final decision ...
@@ -691,6 +693,11 @@ void AutoRecovery::implts_dispatch(const DispatchParams& aParams)
)
{
m_eJob |= AutoRecovery::E_AUTO_SAVE;
+
+ if (bWasUserAutoSaveActive)
+ {
+ m_eJob |= AutoRecovery::E_USER_AUTO_SAVE;
+ }
}
aWriteLock.unlock();
@@ -993,9 +1000,13 @@ void AutoRecovery::implts_readAutoSaveConfig()
m_eTimerType = AutoRecovery::E_NORMAL_AUTOSAVE_INTERVALL;
if (bUserEnabled)
+ {
m_eJob |= AutoRecovery::E_USER_AUTO_SAVE;
+ }
else
+ {
m_eJob &= ~AutoRecovery::E_USER_AUTO_SAVE;
+ }
}
else
{
@@ -2341,7 +2352,6 @@ void AutoRecovery::implts_saveOneDoc(const ::rtl::OUString&
// Mark AutoSave state as "INCOMPLETE" if it failed.
// Because the last temp file is to old and does not include all changes.
Reference< XDocumentRecovery > xDocRecover(rInfo.Document, css::uno::UNO_QUERY_THROW);
- Reference< XStorable > xDocSave(rInfo.Document, css::uno::UNO_QUERY_THROW);
// safe the state about "trying to save"
// ... we need it for recovery if e.g. a crash occures inside next line!
@@ -2359,6 +2369,7 @@ void AutoRecovery::implts_saveOneDoc(const ::rtl::OUString&
// if userautosave is enabled, also save to the original file
if((m_eJob & AutoRecovery::E_USER_AUTO_SAVE) == AutoRecovery::E_USER_AUTO_SAVE)
{
+ Reference< XStorable > xDocSave(rInfo.Document, css::uno::UNO_QUERY_THROW);
xDocSave->store();
}
diff --git a/officecfg/registry/schema/org/openoffice/Office/Recovery.xcs b/officecfg/registry/schema/org/openoffice/Office/Recovery.xcs
index a44e611..c775d08 100644
--- a/officecfg/registry/schema/org/openoffice/Office/Recovery.xcs
+++ b/officecfg/registry/schema/org/openoffice/Office/Recovery.xcs
@@ -135,7 +135,7 @@
<info>
<desc> </desc>
</info>
- <value>true</value>
+ <value>false</value>
</prop>
<prop oor:name="TimeIntervall" oor:type="xs:int" oor:nillable="false">
<info>
diff --git a/unotools/source/config/saveopt.cxx b/unotools/source/config/saveopt.cxx
index 452fcca..513f2a3 100644
--- a/unotools/source/config/saveopt.cxx
+++ b/unotools/source/config/saveopt.cxx
@@ -357,7 +357,6 @@ sal_Bool SvtSaveOptions_Impl::IsReadOnly( SvtSaveOptions::EOption eOption ) cons
#define ODFDEFAULTVERSION 15
#define USESHA1INODF12 16
#define USEBLOWFISHINODF12 17
-#define USERAUTOSAVE 18
Sequence< OUString > GetPropertyNames()
{
@@ -380,8 +379,7 @@ Sequence< OUString > GetPropertyNames()
"WorkingSet",
"ODF/DefaultVersion",
"ODF/UseSHA1InODF12",
- "ODF/UseBlowfishInODF12",
- "Document/UserAutoSave"
+ "ODF/UseBlowfishInODF12"
};
const int nCount = sizeof( aPropNames ) / sizeof( const char* );
@@ -500,10 +498,6 @@ SvtSaveOptions_Impl::SvtSaveOptions_Impl()
bAutoSave = bTemp;
bROAutoSave = pROStates[nProp];
break;
- case USERAUTOSAVE :
- bUserAutoSave = bTemp;
- bROUserAutoSave = pROStates[nProp];
- break;
case PROMPT :
bAutoSavePrompt = bTemp;
bROAutoSavePrompt = pROStates[nProp];
@@ -660,14 +654,6 @@ void SvtSaveOptions_Impl::Commit()
++nRealCount;
}
break;
- case USERAUTOSAVE :
- if (!bROUserAutoSave)
- {
- pValues[nRealCount] <<= bUserAutoSave;
- pNames[nRealCount] = pOrgNames[i];
- ++nRealCount;
- }
- break;
case EDITPROPERTY :
if (!bRODocInfSave)
{
More information about the Libreoffice-commits
mailing list