From bad2f6f5b329d0e17e1df2d6b975ab8e13045922 Mon Sep 17 00:00:00 2001 From: Tyler Renelle Date: Thu, 11 Oct 2012 11:07:16 -0400 Subject: [PATCH] Revert "Just pass in task object to taskClasses view helper", this was breaking deleting tasks - revisit later This reverts commit d04140977f4994d86a4aa0c45eafd98f340908b1. --- lib/app/helpers.js | 5 ++--- src/app/helpers.coffee | 4 ++-- views/app/index.html | 2 +- 3 files changed, 5 insertions(+), 6 deletions(-) diff --git a/lib/app/helpers.js b/lib/app/helpers.js index cbbb0d7313..046e745234 100644 --- a/lib/app/helpers.js +++ b/lib/app/helpers.js @@ -19,9 +19,8 @@ module.exports.dayMapping = dayMapping = { }; module.exports.viewHelpers = function(view) { - view.fn('taskClasses', function(task) { - var classes, completed, repeat, type, value; - type = task.type, completed = task.completed, value = task.value, repeat = task.repeat; + view.fn('taskClasses', function(type, completed, value, repeat) { + var classes; classes = type; if (completed || (repeat && repeat[dayMapping[moment().day()]] === false)) { classes += " completed"; diff --git a/src/app/helpers.coffee b/src/app/helpers.coffee index 69a65ff4d8..29b0bc270f 100644 --- a/src/app/helpers.coffee +++ b/src/app/helpers.coffee @@ -7,8 +7,8 @@ module.exports.daysBetween = (a, b) -> module.exports.dayMapping = dayMapping = {0:'su',1:'m',2:'t',3:'w',4:'th',5:'f',6:'s',7:'su'} module.exports.viewHelpers = (view) -> - view.fn 'taskClasses', (task) -> - {type, completed, value, repeat} = task + view.fn 'taskClasses', (type, completed, value, repeat) -> + #TODO figure out how to just pass in the task model, so i can access all these properties from one object classes = type # show as completed if completed (naturally) or not required for today diff --git a/views/app/index.html b/views/app/index.html index 6721d36516..131d0e64b7 100644 --- a/views/app/index.html +++ b/views/app/index.html @@ -271,7 +271,7 @@ -
  • +
  •