Refactor

common/script/content.coffee

@@ -67,6 +67,7 @@ api.itemList =
   'hatchingPotions' : {localeKey: 'hatchingPotion', isEquipment: false}
   'eggs' : {localeKey: 'eggSingular', isEquipment: false}
   'quests' : {localeKey: 'quest', isEquipment: false}
+  'food' : {localeKey: 'foodText', isEquipment: false}
   'Saddle' : {localeKey: 'foodSaddleText', isEquipment: false}
 
 gear =
@@ -938,6 +939,7 @@ api.dropEggs =
   Dragon:           text: t('dropEggDragonText'), adjective: t('dropEggDragonAdjective')
   Cactus:           text: t('dropEggCactusText'), adjective: t('dropEggCactusAdjective')
   BearCub:          text: t('dropEggBearCubText'),  mountText: t('dropEggBearCubMountText'), adjective: t('dropEggBearCubAdjective')
+
 _.each api.dropEggs, (egg,key) ->
   _.defaults egg,
     canBuy:true

website/public/js/controllers/inventoryCtrl.js

@@ -186,6 +186,7 @@ habitrpg.controller("InventoryCtrl",
       });
       $scope.closeQuest();
     }
+
     $scope.buyQuest = function(quest) {
       var item = Content.quests[quest];
       if (item.lvl && item.lvl > user.stats.lvl)

website/public/js/controllers/rootCtrl.js

@@ -210,38 +210,49 @@ habitrpg.controller("RootCtrl", ['$scope', '$rootScope', '$location', 'User', '$
       return filteredArray;
     }
 
+    // @TODO: Extract equip and purchase into equipment service
+    $rootScope.equip = function(itemKey) {
+      var equipType = user.preferences.costume ? 'costume' : 'equipped';
+      var equipParams = {
+         type: equipType,
+         key: itemKey
+      };
+
+      user.ops.equip({ params: equipParams });
+    }
+
     $rootScope.purchase = function(type, item){
       if (type == 'special') return user.ops.buySpecialSpell({params:{key:item.key}});
 
       var gems = user.balance * 4;
-      var itemName = _getLocalekeyForItem(type);
+      var price = item.value;
 
-      var price = ((((item.specialClass == "wizard") && (item.type == "weapon")) || item.gearSet == "animal") + 1);
+      var itemName = window.env.t(Content.itemList[type].localeKey)
 
-      if (itemList[type].isEquipment) {
-        if (user.items.gear.owned[item.key]) {
-          var equipType = user.preferences.costume ? 'costume' : 'equipped';
-          return user.ops.equip({params:{type: equipType, key: item.key}});
+      if (Content.itemList[type].isEquipment) {
+        var eligibleForPurchase = _canBuyEquipment(item.key);
+        if (!eligibleForPurchase) return false;
+
+        // @TODO: Attach gemValue to content so we don't have to do this
+        price = ((((item.specialClass == "wizard") && (item.type == "weapon")) || item.gearSet == "animal") + 1);
+        type = 'gear';
       }
+
       if (gems < price) return $rootScope.openModal('buyGems');
-        var message = window.env.t('buyThis', {text: itemName, price: price, gems: gems})
-        if($window.confirm(message))
-          user.ops.purchase({params:{type:"gear",key:item.key}});
-      } else {
-        if(gems < item.value) return $rootScope.openModal('buyGems');
-        var message = window.env.t('buyThis', {text: itemName, price: item.value, gems: gems})
+
+      var message = window.env.t('buyThis', {text: itemName, price: price, gems: gems});
       if ($window.confirm(message))
         user.ops.purchase({params:{type:type,key:item.key}});
     }
-    }
 
-    function _getLocalekeyForItem(type) {
-      var itemName = type;
-      if (Content.itemList[type]) {
-        itemName = window.env.t(Content.itemList[type].localeKey)
+    function _canBuyEquipment(itemKey) {
+      if (user.items.gear.owned[itemKey]) {
+        $window.alert(window.env.t('messageAlreadyOwnGear'));
+      } else if (user.items.gear.owned[itemKey] === false) {
+        $window.alert(window.env.t('messageAlreadyPurchasedGear'));
+      } else {
+        return true;
       }
-
-      return itemName;
     }
 
     /*

website/views/options/profile.jade

@@ -62,7 +62,11 @@ mixin customizeProfile(mobile)
           span(ng-hide='#{showPath("user.items.gear.owned", gearGroup("animal"), "&&")}')
             +gemCost(2)
               button.btn.btn-xs(ng-click='#{unlockPath("items.gear.owned", gearGroup("animal"))}')!= env.t('unlockSet', {cost: 5}) + ' <span class="Pet_Currency_Gem1x inline-gems"/>'
-          button.customize-option(ng-repeat='item in ::getGearArray("animal")' ng-class="{locked: user.items.gear.owned[item.key] == undefined, selectableInventory: user.preferences.costume ? user.items.gear.costume.headAccessory == item.key : user.items.gear.equipped.headAccessory == item.key}", popover='{{::item.notes()}}', popover-title='{{::item.text()}}', popover-trigger='mouseenter', popover-placement='right', popover-append-to-body='true', ng-click='purchase(item.type,item)', class='{{::item.key}}')
+          button.customize-option(ng-repeat='item in ::getGearArray("animal")', class='{{::item.key}}',
+            ng-class="{locked: user.items.gear.owned[item.key] == undefined, selectableInventory: user.preferences.costume ? user.items.gear.costume.headAccessory == item.key : user.items.gear.equipped.headAccessory == item.key}",
+            popover='{{::item.notes()}}', popover-title='{{::item.text()}}', popover-trigger='mouseenter',
+            popover-placement='right', popover-append-to-body='true',
+            ng-click='user.items.gear.owned[item.key] ? equip(item.key) : purchase(item.type,item)')
 
     .col-md-4
         h3(class=mobile?'item item-divider':'')=env.t('bodyHead')