archive
/
tasks
mirror of https://github.com/tasks/tasks
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

Merge remote branch 'upstream/master' into dev

Browse Source
pull/14/head
Arne Jans 16 years ago
parent 7b5a6ebae2 f9eb247ef6
commit 149bdc382a
24 changed files with 207 additions and 96 deletions
Show Stats Download Patch File Download Diff File

10
astrid/AndroidManifest.xml
Unescape Escape View File

@ -1,8 +1,8 @@
<?xml version="1.0" encoding="utf-8"?>
<manifest xmlns:android="http://schemas.android.com/apk/res/android"
package="com.timsu.astrid"
android:versionName="3.2.3 (build your own filters, easy sorting, customizable widget, ui improvements)"
android:versionCode="150">
android:versionName="3.2.5 (build your own filters, easy sorting, customizable widget, ui improvements)"
android:versionCode="152">
<!-- widgets, alarms, and services will break if Astrid is installed on SD card -->
<!-- android:installLocation="internalOnly"> -->
@ -343,6 +343,12 @@
</intent-filter>
</activity>
<service android:name="com.todoroo.astrid.producteev.ProducteevBackgroundService"/>
<receiver android:name="com.todoroo.astrid.producteev.ProducteevStartupReceiver">
<intent-filter>
<action android:name="android.intent.action.BOOT_COMPLETED" />
<category android:name="android.intent.category.DEFAULT" />
</intent-filter>
</receiver>
<!-- rmilk -->
<receiver android:name="com.todoroo.astrid.rmilk.MilkFilterExposer">

3
astrid/plugin-src/com/todoroo/astrid/backup/BackupService.java
Unescape Escape View File

@ -15,6 +15,7 @@ import android.util.Log;
import com.timsu.astrid.R;
import com.todoroo.andlib.service.ContextManager;
import com.todoroo.andlib.utility.DateUtilities;
import com.todoroo.astrid.core.PluginServices;
import com.todoroo.astrid.utility.Preferences;
@ -92,7 +93,7 @@ public class BackupService extends Service {
if (!Preferences.getBoolean(R.string.backup_BPr_auto_key, true)) {
return;
}
am.setInexactRepeating(AlarmManager.RTC, System.currentTimeMillis() + BACKUP_OFFSET,
am.setInexactRepeating(AlarmManager.RTC, DateUtilities.now() + BACKUP_OFFSET,
BACKUP_INTERVAL, pendingIntent);
}

4
astrid/plugin-src/com/todoroo/astrid/backup/TasksXmlImporter.java
Unescape Escape View File

@ -249,7 +249,7 @@ public class TasksXmlImporter {
currentTask.setId(Task.NO_ID);
// Save the task to the database.
taskService.save(currentTask, false);
taskService.save(currentTask);
importCount++;
}
@ -461,7 +461,7 @@ public class TasksXmlImporter {
}
// Save the task to the database.
taskService.save(task, false);
taskService.save(task);
importCount++;
return task;
}

26
astrid/plugin-src/com/todoroo/astrid/producteev/ProducteevStartupReceiver.java
Unescape Escape View File

@ -0,0 +1,26 @@
/**
* See the file "LICENSE" for the full license governing this code.
*/
package com.todoroo.astrid.producteev;
import android.content.BroadcastReceiver;
import android.content.Context;
import android.content.Intent;
import com.todoroo.andlib.service.ContextManager;
import com.todoroo.astrid.service.AstridDependencyInjector;
public class ProducteevStartupReceiver extends BroadcastReceiver {
static {
AstridDependencyInjector.initialize();
}
@Override
/** Called when device is restarted */
public void onReceive(final Context context, Intent intent) {
ContextManager.setContext(context);
ProducteevBackgroundService.scheduleService();
}
}

2
astrid/plugin-src/com/todoroo/astrid/producteev/sync/ProducteevDataService.java
Unescape Escape View File

@ -138,7 +138,7 @@ public final class ProducteevDataService {
* @param task
*/
public void saveTaskAndMetadata(ProducteevTaskContainer task) {
taskDao.save(task.task, true);
taskDao.save(task.task);
task.metadata.add(task.pdvTask);
metadataService.synchronizeMetadata(task.task.getId(), task.metadata,

4
astrid/plugin-src/com/todoroo/astrid/repeats/RepeatTaskCompleteListener.java
Unescape Escape View File

@ -58,11 +58,11 @@ public class RepeatTaskCompleteListener extends BroadcastReceiver {
clone.setValue(Task.COMPLETION_DATE, 0L);
clone.setValue(Task.TIMER_START, 0L);
clone.setValue(Task.ELAPSED_SECONDS, 0);
PluginServices.getTaskService().save(clone, false);
PluginServices.getTaskService().save(clone);
// clear recurrence from completed task so it can be re-completed
task.setValue(Task.RECURRENCE, ""); //$NON-NLS-1$
PluginServices.getTaskService().save(task, false);
PluginServices.getTaskService().save(task);
// send a broadcast
Intent broadcastIntent = new Intent(AstridApiConstants.BROADCAST_EVENT_TASK_REPEATED);

2
astrid/plugin-src/com/todoroo/astrid/rmilk/MilkBackgroundService.java
Unescape Escape View File

@ -33,7 +33,7 @@ public class MilkBackgroundService extends Service {
// --- BroadcastReceiver abstract methods
/** Receive the alarm - start the synchronize service! */
/** start the synchronization service. sits in the background */
@Override
public void onStart(Intent intent, int startId) {
try {

5
astrid/plugin-src/com/todoroo/astrid/rmilk/MilkLoginActivity.java
Unescape Escape View File

@ -86,7 +86,7 @@ public class MilkLoginActivity extends Activity {
setContentView(R.layout.rmilk_login_activity);
setTitle(R.string.rmilk_MPr_header);
String urlParam = getIntent().getStringExtra(URL_TOKEN);
final String urlParam = getIntent().getStringExtra(URL_TOKEN);
final WebView webView = (WebView)findViewById(R.id.browser);
Button done = (Button)findViewById(R.id.done);
@ -119,11 +119,12 @@ public class MilkLoginActivity extends Activity {
if(result == null) {
finish();
} else {
// display the error
// display the error, re-load url
handler.post(new Runnable() {
public void run() {
dialogUtilities.okDialog(MilkLoginActivity.this,
result, null);
webView.loadUrl(urlParam);
}
});
}

33
astrid/plugin-src/com/todoroo/astrid/rmilk/data/MilkDataService.java
Unescape Escape View File

@ -23,7 +23,6 @@ import com.todoroo.andlib.sql.Join;
import com.todoroo.andlib.sql.Order;
import com.todoroo.andlib.sql.Query;
import com.todoroo.andlib.utility.SoftHashMap;
import com.todoroo.astrid.dao.MetadataDao;
import com.todoroo.astrid.dao.MetadataDao.MetadataCriteria;
import com.todoroo.astrid.dao.TaskDao;
import com.todoroo.astrid.dao.TaskDao.TaskCriteria;
@ -34,6 +33,7 @@ import com.todoroo.astrid.rmilk.MilkUtilities.ListContainer;
import com.todoroo.astrid.rmilk.api.data.RtmList;
import com.todoroo.astrid.rmilk.api.data.RtmLists;
import com.todoroo.astrid.rmilk.sync.RTMTaskContainer;
import com.todoroo.astrid.service.MetadataService;
import com.todoroo.astrid.tags.TagService;
public final class MilkDataService {
@ -65,7 +65,7 @@ public final class MilkDataService {
private TaskDao taskDao;
@Autowired
private MetadataDao metadataDao;
private MetadataService metadataService;
static final Random random = new Random();
@ -82,7 +82,7 @@ public final class MilkDataService {
* Clears RTM metadata information. Used when user logs out of RTM
*/
public void clearMetadata() {
metadataDao.deleteWhere(Metadata.KEY.eq(MilkTask.METADATA_KEY));
metadataService.deleteWhere(Metadata.KEY.eq(MilkTask.METADATA_KEY));
}
/**
@ -139,17 +139,18 @@ public final class MilkDataService {
* @param task
*/
public void saveTaskAndMetadata(RTMTaskContainer task) {
taskDao.save(task.task, true);
metadataDao.deleteWhere(Criterion.and(MetadataCriteria.byTask(task.task.getId()),
taskDao.save(task.task);
Metadata metadata = MilkTask.create(task);
Metadata foundMetadata = task.findMetadata(MilkTask.METADATA_KEY);
if(foundMetadata != null)
foundMetadata.mergeWith(metadata.getMergedValues());
else
task.metadata.add(metadata);
metadataService.synchronizeMetadata(task.task.getId(), task.metadata,
Criterion.or(MetadataCriteria.withKey(MilkTask.METADATA_KEY),
MetadataCriteria.withKey(MilkNote.METADATA_KEY),
MetadataCriteria.withKey(TagService.KEY))));
task.metadata.add(MilkTask.create(task));
for(Metadata metadata : task.metadata) {
metadata.setValue(Metadata.TASK, task.task.getId());
metadataDao.createNew(metadata);
}
MetadataCriteria.withKey(TagService.KEY)));
}
/**
@ -162,7 +163,7 @@ public final class MilkDataService {
// read tags, notes, etc
ArrayList<Metadata> metadata = new ArrayList<Metadata>();
TodorooCursor<Metadata> metadataCursor = metadataDao.query(Query.select(Metadata.PROPERTIES).
TodorooCursor<Metadata> metadataCursor = metadataService.query(Query.select(Metadata.PROPERTIES).
where(Criterion.and(MetadataCriteria.byTask(task.getId()),
Criterion.or(MetadataCriteria.withKey(TagService.KEY),
MetadataCriteria.withKey(MilkTask.METADATA_KEY),
@ -183,7 +184,7 @@ public final class MilkDataService {
* @return null if no metadata found
*/
public Metadata getTaskMetadata(long taskId) {
TodorooCursor<Metadata> cursor = metadataDao.query(Query.select(
TodorooCursor<Metadata> cursor = metadataService.query(Query.select(
MilkTask.LIST_ID, MilkTask.TASK_SERIES_ID, MilkTask.TASK_ID, MilkTask.REPEATING).where(
MetadataCriteria.byTaskAndwithKey(taskId, MilkTask.METADATA_KEY)));
try {
@ -200,7 +201,7 @@ public final class MilkDataService {
* Reads task notes out of a task
*/
public TodorooCursor<Metadata> getTaskNotesCursor(long taskId) {
TodorooCursor<Metadata> cursor = metadataDao.query(Query.select(Metadata.PROPERTIES).
TodorooCursor<Metadata> cursor = metadataService.query(Query.select(Metadata.PROPERTIES).
where(MetadataCriteria.byTaskAndwithKey(taskId, MilkNote.METADATA_KEY)));
return cursor;
}
@ -264,7 +265,7 @@ public final class MilkDataService {
}
// read all list counts
TodorooCursor<Metadata> cursor = metadataDao.query(Query.select(MilkTask.LIST_ID, COUNT).
TodorooCursor<Metadata> cursor = metadataService.query(Query.select(MilkTask.LIST_ID, COUNT).
join(Join.inner(Task.TABLE, Metadata.TASK.eq(Task.ID))).
where(Criterion.and(TaskCriteria.isVisible(), TaskCriteria.isActive())).
groupBy(MilkTask.LIST_ID));

2
astrid/plugin-src/com/todoroo/astrid/tags/TagFilterExposer.java
Unescape Escape View File

@ -111,7 +111,7 @@ public class TagFilterExposer extends BroadcastReceiver {
Filter[] filters = new Filter[sortedTagSet.size()];
int index = 0;
for(Tag tag : sortedTagSet) {
filters[index++] = filterFromTag(context, tag, TaskCriteria.isActive());
filters[index++] = filterFromTag(context, tag, TaskCriteria.activeAndVisible());
}
FilterCategory tagsFilter = new FilterCategory(context.getString(R.string.tag_FEx_by_size), filters);
list[2] = tagsFilter;

2
astrid/plugin-src/com/todoroo/astrid/timers/TimerPlugin.java
Unescape Escape View File

@ -52,7 +52,7 @@ public class TimerPlugin extends BroadcastReceiver {
task.getValue(Task.ELAPSED_SECONDS) + newElapsed);
}
}
PluginServices.getTaskService().save(task, true);
PluginServices.getTaskService().save(task);
TimerDecorationExposer.removeFromCache(task.getId());
// transmit new intents

2
astrid/src/com/todoroo/astrid/activity/EditPreferences.java
Unescape Escape View File

@ -130,7 +130,7 @@ public class EditPreferences extends TodorooPreferences {
for(int i = 0; i < 100; i++) {
task.clear();
task.setValue(Task.TITLE, Integer.toString(i));
taskService.save(task, false);
taskService.save(task);
}
dialogUtilities.okDialog(EditPreferences.this, "done", null);
return false;

16
astrid/src/com/todoroo/astrid/activity/ShortcutActivity.java
Unescape Escape View File

@ -26,10 +26,12 @@ import android.content.ContentValues;
import android.content.Intent;
import android.os.Bundle;
import com.timsu.astrid.R;
import com.todoroo.andlib.service.ContextManager;
import com.todoroo.andlib.sql.QueryTemplate;
import com.todoroo.andlib.utility.AndroidUtilities;
import com.todoroo.astrid.api.Filter;
import com.todoroo.astrid.model.Task;
/**
* This activity is launched when a user opens up a notification from the
@ -42,6 +44,9 @@ public class ShortcutActivity extends Activity {
// --- constants
/** token for passing a task id through extras for viewing a single task */
public static final String TOKEN_SINGLE_TASK = "id"; //$NON-NLS-1$
/** token for passing a {@link Filter}'s title through extras */
public static final String TOKEN_FILTER_TITLE = "title"; //$NON-NLS-1$
@ -105,13 +110,20 @@ public class ShortcutActivity extends Activity {
}
}
Filter filter = new Filter("", title, new QueryTemplate(), values); //$NON-NLS-1$
filter.sqlQuery = sql;
Filter filter = new Filter("", title, sql, values); //$NON-NLS-1$
Intent taskListIntent = new Intent(this, TaskListActivity.class);
taskListIntent.putExtra(TaskListActivity.TOKEN_FILTER, filter);
startActivity(taskListIntent);
} else if(extras != null && extras.containsKey(TOKEN_SINGLE_TASK)) {
Filter filter = new Filter("", getString(R.string.TLA_custom), //$NON-NLS-1$
new QueryTemplate().where(Task.ID.eq(extras.getLong(TOKEN_SINGLE_TASK, -1))), null);
Intent taskListIntent = new Intent(this, TaskListActivity.class);
taskListIntent.putExtra(TaskListActivity.TOKEN_FILTER, filter);
startActivity(taskListIntent);
}
finish();
}

57
astrid/src/com/todoroo/astrid/activity/TaskEditActivity.java
Unescape Escape View File

@ -104,13 +104,18 @@ public final class TaskEditActivity extends TabActivity {
/**
* Task ID
*/
public static final String TOKEN_ID = "i"; //$NON-NLS-1$
public static final String TOKEN_ID = "id"; //$NON-NLS-1$
/**
* Content Values to set
*/
public static final String TOKEN_VALUES = "v"; //$NON-NLS-1$
/**
* Task in progress (during orientation change)
*/
private static final String TASK_IN_PROGRESS = "task_in_progress"; //$NON-NLS-1$
// --- request codes
@SuppressWarnings("unused")
@ -185,6 +190,16 @@ public final class TaskEditActivity extends TabActivity {
// disable keyboard until user requests it
AndroidUtilities.suppressVirtualKeyboard(title);
// if we were editing a task already, restore it
if(savedInstanceState != null && savedInstanceState.containsKey(TASK_IN_PROGRESS)) {
Task task = savedInstanceState.getParcelable(TASK_IN_PROGRESS);
if(task != null) {
model = task;
}
}
setResult(RESULT_OK);
}
/* ======================================================================
@ -304,6 +319,12 @@ public final class TaskEditActivity extends TabActivity {
*/
@SuppressWarnings("nls")
protected void loadItem(Intent intent) {
if(model != null) {
// came from bundle
isNewTask = (model.getValue(Task.TITLE).length() == 0);
return;
}
long idParam = intent.getLongExtra(TOKEN_ID, -1L);
database.openForReading();
@ -323,6 +344,9 @@ public final class TaskEditActivity extends TabActivity {
if(model.getValue(Task.TITLE).length() == 0) {
FlurryAgent.onEvent("create-task");
isNewTask = true;
// set deletion date until task gets a title
model.setValue(Task.DELETION_DATE, DateUtilities.now());
} else {
FlurryAgent.onEvent("edit-task");
}
@ -354,7 +378,10 @@ public final class TaskEditActivity extends TabActivity {
for(TaskEditControlSet controlSet : controls)
controlSet.writeToModel(model);
if(taskService.save(model, false))
if(title.getText().length() > 0)
model.setValue(Task.DELETION_DATE, 0L);
if(taskService.save(model) && title.getText().length() > 0)
showSaveToast();
}
@ -362,11 +389,11 @@ public final class TaskEditActivity extends TabActivity {
public void finish() {
super.finish();
// abandon editing in this case
if(title.getText().length() == 0 && isNewTask) {
// abandon editing and delete the newly created task if
// no title was entered
if(title.getText().length() == 0 && isNewTask && model.isSaved()) {
taskService.delete(model);
showCancelToast();
setResult(RESULT_CANCELED);
}
}
@ -418,10 +445,6 @@ public final class TaskEditActivity extends TabActivity {
* precision
*/
private void showSaveToast() {
// if we have no title, or nothing's changed, don't show toast
if(isNewTask)
return;
int stringResource;
long due = model.getValue(Task.DUE_DATE);
@ -492,7 +515,6 @@ public final class TaskEditActivity extends TabActivity {
Toast.LENGTH_SHORT).show();
}
@Override
public boolean onMenuItemSelected(int featureId, MenuItem item) {
switch(item.getItemId()) {
@ -532,10 +554,11 @@ public final class TaskEditActivity extends TabActivity {
@Override
protected void onPause() {
if(shouldSaveState)
save();
super.onPause();
unregisterReceiver(controlReceiver);
if(shouldSaveState)
save();
}
@Override
@ -546,6 +569,14 @@ public final class TaskEditActivity extends TabActivity {
populateFields();
}
@Override
protected void onSaveInstanceState(Bundle outState) {
super.onSaveInstanceState(outState);
// stick our task into the outState
outState.putParcelable(TASK_IN_PROGRESS, model);
}
@Override
protected void onStart() {
super.onStart();

6
astrid/src/com/todoroo/astrid/activity/TaskListActivity.java
Unescape Escape View File

@ -290,7 +290,7 @@ public class TaskListActivity extends ListActivity implements OnScrollListener,
int importance = event.getNumber() - '1';
Task task = ((ViewHolder)selected.getTag()).task;
task.setValue(Task.IMPORTANCE, importance);
taskService.save(task, false);
taskService.save(task);
taskAdapter.setFieldContentsAndVisibility(selected);
}
// filter
@ -687,7 +687,7 @@ public class TaskListActivity extends ListActivity implements OnScrollListener,
}
task.mergeWith(forTask);
}
taskService.save(task, false);
taskService.save(task);
if(forMetadata != null && forMetadata.size() > 0) {
Metadata metadata = new Metadata();
metadata.setValue(Metadata.TASK, task.getId());
@ -831,7 +831,7 @@ public class TaskListActivity extends ListActivity implements OnScrollListener,
Task task = new Task();
task.setId(itemId);
task.setValue(Task.DELETION_DATE, 0L);
taskService.save(task, false);
taskService.save(task);
loadTaskListContent(true);
return true;
}

5
astrid/src/com/todoroo/astrid/dao/TaskDao.java
Unescape Escape View File

@ -146,12 +146,9 @@ public class TaskDao extends GenericDao<Task> {
* exist. Returns true on success.
*
* @param task
* @param skipHooks
* Whether pre and post hooks should run. This should be set
* to true if tasks are created as part of synchronization
* @return true if save occurred, false otherwise (i.e. nothing changed)
*/
public boolean save(Task task, boolean skipHooks) {
public boolean save(Task task) {
boolean saveSuccessful;
ContentValues values = task.getSetValues();

9
astrid/src/com/todoroo/astrid/service/StartupService.java
Unescape Escape View File

@ -20,8 +20,11 @@ import com.todoroo.andlib.service.ContextManager;
import com.todoroo.andlib.service.DependencyInjectionService;
import com.todoroo.andlib.service.ExceptionService;
import com.todoroo.andlib.service.ExceptionService.TodorooUncaughtExceptionHandler;
import com.todoroo.astrid.backup.BackupService;
import com.todoroo.astrid.dao.Database;
import com.todoroo.astrid.producteev.ProducteevBackgroundService;
import com.todoroo.astrid.producteev.ProducteevUtilities;
import com.todoroo.astrid.rmilk.MilkBackgroundService;
import com.todoroo.astrid.rmilk.MilkUtilities;
import com.todoroo.astrid.utility.Constants;
import com.todoroo.astrid.utility.Preferences;
@ -95,6 +98,8 @@ public class StartupService {
Preferences.setCurrentVersion(version);
}
upgradeService.performSecondaryUpgrade(context);
// perform startup activities in a background thread
new Thread(new Runnable() {
public void run() {
@ -117,6 +122,10 @@ public class StartupService {
MilkUtilities.stopOngoing();
ProducteevUtilities.INSTANCE.stopOngoing();
ProducteevBackgroundService.scheduleService();
BackupService.scheduleService(context);
MilkBackgroundService.scheduleService();
// check for task killers
if(!Constants.OEM)
showTaskKillerHelp(context);

11
astrid/src/com/todoroo/astrid/service/TaskService.java
Unescape Escape View File

@ -67,7 +67,7 @@ public class TaskService {
item.setValue(Task.COMPLETION_DATE, DateUtilities.now());
else
item.setValue(Task.COMPLETION_DATE, 0L);
taskDao.save(item, false);
taskDao.save(item);
}
/**
@ -78,8 +78,8 @@ public class TaskService {
* Whether pre and post hooks should run. This should be set
* to true if tasks are created as part of synchronization
*/
public boolean save(Task item, boolean runHooks) {
return taskDao.save(item, runHooks);
public boolean save(Task item) {
return taskDao.save(item);
}
/**
@ -122,12 +122,13 @@ public class TaskService {
return;
else if(item.containsValue(Task.TITLE) && item.getValue(Task.TITLE).length() == 0) {
taskDao.delete(item.getId());
item.setId(Task.NO_ID);
} else {
long id = item.getId();
item.clear();
item.setId(id);
item.setValue(Task.DELETION_DATE, DateUtilities.now());
taskDao.save(item, false);
taskDao.save(item);
}
}
@ -224,7 +225,7 @@ public class TaskService {
try {
for(cursor.moveToFirst(); !cursor.isAfterLast(); cursor.moveToNext()) {
taskValues.setValue(Task.ID, cursor.get(Task.ID));
taskDao.save(taskValues, false);
taskDao.save(taskValues);
}
return cursor.getCount();
} finally {

34
astrid/src/com/todoroo/astrid/service/UpgradeService.java
Unescape Escape View File

@ -11,18 +11,25 @@ import com.todoroo.andlib.service.Autowired;
import com.todoroo.andlib.service.DependencyInjectionService;
import com.todoroo.andlib.utility.DialogUtilities;
import com.todoroo.astrid.activity.TaskListActivity;
import com.todoroo.astrid.dao.Database;
public final class UpgradeService {
private static final int V3_2_4 = 151;
private static final int V3_2_3 = 150;
private static final int V3_1_0 = 146;
private static final int V3_0_6 = 145;
private static final int V3_0_5 = 144;
private static final int V3_0_0 = 136;
private static final int V2_14_4 = 135;
@Autowired
private DialogUtilities dialogUtilities;
@Autowired
private Database database;
public UpgradeService() {
DependencyInjectionService.getInstance().inject(this);
}
@ -102,6 +109,15 @@ public final class UpgradeService {
"If you liked the old version, you can also go back by " +
"<a href='http://bit.ly/oldastrid'>clicking here</a>",
});
if(from > V2_14_4 && from <= V3_2_4)
newVersionString(changeLog, "3.2.5 (8/18/10)", new String[] {
"Fix for duplicated tasks created in RTM",
});
if(from > V2_14_4 && from <= V3_2_3)
newVersionString(changeLog, "3.2.5 (8/18/10)", new String[] {
"Fix for duplicated tasks created in Producteev",
"Fix for being able to create tasks without title",
});
if(from > V2_14_4 && from <= V3_1_0)
newVersionString(changeLog, "3.2.0 (8/16/10)", new String[] {
"Build your own custom filters from the Filter page",
@ -110,7 +126,6 @@ public final class UpgradeService {
"Synchronize with Producteev! (producteev.com)",
"Select tags by drop-down box",
"Cosmetic improvements, calendar & sync bug fixes",
"... enjoy! - we <3 astrid team",
});
if(from > V2_14_4 && from <= V3_0_6)
newVersionString(changeLog, "3.1.0 (8/9/10)", new String[] {
@ -121,7 +136,6 @@ public final class UpgradeService {
"Restored tag hiding when tag begins with underscore (_)",
"FROYO: disabled moving app to SD card, it would break alarms and widget",
"Also gone: a couple force closes, bugs with repeating tasks",
"... enjoy! - we <3 astrid team",
});
if(from > V2_14_4 && from <= V3_0_5)
newVersionString(changeLog, "3.0.6 (8/4/10)", new String[] {
@ -134,7 +148,7 @@ public final class UpgradeService {
if(changeLog.length() == 0)
return;
changeLog.append("</body></html>");
changeLog.append("Enjoy!</body></html>");
String changeLogHtml = "<html><body style='color: white'>" + changeLog;
WebView webView = new WebView(context);
@ -163,6 +177,18 @@ public final class UpgradeService {
changeLog.append("</ul>");
}
// --- database upgrade logic
// --- secondary upgrade
/**
* If primary upgrade doesn't work for some reason (corrupt SharedPreferences,
* for example), this will catch some cases
*/
public void performSecondaryUpgrade(Context context) {
if(!context.getDatabasePath(database.getName()).exists() &&
context.getDatabasePath("tasks").exists()) { //$NON-NLS-1$
new Astrid2To3UpgradeHelper().upgrade2To3(context, 1);
}
}
}

2
astrid/src/com/todoroo/astrid/widget/TasksWidget.java
Unescape Escape View File

@ -188,7 +188,7 @@ public class TasksWidget extends AppWidgetProvider {
views.setOnClickPendingIntent(R.id.taskbody, pendingIntent);
Intent editIntent = new Intent(context, TaskEditActivity.class);
editIntent.setFlags(Intent.FLAG_ACTIVITY_NEW_TASK);
editIntent.setFlags(Intent.FLAG_ACTIVITY_NEW_TASK | Intent.FLAG_ACTIVITY_CLEAR_TOP);
if(filter != null && filter.valuesForNewTasks != null) {
String values = AndroidUtilities.contentValuesToSerializedString(filter.valuesForNewTasks);
editIntent.putExtra(TaskEditActivity.TOKEN_VALUES, values);

22
tests/src/com/todoroo/astrid/dao/TaskDaoTests.java
Unescape Escape View File

@ -32,7 +32,7 @@ public class TaskDaoTests extends DatabaseTestCase {
// create task "happy"
Task task = new Task();
task.setValue(Task.TITLE, "happy");
assertTrue(taskDao.save(task, false));
assertTrue(taskDao.save(task));
cursor = taskDao.query(
Query.select(IDS));
assertEquals(1, cursor.getCount());
@ -45,7 +45,7 @@ public class TaskDaoTests extends DatabaseTestCase {
// create task "sad"
task = new Task();
task.setValue(Task.TITLE, "sad");
assertTrue(taskDao.save(task, false));
assertTrue(taskDao.save(task));
cursor = taskDao.query(
Query.select(IDS));
assertEquals(2, cursor.getCount());
@ -55,7 +55,7 @@ public class TaskDaoTests extends DatabaseTestCase {
long sadId = task.getId();
assertNotSame(Task.NO_ID, sadId);
task.setValue(Task.TITLE, "melancholy");
assertTrue(taskDao.save(task, false));
assertTrue(taskDao.save(task));
cursor = taskDao.query(
Query.select(IDS));
assertEquals(2, cursor.getCount());
@ -75,35 +75,35 @@ public class TaskDaoTests extends DatabaseTestCase {
// create normal task
Task task = new Task();
task.setValue(Task.TITLE, "normal");
assertTrue(taskDao.save(task, false));
assertTrue(taskDao.save(task));
// create blank task
task = new Task();
task.setValue(Task.TITLE, "");
assertTrue(taskDao.save(task, false));
assertTrue(taskDao.save(task));
// create hidden task
task = new Task();
task.setValue(Task.TITLE, "hidden");
task.setValue(Task.HIDE_UNTIL, DateUtilities.now() + 10000);
assertTrue(taskDao.save(task, false));
assertTrue(taskDao.save(task));
// create task with deadlines
task = new Task();
task.setValue(Task.TITLE, "deadlineInFuture");
task.setValue(Task.DUE_DATE, DateUtilities.now() + 10000);
assertTrue(taskDao.save(task, false));
assertTrue(taskDao.save(task));
task = new Task();
task.setValue(Task.TITLE, "deadlineInPast");
task.setValue(Task.DUE_DATE, DateUtilities.now() - 10000);
assertTrue(taskDao.save(task, false));
assertTrue(taskDao.save(task));
// create completed task
task = new Task();
task.setValue(Task.TITLE, "completed");
task.setValue(Task.COMPLETION_DATE, DateUtilities.now() - 10000);
assertTrue(taskDao.save(task, false));
assertTrue(taskDao.save(task));
// check has no name
TodorooCursor<Task> cursor = taskDao.query(
@ -165,7 +165,7 @@ public class TaskDaoTests extends DatabaseTestCase {
// create task "happy"
Task task = new Task();
task.setValue(Task.TITLE, "happy");
assertTrue(taskDao.save(task, false));
assertTrue(taskDao.save(task));
cursor = taskDao.query(
Query.select(IDS));
assertEquals(1, cursor.getCount());
@ -191,7 +191,7 @@ public class TaskDaoTests extends DatabaseTestCase {
task.setValue(Task.TITLE, "happy");
task.setValue(Task.ID, 1L);
assertFalse(taskDao.save(task, false));
assertFalse(taskDao.save(task));
cursor = taskDao.query(
Query.select(IDS));

2
tests/src/com/todoroo/astrid/model/TaskTests.java
Unescape Escape View File

@ -46,7 +46,7 @@ public class TaskTests extends DatabaseTestCase {
/** Check task gets a creation date at some point */
public void checkCreationDate() {
Task task = new Task();
taskService.save(task, false);
taskService.save(task);
assertTrue(task.getValue(Task.CREATION_DATE) > 0);
}

16
tests/src/com/todoroo/astrid/reminders/ReminderServiceTests.java
Unescape Escape View File

@ -37,7 +37,7 @@ public class ReminderServiceTests extends DatabaseTestCase {
Task task = new Task();
task.setValue(Task.TITLE, "water");
task.setValue(Task.REMINDER_FLAGS, 0);
taskDao.save(task, false);
taskDao.save(task);
service.scheduleAlarm(task);
}
@ -49,7 +49,7 @@ public class ReminderServiceTests extends DatabaseTestCase {
task.setValue(Task.TITLE, "water");
task.setValue(Task.DUE_DATE, DateUtilities.now() - DateUtilities.ONE_DAY);
task.setValue(Task.REMINDER_FLAGS, Task.NOTIFY_AT_DEADLINE);
taskDao.save(task, false);
taskDao.save(task);
// test due date in the future
task.setValue(Task.DUE_DATE, DateUtilities.now() + DateUtilities.ONE_DAY);
@ -61,7 +61,7 @@ public class ReminderServiceTests extends DatabaseTestCase {
assertEquals(type, ReminderService.TYPE_DUE);
}
});
taskDao.save(task, false);
taskDao.save(task);
assertTrue(((AlarmExpected)service.getScheduler()).alarmCreated);
}
@ -80,7 +80,7 @@ public class ReminderServiceTests extends DatabaseTestCase {
assertEquals(type, ReminderService.TYPE_RANDOM);
}
});
taskDao.save(task, false);
taskDao.save(task);
assertTrue(((AlarmExpected)service.getScheduler()).alarmCreated);
}
@ -92,7 +92,7 @@ public class ReminderServiceTests extends DatabaseTestCase {
task.setValue(Task.TITLE, "water");
task.setValue(Task.DUE_DATE, DateUtilities.now() + DateUtilities.ONE_DAY);
task.setValue(Task.REMINDER_FLAGS, Task.NOTIFY_AFTER_DEADLINE);
taskDao.save(task, false);
taskDao.save(task);
// test due date in the past
task.setValue(Task.DUE_DATE, DateUtilities.now() - DateUtilities.ONE_DAY);
@ -105,7 +105,7 @@ public class ReminderServiceTests extends DatabaseTestCase {
assertEquals(type, ReminderService.TYPE_OVERDUE);
}
});
taskDao.save(task, false);
taskDao.save(task);
assertTrue(((AlarmExpected)service.getScheduler()).alarmCreated);
}
@ -126,7 +126,7 @@ public class ReminderServiceTests extends DatabaseTestCase {
assertEquals(type, ReminderService.TYPE_RANDOM);
}
});
taskDao.save(task, false);
taskDao.save(task);
assertTrue(((AlarmExpected)service.getScheduler()).alarmCreated);
// now set the due date in the past
@ -145,7 +145,7 @@ public class ReminderServiceTests extends DatabaseTestCase {
assertEquals(type, ReminderService.TYPE_DUE);
}
});
taskDao.save(task, false);
taskDao.save(task);
assertTrue(((AlarmExpected)service.getScheduler()).alarmCreated);
}

28
tests/src/com/todoroo/astrid/repeats/RepeatTests.java
Unescape Escape View File

@ -36,10 +36,10 @@ public class RepeatTests extends DatabaseTestCase {
public void testNoRepeats() throws Exception{
Task task = new Task();
task.setValue(Task.TITLE, "nothing");
taskDao.save(task, false);
taskDao.save(task);
task.setValue(Task.COMPLETION_DATE, DateUtilities.now());
taskDao.save(task, false);
taskDao.save(task);
// wait for repeat handler
Thread.sleep(REPEAT_WAIT);
@ -60,10 +60,10 @@ public class RepeatTests extends DatabaseTestCase {
rrule.setInterval(5);
rrule.setFreq(Frequency.DAILY);
task.setValue(Task.RECURRENCE, rrule.toIcal());
taskDao.save(task, false);
taskDao.save(task);
task.setValue(Task.COMPLETION_DATE, DateUtilities.now());
taskDao.save(task, false);
taskDao.save(task);
// wait for repeat handler
Thread.sleep(REPEAT_WAIT);
@ -102,10 +102,10 @@ public class RepeatTests extends DatabaseTestCase {
task.setValue(Task.RECURRENCE, rrule.toIcal());
long originalDueDate = (DateUtilities.now() - 3 * DateUtilities.ONE_DAY) / 1000L * 1000L;
task.setValue(Task.DUE_DATE, task.createDueDate(Task.URGENCY_SPECIFIC_DAY_TIME, originalDueDate));
taskDao.save(task, false);
taskDao.save(task);
task.setValue(Task.COMPLETION_DATE, DateUtilities.now());
taskDao.save(task, false);
taskDao.save(task);
// wait for repeat handler
Thread.sleep(REPEAT_WAIT);
@ -144,10 +144,10 @@ public class RepeatTests extends DatabaseTestCase {
task.setValue(Task.RECURRENCE, rrule.toIcal());
long originalDueDate = (DateUtilities.now() + DateUtilities.ONE_DAY) / 1000L * 1000L;
task.setValue(Task.DUE_DATE, task.createDueDate(Task.URGENCY_SPECIFIC_DAY_TIME, originalDueDate));
taskDao.save(task, false);
taskDao.save(task);
task.setValue(Task.COMPLETION_DATE, DateUtilities.now());
taskDao.save(task, false);
taskDao.save(task);
// wait for repeat handler
Thread.sleep(2 * REPEAT_WAIT);
@ -187,10 +187,10 @@ public class RepeatTests extends DatabaseTestCase {
long originalDueDate = (DateUtilities.now() - 3 * DateUtilities.ONE_DAY) / 1000L * 1000L;
task.setValue(Task.DUE_DATE, task.createDueDate(Task.URGENCY_SPECIFIC_DAY, originalDueDate));
task.setFlag(Task.FLAGS, Task.FLAG_REPEAT_AFTER_COMPLETION, true);
taskDao.save(task, false);
taskDao.save(task);
task.setValue(Task.COMPLETION_DATE, DateUtilities.now());
taskDao.save(task, false);
taskDao.save(task);
// wait for repeat handler
Thread.sleep(REPEAT_WAIT);
@ -226,7 +226,7 @@ public class RepeatTests extends DatabaseTestCase {
rrule.setInterval(5);
rrule.setFreq(Frequency.DAILY);
task.setValue(Task.RECURRENCE, rrule.toIcal());
taskDao.save(task, false);
taskDao.save(task);
Metadata metadata = new Metadata();
metadata.setValue(Metadata.KEY, "special");
@ -235,7 +235,7 @@ public class RepeatTests extends DatabaseTestCase {
metadataDao.persist(metadata);
task.setValue(Task.COMPLETION_DATE, DateUtilities.now());
taskDao.save(task, false);
taskDao.save(task);
// wait for repeat handler
Thread.sleep(REPEAT_WAIT);
@ -266,10 +266,10 @@ public class RepeatTests extends DatabaseTestCase {
task.setValue(Task.RECURRENCE, rrule.toIcal());
task.setValue(Task.DUE_DATE, task.createDueDate(Task.URGENCY_TODAY, 0));
task.setValue(Task.HIDE_UNTIL, task.createHideUntil(Task.HIDE_UNTIL_DAY_BEFORE, 0));
taskDao.save(task, false);
taskDao.save(task);
task.setValue(Task.COMPLETION_DATE, DateUtilities.now());
taskDao.save(task, false);
taskDao.save(task);
// wait for repeat handler
Thread.sleep(REPEAT_WAIT);

Write Preview
Loading…
Cancel
Save