use model events to handle stats notifications. not perfect, need to

fix handling of "undo exp/gp" operation
2025-12-19 15:48:04 +01:00 · 2012-09-24 08:48:05 -04:00
parent 0ba7ccd370
commit d929905c48
6 changed files with 95 additions and 59 deletions
--- a/lib/app/index.js
+++ b/lib/app/index.js
@@ -53,7 +53,7 @@ get('/:uidParam?', function(page, model, _arg, next) {

 ready(function(model) {
  var setupSortable, step, tour, type, _i, _j, _len, _len1, _ref1, _ref2;
-  scoring.setUser(model.at('_user'));
+  scoring.setModel(model);
  $('[rel=tooltip]').tooltip();
  $('[rel=popover]').popover();
  model.on('set', '*', function() {
--- a/lib/app/scoring.js
+++ b/lib/app/scoring.js
@@ -1,5 +1,5 @@
 // Generated by CoffeeScript 1.3.3
-var MODIFIER, content, cron, expModifier, helpers, hpModifier, score, setUser, statsNotification, tally, updateStats, user;
+var MODIFIER, content, cron, expModifier, helpers, hpModifier, model, score, setModel, setupNotifications, tally, updateStats, user;

 content = require('./content');

@@ -9,22 +9,53 @@ MODIFIER = .03;

 user = void 0;

-setUser = function(u) {
-  return user = u;
+model = void 0;
+
+setModel = function(m) {
+  model = m;
+  user = model.at('_user');
+  return setupNotifications();
 };

-statsNotification = function(html, type) {
-  if (user.get('stats.lvl') === 0) {
-    return;
-  }
-  return $.bootstrapGrowl(html, {
-    type: type,
-    top_offset: 20,
-    align: 'right',
-    width: 250,
-    delay: 3000,
-    allow_dismiss: true,
-    stackup_spacing: 10
+setupNotifications = function() {
+  var statsNotification;
+  statsNotification = function(html, type) {
+    if (user.get('stats.lvl') === 0) {
+      return;
+    }
+    return $.bootstrapGrowl(html, {
+      type: type,
+      top_offset: 20,
+      align: 'right',
+      width: 250,
+      delay: 3000,
+      allow_dismiss: true,
+      stackup_spacing: 10
+    });
+  };
+  user.on('set', 'stats.hp', function(captures, args, out, isLocal, passed) {
+    var num, rounded;
+    num = captures - args;
+    rounded = Math.abs(num.toFixed(1));
+    if (num < 0) {
+      return statsNotification("<i class='icon-heart'></i>HP -" + rounded, 'error');
+    }
+  });
+  user.on('set', 'stats.money', function(captures, args, out, isLocal, passed) {
+    var num, rounded;
+    num = captures - args;
+    rounded = Math.abs(num.toFixed(1));
+    if (num < 0) {
+      return statsNotification("<i class='icon-star'></i>GP -" + rounded, 'success');
+    } else if (num > 0) {
+      num = Math.abs(num);
+      return statsNotification("<i class='icon-star'></i>Exp,GP +" + rounded, 'success');
+    }
+  });
+  return user.on('set', 'stats.lvl', function(captures, args, out, isLocal, passed) {
+    if (captures > args) {
+      return statsNotification('<i class="icon-chevron-up"></i> Level Up!', 'info');
+    }
  });
 };

@@ -64,7 +95,6 @@ updateStats = function(stats) {
      stats.exp -= tnl;
      user.set('stats.lvl', user.get('stats.lvl') + 1);
      user.set('stats.hp', 50);
-      statsNotification('<i class="icon-chevron-up"></i> Level Up!', 'info');
    }
    if (!user.get('items.itemsEnabled') && stats.exp >= 15) {
      user.set('items.itemsEnabled', true);
@@ -138,10 +168,8 @@ score = function(spec) {
  if (type === 'reward') {
    money -= task.get('value');
    num = parseFloat(task.get('value')).toFixed(2);
-    statsNotification("<i class='icon-star'></i>GP -" + num, 'success');
    if (money < 0) {
      hp += money;
-      statsNotification("<i class='icon-heart'></i>HP " + (money.toFixed(2)), 'error');
      money = 0;
    }
  }
@@ -149,15 +177,9 @@ score = function(spec) {
    modified = expModifier(delta);
    exp += modified;
    money += modified;
-    if (modified > 0) {
-      statsNotification("<i class='icon-star'></i>Exp,GP +" + (modified.toFixed(2)), 'success');
-    } else {
-      statsNotification("<i class='icon-star'></i>Exp,GP " + (modified.toFixed(2)), 'warning');
-    }
  } else if (type !== 'reward' && type !== 'todo') {
    modified = hpModifier(delta);
    hp += modified;
-    statsNotification("<i class='icon-heart'></i>HP " + (modified.toFixed(2)), 'error');
  }
  updateStats({
    hp: hp,
@@ -248,7 +270,7 @@ tally = function(momentDate) {

 module.exports = {
  MODIFIER: MODIFIER,
-  setUser: setUser,
+  setModel: setModel,
  score: score,
  cron: cron
 };
--- a/lib/server/serverRoutes.js
+++ b/lib/server/serverRoutes.js
@@ -12,7 +12,7 @@ module.exports = function(expressApp, root, derby) {
      if (err || !user.get()) {
        return;
      }
-      scoring.setUser(user);
+      scoring.setModel(model);
      return scoring.score({
        direction: 'up'
      });
@@ -27,7 +27,7 @@ module.exports = function(expressApp, root, derby) {
      if (err || !user.get()) {
        return;
      }
-      scoring.setUser(user);
+      scoring.setModel(model);
      return scoring.score({
        direction: 'down'
      });
--- a/src/app/index.coffee
+++ b/src/app/index.coffee
@@ -51,7 +51,7 @@ get '/:uidParam?', (page, model, {uidParam}, next) ->
 # ========== CONTROLLER FUNCTIONS ==========

 ready (model) ->
-  scoring.setUser(model.at('_user'))
+  scoring.setModel(model)
    
  $('[rel=tooltip]').tooltip()
  $('[rel=popover]').popover()
--- a/src/app/scoring.coffee
+++ b/src/app/scoring.coffee
@@ -2,25 +2,50 @@ content = require('./content')
 helpers = require('./helpers')
 MODIFIER = .03 # each new level, armor, weapon add 3% modifier (this number may change) 
 user = undefined
+model = undefined

 # This is required by all the functions, make sure it's set before anythign else is called
-setUser = (u) ->
-  user = u
-
-# FIXME move to index.coffee as module.on('set','*')
-statsNotification = (html, type) ->
-  #don't show notifications if user dead
-  return if user.get('stats.lvl') == 0
+setModel = (m) ->
+  model = m
+  user = model.at('_user')
+  setupNotifications()
  
-  $.bootstrapGrowl html, {
-    type: type # (null, 'info', 'error', 'success')
-    top_offset: 20
-    align: 'right' # ('left', 'right', or 'center')
-    width: 250 # (integer, or 'auto')
-    delay: 3000
-    allow_dismiss: true
-    stackup_spacing: 10 # spacing between consecutive stacecked growls.
-  }
+setupNotifications = ->
+  statsNotification = (html, type) ->
+    #don't show notifications if user dead
+    return if user.get('stats.lvl') == 0
+    $.bootstrapGrowl html, {
+      type: type # (null, 'info', 'error', 'success')
+      top_offset: 20
+      align: 'right' # ('left', 'right', or 'center')
+      width: 250 # (integer, or 'auto')
+      delay: 3000
+      allow_dismiss: true
+      stackup_spacing: 10 # spacing between consecutive stacecked growls.
+    }
+    
+  # Setup listeners which trigger notifications
+  user.on 'set', 'stats.hp', (captures, args, out, isLocal, passed) ->
+    num = captures - args
+    rounded = Math.abs(num.toFixed(1))
+    if num < 0
+      statsNotification "<i class='icon-heart'></i>HP -#{rounded}", 'error' # lost hp from purchase
+    
+  user.on 'set', 'stats.money', (captures, args, out, isLocal, passed) ->
+    num = captures - args
+    rounded = Math.abs(num.toFixed(1))
+    # made purchase
+    if num < 0
+      # FIXME use 'warning' when unchecking an accidently completed daily/todo, and notify of exp too
+      statsNotification "<i class='icon-star'></i>GP -#{rounded}", 'success'
+    # gained money (and thereby exp)
+    else if num > 0
+      num = Math.abs(num)
+      statsNotification "<i class='icon-star'></i>Exp,GP +#{rounded}", 'success'
+    
+  user.on 'set', 'stats.lvl', (captures, args, out, isLocal, passed) ->
+    if captures > args
+      statsNotification('<i class="icon-chevron-up"></i> Level Up!', 'info')
  
 # Calculates Exp modification based on weapon & lvl
 expModifier = (value) ->
@@ -57,7 +82,6 @@ updateStats = (stats) ->
      stats.exp -= tnl
      user.set 'stats.lvl', user.get('stats.lvl') + 1
      user.set 'stats.hp', 50
-      statsNotification('<i class="icon-chevron-up"></i> Level Up!', 'info')
    if !user.get('items.itemsEnabled') and stats.exp >=15
      user.set 'items.itemsEnabled', true
      $('ul.items').popover
@@ -87,11 +111,9 @@ score = (spec = {task:null, direction:null, cron:null}) ->
      modified = expModifier(1)
      money += modified
      exp += modified
-      # statsNotification "<i class='icon-star'></i>Exp,GP +#{modified.toFixed(2)}", 'success'
    else
      modified = hpModifier(1)
      hp -= modified
-      # statsNotification "<i class='icon-heart'></i>HP #{modified.toFixed(2)}", 'error'
    updateStats({hp: hp, exp: exp, money: money})
    return
    
@@ -123,11 +145,9 @@ score = (spec = {task:null, direction:null, cron:null}) ->
    # purchase item
    money -= task.get('value')
    num = parseFloat(task.get('value')).toFixed(2)
-    statsNotification "<i class='icon-star'></i>GP -#{num}", 'success'
    # if too expensive, reduce health & zero money
    if money < 0
      hp += money# hp - money difference
-      statsNotification "<i class='icon-heart'></i>HP #{money.toFixed(2)}", 'error'
      money = 0
      
  # Add points to exp & money if positive delta
@@ -136,16 +156,10 @@ score = (spec = {task:null, direction:null, cron:null}) ->
    modified = expModifier(delta)
    exp += modified
    money += modified
-    if modified > 0
-      statsNotification "<i class='icon-star'></i>Exp,GP +#{modified.toFixed(2)}", 'success'
-    else
-      # unchecking an accidently completed daily/todo
-      statsNotification "<i class='icon-star'></i>Exp,GP #{modified.toFixed(2)}", 'warning'
  # Deduct from health (rewards case handled above)
  else unless type in ['reward', 'todo']
    modified = hpModifier(delta)
    hp += modified
-    statsNotification "<i class='icon-heart'></i>HP #{modified.toFixed(2)}", 'error'

  updateStats({hp: hp, exp: exp, money: money})
  
@@ -198,7 +212,7 @@ tally = (momentDate) ->

 module.exports = {
  MODIFIER: MODIFIER
-  setUser: setUser
+  setModel: setModel
  score: score
  cron: cron
 }
--- a/src/server/serverRoutes.coffee
+++ b/src/server/serverRoutes.coffee
@@ -7,7 +7,7 @@ module.exports = (expressApp, root, derby) ->
    model = req.getModel()
    model.fetch "users.#{req.params.uid}", (err, user) ->
      return if err || !user.get()
-      scoring.setUser(user)
+      scoring.setModel(model)
      scoring.score({direction:'up'})
    res.send(200)

@@ -16,7 +16,7 @@ module.exports = (expressApp, root, derby) ->
    model = req.getModel()
    model.fetch "users.#{req.params.uid}", (err, user) ->
      return if err || !user.get()
-      scoring.setUser(user)
+      scoring.setModel(model)
      scoring.score({direction:'down'})
    res.send(200)