Track repeat snackbar exception

pull/384/head
Alex Baker 10 years ago
parent fd0d1150a0
commit 698bf7f574

@ -65,10 +65,10 @@ public class TaskListActivity extends InjectingAppCompatActivity implements
@Inject SubtasksHelper subtasksHelper; @Inject SubtasksHelper subtasksHelper;
@Inject TaskService taskService; @Inject TaskService taskService;
@Inject TaskEditControlSetFragmentManager taskEditControlSetFragmentManager; @Inject TaskEditControlSetFragmentManager taskEditControlSetFragmentManager;
@Inject RepeatConfirmationReceiver repeatConfirmationReceiver;
public static final int REQUEST_UPGRADE = 505; public static final int REQUEST_UPGRADE = 505;
private final RepeatConfirmationReceiver repeatConfirmationReceiver = new RepeatConfirmationReceiver(this);
private NavigationDrawerFragment navigationDrawer; private NavigationDrawerFragment navigationDrawer;
/** For indicating the new list screen should be launched at fragment setup time */ /** For indicating the new list screen should be launched at fragment setup time */

@ -34,8 +34,7 @@ import dagger.Module;
FirstLaunchReceiver.class, FirstLaunchReceiver.class,
MyPackageReplacedReceiver.class, MyPackageReplacedReceiver.class,
RefreshReceiver.class, RefreshReceiver.class,
CompleteTaskReceiver.class, CompleteTaskReceiver.class
RepeatConfirmationReceiver.class
}) })
public class BroadcastModule { public class BroadcastModule {
} }

@ -1,6 +1,7 @@
package org.tasks.receivers; package org.tasks.receivers;
import android.app.Activity; import android.app.Activity;
import android.content.BroadcastReceiver;
import android.content.Context; import android.content.Context;
import android.content.Intent; import android.content.Intent;
import android.support.design.widget.Snackbar; import android.support.design.widget.Snackbar;
@ -12,16 +13,15 @@ import com.todoroo.andlib.utility.DateUtilities;
import com.todoroo.astrid.api.AstridApiConstants; import com.todoroo.astrid.api.AstridApiConstants;
import com.todoroo.astrid.data.Task; import com.todoroo.astrid.data.Task;
import com.todoroo.astrid.service.TaskService; import com.todoroo.astrid.service.TaskService;
import com.todoroo.astrid.utility.Flags;
import org.tasks.R; import org.tasks.R;
import org.tasks.injection.InjectingBroadcastReceiver; import org.tasks.analytics.Tracker;
import javax.inject.Inject; import javax.inject.Inject;
import timber.log.Timber; import timber.log.Timber;
public class RepeatConfirmationReceiver extends InjectingBroadcastReceiver { public class RepeatConfirmationReceiver extends BroadcastReceiver {
private final Property<?>[] REPEAT_RESCHEDULED_PROPERTIES = private final Property<?>[] REPEAT_RESCHEDULED_PROPERTIES =
new Property<?>[]{ new Property<?>[]{
@ -32,18 +32,19 @@ public class RepeatConfirmationReceiver extends InjectingBroadcastReceiver {
Task.REPEAT_UNTIL Task.REPEAT_UNTIL
}; };
@Inject TaskService taskService; private final TaskService taskService;
private final Activity activity; private final Activity activity;
private Tracker tracker;
public RepeatConfirmationReceiver(Activity activity) { @Inject
public RepeatConfirmationReceiver(TaskService taskService, Activity activity, Tracker tracker) {
this.taskService = taskService;
this.activity = activity; this.activity = activity;
this.tracker = tracker;
} }
@Override @Override
public void onReceive(final Context context, final Intent intent) { public void onReceive(final Context context, final Intent intent) {
super.onReceive(context, intent);
long taskId = intent.getLongExtra(AstridApiConstants.EXTRAS_TASK_ID, 0); long taskId = intent.getLongExtra(AstridApiConstants.EXTRAS_TASK_ID, 0);
if (taskId > 0) { if (taskId > 0) {
@ -61,6 +62,9 @@ public class RepeatConfirmationReceiver extends InjectingBroadcastReceiver {
context.sendBroadcast(intent); context.sendBroadcast(intent);
} }
}.start(); }.start();
} catch (Exception e) {
Timber.e(e, e.getMessage());
tracker.reportException(e);
} }
} }
} }

Loading…
Cancel
Save