sudoxreboot/habitica
1
0
Fork 0
mirror of https://github.com/sudoxnym/habitica.git synced 2026-08-01 23:40:25 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

prevent long-standing red tasks from being deleted by hurting on delete

Browse source
This commit is contained in:
Tyler Renelle 2012-09-01 19:10:51 -04:00
parent 7a8a7d4566
commit 1dea15f551
2 changed files with 26 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

16
lib/app/index.js
View file

@ -184,8 +184,22 @@ ready(function(model) {
}
};
exports.del = function(e, el) {
var task;
var history, result, task;
task = model.at(e.target);
history = task.get('history');
if (history && history.length > 3 && task.get('value') < 0) {
result = confirm("Are you sure? Deleting this task will hurt you (to prevent deleting, then re-creating red tasks).");
if (result !== true) {
return;
} else {
task.set('type', 'habit');
scoring.score({
user: model.at('_user'),
task: task,
direction: 'down'
});
}
}
model.del('_user.tasks.' + task.get('id'));
return task.remove();
};

11
src/app/index.coffee
View file

@ -153,6 +153,17 @@ ready (model) ->
exports.del = (e, el) ->
# Derby extends model.at to support creation from DOM nodes
task = model.at(e.target)
# prevent long-standing red tasks from being deleted
history = task.get('history')
if history and history.length>3 and task.get('value')<0
result = confirm("Are you sure? Deleting this task will hurt you (to prevent deleting, then re-creating red tasks).")
if result != true
return # Cancel. Don't delete, don't hurt user
else
task.set('type','habit') # hack to make sure it hits HP, instead of performing "undo checkbox"
scoring.score({user:model.at('_user'), task:task, direction:'down'})
#TODO bug where I have to delete from _users.tasks AND _{type}List,
# fix when query subscriptions implemented properly
model.del('_user.tasks.'+task.get('id'))