From c1e764e7c709f3d99c9603c1f09e685759d0d245 Mon Sep 17 00:00:00 2001 From: misaochan Date: Wed, 14 Dec 2016 16:59:03 +1300 Subject: [PATCH] Add checks for result Nodes and fileExists --- .../free/nrw/commons/upload/ExistingFileAsync.java | 12 +++++++++++- 1 file changed, 11 insertions(+), 1 deletion(-) diff --git a/app/src/main/java/fr/free/nrw/commons/upload/ExistingFileAsync.java b/app/src/main/java/fr/free/nrw/commons/upload/ExistingFileAsync.java index 4f2a39e2a..61e5f3c2d 100644 --- a/app/src/main/java/fr/free/nrw/commons/upload/ExistingFileAsync.java +++ b/app/src/main/java/fr/free/nrw/commons/upload/ExistingFileAsync.java @@ -51,7 +51,17 @@ public class ExistingFileAsync extends AsyncTask { } ArrayList resultNodes = result.getNodes("/api/query/allimages/"); - boolean fileExists = resultNodes!=null; + + Log.d(TAG, "Result nodes: " + resultNodes); + + boolean fileExists; + if (resultNodes!=null) { + fileExists = true; + } else { + fileExists = false; + } + + //FIXME: This always returns false even when file (and nodes) exists, why? Log.d(TAG, "File already exists in Commons:" + fileExists); return fileExists;