fix(inventory): users were getting their equipment buffed every fight

we were using expToLevel to check if the equipment had leveled up, but
we should have been using levelFromExp.

diff --git a/package.json b/package.json
index 7150b7b4d4882c51f9156573f51cb0ff50620c43..dc9e8ffe92d8e9a266576613139b200267606957 100644 (file)
--- a/package.json
+++ b/package.json
@@ -18,7 +18,8 @@
     "copy-changelog": "cp ./CHANGELOG.md ~/repos/xangelo.ca/static/",
     "release:minor": "npx standard-version --release-as minor",
     "release:patch": "npx standard-version --release-as patch",
-    "release:major": "npx standard-version --release-as major"
+    "release:major": "npx standard-version --release-as major",
+    "dev:debug": "node --inspect -r ts-node/register -r tsconfig-paths/register src/server/api.ts"
   },
   "devDependencies": {
     "@commitlint/cli": "^17.6.6",

diff --git a/src/server/inventory.ts b/src/server/inventory.ts
index 560ad30d3c7590a453d67867adc22a78287e17f1..a4a0ea1f171c8ec16853cba0626123172018301f 100644 (file)
--- a/src/server/inventory.ts
+++ b/src/server/inventory.ts
@@ -129,7 +129,7 @@ export async function increaseExp(player_id: string, payload: Record<string, num
       'ELSE 0 END as previous_exp')]);
 
   const itemsToUpdate = updates.filter(item => {
-    const previousLevel = expToLevel(item.previous_exp);
+    const previousLevel = levelFromExp(item.previous_exp);
     const newLevel = levelFromExp(item.new_exp);
 
     return previousLevel !== newLevel;