From 839a3e58c758416a8611f544d29657f2a1a25ce9 Mon Sep 17 00:00:00 2001
From: Alex Baker <baker.alex@gmail.com>
Date: Wed, 11 Mar 2015 14:51:19 -0500
Subject: [PATCH] Add CompleteTaskReceiver

---
 src/main/AndroidManifest.xml                  |  1 +
 .../reminders/NotificationFragment.java       | 12 ++-----
 .../astrid/reminders/ReminderDialog.java      |  8 ++---
 .../todoroo/astrid/widget/TasksWidget.java    |  8 ++---
 src/main/java/org/tasks/Broadcaster.java      | 16 +++++++++
 .../org/tasks/injection/BroadcastModule.java  |  4 ++-
 .../tasks/receivers/CompleteTaskReceiver.java | 34 +++++++++++++++++++
 7 files changed, 63 insertions(+), 20 deletions(-)
 create mode 100644 src/main/java/org/tasks/receivers/CompleteTaskReceiver.java

diff --git a/src/main/AndroidManifest.xml b/src/main/AndroidManifest.xml
index 94a640b60..f6763cfd4 100644
--- a/src/main/AndroidManifest.xml
+++ b/src/main/AndroidManifest.xml
@@ -475,6 +475,7 @@
         </receiver>
 
         <receiver android:name=".receivers.FirstLaunchReceiver"/>
+        <receiver android:name=".receivers.CompleteTaskReceiver"/>
 
         <receiver
             android:name=".receivers.PackageReplacedReceiver"
diff --git a/src/main/java/com/todoroo/astrid/reminders/NotificationFragment.java b/src/main/java/com/todoroo/astrid/reminders/NotificationFragment.java
index 8f299bc85..0b1799a1b 100644
--- a/src/main/java/com/todoroo/astrid/reminders/NotificationFragment.java
+++ b/src/main/java/com/todoroo/astrid/reminders/NotificationFragment.java
@@ -9,6 +9,7 @@ import com.todoroo.astrid.activity.AstridActivity;
 import com.todoroo.astrid.activity.TaskListFragment;
 import com.todoroo.astrid.service.TaskService;
 
+import org.tasks.Broadcaster;
 import org.tasks.preferences.Preferences;
 
 import javax.inject.Inject;
@@ -22,13 +23,9 @@ import javax.inject.Inject;
  */
 public class NotificationFragment extends TaskListFragment {
 
-    // --- constants
-
-    /** task id from notification */
     public static final String TOKEN_ID = "id"; //$NON-NLS-1$
 
-    // --- implementation
-
+    @Inject Broadcaster broadcaster;
     @Inject TaskService taskService;
     @Inject Preferences preferences;
 
@@ -38,12 +35,9 @@ public class NotificationFragment extends TaskListFragment {
         super.initializeData();
     }
 
-    /**
-     * Set up the UI for this activity
-     */
     private void displayNotificationPopup() {
         String title = extras.getString(Notifications.EXTRAS_TEXT);
         long taskId = extras.getLong(TOKEN_ID);
-        new ReminderDialog(preferences, taskService, (AstridActivity) getActivity(), taskId, title).show();
+        new ReminderDialog(preferences, broadcaster, taskService, (AstridActivity) getActivity(), taskId, title).show();
     }
 }
diff --git a/src/main/java/com/todoroo/astrid/reminders/ReminderDialog.java b/src/main/java/com/todoroo/astrid/reminders/ReminderDialog.java
index 4536ecb08..32083a842 100644
--- a/src/main/java/com/todoroo/astrid/reminders/ReminderDialog.java
+++ b/src/main/java/com/todoroo/astrid/reminders/ReminderDialog.java
@@ -20,6 +20,7 @@ import com.todoroo.astrid.activity.AstridActivity;
 import com.todoroo.astrid.data.Task;
 import com.todoroo.astrid.service.TaskService;
 
+import org.tasks.Broadcaster;
 import org.tasks.R;
 import org.tasks.preferences.Preferences;
 
@@ -37,7 +38,7 @@ public class ReminderDialog extends Dialog {
 
     private final Preferences preferences;
 
-    public ReminderDialog(Preferences preferences, final TaskService taskService, final AstridActivity activity, final long taskId,
+    public ReminderDialog(Preferences preferences, final Broadcaster broadcaster, final TaskService taskService, final AstridActivity activity, final long taskId,
             String title) {
         super(activity, R.style.ReminderDialog);
         this.preferences = preferences;
@@ -87,10 +88,7 @@ public class ReminderDialog extends Dialog {
         findViewById(R.id.reminder_complete).setOnClickListener(new View.OnClickListener() {
             @Override
             public void onClick(View arg0) {
-                Task task = taskService.fetchById(taskId, Task.ID, Task.REMINDER_LAST);
-                if (task != null) {
-                    taskService.setComplete(task, true);
-                }
+                broadcaster.completeTask(taskId);
                 dismiss();
             }
         });
diff --git a/src/main/java/com/todoroo/astrid/widget/TasksWidget.java b/src/main/java/com/todoroo/astrid/widget/TasksWidget.java
index 3fd29b208..bb840d1d8 100644
--- a/src/main/java/com/todoroo/astrid/widget/TasksWidget.java
+++ b/src/main/java/com/todoroo/astrid/widget/TasksWidget.java
@@ -16,11 +16,10 @@ import com.todoroo.andlib.utility.DateUtilities;
 import com.todoroo.astrid.activity.TaskEditActivity;
 import com.todoroo.astrid.activity.TaskEditFragment;
 import com.todoroo.astrid.activity.TaskListActivity;
-import com.todoroo.astrid.data.Task;
-import com.todoroo.astrid.service.TaskService;
 
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
+import org.tasks.Broadcaster;
 import org.tasks.R;
 import org.tasks.injection.InjectingAppWidgetProvider;
 import org.tasks.preferences.ActivityPreferences;
@@ -35,7 +34,7 @@ public class TasksWidget extends InjectingAppWidgetProvider {
 
     private static final Logger log = LoggerFactory.getLogger(TasksWidget.class);
 
-    @Inject TaskService taskService;
+    @Inject Broadcaster broadcaster;
     @Inject WidgetHelper widgetHelper;
 
     public static final String COMPLETE_TASK = "COMPLETE_TASK";
@@ -50,8 +49,7 @@ public class TasksWidget extends InjectingAppWidgetProvider {
 
         switch(intent.getAction()) {
             case COMPLETE_TASK:
-                Task task = taskService.fetchById(intent.getLongExtra(TaskEditFragment.TOKEN_ID, 0), Task.ID, Task.COMPLETION_DATE);
-                taskService.setComplete(task, !task.isCompleted());
+                broadcaster.flipCompleteState(intent.getLongExtra(TaskEditFragment.TOKEN_ID, 0));
                 break;
             case EDIT_TASK:
                 if(ActivityPreferences.isTabletSized(context)) {
diff --git a/src/main/java/org/tasks/Broadcaster.java b/src/main/java/org/tasks/Broadcaster.java
index 99b5f6b50..00eb916ab 100644
--- a/src/main/java/org/tasks/Broadcaster.java
+++ b/src/main/java/org/tasks/Broadcaster.java
@@ -9,6 +9,7 @@ import com.todoroo.astrid.reminders.Notifications;
 import com.todoroo.astrid.utility.Constants;
 
 import org.tasks.injection.ForApplication;
+import org.tasks.receivers.CompleteTaskReceiver;
 import org.tasks.receivers.FirstLaunchReceiver;
 
 import javax.inject.Inject;
@@ -26,6 +27,21 @@ public class Broadcaster {
         this.context = context;
     }
 
+    public void flipCompleteState(long taskId) {
+        completeTask(taskId, true);
+    }
+
+    public void completeTask(long taskId) {
+        completeTask(taskId, false);
+    }
+
+    private void completeTask(final long taskId, final boolean flipState) {
+        sendOrderedBroadcast(new Intent(context, CompleteTaskReceiver.class) {{
+            putExtra(CompleteTaskReceiver.TASK_ID, taskId);
+            putExtra(CompleteTaskReceiver.FLIP_STATE, flipState);
+        }});
+    }
+
     public void requestNotification(final long taskId, final Intent intent, final int type,
                                     final String title, final String text, final int ringTimes) {
         sendOrderedBroadcast(new Intent(BROADCAST_IN_APP_NOTIFY) {{
diff --git a/src/main/java/org/tasks/injection/BroadcastModule.java b/src/main/java/org/tasks/injection/BroadcastModule.java
index 8ca9df7cd..0838125e4 100644
--- a/src/main/java/org/tasks/injection/BroadcastModule.java
+++ b/src/main/java/org/tasks/injection/BroadcastModule.java
@@ -14,6 +14,7 @@ import com.todoroo.astrid.timers.TimerTaskCompleteListener;
 import com.todoroo.astrid.widget.TasksWidget;
 
 import org.tasks.receivers.BootCompletedReceiver;
+import org.tasks.receivers.CompleteTaskReceiver;
 import org.tasks.receivers.FirstLaunchReceiver;
 import org.tasks.receivers.MyPackageReplacedReceiver;
 import org.tasks.receivers.PackageReplacedReceiver;
@@ -39,7 +40,8 @@ import dagger.Module;
                 FirstLaunchReceiver.class,
                 MyPackageReplacedReceiver.class,
                 PackageReplacedReceiver.class,
-                RefreshReceiver.class
+                RefreshReceiver.class,
+                CompleteTaskReceiver.class
         })
 public class BroadcastModule {
 }
diff --git a/src/main/java/org/tasks/receivers/CompleteTaskReceiver.java b/src/main/java/org/tasks/receivers/CompleteTaskReceiver.java
new file mode 100644
index 000000000..eedb509cc
--- /dev/null
+++ b/src/main/java/org/tasks/receivers/CompleteTaskReceiver.java
@@ -0,0 +1,34 @@
+package org.tasks.receivers;
+
+import android.content.Context;
+import android.content.Intent;
+
+import com.todoroo.astrid.data.Task;
+import com.todoroo.astrid.service.TaskService;
+
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
+import org.tasks.injection.InjectingBroadcastReceiver;
+
+import javax.inject.Inject;
+
+public class CompleteTaskReceiver extends InjectingBroadcastReceiver {
+
+    private static final Logger log = LoggerFactory.getLogger(CompleteTaskReceiver.class);
+
+    public static final String TASK_ID = "id";
+    public static final String  FLIP_STATE = "flip_state";
+
+    @Inject TaskService taskService;
+
+    @Override
+    public void onReceive(Context context, Intent intent) {
+        super.onReceive(context, intent);
+
+        long taskId = intent.getLongExtra(TASK_ID, 0);
+        boolean flipState = intent.getBooleanExtra(FLIP_STATE, false);
+        log.info("Completing {}", taskId);
+        Task task = taskService.fetchById(taskId, Task.ID, Task.COMPLETION_DATE);
+        taskService.setComplete(task, !flipState || !task.isCompleted());
+    }
+}