[Libreoffice-commits] core.git: android/experimental

Miklos Vajna vmiklos at collabora.co.uk
Tue Dec 9 07:20:16 PST 2014


 android/experimental/LOAndroid3/src/java/org/libreoffice/LOKitTileProvider.java |   22 +++++++---
 1 file changed, 16 insertions(+), 6 deletions(-)

New commits:
commit 37280590cdb0a0654a6c618eb06f67aa48b27f31
Author: Miklos Vajna <vmiklos at collabora.co.uk>
Date:   Tue Dec 9 16:03:29 2014 +0100

    Related: fdo#85286 android: show error message given by LOK on the UI
    
    Change-Id: Ide8e6bc7e719968bacf86572036b15c4ef834761

diff --git a/android/experimental/LOAndroid3/src/java/org/libreoffice/LOKitTileProvider.java b/android/experimental/LOAndroid3/src/java/org/libreoffice/LOKitTileProvider.java
index ec6d1d1..81de8ba 100644
--- a/android/experimental/LOAndroid3/src/java/org/libreoffice/LOKitTileProvider.java
+++ b/android/experimental/LOAndroid3/src/java/org/libreoffice/LOKitTileProvider.java
@@ -123,22 +123,30 @@ public class LOKitTileProvider implements TileProvider {
     }
 
     private boolean checkDocument() {
+        String error = null;
+        boolean ret;
+
         if (mDocument == null || !mOffice.getError().isEmpty()) {
-            Log.e(LOGTAG, "Error at loading: " + mOffice.getError());
-            return false;
+            error = "Cannot open " + mInputFile + ": " + mOffice.getError();
+            ret = false;
+        } else {
+            ret = resetDocumentSize();
+            if (!ret) {
+                error = "Document returned an invalid size or the document is empty!";
+            }
         }
 
-        boolean result = resetDocumentSize();
-        if (!result) {
+        if (!ret) {
+            final String message = error;
             LOKitShell.getMainHandler().post(new Runnable() {
                 @Override
                 public void run() {
-                    LibreOfficeMainActivity.mAppContext.showAlertDialog("Document returned an invalid size or the document is empty!");
+                    LibreOfficeMainActivity.mAppContext.showAlertDialog(message);
                 }
             });
         }
 
-        return result;
+        return ret;
     }
 
     private boolean resetDocumentSize() {
@@ -247,3 +255,5 @@ public class LOKitTileProvider implements TileProvider {
         resetDocumentSize();
     }
 }
+
+// vim:set shiftwidth=4 softtabstop=4 expandtab:


More information about the Libreoffice-commits mailing list