diff options
| author | Katarina Behrens <Katarina.Behrens@cib.de> | 2017-06-19 15:08:20 +0200 |
|---|---|---|
| committer | Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de> | 2017-06-23 09:15:30 +0200 |
| commit | 7be0d5490517d41c20f99a006edb9fa651ce3d85 (patch) | |
| tree | 405c4a280e7ae3ea5d711871b954921a77fa931f /xmlsecurity/source/dialogs/digitalsignaturesdialog.cxx | |
| parent | 72f84c1a0efbce8da02cbf6d5947d2ddfd45c533 (diff) | |
gpg4libre: Warn before removing document signatures
simple version, no "do not show this message again" checkbox
Change-Id: Iba8ac8cda51acac748174c864aa5c205f2efcc8f
Reviewed-on: https://gerrit.libreoffice.org/38962
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de>
Diffstat (limited to 'xmlsecurity/source/dialogs/digitalsignaturesdialog.cxx')
| -rw-r--r-- | xmlsecurity/source/dialogs/digitalsignaturesdialog.cxx | 13 |
1 files changed, 12 insertions, 1 deletions
diff --git a/xmlsecurity/source/dialogs/digitalsignaturesdialog.cxx b/xmlsecurity/source/dialogs/digitalsignaturesdialog.cxx index f52b65dc9917..5f4464b8b957 100644 --- a/xmlsecurity/source/dialogs/digitalsignaturesdialog.cxx +++ b/xmlsecurity/source/dialogs/digitalsignaturesdialog.cxx @@ -263,6 +263,8 @@ void DigitalSignaturesDialog::SetSignatureStream( const css::uno::Reference < cs bool DigitalSignaturesDialog::canAddRemove() { + //FIXME: this func needs some cleanup, such as real split between + //'canAdd' and 'canRemove' case bool ret = true; if (!maSignatureManager.mxStore.is()) @@ -317,7 +319,16 @@ bool DigitalSignaturesDialog::canAdd() bool DigitalSignaturesDialog::canRemove() { - return canAddRemove(); + bool bRet = true; + + if ( maSignatureManager.meSignatureMode == DocumentSignatureMode::Content ) + { + short nDlgRet = ScopedVclPtrInstance<MessageDialog>( + nullptr, XsResId(STR_XMLSECDLG_QUERY_REALLYREMOVE), VclMessageType::Question, VclButtonsType::YesNo)->Execute(); + bRet = ( nDlgRet == RET_YES ); + } + + return (bRet && canAddRemove()); } short DigitalSignaturesDialog::Execute() |