Commit 1f194fd4 authored by Vance Turnewitsch's avatar Vance Turnewitsch

Issue 114 update. More error messages changed to info.

parent 51efa8f7
...@@ -89,7 +89,9 @@ public class BuildCommand extends ChainableCommand { ...@@ -89,7 +89,9 @@ public class BuildCommand extends ChainableCommand {
break; break;
default: default:
String errorMsg = result.getError(); String errorMsg = result.getError();
ErrorReporter.reportError(MESSAGES.buildFailedError() + // This is not an internal App Inventor bug. The error is reported as info so that
// the red background is not shown.
ErrorReporter.reportInfo(MESSAGES.buildFailedError() +
(errorMsg.isEmpty() ? "" : " " + errorMsg)); (errorMsg.isEmpty() ? "" : " " + errorMsg));
break; break;
} }
......
...@@ -36,7 +36,15 @@ public class EnsurePhoneConnectedCommand extends ChainableCommand { ...@@ -36,7 +36,15 @@ public class EnsurePhoneConnectedCommand extends ChainableCommand {
AsyncCallback<Boolean> callback = new AsyncCallback<Boolean>() { AsyncCallback<Boolean> callback = new AsyncCallback<Boolean>() {
@Override @Override
public void onFailure(Throwable caught) { public void onFailure(Throwable caught) {
if (caught.getMessage().equals(MESSAGES.noCodeblocksConnection())) {
// This error is not an App Inventor bug; therefore, it
// should be reported as an info message.
ErrorReporter.reportInfo(caught.getMessage());
}else{
// The error is something else; therefore, report as
// an error message.
ErrorReporter.reportError(caught.getMessage()); ErrorReporter.reportError(caught.getMessage());
}
executionFailedOrCanceled(); executionFailedOrCanceled();
} }
......
...@@ -91,7 +91,9 @@ public class WaitForBuildResultCommand extends ChainableCommand { ...@@ -91,7 +91,9 @@ public class WaitForBuildResultCommand extends ChainableCommand {
} else if (result.getResult() == 1) { } else if (result.getResult() == 1) {
// General build error // General build error
String errorMsg = result.getError(); String errorMsg = result.getError();
ErrorReporter.reportError(MESSAGES.buildFailedError() + // This is not an internal App Inventor bug. The error should be
// reported in a yellow info box.
ErrorReporter.reportInfo(MESSAGES.buildFailedError() +
(errorMsg.isEmpty() ? "" : " " + errorMsg)); (errorMsg.isEmpty() ? "" : " " + errorMsg));
executionFailedOrCanceled(); executionFailedOrCanceled();
} else if (result.getResult() == 2) { } else if (result.getResult() == 2) {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment