From 7a5a4819e6e80ee7d36e1a2180099d2deef0976c Mon Sep 17 00:00:00 2001 From: Tyler Renelle Date: Mon, 25 Mar 2013 09:42:30 -0600 Subject: [PATCH] pets: bug fix - user.remove(path, idx, count) was wiping the whole eggs/food arrays on page refresh, using array.splice & user.set('items.food', splicedFood), etc --- src/app/pets.coffee | 11 +++++++++-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/src/app/pets.coffee b/src/app/pets.coffee index 69e6308ec8..d477927b6e 100644 --- a/src/app/pets.coffee +++ b/src/app/pets.coffee @@ -38,8 +38,15 @@ module.exports.app = (appExports, model) -> return alert "You already have that pet." if user.get('items.pets').indexOf("#{egg.name}-#{foodName}") != -1 user.push 'items.pets', egg.name + '-' + foodName - user.remove 'items.food', foodIdx, 1 - user.remove 'items.eggs', eggIdx, 1 + + eggs.splice eggIdx, 1 + myFood.splice foodIdx, 1 + user.set 'items.eggs', eggs + user.set 'items.food', myFood + + #FIXME Bug: this removes from the array properly in the browser, but on refresh is has removed all items from the arrays +# user.remove 'items.food', foodIdx, 1 +# user.remove 'items.eggs', eggIdx, 1 appExports.choosePet = (e, el) -> petArray = $(el).attr('data-pet').split '-'