Handled exceptions with crittercism

pull/14/head
Sam Bosley 14 years ago
parent 4e3ebbde05
commit 9d23ba2c34

@ -28,6 +28,7 @@ import android.text.TextUtils;
import android.util.Log; import android.util.Log;
import android.widget.Toast; import android.widget.Toast;
import com.crittercism.app.Crittercism;
import com.timsu.astrid.R; import com.timsu.astrid.R;
import com.todoroo.andlib.data.AbstractModel; import com.todoroo.andlib.data.AbstractModel;
import com.todoroo.andlib.data.DatabaseDao; import com.todoroo.andlib.data.DatabaseDao;
@ -427,9 +428,12 @@ public final class ActFmSyncService {
JsonHelper.taskFromJson(result, task, metadata); JsonHelper.taskFromJson(result, task, metadata);
} catch (JSONException e) { } catch (JSONException e) {
handleException("task-save-json", e); handleException("task-save-json", e);
Crittercism.logHandledException(e);
} catch (IOException e) { } catch (IOException e) {
if (notPermanentError(e)) if (notPermanentError(e))
addFailedPush(new FailedPush(PUSH_TYPE_TASK, task.getId())); addFailedPush(new FailedPush(PUSH_TYPE_TASK, task.getId()));
else
Crittercism.logHandledException(e);
handleException("task-save-io", e); handleException("task-save-io", e);
task.setValue(Task.LAST_SYNC, DateUtilities.now() + 1000L); task.setValue(Task.LAST_SYNC, DateUtilities.now() + 1000L);
} }
@ -1090,8 +1094,10 @@ public final class ActFmSyncService {
handler.handleException("io-exception-list-" + model, e); handler.handleException("io-exception-list-" + model, e);
else else
handleException("io-exception-list-" + model, e); handleException("io-exception-list-" + model, e);
Crittercism.logHandledException(e);
} catch (JSONException e) { } catch (JSONException e) {
handleException("json: " + result.toString(), e); handleException("json: " + result.toString(), e);
Crittercism.logHandledException(e);
} }
} }
}).start(); }).start();

@ -11,6 +11,7 @@ import java.util.concurrent.atomic.AtomicInteger;
import org.json.JSONException; import org.json.JSONException;
import com.crittercism.app.Crittercism;
import com.timsu.astrid.C2DMReceiver; import com.timsu.astrid.C2DMReceiver;
import com.timsu.astrid.R; import com.timsu.astrid.R;
import com.todoroo.andlib.data.TodorooCursor; import com.todoroo.andlib.data.TodorooCursor;
@ -132,8 +133,10 @@ public class ActFmSyncV2Provider extends SyncV2Provider {
Preferences.setInt(LAST_TAG_FETCH_TIME, time); Preferences.setInt(LAST_TAG_FETCH_TIME, time);
} catch (JSONException e) { } catch (JSONException e) {
handler.handleException("actfm-sync", e); //$NON-NLS-1$ handler.handleException("actfm-sync", e); //$NON-NLS-1$
Crittercism.logHandledException(e);
} catch (IOException e) { } catch (IOException e) {
handler.handleException("actfm-sync", e); //$NON-NLS-1$ handler.handleException("actfm-sync", e); //$NON-NLS-1$
Crittercism.logHandledException(e);
} finally { } finally {
callback.incrementProgress(20); callback.incrementProgress(20);
if(finisher.decrementAndGet() == 0) { if(finisher.decrementAndGet() == 0) {

@ -7,6 +7,7 @@ import android.content.ContentValues;
import android.text.TextUtils; import android.text.TextUtils;
import android.util.Log; import android.util.Log;
import com.crittercism.app.Crittercism;
import com.todoroo.andlib.data.DatabaseDao.ModelUpdateListener; import com.todoroo.andlib.data.DatabaseDao.ModelUpdateListener;
import com.todoroo.andlib.data.Property; import com.todoroo.andlib.data.Property;
import com.todoroo.andlib.service.Autowired; import com.todoroo.andlib.service.Autowired;
@ -87,7 +88,8 @@ public final class GtasksSyncService {
pushMetadataOnSave(move.metadata, invoker); pushMetadataOnSave(move.metadata, invoker);
} }
} }
} catch (IOException e){ } catch (IOException e) {
Crittercism.logHandledException(e);
Log.w("gtasks-sync-error", "Sync on save failed", e); Log.w("gtasks-sync-error", "Sync on save failed", e);
} }
} }

@ -11,6 +11,7 @@ import org.json.JSONException;
import android.text.TextUtils; import android.text.TextUtils;
import com.crittercism.app.Crittercism;
import com.google.api.services.tasks.model.Tasks; import com.google.api.services.tasks.model.Tasks;
import com.timsu.astrid.R; import com.timsu.astrid.R;
import com.todoroo.andlib.data.AbstractModel; import com.todoroo.andlib.data.AbstractModel;
@ -112,6 +113,7 @@ public class GtasksSyncV2Provider extends SyncV2Provider {
gtasksListService.updateLists(invoker.allGtaskLists()); gtasksListService.updateLists(invoker.allGtaskLists());
} catch (IOException e) { } catch (IOException e) {
handler.handleException("gtasks-sync=io", e); //$NON-NLS-1$ handler.handleException("gtasks-sync=io", e); //$NON-NLS-1$
Crittercism.logHandledException(e);
} }
StoreObject[] lists = gtasksListService.getLists(); StoreObject[] lists = gtasksListService.getLists();
@ -153,6 +155,7 @@ public class GtasksSyncV2Provider extends SyncV2Provider {
gtasksSyncService.pushTaskOnSave(task, task.getMergedValues(), invoker, false); gtasksSyncService.pushTaskOnSave(task, task.getMergedValues(), invoker, false);
} catch (IOException e) { } catch (IOException e) {
handler.handleException("gtasks-sync-io", e); //$NON-NLS-1$ handler.handleException("gtasks-sync-io", e); //$NON-NLS-1$
Crittercism.logHandledException(e);
} finally { } finally {
callback.incrementProgress(10); callback.incrementProgress(10);
} }

Loading…
Cancel
Save