Skip to content

Commit

Permalink
Bug 737968: Weapons looted while under the firepotion effect are corr…
Browse files Browse the repository at this point in the history
…ectly saved
  • Loading branch information
sork committed Mar 23, 2012
1 parent d51a4cb commit e0ab40c
Show file tree
Hide file tree
Showing 4 changed files with 18 additions and 6 deletions.
1 change: 1 addition & 0 deletions .gitignore
Expand Up @@ -4,3 +4,4 @@ build.txt
config_build.json
config_local*
client/audio
.DS_Store
8 changes: 3 additions & 5 deletions client/js/game.js
Expand Up @@ -1051,11 +1051,9 @@ function(InfoManager, BubbleManager, Renderer, Map, Animation, Sprite, AnimatedT
});

self.player.onSwitchItem(function() {
if(self.player.getSpriteName() !== "firefox") {
self.storage.savePlayer(self.renderer.getPlayerImage(),
self.player.getSpriteName(),
self.player.getWeaponName());
}
self.storage.savePlayer(self.renderer.getPlayerImage(),
self.player.getArmorName(),
self.player.getWeaponName());
if(self.equipment_callback) {
self.equipment_callback();
}
Expand Down
13 changes: 13 additions & 0 deletions client/js/player.js
Expand Up @@ -71,6 +71,19 @@ define(['character', 'exceptions'], function(Character, Exceptions) {
setSpriteName: function(name) {
this.spriteName = name;
},

getArmorName: function() {
var sprite = this.getArmorSprite();
return sprite.id;
},

getArmorSprite: function() {
if(this.invincible) {
return this.currentArmorSprite;
} else {
return this.sprite;
}
},

getWeaponName: function() {
return this.weaponName;
Expand Down
2 changes: 1 addition & 1 deletion client/js/renderer.js
Expand Up @@ -670,7 +670,7 @@ function(Camera, Item, Character, Player, Timer) {
ctx = canvas.getContext('2d'),
os = this.upscaledRendering ? 1 : this.scale,
player = this.game.player,
sprite = player.sprite,
sprite = player.getArmorSprite(),
spriteAnim = sprite.animationData["idle_down"],
// character
row = spriteAnim.row,
Expand Down

0 comments on commit e0ab40c

Please sign in to comment.