search

jpcsupplies / Economy_mod

Basic Economy System for Space Engineers
13 stars 12 forks source link

Exception using /buy and /value and balances reset #72

Closed jpcsupplies closed 8 years ago

jpcsupplies commented 8 years ago

On Ramblers Frontier server getting following error reports from players

zannador: An exception has been logged in the Economy client file zannador: EconomyClient.log

Not so concerned about balances being reset, although we probably should improve data integrity there somehow.. (maybe have a bank balance file seperate so item changes dont risk curruption if we need to do a database reset/delete for items)

But in the workshop test build the following commands cause errors. /buy 1 uranium ingot

/buy 100 interior

/value Steel Plate

/value ice

About to test here locally to get that log file.

jpcsupplies commented 8 years ago

log here https://gist.github.com/jpcsupplies/15419d69d1f0a656e000

/buy 1 ice (error)

/value ice (error)

/pay zannador 1 test (worked fine)

Looks to be in the itemlist lookup..

midspace commented 8 years ago

yeah, I thought I had done something wrong with that code. I'm fixing it now.

midspace commented 8 years ago

Change https://github.com/jpcsupplies/Economy_mod/commit/ce146daede53fc1d6deb3556e201f33eba187c55 fixes this issue.

jpcsupplies commented 8 years ago

If you have pushed that fix to the workshop testing release, remember to restart the ramblers server or ask dragon or zannador to do it when server is empty next - i m still shaped so the console wont load here i would have to do it tomorrow at work

midspace commented 8 years ago

It only affects the client. Server side is fine.