[Libreoffice-commits] core.git: basic/source include/svtools include/tools sc/source sd/source sfx2/source svtools/source svx/source sw/source tools/source uui/source
Noel Grandin
noel.grandin at collabora.co.uk
Tue Feb 7 12:04:57 UTC 2017
basic/source/basmgr/basicmanagerrepository.cxx | 2
basic/source/basmgr/basmgr.cxx | 18 +++---
include/svtools/ehdl.hxx | 4 -
include/svtools/sfxecode.hxx | 1
include/tools/errcode.hxx | 19 -------
include/tools/errinf.hxx | 50 +++++++++++++++----
sc/source/filter/xml/xmlwrap.cxx | 4 -
sc/source/ui/docshell/docsh.cxx | 2
sc/source/ui/docshell/docsh8.cxx | 4 -
sd/source/filter/xml/sdxmlwrp.cxx | 4 -
sfx2/source/doc/objstor.cxx | 2
svtools/source/misc/ehdl.cxx | 65 ++++++++++++-------------
svx/source/src/errtxt.src | 5 -
sw/source/core/frmedt/fetab.cxx | 20 +++----
sw/source/filter/html/swhtml.cxx | 2
sw/source/filter/xml/swxml.cxx | 4 -
sw/source/filter/xml/wrtxml.cxx | 4 -
sw/source/uibase/misc/glosdoc.cxx | 4 -
tools/source/ref/errinf.cxx | 54 ++++++++++----------
uui/source/iahndl-authentication.cxx | 26 +++++-----
uui/source/iahndl-errorhandler.cxx | 32 ++++++------
uui/source/iahndl.cxx | 26 +++++-----
uui/source/loginerr.hxx | 10 +--
23 files changed, 188 insertions(+), 174 deletions(-)
New commits:
commit 05e78bde26d8b8b257ed22a0ce20c5b386a629d1
Author: Noel Grandin <noel.grandin at collabora.co.uk>
Date: Tue Feb 7 11:26:27 2017 +0200
create ErrorHandlerFlags scoped enum for error handling flags
Change-Id: I30c80979c87bad7bc98f36a158c31c88d80d1caf
Reviewed-on: https://gerrit.libreoffice.org/33991
Tested-by: Jenkins <ci at libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin at collabora.co.uk>
diff --git a/basic/source/basmgr/basicmanagerrepository.cxx b/basic/source/basmgr/basicmanagerrepository.cxx
index f7bff18..500f314 100644
--- a/basic/source/basmgr/basicmanagerrepository.cxx
+++ b/basic/source/basmgr/basicmanagerrepository.cxx
@@ -441,7 +441,7 @@ namespace basic
for(const auto& rError : aErrors)
{
// show message to user
- if ( ERRCODE_BUTTON_CANCEL == ErrorHandler::HandleError( rError.GetErrorId() ) )
+ if ( ErrorHandlerFlags::ButtonsCancel == ErrorHandler::HandleError( rError.GetErrorId() ) )
{
// user wants to break loading of BASIC-manager
delete _out_rpBasicManager;
diff --git a/basic/source/basmgr/basmgr.cxx b/basic/source/basmgr/basmgr.cxx
index 286a9c4..2a75286 100644
--- a/basic/source/basmgr/basmgr.cxx
+++ b/basic/source/basmgr/basmgr.cxx
@@ -636,7 +636,7 @@ void BasicManager::ImpMgrNotLoaded( const OUString& rStorageName )
{
// pErrInf is only destroyed if the error is processed by an
// ErrorHandler
- StringErrorInfo* pErrInf = new StringErrorInfo( ERRCODE_BASMGR_MGROPEN, rStorageName, ERRCODE_BUTTON_OK );
+ StringErrorInfo* pErrInf = new StringErrorInfo( ERRCODE_BASMGR_MGROPEN, rStorageName, ErrorHandlerFlags::ButtonsOk );
aErrors.push_back(BasicError(pErrInf->GetErrorCode(), BasicErrorReason::OPENMGRSTREAM));
// Create a stdlib otherwise we crash!
@@ -776,7 +776,7 @@ void BasicManager::LoadOldBasicManager( SotStorage& rStorage )
xManagerStream->Seek( nBasicStartOff );
if (!ImplLoadBasic( *xManagerStream, mpImpl->aLibs.front()->GetLibRef() ))
{
- StringErrorInfo* pErrInf = new StringErrorInfo( ERRCODE_BASMGR_MGROPEN, aStorName, ERRCODE_BUTTON_OK );
+ StringErrorInfo* pErrInf = new StringErrorInfo( ERRCODE_BASMGR_MGROPEN, aStorName, ErrorHandlerFlags::ButtonsOk );
aErrors.push_back(BasicError(pErrInf->GetErrorCode(), BasicErrorReason::OPENMGRSTREAM));
// and it proceeds ...
}
@@ -824,7 +824,7 @@ void BasicManager::LoadOldBasicManager( SotStorage& rStorage )
}
else
{
- StringErrorInfo* pErrInf = new StringErrorInfo( ERRCODE_BASMGR_LIBLOAD, aStorName, ERRCODE_BUTTON_OK );
+ StringErrorInfo* pErrInf = new StringErrorInfo( ERRCODE_BASMGR_LIBLOAD, aStorName, ErrorHandlerFlags::ButtonsOk );
aErrors.push_back(BasicError(pErrInf->GetErrorCode(), BasicErrorReason::STORAGENOTFOUND));
}
}
@@ -907,7 +907,7 @@ bool BasicManager::ImpLoadLibrary( BasicLibInfo* pLibInfo, SotStorage* pCurStora
if ( !xBasicStorage.is() || xBasicStorage->GetError() )
{
- StringErrorInfo* pErrInf = new StringErrorInfo( ERRCODE_BASMGR_MGROPEN, xStorage->GetName(), ERRCODE_BUTTON_OK );
+ StringErrorInfo* pErrInf = new StringErrorInfo( ERRCODE_BASMGR_MGROPEN, xStorage->GetName(), ErrorHandlerFlags::ButtonsOk );
aErrors.push_back(BasicError(pErrInf->GetErrorCode(), BasicErrorReason::OPENLIBSTORAGE));
}
else
@@ -916,7 +916,7 @@ bool BasicManager::ImpLoadLibrary( BasicLibInfo* pLibInfo, SotStorage* pCurStora
tools::SvRef<SotStorageStream> xBasicStream = xBasicStorage->OpenSotStream( pLibInfo->GetLibName(), eStreamReadMode );
if ( !xBasicStream.is() || xBasicStream->GetError() )
{
- StringErrorInfo* pErrInf = new StringErrorInfo( ERRCODE_BASMGR_LIBLOAD , pLibInfo->GetLibName(), ERRCODE_BUTTON_OK );
+ StringErrorInfo* pErrInf = new StringErrorInfo( ERRCODE_BASMGR_LIBLOAD , pLibInfo->GetLibName(), ErrorHandlerFlags::ButtonsOk );
aErrors.push_back(BasicError(pErrInf->GetErrorCode(), BasicErrorReason::OPENLIBSTREAM));
}
else
@@ -939,7 +939,7 @@ bool BasicManager::ImpLoadLibrary( BasicLibInfo* pLibInfo, SotStorage* pCurStora
}
if ( !bLoaded )
{
- StringErrorInfo* pErrInf = new StringErrorInfo( ERRCODE_BASMGR_LIBLOAD, pLibInfo->GetLibName(), ERRCODE_BUTTON_OK );
+ StringErrorInfo* pErrInf = new StringErrorInfo( ERRCODE_BASMGR_LIBLOAD, pLibInfo->GetLibName(), ErrorHandlerFlags::ButtonsOk );
aErrors.push_back(BasicError(pErrInf->GetErrorCode(), BasicErrorReason::BASICLOADERROR));
}
else
@@ -1123,7 +1123,7 @@ bool BasicManager::RemoveLib( sal_uInt16 nLib, bool bDelBasicFromStorage )
if( !nLib || nLib < mpImpl->aLibs.size() )
{
- StringErrorInfo* pErrInf = new StringErrorInfo( ERRCODE_BASMGR_REMOVELIB, OUString(), ERRCODE_BUTTON_OK );
+ StringErrorInfo* pErrInf = new StringErrorInfo( ERRCODE_BASMGR_REMOVELIB, OUString(), ErrorHandlerFlags::ButtonsOk );
aErrors.push_back(BasicError(pErrInf->GetErrorCode(), BasicErrorReason::STDLIB));
return false;
}
@@ -1159,7 +1159,7 @@ bool BasicManager::RemoveLib( sal_uInt16 nLib, bool bDelBasicFromStorage )
if ( !xBasicStorage.is() || xBasicStorage->GetError() )
{
- StringErrorInfo* pErrInf = new StringErrorInfo( ERRCODE_BASMGR_REMOVELIB, OUString(), ERRCODE_BUTTON_OK );
+ StringErrorInfo* pErrInf = new StringErrorInfo( ERRCODE_BASMGR_REMOVELIB, OUString(), ErrorHandlerFlags::ButtonsOk );
aErrors.push_back(BasicError(pErrInf->GetErrorCode(), BasicErrorReason::OPENLIBSTORAGE));
}
else if (xBasicStorage->IsStream((*itLibInfo)->GetLibName()))
@@ -1294,7 +1294,7 @@ bool BasicManager::LoadLib( sal_uInt16 nLib )
}
else
{
- StringErrorInfo* pErrInf = new StringErrorInfo( ERRCODE_BASMGR_LIBLOAD, OUString(), ERRCODE_BUTTON_OK );
+ StringErrorInfo* pErrInf = new StringErrorInfo( ERRCODE_BASMGR_LIBLOAD, OUString(), ErrorHandlerFlags::ButtonsOk );
aErrors.push_back(BasicError(pErrInf->GetErrorCode(), BasicErrorReason::LIBNOTFOUND));
}
return bDone;
diff --git a/include/svtools/ehdl.hxx b/include/svtools/ehdl.hxx
index cb73990..2865e26 100644
--- a/include/svtools/ehdl.hxx
+++ b/include/svtools/ehdl.hxx
@@ -54,7 +54,7 @@ public:
virtual ~SfxErrorHandler() override;
protected:
- bool GetErrorString(sal_uLong lErrId, OUString &, sal_uInt16&) const;
+ bool GetErrorString(sal_uLong lErrId, OUString &, ErrorHandlerFlags&) const;
private:
@@ -66,7 +66,7 @@ private:
pFreeMgr;
SVT_DLLPRIVATE static void GetClassString(sal_uLong lErrId, OUString &);
- virtual bool CreateString( const ErrorInfo *, OUString &, sal_uInt16 &) const override;
+ virtual bool CreateString( const ErrorInfo *, OUString &, ErrorHandlerFlags &) const override;
};
#endif
diff --git a/include/svtools/sfxecode.hxx b/include/svtools/sfxecode.hxx
index 99083f2..77c3ee4 100644
--- a/include/svtools/sfxecode.hxx
+++ b/include/svtools/sfxecode.hxx
@@ -34,7 +34,6 @@
#define ERRCODE_SFX_WRONGPASSWORD (ERRCODE_AREA_SFX|ERRCODE_CLASS_READ|11)
#define ERRCODE_SFX_DOCUMENTREADONLY (ERRCODE_AREA_SFX|ERRCODE_CLASS_WRITE|12)
#define ERRCODE_SFX_OLEGENERAL (ERRCODE_AREA_SFX|ERRCODE_CLASS_NONE|14)
-#define ERRCODE_SFXMSG_STYLEREPLACE (ERRCODE_WARNING_MASK|ERRCODE_AREA_SFX|ERRCODE_CLASS_NONE|13)
#define ERRCODE_SFX_TEMPLATENOTFOUND (ERRCODE_AREA_SFX|ERRCODE_CLASS_NOTEXISTS|15)
#define ERRCODE_SFX_ISRELATIVE (ERRCODE_WARNING_MASK|ERRCODE_AREA_SFX|ERRCODE_CLASS_NOTEXISTS|16)
#define ERRCODE_SFX_FORCEDOCLOAD (ERRCODE_WARNING_MASK|ERRCODE_AREA_SFX|ERRCODE_CLASS_NONE|17)
diff --git a/include/tools/errcode.hxx b/include/tools/errcode.hxx
index 5a8f57a..f9859d38 100644
--- a/include/tools/errcode.hxx
+++ b/include/tools/errcode.hxx
@@ -46,25 +46,6 @@ Warning || || |
Code
*/
-#define ERRCODE_BUTTON_OK 0x01
-#define ERRCODE_BUTTON_CANCEL 0x02
-#define ERRCODE_BUTTON_RETRY 0x04
-#define ERRCODE_BUTTON_OK_CANCEL 0x03
-#define ERRCODE_BUTTON_NO 0x08
-#define ERRCODE_BUTTON_YES 0x10
-#define ERRCODE_BUTTON_YES_NO 0x18
-#define ERRCODE_BUTTON_YES_NO_CANCEL 0x1a
-
-#define ERRCODE_BUTTON_DEF_OK 0x100
-#define ERRCODE_BUTTON_DEF_CANCEL 0x200
-#define ERRCODE_BUTTON_DEF_YES 0x300
-#define ERRCODE_BUTTON_DEF_NO 0x400
-
-#define ERRCODE_MSG_ERROR 0x1000
-#define ERRCODE_MSG_WARNING 0x2000
-#define ERRCODE_MSG_INFO 0x3000
-#define ERRCODE_MSG_QUERY 0x4000
-
#define ERRCODE_ERROR_MASK 0x3fffffffUL
#define ERRCODE_WARNING_MASK 0x80000000UL
#define ERRCODE_RES_MASK 0x7fff
diff --git a/include/tools/errinf.hxx b/include/tools/errinf.hxx
index 2b9f6b5..d83fa54 100644
--- a/include/tools/errinf.hxx
+++ b/include/tools/errinf.hxx
@@ -26,6 +26,7 @@
#include <rtl/ustring.hxx>
#include <tools/errcode.hxx>
#include <tools/toolsdllapi.h>
+#include <o3tl/typed_flags_set.hxx>
#include <memory>
// FIXME: horrible legacy dependency on VCL from tools.
@@ -34,6 +35,35 @@ namespace vcl { class Window; }
class DynamicErrorInfo_Impl;
class ErrorHandler_Impl;
+enum class ErrorHandlerFlags
+{
+ NONE = 0x0000,
+ ButtonsOk = 0x0001,
+ ButtonsCancel = 0x0002,
+ ButtonsRetry = 0x0004,
+ ButtonsOkCancel = 0x0003,
+ ButtonsNo = 0x0008,
+ ButtonsYes = 0x0010,
+ ButtonsYesNo = 0x0018,
+ ButtonsYesNoCancel = 0x001a,
+
+ ButtonDefaultsOk = 0x0100,
+ ButtonDefaultsCancel = 0x0200,
+ ButtonDefaultsYes = 0x0300,
+ ButtonDefaultsNo = 0x0400,
+
+ MessageError = 0x1000,
+ MessageWarning = 0x2000,
+ MessageInfo = 0x3000,
+ MessageQuery = 0x4000,
+
+ MAX = USHRT_MAX,
+};
+namespace o3tl
+{
+ template<> struct typed_flags<ErrorHandlerFlags> : is_typed_flags<ErrorHandlerFlags, 0xffff> {};
+}
+
class SAL_WARN_UNUSED TOOLS_DLLPUBLIC ErrorInfo
{
private:
@@ -59,11 +89,11 @@ private:
public:
- DynamicErrorInfo(sal_uIntPtr lUserId, sal_uInt16 nMask);
+ DynamicErrorInfo(sal_uIntPtr lUserId, ErrorHandlerFlags nMask);
virtual ~DynamicErrorInfo() override;
sal_uIntPtr GetErrorCode() const;
- sal_uInt16 GetDialogMask() const;
+ ErrorHandlerFlags GetDialogMask() const;
};
class SAL_WARN_UNUSED TOOLS_DLLPUBLIC StringErrorInfo : public DynamicErrorInfo
@@ -75,7 +105,7 @@ public:
StringErrorInfo( sal_uIntPtr lUserId,
const OUString& aStringP,
- sal_uInt16 nMask = 0);
+ ErrorHandlerFlags nMask = ErrorHandlerFlags::NONE);
const OUString& GetErrorString() const { return aString; }
};
@@ -88,7 +118,7 @@ private:
public:
TwoStringErrorInfo(sal_uIntPtr nUserID, const OUString & rTheArg1,
- const OUString & rTheArg2, sal_uInt16 nMask):
+ const OUString & rTheArg2, ErrorHandlerFlags nMask):
DynamicErrorInfo(nUserID, nMask), aArg1(rTheArg1), aArg2(rTheArg2) {}
const OUString& GetArg1() const { return aArg1; }
@@ -113,8 +143,8 @@ public:
static ErrorContext* GetContext();
};
-typedef sal_uInt16 WindowDisplayErrorFunc(
- vcl::Window *, sal_uInt16 nMask, const OUString &rErr, const OUString &rAction);
+typedef ErrorHandlerFlags WindowDisplayErrorFunc(
+ vcl::Window *, ErrorHandlerFlags nMask, const OUString &rErr, const OUString &rAction);
typedef void BasicDisplayErrorFunc(
const OUString &rErr, const OUString &rAction);
@@ -124,19 +154,19 @@ class SAL_WARN_UNUSED TOOLS_DLLPUBLIC ErrorHandler
friend class ErrorHandler_Impl;
private:
- static sal_uInt16 HandleError_Impl( sal_uIntPtr lId,
- sal_uInt16 nFlags,
+ static ErrorHandlerFlags HandleError_Impl( sal_uIntPtr lId,
+ ErrorHandlerFlags nFlags,
bool bJustCreateString,
OUString & rError);
protected:
virtual bool CreateString( const ErrorInfo *,
- OUString &, sal_uInt16& nMask ) const = 0;
+ OUString &, ErrorHandlerFlags& nMask ) const = 0;
public:
ErrorHandler();
virtual ~ErrorHandler();
- static sal_uInt16 HandleError ( sal_uIntPtr lId, sal_uInt16 nMask = USHRT_MAX );
+ static ErrorHandlerFlags HandleError ( sal_uIntPtr lId, ErrorHandlerFlags nMask = ErrorHandlerFlags::MAX );
static bool GetErrorString( sal_uIntPtr lId, OUString& rStr );
static void RegisterDisplay( BasicDisplayErrorFunc* );
diff --git a/sc/source/filter/xml/xmlwrap.cxx b/sc/source/filter/xml/xmlwrap.cxx
index 82c90df..0bf1475 100644
--- a/sc/source/filter/xml/xmlwrap.cxx
+++ b/sc/source/filter/xml/xmlwrap.cxx
@@ -220,13 +220,13 @@ sal_uInt32 ScXMLImportWrapper::ImportFromComponent(const uno::Reference<uno::XCo
(bMustBeSuccessfull ? SCERR_IMPORT_FILE_ROWCOL
: SCWARN_IMPORT_FILE_ROWCOL),
sDocName, sErr,
- ERRCODE_BUTTON_OK | ERRCODE_MSG_ERROR ))->GetErrorCode();
+ ErrorHandlerFlags::ButtonsOk | ErrorHandlerFlags::MessageError ))->GetErrorCode();
}
else
{
OSL_ENSURE( bMustBeSuccessfull, "Warnings are not supported" );
nReturn = (new StringErrorInfo( SCERR_IMPORT_FORMAT_ROWCOL, sErr,
- ERRCODE_BUTTON_OK | ERRCODE_MSG_ERROR ))->GetErrorCode();
+ ErrorHandlerFlags::ButtonsOk | ErrorHandlerFlags::MessageError ))->GetErrorCode();
}
}
}
diff --git a/sc/source/ui/docshell/docsh.cxx b/sc/source/ui/docshell/docsh.cxx
index 1a7fc52..9fe756c 100644
--- a/sc/source/ui/docshell/docsh.cxx
+++ b/sc/source/ui/docshell/docsh.cxx
@@ -2468,7 +2468,7 @@ bool ScDocShell::ConvertTo( SfxMedium &rMed )
SetError((new StringErrorInfo(
SCWARN_EXPORT_NONCONVERTIBLE_CHARS,
aImExport.GetNonConvertibleChars(),
- ERRCODE_BUTTON_OK | ERRCODE_MSG_INFO))->GetErrorCode(), OSL_LOG_PREFIX);
+ ErrorHandlerFlags::ButtonsOk | ErrorHandlerFlags::MessageInfo))->GetErrorCode(), OSL_LOG_PREFIX);
}
}
}
diff --git a/sc/source/ui/docshell/docsh8.cxx b/sc/source/ui/docshell/docsh8.cxx
index 72dde81..29aa6b8 100644
--- a/sc/source/ui/docshell/docsh8.cxx
+++ b/sc/source/ui/docshell/docsh8.cxx
@@ -1090,10 +1090,10 @@ sal_uLong ScDocShell::DBaseExport( const OUString& rFullFileName, rtl_TextEncodi
OUString sEncoding( SvxTextEncodingTable().GetTextString( eCharSet));
nErr = (new TwoStringErrorInfo( (bEncErr ? SCERR_EXPORT_ENCODING :
SCERR_EXPORT_FIELDWIDTH), sPosition, sEncoding,
- ERRCODE_BUTTON_OK | ERRCODE_MSG_ERROR))->GetErrorCode();
+ ErrorHandlerFlags::ButtonsOk | ErrorHandlerFlags::MessageError))->GetErrorCode();
}
else if ( !aException.Message.isEmpty() )
- nErr = (new StringErrorInfo( SCERR_EXPORT_SQLEXCEPTION, aException.Message, ERRCODE_BUTTON_OK | ERRCODE_MSG_ERROR))->GetErrorCode();
+ nErr = (new StringErrorInfo( SCERR_EXPORT_SQLEXCEPTION, aException.Message, ErrorHandlerFlags::ButtonsOk | ErrorHandlerFlags::MessageError))->GetErrorCode();
else
nErr = SCERR_EXPORT_DATA;
}
diff --git a/sd/source/filter/xml/sdxmlwrp.cxx b/sd/source/filter/xml/sdxmlwrp.cxx
index d79ff01..9f8c176 100644
--- a/sd/source/filter/xml/sdxmlwrp.cxx
+++ b/sd/source/filter/xml/sdxmlwrp.cxx
@@ -249,13 +249,13 @@ sal_Int32 ReadThroughComponent(
(bMustBeSuccessfull ? ERR_FORMAT_FILE_ROWCOL
: WARN_FORMAT_FILE_ROWCOL),
rStreamName, sErr,
- ERRCODE_BUTTON_OK | ERRCODE_MSG_ERROR ))->GetErrorCode();
+ ErrorHandlerFlags::ButtonsOk | ErrorHandlerFlags::MessageError ))->GetErrorCode();
}
else
{
DBG_ASSERT( bMustBeSuccessfull, "Warnings are not supported" );
return (new StringErrorInfo( ERR_FORMAT_ROWCOL, sErr,
- ERRCODE_BUTTON_OK | ERRCODE_MSG_ERROR ))->GetErrorCode();
+ ErrorHandlerFlags::ButtonsOk | ErrorHandlerFlags::MessageError ))->GetErrorCode();
}
}
catch (const xml::sax::SAXException& r)
diff --git a/sfx2/source/doc/objstor.cxx b/sfx2/source/doc/objstor.cxx
index b0c4687..4135b54 100644
--- a/sfx2/source/doc/objstor.cxx
+++ b/sfx2/source/doc/objstor.cxx
@@ -2297,7 +2297,7 @@ bool SfxObjectShell::ImportFrom(SfxMedium& rMedium,
if (rWrapped.TargetException >>= e)
{
SetError((new StringErrorInfo(ERRCODE_SFX_FORMAT_ROWCOL,
- e.Message, ERRCODE_BUTTON_OK | ERRCODE_MSG_ERROR ))->GetErrorCode(), "");
+ e.Message, ErrorHandlerFlags::ButtonsOk | ErrorHandlerFlags::MessageError ))->GetErrorCode(), "");
}
}
catch(...)
diff --git a/svtools/source/misc/ehdl.cxx b/svtools/source/misc/ehdl.cxx
index 577953b..6a3481f 100644
--- a/svtools/source/misc/ehdl.cxx
+++ b/svtools/source/misc/ehdl.cxx
@@ -31,9 +31,9 @@
#include <memory>
-static sal_uInt16 aWndFunc(
+static ErrorHandlerFlags aWndFunc(
vcl::Window *pWin, // Parent of the dialog
- sal_uInt16 nFlags,
+ ErrorHandlerFlags nFlags,
const OUString &rErr, // error text
const OUString &rAction) // action text
@@ -52,34 +52,35 @@ static sal_uInt16 aWndFunc(
// determine necessary WinBits from the flags
WinBits eBits=0;
- if ( (ERRCODE_BUTTON_CANCEL|ERRCODE_BUTTON_RETRY) == (nFlags & (ERRCODE_BUTTON_CANCEL|ERRCODE_BUTTON_RETRY)) )
+ if ( nFlags & (ErrorHandlerFlags::ButtonsCancel | ErrorHandlerFlags::ButtonsRetry) )
eBits = WB_RETRY_CANCEL;
- else if ( ERRCODE_BUTTON_OK_CANCEL == (nFlags & ERRCODE_BUTTON_OK_CANCEL) )
+ else if ( nFlags & ErrorHandlerFlags::ButtonsOkCancel )
eBits = WB_OK_CANCEL;
- else if ( ERRCODE_BUTTON_OK == (nFlags & ERRCODE_BUTTON_OK) )
+ else if ( nFlags & ErrorHandlerFlags::ButtonsOk )
eBits = WB_OK;
- else if ( ERRCODE_BUTTON_YES_NO_CANCEL == (nFlags & ERRCODE_BUTTON_YES_NO_CANCEL) )
+ else if ( nFlags & ErrorHandlerFlags::ButtonsYesNoCancel )
eBits = WB_YES_NO_CANCEL;
- else if ( ERRCODE_BUTTON_YES_NO == (nFlags & ERRCODE_BUTTON_YES_NO) )
+ else if ( nFlags & ErrorHandlerFlags::ButtonsYesNo )
eBits = WB_YES_NO;
- switch(nFlags & 0x0f00)
+ switch(nFlags & ErrorHandlerFlags(0x0f00))
{
- case ERRCODE_BUTTON_DEF_OK:
+ case ErrorHandlerFlags::ButtonDefaultsOk:
eBits |= WB_DEF_OK;
break;
- case ERRCODE_BUTTON_DEF_CANCEL:
+ case ErrorHandlerFlags::ButtonDefaultsCancel:
eBits |= WB_DEF_CANCEL;
break;
- case ERRCODE_BUTTON_DEF_YES:
+ case ErrorHandlerFlags::ButtonDefaultsYes:
eBits |= WB_DEF_YES;
break;
- case ERRCODE_BUTTON_DEF_NO:
+ case ErrorHandlerFlags::ButtonDefaultsNo:
eBits |= WB_DEF_NO;
break;
+ default: break;
}
OUString aErr(SvtResId(STR_ERR_HDLMESS).toString());
@@ -90,48 +91,48 @@ static sal_uInt16 aWndFunc(
aErr = aErr.replaceAll("$(ERROR)", rErr);
VclPtr<MessBox> pBox;
- switch ( nFlags & 0xf000 )
+ switch ( nFlags & ErrorHandlerFlags(0xf000) )
{
- case ERRCODE_MSG_ERROR:
+ case ErrorHandlerFlags::MessageError:
pBox.reset(VclPtr<ErrorBox>::Create(pWin, eBits, aErr));
break;
- case ERRCODE_MSG_WARNING:
+ case ErrorHandlerFlags::MessageWarning:
pBox.reset(VclPtr<WarningBox>::Create(pWin, eBits, aErr));
break;
- case ERRCODE_MSG_INFO:
+ case ErrorHandlerFlags::MessageInfo:
pBox.reset(VclPtr<InfoBox>::Create(pWin, aErr));
break;
- case ERRCODE_MSG_QUERY:
+ case ErrorHandlerFlags::MessageQuery:
pBox.reset(VclPtr<QueryBox>::Create(pWin, eBits, aErr));
break;
default:
{
SAL_WARN( "svtools.misc", "no MessBox type");
- return ERRCODE_BUTTON_OK;
+ return ErrorHandlerFlags::ButtonsOk;
}
}
- sal_uInt16 nRet = RET_CANCEL;
+ ErrorHandlerFlags nRet = ErrorHandlerFlags::NONE;
switch ( pBox->Execute() )
{
case RET_OK:
- nRet = ERRCODE_BUTTON_OK;
+ nRet = ErrorHandlerFlags::ButtonsOk;
break;
case RET_CANCEL:
- nRet = ERRCODE_BUTTON_CANCEL;
+ nRet = ErrorHandlerFlags::ButtonsCancel;
break;
case RET_RETRY:
- nRet = ERRCODE_BUTTON_RETRY;
+ nRet = ErrorHandlerFlags::ButtonsRetry;
break;
case RET_YES:
- nRet = ERRCODE_BUTTON_YES;
+ nRet = ErrorHandlerFlags::ButtonsYes;
break;
case RET_NO:
- nRet = ERRCODE_BUTTON_NO;
+ nRet = ErrorHandlerFlags::ButtonsNo;
break;
default:
SAL_WARN( "svtools.misc", "Unknown MessBox return value" );
@@ -162,7 +163,7 @@ SfxErrorHandler::~SfxErrorHandler()
bool SfxErrorHandler::CreateString(
- const ErrorInfo *pErr, OUString &rStr, sal_uInt16& nFlags) const
+ const ErrorInfo *pErr, OUString &rStr, ErrorHandlerFlags& nFlags) const
/* [Description]
@@ -206,9 +207,9 @@ class ResString: public OUString
*/
{
- sal_uInt16 nFlags;
+ ErrorHandlerFlags nFlags;
public:
- sal_uInt16 GetFlags() const {return nFlags;}
+ ErrorHandlerFlags GetFlags() const {return nFlags;}
const OUString & GetString() const {return *this;}
explicit ResString( ResId &rId);
};
@@ -216,12 +217,12 @@ class ResString: public OUString
ResString::ResString(ResId & rId):
OUString(rId.SetAutoRelease(false).toString()),
- nFlags(0)
+ nFlags(ErrorHandlerFlags::NONE)
{
ResMgr * pResMgr = rId.GetResMgr();
// String ctor temporarily sets global ResManager
if (pResMgr->GetRemainSize())
- nFlags = sal_uInt16(pResMgr->ReadShort());
+ nFlags = ErrorHandlerFlags(pResMgr->ReadShort());
rId.SetAutoRelease(true);
pResMgr->PopContext();
}
@@ -273,7 +274,7 @@ void SfxErrorHandler::GetClassString(sal_uLong lClassId, OUString &rStr)
bool SfxErrorHandler::GetErrorString(
- sal_uLong lErrId, OUString &rStr, sal_uInt16 &nFlags) const
+ sal_uLong lErrId, OUString &rStr, ErrorHandlerFlags &nFlags) const
/* [Description]
@@ -295,8 +296,8 @@ bool SfxErrorHandler::GetErrorString(
{
ResString aErrorString(aEr.GetResString());
- sal_uInt16 nResFlags = aErrorString.GetFlags();
- if ( nResFlags )
+ ErrorHandlerFlags nResFlags = aErrorString.GetFlags();
+ if ( nResFlags != ErrorHandlerFlags::NONE )
nFlags = nResFlags;
rStr = rStr.replaceAll("$(ERROR)", aErrorString.GetString());
bRet = true;
diff --git a/svx/source/src/errtxt.src b/svx/source/src/errtxt.src
index 40ff546..af5e40e 100644
--- a/svx/source/src/errtxt.src
+++ b/svx/source/src/errtxt.src
@@ -338,11 +338,6 @@ Resource RID_ERRHDL
{
Text [ en-US ] = "The requested Internet data is not available in the cache and cannot be transmitted as the Online mode has not be activated." ;
};
- String ERRCODE_SFXMSG_STYLEREPLACE
- {
- ExtraData = ERRCODE_MSG_ERROR | ERRCODE_BUTTON_OK_CANCEL ;
- Text [ en-US ] = "Should the $(ARG1) Style be replaced?" ;
- };
String ERRCODE_SFX_NOFILTER
{
Text [ en-US ] = "A filter has not been found." ;
diff --git a/sw/source/core/frmedt/fetab.cxx b/sw/source/core/frmedt/fetab.cxx
index 86c4fd7..c795569 100644
--- a/sw/source/core/frmedt/fetab.cxx
+++ b/sw/source/core/frmedt/fetab.cxx
@@ -189,7 +189,7 @@ bool SwFEShell::InsertRow( sal_uInt16 nCnt, bool bBehind )
if( dynamic_cast< const SwDDETable* >(pFrame->ImplFindTabFrame()->GetTable()) != nullptr )
{
ErrorHandler::HandleError( ERR_TBLDDECHG_ERROR,
- ERRCODE_MSG_INFO | ERRCODE_BUTTON_DEF_OK );
+ ErrorHandlerFlags::MessageInfo | ErrorHandlerFlags::ButtonDefaultsOk );
return false;
}
@@ -230,7 +230,7 @@ bool SwFEShell::InsertCol( sal_uInt16 nCnt, bool bBehind )
if( dynamic_cast< const SwDDETable* >(pFrame->ImplFindTabFrame()->GetTable()) != nullptr )
{
ErrorHandler::HandleError( ERR_TBLDDECHG_ERROR,
- ERRCODE_MSG_INFO | ERRCODE_BUTTON_DEF_OK );
+ ErrorHandlerFlags::MessageInfo | ErrorHandlerFlags::ButtonDefaultsOk );
return false;
}
@@ -239,7 +239,7 @@ bool SwFEShell::InsertCol( sal_uInt16 nCnt, bool bBehind )
if( !CheckSplitCells( *this, nCnt + 1, nsSwTableSearchType::TBLSEARCH_COL ) )
{
ErrorHandler::HandleError( ERR_TBLINSCOL_ERROR,
- ERRCODE_MSG_INFO | ERRCODE_BUTTON_DEF_OK );
+ ErrorHandlerFlags::MessageInfo | ErrorHandlerFlags::ButtonDefaultsOk );
return false;
}
@@ -285,7 +285,7 @@ bool SwFEShell::DeleteCol()
if( dynamic_cast< const SwDDETable* >(pFrame->ImplFindTabFrame()->GetTable()) != nullptr )
{
ErrorHandler::HandleError( ERR_TBLDDECHG_ERROR,
- ERRCODE_MSG_INFO | ERRCODE_BUTTON_DEF_OK );
+ ErrorHandlerFlags::MessageInfo | ErrorHandlerFlags::ButtonDefaultsOk );
return false;
}
@@ -336,7 +336,7 @@ bool SwFEShell::DeleteRow(bool bCompleteTable)
if( dynamic_cast< const SwDDETable* >(pFrame->ImplFindTabFrame()->GetTable()) != nullptr )
{
ErrorHandler::HandleError( ERR_TBLDDECHG_ERROR,
- ERRCODE_MSG_INFO | ERRCODE_BUTTON_DEF_OK );
+ ErrorHandlerFlags::MessageInfo | ErrorHandlerFlags::ButtonDefaultsOk );
return false;
}
@@ -455,7 +455,7 @@ sal_uInt16 SwFEShell::MergeTab()
if( dynamic_cast< const SwDDETable* >(&pTableNd->GetTable()) != nullptr )
{
ErrorHandler::HandleError( ERR_TBLDDECHG_ERROR,
- ERRCODE_MSG_INFO | ERRCODE_BUTTON_DEF_OK );
+ ErrorHandlerFlags::MessageInfo | ErrorHandlerFlags::ButtonDefaultsOk );
}
else
{
@@ -486,7 +486,7 @@ bool SwFEShell::SplitTab( bool bVert, sal_uInt16 nCnt, bool bSameHeight )
if( dynamic_cast< const SwDDETable* >(pFrame->ImplFindTabFrame()->GetTable()) != nullptr )
{
ErrorHandler::HandleError( ERR_TBLDDECHG_ERROR,
- ERRCODE_MSG_INFO | ERRCODE_BUTTON_DEF_OK );
+ ErrorHandlerFlags::MessageInfo | ErrorHandlerFlags::ButtonDefaultsOk );
return false;
}
@@ -495,7 +495,7 @@ bool SwFEShell::SplitTab( bool bVert, sal_uInt16 nCnt, bool bSameHeight )
if( bVert && !CheckSplitCells( *this, nCnt + 1 ) )
{
ErrorHandler::HandleError( ERR_TBLSPLIT_ERROR,
- ERRCODE_MSG_INFO | ERRCODE_BUTTON_DEF_OK );
+ ErrorHandlerFlags::MessageInfo | ErrorHandlerFlags::ButtonDefaultsOk );
return false;
}
StartAllAction();
@@ -1295,7 +1295,7 @@ bool SwFEShell::DeleteTableSel()
if( dynamic_cast< const SwDDETable* >(pFrame->ImplFindTabFrame()->GetTable()) != nullptr )
{
ErrorHandler::HandleError( ERR_TBLDDECHG_ERROR,
- ERRCODE_MSG_INFO | ERRCODE_BUTTON_DEF_OK );
+ ErrorHandlerFlags::MessageInfo | ErrorHandlerFlags::ButtonDefaultsOk );
return false;
}
@@ -2162,7 +2162,7 @@ bool SwFEShell::SetColRowWidthHeight( sal_uInt16 eType, sal_uInt16 nDiff )
dynamic_cast< const SwDDETable* >(pFrame->ImplFindTabFrame()->GetTable()) != nullptr )
{
ErrorHandler::HandleError( ERR_TBLDDECHG_ERROR,
- ERRCODE_MSG_INFO | ERRCODE_BUTTON_DEF_OK );
+ ErrorHandlerFlags::MessageInfo | ErrorHandlerFlags::ButtonDefaultsOk );
return false;
}
diff --git a/sw/source/filter/html/swhtml.cxx b/sw/source/filter/html/swhtml.cxx
index 830a9b0..3df726b 100644
--- a/sw/source/filter/html/swhtml.cxx
+++ b/sw/source/filter/html/swhtml.cxx
@@ -227,7 +227,7 @@ sal_uLong HTMLReader::Read( SwDoc &rDoc, const OUString& rBaseURL, SwPaM &rPam,
// den Stream als Fehlernummer Transporter benutzen
nRet = (new StringErrorInfo( ERR_FORMAT_ROWCOL, sErr,
- ERRCODE_BUTTON_OK | ERRCODE_MSG_ERROR ))->GetErrorCode();
+ ErrorHandlerFlags::ButtonsOk | ErrorHandlerFlags::MessageError ))->GetErrorCode();
}
return nRet;
diff --git a/sw/source/filter/xml/swxml.cxx b/sw/source/filter/xml/swxml.cxx
index 374029e..675599e 100644
--- a/sw/source/filter/xml/swxml.cxx
+++ b/sw/source/filter/xml/swxml.cxx
@@ -210,13 +210,13 @@ sal_Int32 ReadThroughComponent(
(bMustBeSuccessfull ? ERR_FORMAT_FILE_ROWCOL
: WARN_FORMAT_FILE_ROWCOL),
rStreamName, sErr,
- ERRCODE_BUTTON_OK | ERRCODE_MSG_ERROR ))->GetErrorCode();
+ ErrorHandlerFlags::ButtonsOk | ErrorHandlerFlags::MessageError ))->GetErrorCode();
}
else
{
OSL_ENSURE( bMustBeSuccessfull, "Warnings are not supported" );
return (new StringErrorInfo( ERR_FORMAT_ROWCOL, sErr,
- ERRCODE_BUTTON_OK | ERRCODE_MSG_ERROR ))->GetErrorCode();
+ ErrorHandlerFlags::ButtonsOk | ErrorHandlerFlags::MessageError ))->GetErrorCode();
}
}
catch(const xml::sax::SAXException& r)
diff --git a/sw/source/filter/xml/wrtxml.cxx b/sw/source/filter/xml/wrtxml.cxx
index a2494ef..92081a4 100644
--- a/sw/source/filter/xml/wrtxml.cxx
+++ b/sw/source/filter/xml/wrtxml.cxx
@@ -414,14 +414,14 @@ sal_uInt32 SwXMLWriter::Write_( const uno::Reference < task::XStatusIndicator >&
{
if( !sErrFile.isEmpty() )
return (new StringErrorInfo( ERR_WRITE_ERROR_FILE, sErrFile,
- ERRCODE_BUTTON_OK | ERRCODE_MSG_ERROR ))->GetErrorCode();
+ ErrorHandlerFlags::ButtonsOk | ErrorHandlerFlags::MessageError ))->GetErrorCode();
return ERR_SWG_WRITE_ERROR;
}
else if( bWarn )
{
if( !sWarnFile.isEmpty() )
return (new StringErrorInfo( WARN_WRITE_ERROR_FILE, sWarnFile,
- ERRCODE_BUTTON_OK | ERRCODE_MSG_ERROR ))->GetErrorCode();
+ ErrorHandlerFlags::ButtonsOk | ErrorHandlerFlags::MessageError ))->GetErrorCode();
return WARN_SWG_FEATURES_LOST;
}
diff --git a/sw/source/uibase/misc/glosdoc.cxx b/sw/source/uibase/misc/glosdoc.cxx
index 513aba7..25210bc 100644
--- a/sw/source/uibase/misc/glosdoc.cxx
+++ b/sw/source/uibase/misc/glosdoc.cxx
@@ -400,7 +400,7 @@ void SwGlossaries::UpdateGlosPath(bool bFull)
ErrorHandler::HandleError( (new StringErrorInfo(
ERR_AUTOPATH_ERROR, lcl_makePath(m_aInvalidPaths),
- ERRCODE_BUTTON_OK | ERRCODE_MSG_ERROR ))->GetErrorCode());
+ ErrorHandlerFlags::ButtonsOk | ErrorHandlerFlags::MessageError ))->GetErrorCode());
m_bError = true;
}
else
@@ -420,7 +420,7 @@ void SwGlossaries::UpdateGlosPath(bool bFull)
void SwGlossaries::ShowError()
{
sal_uInt32 nPathError = (new StringErrorInfo(ERR_AUTOPATH_ERROR,
- lcl_makePath(m_aInvalidPaths), ERRCODE_BUTTON_OK ))->GetErrorCode();
+ lcl_makePath(m_aInvalidPaths), ErrorHandlerFlags::ButtonsOk ))->GetErrorCode();
ErrorHandler::HandleError( nPathError );
}
diff --git a/tools/source/ref/errinf.cxx b/tools/source/ref/errinf.cxx
index 539151c..754a888 100644
--- a/tools/source/ref/errinf.cxx
+++ b/tools/source/ref/errinf.cxx
@@ -47,8 +47,8 @@ struct TheEDcrData: public rtl::Static<EDcrData, TheEDcrData> {};
class DynamicErrorInfo_Impl
{
- sal_uIntPtr lErrId;
- sal_uInt16 nMask;
+ ErrCode lErrId;
+ ErrorHandlerFlags nMask;
void RegisterEDcr(DynamicErrorInfo *);
static void UnRegisterEDcr(DynamicErrorInfo const *);
@@ -110,7 +110,7 @@ sal_uIntPtr DynamicErrorInfo::GetErrorCode() const
return pImpl->lErrId;
}
-DynamicErrorInfo::DynamicErrorInfo(sal_uIntPtr lArgUserId, sal_uInt16 nMask)
+DynamicErrorInfo::DynamicErrorInfo(sal_uIntPtr lArgUserId, ErrorHandlerFlags nMask)
: ErrorInfo(lArgUserId),
pImpl(new DynamicErrorInfo_Impl)
{
@@ -133,13 +133,13 @@ ErrorInfo* DynamicErrorInfo_Impl::GetDynamicErrorInfo(sal_uIntPtr lId)
return new ErrorInfo(lId & ~ERRCODE_DYNAMIC_MASK);
}
-sal_uInt16 DynamicErrorInfo::GetDialogMask() const
+ErrorHandlerFlags DynamicErrorInfo::GetDialogMask() const
{
return pImpl->nMask;
}
StringErrorInfo::StringErrorInfo(
- sal_uIntPtr UserId, const OUString& aStringP, sal_uInt16 nMask)
+ sal_uIntPtr UserId, const OUString& aStringP, ErrorHandlerFlags nMask)
: DynamicErrorInfo(UserId, nMask), aString(aStringP)
{
}
@@ -147,7 +147,7 @@ StringErrorInfo::StringErrorInfo(
class ErrorHandler_Impl
{
public:
- static bool CreateString(const ErrorInfo*, OUString&, sal_uInt16&);
+ static bool CreateString(const ErrorInfo*, OUString&, ErrorHandlerFlags&);
};
static void aDspFunc(const OUString &rErr, const OUString &rAction)
@@ -226,25 +226,25 @@ void ErrorHandler::RegisterDisplay(BasicDisplayErrorFunc *aDsp)
1. nFlags,
2. Resource Flags
3. Dynamic Flags
- 4. Default ERRCODE_BUTTON_OK, ERRCODE_MSG_ERROR
+ 4. Default ButtonsOk, MessageError
- @param lId error id
+ @param nErrCodeId error id
@param nFlags error flags.
@param bJustCreateString ???
@param rError ???
@return ???
*/
-sal_uInt16 ErrorHandler::HandleError_Impl(
- sal_uIntPtr lId, sal_uInt16 nFlags, bool bJustCreateString, OUString & rError)
+ErrorHandlerFlags ErrorHandler::HandleError_Impl(
+ sal_uIntPtr nErrCodeId, ErrorHandlerFlags nFlags, bool bJustCreateString, OUString & rError)
{
OUString aErr;
OUString aAction;
- if(!lId || lId == ERRCODE_ABORT)
- return 0;
+ if(!nErrCodeId || nErrCodeId == ERRCODE_ABORT)
+ return ErrorHandlerFlags::NONE;
EDcrData &rData = TheEDcrData::get();
vcl::Window *pParent = nullptr;
- ErrorInfo *pInfo = ErrorInfo::GetErrorInfo(lId);
+ ErrorInfo *pInfo = ErrorInfo::GetErrorInfo(nErrCodeId);
if (!rData.contexts.empty())
{
rData.contexts.front()->GetString(pInfo->GetErrorCode(), aAction);
@@ -257,18 +257,18 @@ sal_uInt16 ErrorHandler::HandleError_Impl(
}
}
- bool bWarning = ((lId & ERRCODE_WARNING_MASK) == ERRCODE_WARNING_MASK);
- sal_uInt16 nErrFlags = ERRCODE_BUTTON_DEF_OK | ERRCODE_BUTTON_OK;
+ bool bWarning = ((nErrCodeId & ERRCODE_WARNING_MASK) == ERRCODE_WARNING_MASK);
+ ErrorHandlerFlags nErrFlags = ErrorHandlerFlags::ButtonDefaultsOk | ErrorHandlerFlags::ButtonsOk;
if (bWarning)
- nErrFlags |= ERRCODE_MSG_WARNING;
+ nErrFlags |= ErrorHandlerFlags::MessageWarning;
else
- nErrFlags |= ERRCODE_MSG_ERROR;
+ nErrFlags |= ErrorHandlerFlags::MessageError;
DynamicErrorInfo* pDynPtr=dynamic_cast<DynamicErrorInfo*>(pInfo);
if(pDynPtr)
{
- sal_uInt16 nDynFlags = pDynPtr->GetDialogMask();
- if( nDynFlags )
+ ErrorHandlerFlags nDynFlags = pDynPtr->GetDialogMask();
+ if( nDynFlags != ErrorHandlerFlags::NONE )
nErrFlags = nDynFlags;
}
@@ -277,7 +277,7 @@ sal_uInt16 ErrorHandler::HandleError_Impl(
if (bJustCreateString)
{
rError = aErr;
- return 1;
+ return ErrorHandlerFlags::ButtonsOk;
}
else
{
@@ -295,11 +295,11 @@ sal_uInt16 ErrorHandler::HandleError_Impl(
if(!rData.bIsWindowDsp)
{
(*reinterpret_cast<BasicDisplayErrorFunc*>(rData.pDsp))(aErr,aAction);
- return 0;
+ return ErrorHandlerFlags::NONE;
}
else
{
- if (nFlags != USHRT_MAX)
+ if (nFlags != ErrorHandlerFlags::MAX)
nErrFlags = nFlags;
return (*reinterpret_cast<WindowDisplayErrorFunc*>(rData.pDsp))(
pParent, nErrFlags, aErr, aAction);
@@ -311,34 +311,34 @@ sal_uInt16 ErrorHandler::HandleError_Impl(
// Error 1 is General Error in the Sfx
if(pInfo->GetErrorCode()!=1)
{
- HandleError_Impl(1, USHRT_MAX, bJustCreateString, rError);
+ HandleError_Impl(1, ErrorHandlerFlags::MAX, bJustCreateString, rError);
}
else
{
OSL_FAIL("Error 1 nicht gehandeled");
}
delete pInfo;
- return 0;
+ return ErrorHandlerFlags::NONE;
}
// static
bool ErrorHandler::GetErrorString(sal_uIntPtr lId, OUString& rStr)
{
- return (bool)HandleError_Impl( lId, USHRT_MAX, true, rStr );
+ return HandleError_Impl( lId, ErrorHandlerFlags::MAX, true, rStr ) != ErrorHandlerFlags::NONE;
}
/** Handles an error.
@see ErrorHandler::HandleError_Impl
*/
-sal_uInt16 ErrorHandler::HandleError(sal_uIntPtr lId, sal_uInt16 nFlags)
+ErrorHandlerFlags ErrorHandler::HandleError(sal_uIntPtr lId, ErrorHandlerFlags nFlags)
{
OUString aDummy;
return HandleError_Impl( lId, nFlags, false, aDummy );
}
bool ErrorHandler_Impl::CreateString( const ErrorInfo* pInfo, OUString& pStr,
- sal_uInt16 &rFlags)
+ ErrorHandlerFlags &rFlags)
{
for(const ErrorHandler *pHdl : TheEDcrData::get().errorHandlers)
{
diff --git a/uui/source/iahndl-authentication.cxx b/uui/source/iahndl-authentication.cxx
index 9e07550..3701f73 100644
--- a/uui/source/iahndl-authentication.cxx
+++ b/uui/source/iahndl-authentication.cxx
@@ -113,8 +113,8 @@ executeLoginDialog(
if ( bCanUseSysCreds )
xDialog->SetUseSystemCredentials( rInfo.GetIsUseSystemCredentials() );
- rInfo.SetResult(xDialog->Execute() == RET_OK ? ERRCODE_BUTTON_OK :
- ERRCODE_BUTTON_CANCEL);
+ rInfo.SetResult(xDialog->Execute() == RET_OK ? ErrorHandlerFlags::ButtonsOk :
+ ErrorHandlerFlags::ButtonsCancel);
rInfo.SetUserName(xDialog->GetName());
rInfo.SetPassword(xDialog->GetPassword());
rInfo.SetAccount(xDialog->GetAccount());
@@ -276,7 +276,7 @@ handleAuthenticationRequest_(
rRequest.HasRealm ? rRequest.Realm : OUString());
switch (aInfo.GetResult())
{
- case ERRCODE_BUTTON_OK:
+ case ErrorHandlerFlags::ButtonsOk:
if (xSupplyAuthentication.is())
{
if (xSupplyAuthentication->canSetUserName())
@@ -390,7 +390,7 @@ handleAuthenticationRequest_(
}
break;
- case ERRCODE_BUTTON_RETRY:
+ case ErrorHandlerFlags::ButtonsRetry:
if (xRetry.is())
xRetry->select();
break;
@@ -419,7 +419,7 @@ executeMasterPasswordDialog(
ScopedVclPtrInstance< MasterPasswordCreateDialog > xDialog(
pParent, xManager.get());
rInfo.SetResult(xDialog->Execute()
- == RET_OK ? ERRCODE_BUTTON_OK : ERRCODE_BUTTON_CANCEL);
+ == RET_OK ? ErrorHandlerFlags::ButtonsOk : ErrorHandlerFlags::ButtonsCancel);
aMaster = OUStringToOString(
xDialog->GetMasterPassword(), RTL_TEXTENCODING_UTF8);
}
@@ -428,7 +428,7 @@ executeMasterPasswordDialog(
ScopedVclPtrInstance< MasterPasswordDialog > xDialog(
pParent, nMode, xManager.get());
rInfo.SetResult(xDialog->Execute()
- == RET_OK ? ERRCODE_BUTTON_OK : ERRCODE_BUTTON_CANCEL);
+ == RET_OK ? ErrorHandlerFlags::ButtonsOk : ErrorHandlerFlags::ButtonsCancel);
aMaster = OUStringToOString(
xDialog->GetMasterPassword(), RTL_TEXTENCODING_UTF8);
}
@@ -476,7 +476,7 @@ handleMasterPasswordRequest_(
switch (aInfo.GetResult())
{
- case ERRCODE_BUTTON_OK:
+ case ErrorHandlerFlags::ButtonsOk:
if (xSupplyAuthentication.is())
{
if (xSupplyAuthentication->canSetPassword())
@@ -485,7 +485,7 @@ handleMasterPasswordRequest_(
}
break;
- case ERRCODE_BUTTON_RETRY:
+ case ErrorHandlerFlags::ButtonsRetry:
if (xRetry.is())
xRetry->select();
break;
@@ -520,7 +520,7 @@ executePasswordDialog(
bIsPasswordToModify, bIsSimplePasswordRequest);
xDialog->SetMinLen(0);
- rInfo.SetResult(xDialog->Execute() == RET_OK ? ERRCODE_BUTTON_OK : ERRCODE_BUTTON_CANCEL);
+ rInfo.SetResult(xDialog->Execute() == RET_OK ? ErrorHandlerFlags::ButtonsOk : ErrorHandlerFlags::ButtonsCancel);
rInfo.SetPassword(xDialog->GetPassword());
}
else
@@ -531,7 +531,7 @@ executePasswordDialog(
ScopedVclPtr<AbstractPasswordToOpenModifyDialog> const pDialog(
pFact->CreatePasswordToOpenModifyDialog(pParent, nMaxPasswdLen, bIsPasswordToModify));
- rInfo.SetResult( pDialog->Execute() == RET_OK ? ERRCODE_BUTTON_OK : ERRCODE_BUTTON_CANCEL );
+ rInfo.SetResult( pDialog->Execute() == RET_OK ? ErrorHandlerFlags::ButtonsOk : ErrorHandlerFlags::ButtonsCancel );
rInfo.SetPassword( pDialog->GetPasswordToOpen() );
rInfo.SetPasswordToModify( pDialog->GetPasswordToModify() );
rInfo.SetRecommendToOpenReadonly( pDialog->IsRecommendToOpenReadonly() );
@@ -543,7 +543,7 @@ executePasswordDialog(
bIsPasswordToModify, bIsSimplePasswordRequest);
xDialog->SetMinLen(0);
- rInfo.SetResult(xDialog->Execute() == RET_OK ? ERRCODE_BUTTON_OK : ERRCODE_BUTTON_CANCEL);
+ rInfo.SetResult(xDialog->Execute() == RET_OK ? ErrorHandlerFlags::ButtonsOk : ErrorHandlerFlags::ButtonsCancel);
rInfo.SetPassword(bIsPasswordToModify ? OUString() : xDialog->GetPassword());
rInfo.SetPasswordToModify(bIsPasswordToModify ? xDialog->GetPassword() : OUString());
}
@@ -582,7 +582,7 @@ handlePasswordRequest_(
switch (aInfo.GetResult())
{
- case ERRCODE_BUTTON_OK:
+ case ErrorHandlerFlags::ButtonsOk:
OSL_ENSURE( !bIsPasswordToModify || xPassword2.is(), "PasswordToModify is requested, but there is no Interaction!" );
if (xPassword.is())
{
@@ -597,7 +597,7 @@ handlePasswordRequest_(
}
break;
- case ERRCODE_BUTTON_RETRY:
+ case ErrorHandlerFlags::ButtonsRetry:
if (xRetry.is())
xRetry->select();
break;
diff --git a/uui/source/iahndl-errorhandler.cxx b/uui/source/iahndl-errorhandler.cxx
index 167db08..8be4cbb 100644
--- a/uui/source/iahndl-errorhandler.cxx
+++ b/uui/source/iahndl-errorhandler.cxx
@@ -39,7 +39,7 @@ using namespace com::sun::star;
namespace {
-sal_uInt16
+ErrorHandlerFlags
executeErrorDialog(
vcl::Window * pParent,
task::InteractionClassification eClassification,
@@ -100,27 +100,29 @@ executeErrorDialog(
throw uno::RuntimeException("out of memory");
}
- sal_uInt16 aResult = xBox->Execute();
+ sal_uInt16 aMessResult = xBox->Execute();
xBox.disposeAndClear();
- switch( aResult )
+ ErrorHandlerFlags aResult = ErrorHandlerFlags::NONE;
+ switch( aMessResult )
{
case RET_OK:
- aResult = ERRCODE_BUTTON_OK;
+ aResult = ErrorHandlerFlags::ButtonsOk;
break;
case RET_CANCEL:
- aResult = ERRCODE_BUTTON_CANCEL;
+ aResult = ErrorHandlerFlags::ButtonsCancel;
break;
case RET_YES:
- aResult = ERRCODE_BUTTON_YES;
+ aResult = ErrorHandlerFlags::ButtonsYes;
break;
case RET_NO:
- aResult = ERRCODE_BUTTON_NO;
+ aResult = ErrorHandlerFlags::ButtonsNo;
break;
case RET_RETRY:
- aResult = ERRCODE_BUTTON_RETRY;
+ aResult = ErrorHandlerFlags::ButtonsRetry;
break;
+ default: assert(false);
}
return aResult;
@@ -262,12 +264,12 @@ UUIInteractionHelper::handleErrorHandlerRequest(
}
}
- sal_uInt16 nResult = executeErrorDialog(
+ ErrorHandlerFlags nResult = executeErrorDialog(
getParentProperty(), eClassification, aContext, aMessage, nButtonMask );
switch (nResult)
{
- case ERRCODE_BUTTON_OK:
+ case ErrorHandlerFlags::ButtonsOk:
OSL_ENSURE(xApprove.is() || xAbort.is(), "unexpected situation");
if (xApprove.is())
xApprove->select();
@@ -275,29 +277,31 @@ UUIInteractionHelper::handleErrorHandlerRequest(
xAbort->select();
break;
- case ERRCODE_BUTTON_CANCEL:
+ case ErrorHandlerFlags::ButtonsCancel:
OSL_ENSURE(xAbort.is(), "unexpected situation");
if (xAbort.is())
xAbort->select();
break;
- case ERRCODE_BUTTON_RETRY:
+ case ErrorHandlerFlags::ButtonsRetry:
OSL_ENSURE(xRetry.is(), "unexpected situation");
if (xRetry.is())
xRetry->select();
break;
- case ERRCODE_BUTTON_NO:
+ case ErrorHandlerFlags::ButtonsNo:
OSL_ENSURE(xDisapprove.is(), "unexpected situation");
if (xDisapprove.is())
xDisapprove->select();
break;
- case ERRCODE_BUTTON_YES:
+ case ErrorHandlerFlags::ButtonsYes:
OSL_ENSURE(xApprove.is(), "unexpected situation");
if (xApprove.is())
xApprove->select();
break;
+
+ default: break;
}
}
diff --git a/uui/source/iahndl.cxx b/uui/source/iahndl.cxx
index 31074fb..f2915b1 100644
--- a/uui/source/iahndl.cxx
+++ b/uui/source/iahndl.cxx
@@ -987,7 +987,7 @@ UUIInteractionHelper::getInteractionHandler()
namespace {
-sal_uInt16
+ErrorHandlerFlags
executeMessageBox(
vcl::Window * pParent,
OUString const & rTitle,
@@ -998,24 +998,26 @@ executeMessageBox(
ScopedVclPtrInstance< MessBox > xBox(pParent, nButtonMask, rTitle, rMessage);
- sal_uInt16 aResult = xBox->Execute();
- switch( aResult )
+ sal_uInt16 aMessResult = xBox->Execute();
+ ErrorHandlerFlags aResult = ErrorHandlerFlags::NONE;
+ switch( aMessResult )
{
case RET_OK:
- aResult = ERRCODE_BUTTON_OK;
+ aResult = ErrorHandlerFlags::ButtonsOk;
break;
case RET_CANCEL:
- aResult = ERRCODE_BUTTON_CANCEL;
+ aResult = ErrorHandlerFlags::ButtonsCancel;
break;
case RET_YES:
- aResult = ERRCODE_BUTTON_YES;
+ aResult = ErrorHandlerFlags::ButtonsYes;
break;
case RET_NO:
- aResult = ERRCODE_BUTTON_NO;
+ aResult = ErrorHandlerFlags::ButtonsNo;
break;
case RET_RETRY:
- aResult = ERRCODE_BUTTON_RETRY;
+ aResult = ErrorHandlerFlags::ButtonsRetry;
break;
+ default: assert(false);
}
return aResult;
@@ -1270,23 +1272,25 @@ UUIInteractionHelper::handleBrokenPackageRequest(
switch (
executeMessageBox( getParentProperty(), title, aMessage, nButtonMask ) )
{
- case ERRCODE_BUTTON_OK:
+ case ErrorHandlerFlags::ButtonsOk:
OSL_ENSURE( xAbort.is(), "unexpected situation" );
if (xAbort.is())
xAbort->select();
break;
- case ERRCODE_BUTTON_NO:
+ case ErrorHandlerFlags::ButtonsNo:
OSL_ENSURE(xDisapprove.is(), "unexpected situation");
if (xDisapprove.is())
xDisapprove->select();
break;
- case ERRCODE_BUTTON_YES:
+ case ErrorHandlerFlags::ButtonsYes:
OSL_ENSURE(xApprove.is(), "unexpected situation");
if (xApprove.is())
xApprove->select();
break;
+
+ default: break;
}
}
diff --git a/uui/source/loginerr.hxx b/uui/source/loginerr.hxx
index db4ce88..ae0775a 100644
--- a/uui/source/loginerr.hxx
+++ b/uui/source/loginerr.hxx
@@ -43,13 +43,13 @@ private:
OUString m_aPath;
OUString m_aErrorText;
sal_uInt8 m_nFlags;
- sal_uInt16 m_nRet;
+ ErrorHandlerFlags m_nRet;
bool m_bRecommendToOpenReadonly;
public:
LoginErrorInfo()
: m_nFlags(LOGINERROR_FLAG_MODIFY_USER_NAME)
- , m_nRet(ERRCODE_BUTTON_CANCEL)
+ , m_nRet(ErrorHandlerFlags::ButtonsCancel)
, m_bRecommendToOpenReadonly(false)
{
}
@@ -71,8 +71,8 @@ public:
bool GetIsUseSystemCredentials() const
{ return ( m_nFlags & LOGINERROR_FLAG_IS_USE_SYSCREDS ) ==
LOGINERROR_FLAG_IS_USE_SYSCREDS; }
- sal_uInt8 GetFlags() const { return m_nFlags; }
- sal_uInt16 GetResult() const { return m_nRet; }
+ sal_uInt8 GetFlags() const { return m_nFlags; }
+ ErrorHandlerFlags GetResult() const { return m_nRet; }
void SetTitle( const OUString& aTitle )
{ m_aTitle = aTitle; }
@@ -100,7 +100,7 @@ public:
inline void SetModifyAccount( bool bSet );
inline void SetModifyUserName( bool bSet );
- void SetResult( sal_uInt16 nRet )
+ void SetResult( ErrorHandlerFlags nRet )
{ m_nRet = nRet; }
};
More information about the Libreoffice-commits
mailing list