Use the broadcast refresh mechanism for sync finished callbacks in TLA and TVA. This gives better consistency and fewer crashes."

astrid/plugin-src/com/todoroo/astrid/actfm/TagViewActivity.java

@@ -254,9 +254,7 @@ public class TagViewActivity extends TaskListActivity {
                 R.id.progressBar, new Runnable() {
             @Override
             public void run() {
-                setUpMembersGallery();
-                loadTaskListContent(true);
-                ((TextView)taskListView.findViewById(android.R.id.empty)).setText(R.string.TLA_no_items);
+                ContextManager.getContext().sendBroadcast(new Intent(AstridApiConstants.BROADCAST_EVENT_REFRESH));
             }
         }));
         Preferences.setLong(LAST_FETCH_KEY + tagData.getId(), DateUtilities.now());
@@ -467,4 +465,11 @@ public class TagViewActivity extends TaskListActivity {
         return super.onOptionsItemSelected(item);
     }
 
+    @Override
+    protected void refresh() {
+        setUpMembersGallery();
+        loadTaskListContent(true);
+        ((TextView)taskListView.findViewById(android.R.id.empty)).setText(R.string.TLA_no_items);
+    }
+
 }

astrid/src/com/todoroo/astrid/activity/TaskListActivity.java

@@ -686,13 +686,22 @@ public class TaskListActivity extends ListFragment implements OnScrollListener,
             getActivity().runOnUiThread(new Runnable() {
                 @Override
                 public void run() {
-                    taskAdapter.flushCaches();
-                    loadTaskListContent(true);
+                    refresh();
                 }
             });
         }
     }
 
+
+    /**
+     * Called by the RefreshReceiver when the task list receives a refresh
+     * broadcast. Subclasses should override this.
+     */
+    protected void refresh() {
+        taskAdapter.flushCaches();
+        loadTaskListContent(true);
+    }
+
     /**
      * Receiver which receives sync provider intents
      *
@@ -1178,7 +1187,7 @@ public class TaskListActivity extends ListFragment implements OnScrollListener,
                 R.id.progressBar, new Runnable() {
             @Override
             public void run() {
-                loadTaskListContent(true);
+                ContextManager.getContext().sendBroadcast(new Intent(AstridApiConstants.BROADCAST_EVENT_REFRESH));
             }
         }));
         Preferences.setLong(PREF_LAST_AUTO_SYNC, DateUtilities.now());