sfx2: initial PDF document signatures dialog

This just allows the menu entry in Draw to reach
SfxMedium::SignContents_Impl(), and then call signDocumentContent(),
which shows the dialog (where the signature itself isn't visible yet).

Do this by not assuming a ZIP-based storage at various places when we're
not working with an "own" format.

Change-Id: I030d0f636d1aab98369024f3f10a975ea2786d59
Reviewed-on: https://gerrit.libreoffice.org/29774
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk>

3 files changed, 26 insertions, 5 deletions

diff --git a/sfx2/source/doc/docfile.cxx b/sfx2/source/doc/docfile.cxx
index cae3a9e89623..b8efb549e253 100644
--- a/sfx2/source/doc/docfile.cxx
+++ b/sfx2/source/doc/docfile.cxx
@@ -3519,8 +3519,18 @@ bool SfxMedium::SignContents_Impl( bool bScriptingContent, const OUString& aODFV
                 if ( !pImpl->xStream.is() )
                     throw uno::RuntimeException();
 
-                xWriteableZipStor = ::comphelper::OStorageHelper::GetStorageOfFormatFromStream( ZIP_STORAGE_FORMAT_STRING, pImpl->xStream );
-                if ( !xWriteableZipStor.is() )
+                bool bODF = GetFilter()->IsOwnFormat();
+                try
+                {
+                    xWriteableZipStor = ::comphelper::OStorageHelper::GetStorageOfFormatFromStream( ZIP_STORAGE_FORMAT_STRING, pImpl->xStream );
+                }
+                catch (const io::IOException& rException)
+                {
+                    if (bODF)
+                        SAL_WARN("sfx.doc", "ODF stream is not a zip storage: " << rException.Message);
+                }
+
+                if ( !xWriteableZipStor.is() && bODF )
                     throw uno::RuntimeException();
 
                 uno::Reference< embed::XStorage > xMetaInf;
@@ -3581,7 +3591,7 @@ bool SfxMedium::SignContents_Impl( bool bScriptingContent, const OUString& aODFV
                             bChanges = true;
                         }
                     }
-                    else
+                    else if (xWriteableZipStor.is())
                     {
                         // OOXML.
                         uno::Reference<io::XStream> xStream;
@@ -3596,6 +3606,14 @@ bool SfxMedium::SignContents_Impl( bool bScriptingContent, const OUString& aODFV
                             bChanges = true;
                         }
                     }
+                    else
+                    {
+                        // Something not based: e.g. PDF.
+                        SvStream* pStream = utl::UcbStreamHelper::CreateStream(GetName(), StreamMode::READ);
+                        uno::Reference<io::XStream> xStream(new utl::OStreamWrapper(*pStream));
+                        if (xSigner->signDocumentContent(uno::Reference<embed::XStorage>(), xStream))
+                            bChanges = true;
+                    }
                 }
             }
             catch ( const uno::Exception& )
diff --git a/sfx2/source/doc/objserv.cxx b/sfx2/source/doc/objserv.cxx
index f6243ccf5cf4..9e1d8ef978af 100644
--- a/sfx2/source/doc/objserv.cxx
+++ b/sfx2/source/doc/objserv.cxx
@@ -1371,7 +1371,7 @@ void SfxObjectShell::ImplSign( bool bScriptingContent )
         &&  GetMedium()->GetFilter()
         &&  !GetMedium()->GetName().isEmpty()
         &&  (   (!GetMedium()->GetFilter()->IsOwnFormat() && !GetMedium()->GetFilter()->GetSupportsSigning())
-            ||  !GetMedium()->HasStorage_Impl()
+            ||  (GetMedium()->GetFilter()->IsOwnFormat() && !GetMedium()->HasStorage_Impl())
             )
         )
     {
@@ -1382,7 +1382,8 @@ void SfxObjectShell::ImplSign( bool bScriptingContent )
 
     // check whether the document is signed
     ImplGetSignatureState(); // document signature
-    ImplGetSignatureState( true ); // script signature
+    if (GetMedium() && GetMedium()->GetFilter() && GetMedium()->GetFilter()->IsOwnFormat())
+        ImplGetSignatureState( true ); // script signature
     bool bHasSign = ( pImpl->nScriptingSignatureState != SignatureState::NOSIGNATURES || pImpl->nDocumentSignatureState != SignatureState::NOSIGNATURES );
 
     // the target ODF version on saving
diff --git a/sfx2/source/doc/objstor.cxx b/sfx2/source/doc/objstor.cxx
index 6edaf1159291..d4e82fa3d84e 100644
--- a/sfx2/source/doc/objstor.cxx
+++ b/sfx2/source/doc/objstor.cxx
@@ -1871,6 +1871,8 @@ bool SfxObjectShell::ConnectTmpStorage_Impl(
             SetError( ERRCODE_IO_GENERAL, OSL_LOG_PREFIX );
         }
     }
+    else if (!GetMedium()->GetFilter()->IsOwnFormat())
+        bResult = true;
 
     return bResult;
 }