From 7a1451f561517bd4614cd96247b7c698a178e4bd Mon Sep 17 00:00:00 2001 From: Tom Morris Date: Sun, 21 Jun 2020 23:09:47 -0400 Subject: [PATCH] Report errors to user No errors were being reported before. Also add TODO for progress indicator on long uploads --- .../src/com/google/refine/extension/gdata/UploadCommand.java | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/extensions/gdata/src/com/google/refine/extension/gdata/UploadCommand.java b/extensions/gdata/src/com/google/refine/extension/gdata/UploadCommand.java index 15c1cf657..b81f876c5 100644 --- a/extensions/gdata/src/com/google/refine/extension/gdata/UploadCommand.java +++ b/extensions/gdata/src/com/google/refine/extension/gdata/UploadCommand.java @@ -68,6 +68,8 @@ public class UploadCommand extends Command { private static final String METADATA_DESCRIPTION = "OpenRefine project dump"; private static final String METADATA_ICON_FILE = "logo-openrefine-550.png"; + + // TODO: We need a way to provide progress to the user during long uploads @Override public void doPost(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException { @@ -99,7 +101,8 @@ public class UploadCommand extends Command { List exceptions = new LinkedList(); String url = upload(project, engine, params, token, name, exceptions); - if (url != null) { + // The URL can be non-null even if it doesn't fail + if (url != null && exceptions.size() == 0) { writer.writeStringField("status", "ok"); writer.writeStringField("url", url); } else if (exceptions.size() == 0) {