From c5b23318ca785c6f48b85122c8b443b277d9f881 Mon Sep 17 00:00:00 2001
From: Albert Astals Cid <tsdgeos@terra.es>
Date: Tue, 5 Jun 2007 21:25:41 +0000
Subject: [PATCH] Error notyfing when not able to open a file WindowTitle
 settign for remote files

svn path=/trunk/KDE/kdegraphics/okular/; revision=671944
---
 part.cpp | 43 +++++++++++++++++++++++++++++++++----------
 part.h   |  2 ++
 2 files changed, 35 insertions(+), 10 deletions(-)

diff --git a/part.cpp b/part.cpp
index 13e8e1e47..8606bc156 100644
--- a/part.cpp
+++ b/part.cpp
@@ -103,6 +103,11 @@ m_searchStarted(false), m_cliPresentation(false)
 
     // connect the started signal to tell the job the mimetypes we like
     connect(this, SIGNAL(started(KIO::Job *)), this, SLOT(setMimeTypes(KIO::Job *)));
+
+    // connect the completed signal so we can put the window caption when loading remote files
+    connect(this, SIGNAL(completed()), this, SLOT(setWindowTitleFromDocument()));
+    connect(this, SIGNAL(canceled(const QString &)), this, SLOT(loadCancelled(const QString &)));
+
     // load catalog for translation
     KGlobal::locale()->insertCatalog("okular");
 
@@ -525,6 +530,33 @@ void Part::readMimeType(KIO::Job *, const QString &mime)
     m_jobMime = mime;
 }
 
+void Part::loadCancelled(const QString &reason)
+{
+    emit setWindowCaption( QString() );
+    if (!reason.isEmpty())
+    {
+        KMessageBox::error( widget(), i18n("Could not open %1. Reason: %2", url().prettyUrl(), reason ) );
+    }
+    else
+    {
+        KMessageBox::error( widget(), i18n("Could not open %1", url().prettyUrl() ) );
+    }
+}
+
+void Part::setWindowTitleFromDocument()
+{
+    // if the document have a 'DocumentTitle' flag set (and it is not empty), set it as title
+    QString title = m_document->metaData( "DocumentTitle" ).toString();
+    if ( !title.isEmpty() && !title.trimmed().isEmpty() )
+    {
+        emit setWindowCaption( title );
+    }
+    else
+    {
+        emit setWindowCaption( url().fileName() );
+    }
+}
+
 void Part::slotGeneratorPreferences( )
 {
     // an instance the dialog could be already created and could be cached,
@@ -772,16 +804,7 @@ bool Part::openUrl(const KUrl &url)
     {
         m_viewportDirty.pageNumber = -1;
 
-        // if the document have a 'DocumentTitle' flag set (and it is not empty), set it as title
-        QString title = m_document->metaData( "DocumentTitle" ).toString();
-        if ( !title.isEmpty() && !title.trimmed().isEmpty() )
-        {
-            emit setWindowCaption( title );
-        }
-        else
-        {
-            emit setWindowCaption( url.fileName() );
-        }
+        setWindowTitleFromDocument();
     }
 
     emit enablePrintAction(openOk);
diff --git a/part.h b/part.h
index 0af7455f3..d379fccc0 100644
--- a/part.h
+++ b/part.h
@@ -148,6 +148,8 @@ class Part : public KParts::ReadOnlyPart, public Okular::DocumentObserver, publi
         void slotHideFindBar();
         void setMimeTypes(KIO::Job *job);
         void readMimeType(KIO::Job *job, const QString &mime);
+        void loadCancelled(const QString &reason);
+        void setWindowTitleFromDocument();
         void saveSplitterSize();
         // can be connected to widget elements
         void updateViewActions();