|
 |
135360 |
From f752c0f12b0e903eb958e48ec1b88e4bc70778f9 Mon Sep 17 00:00:00 2001
|
|
|
135360 |
From: Miklos Vajna <vmiklos@collabora.co.uk>
|
|
|
135360 |
Date: Fri, 18 Sep 2015 10:58:24 +0200
|
|
|
135360 |
Subject: [PATCH 152/398] gtktiledviewer: don't crash on opening non-existing
|
|
|
135360 |
files
|
|
|
135360 |
|
|
|
135360 |
Change-Id: Ic48adaf038e8fbcc86a94b5e351d2f963fcfcd16
|
|
|
135360 |
(cherry picked from commit 43b2103f932c92374e2ba2ab0ec1006582a0f646)
|
|
|
135360 |
---
|
|
|
135360 |
libreofficekit/qa/gtktiledviewer/gtktiledviewer.cxx | 16 ++++++++++++++--
|
|
|
135360 |
libreofficekit/source/gtk/lokdocview.cxx | 2 --
|
|
|
135360 |
2 files changed, 14 insertions(+), 4 deletions(-)
|
|
|
135360 |
|
|
|
135360 |
diff --git a/libreofficekit/qa/gtktiledviewer/gtktiledviewer.cxx b/libreofficekit/qa/gtktiledviewer/gtktiledviewer.cxx
|
|
|
135360 |
index 5a18aee50bf3..8e64972be17d 100644
|
|
|
135360 |
--- a/libreofficekit/qa/gtktiledviewer/gtktiledviewer.cxx
|
|
|
135360 |
+++ b/libreofficekit/qa/gtktiledviewer/gtktiledviewer.cxx
|
|
|
135360 |
@@ -553,8 +553,20 @@ static void openDocumentCallback (GObject* source_object, GAsyncResult* res, gpo
|
|
|
135360 |
|
|
|
135360 |
if (!lok_doc_view_open_document_finish(pDocView, res, &error))
|
|
|
135360 |
{
|
|
|
135360 |
- g_warning ("Error occurred while opening the document : %s", error->message);
|
|
|
135360 |
- g_error_free (error);
|
|
|
135360 |
+ GtkDialogFlags eFlags = GTK_DIALOG_DESTROY_WITH_PARENT;
|
|
|
135360 |
+ GtkWidget* pDialog = gtk_message_dialog_new(GTK_WINDOW(gtk_widget_get_toplevel(GTK_WIDGET(pDocView))),
|
|
|
135360 |
+ eFlags,
|
|
|
135360 |
+ GTK_MESSAGE_ERROR,
|
|
|
135360 |
+ GTK_BUTTONS_CLOSE,
|
|
|
135360 |
+ "Error occurred while opening the document: '%s'",
|
|
|
135360 |
+ error->message);
|
|
|
135360 |
+ gtk_dialog_run(GTK_DIALOG(pDialog));
|
|
|
135360 |
+ gtk_widget_destroy(pDialog);
|
|
|
135360 |
+
|
|
|
135360 |
+ g_error_free(error);
|
|
|
135360 |
+ gtk_widget_destroy(GTK_WIDGET(pDocView));
|
|
|
135360 |
+ gtk_main_quit();
|
|
|
135360 |
+ return;
|
|
|
135360 |
}
|
|
|
135360 |
|
|
|
135360 |
populatePartSelector(pDocView);
|
|
|
135360 |
diff --git a/libreofficekit/source/gtk/lokdocview.cxx b/libreofficekit/source/gtk/lokdocview.cxx
|
|
|
135360 |
index f8202c339587..60304b4ff59c 100644
|
|
|
135360 |
--- a/libreofficekit/source/gtk/lokdocview.cxx
|
|
|
135360 |
+++ b/libreofficekit/source/gtk/lokdocview.cxx
|
|
|
135360 |
@@ -1225,9 +1225,7 @@ openDocumentInThread (gpointer data)
|
|
|
135360 |
priv->m_pDocument = priv->m_pOffice->pClass->documentLoad( priv->m_pOffice, priv->m_aDocPath );
|
|
|
135360 |
if ( !priv->m_pDocument )
|
|
|
135360 |
{
|
|
|
135360 |
- // FIXME: should have a GError parameter and populate it.
|
|
|
135360 |
char *pError = priv->m_pOffice->pClass->getError( priv->m_pOffice );
|
|
|
135360 |
- fprintf( stderr, "Error opening document '%s'\n", pError );
|
|
|
135360 |
g_task_return_new_error(task, 0, 0, "%s", pError);
|
|
|
135360 |
}
|
|
|
135360 |
else
|
|
|
135360 |
--
|
|
|
135360 |
2.12.0
|
|
|
135360 |
|