karol/OpenTracks
1
0
Fork 0
forked from upstream-mirrors/OpenTracks
Code Pull Requests Actions Releases 2 Activity

Add a better status message to the exception wrapper.

Browse Source 
Be more paranoid about exception messages.
This commit is contained in:
Matthew Simmons
2011-02-09 11:34:46 -05:00
parent 43aa3c857d
commit 95cbaf1993
2 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
MyTracks/src/com/google/android/apps/mytracks/io/SendToFusionTables.java
+1 -1
View File
@@ -670,7 +670,7 @@ public class SendToFusionTables implements Runnable {
try { try {
response = request.execute(); response = request.execute();
} catch (HttpResponseException e) { } catch (HttpResponseException e) {
throw new GDataWrapper.HttpException(e.response.statusCode, e.getMessage()); throw new GDataWrapper.HttpException(e.response.statusCode, e.response.statusMessage);
} }
boolean success = response.isSuccessStatusCode; boolean success = response.isSuccessStatusCode;
if (success) { if (success) {
MyTracks/src/com/google/android/apps/mytracks/io/gdata/GDataWrapper.java
+2 -2
View File
@@ -212,7 +212,7 @@ public class GDataWrapper<C> {
} catch (HttpException e) { } catch (HttpException e) {
Log.e(MyTracksConstants.TAG, "HttpException", e); Log.e(MyTracksConstants.TAG, "HttpException", e);
errorMessage = e.getMessage(); errorMessage = e.getMessage();
if (errorMessage.contains("401")) { if (errorMessage != null && errorMessage.contains("401")) {
errorType = ERROR_AUTH; errorType = ERROR_AUTH;
} else { } else {
errorType = ERROR_CONNECTION; errorType = ERROR_CONNECTION;
@@ -224,7 +224,7 @@ public class GDataWrapper<C> {
} catch (IOException e) { } catch (IOException e) {
Log.e(MyTracksConstants.TAG, "Exception", e); Log.e(MyTracksConstants.TAG, "Exception", e);
errorMessage = e.getMessage(); errorMessage = e.getMessage();
if (errorMessage.contains("503")) { if (errorMessage != null && errorMessage.contains("503")) {
errorType = ERROR_INTERNAL; errorType = ERROR_INTERNAL;
} else { } else {
errorType = ERROR_CONNECTION; errorType = ERROR_CONNECTION;