From 0de27373ad7e856bf6697d7c540ee192124ccd74 Mon Sep 17 00:00:00 2001 From: Tyler Renelle Date: Sun, 14 Apr 2013 21:44:30 -0400 Subject: [PATCH] rest bug - lastCron must be set before "return if resting", otherwise when they unrest they'll get piled by all the days they missed --- src/app/scoring.coffee | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/app/scoring.coffee b/src/app/scoring.coffee index ab4b30f331..5aa082c55a 100644 --- a/src/app/scoring.coffee +++ b/src/app/scoring.coffee @@ -258,13 +258,14 @@ updateStats = (model, newStats, batch) -> cron = (model) -> user = model.at '_user' today = +new Date - return if user.get 'flags.rest' daysPassed = helpers.daysBetween(user.get('lastCron'), today, user.get('preferences.dayStart')) if daysPassed > 0 + user.set 'lastCron', today # at least this must be called, even if user resting + return if user.get 'flags.rest' + batch = new character.BatchUpdate(model) batch.startTransaction() - batch.set 'lastCron', today obj = batch.obj() hpBefore = obj.stats.hp #we'll use this later so we can animate hp loss # Tally each task