Merge branch 'runejs:develop' into no-zero-items

Author: src52

package-lock.json

@@ -140,8 +140,7 @@ export class DialogueAction {
             this.p.interfaceState.openWidget(widgetId, {
                 slot: 'chatbox'
             })
-
-            const sub = this.p.dialogueInteractionEvent.subscribe(action => {
+            const sub = this.p.interfaceState.closed.subscribe(action => {
                 sub.unsubscribe();
                 this._action = action;
                 resolve(this);

src/game-engine/world/actor/player/dialogue-action.ts

@@ -170,9 +170,7 @@ export const withdrawItem: itemInteractionActionHandler = (details) => {
         amount: removeFromContainer(playerBank, details.itemId, countToRemove)
     };
 
-    for (let i = 0; i < itemToAdd.amount; i++) {
-        playerInventory.add({ itemId: itemIdToAdd, amount: 1 });
-    }
+    playerInventory.add({ itemId: itemToAdd.itemId, amount: itemToAdd.amount });
 
     updateBankingInterface(details.player);
 };

src/plugins/objects/bank/bank.plugin.ts

@@ -39,6 +39,11 @@ const doors = [
         closed: 11993,
         open: 11994,
         hinge: 'RIGHT'
+    },
+    {
+        closed: 13001,
+        open: 13002,
+        hinge: 'RIGHT'
     }
 ];
 
@@ -98,7 +103,7 @@ export default {
         {
             type: 'object_interaction',
             objectIds: [ 1530, 4465, 4467, 3014, 3017, 3018,
-                3019, 1536, 1537, 1533, 1531, 1534, 12348, 11993, 11994 ],
+                3019, 1536, 1537, 1533, 1531, 1534, 12348, 11993, 11994, 13001, 13002 ],
             options: [ 'open', 'close' ],
             walkTo: true,
             handler: action

src/plugins/objects/doors/door.plugin.ts

@@ -11,7 +11,6 @@ const validate: (level: number) => boolean = (level) => {
 
 export const action: objectInteractionActionHandler = (details) => {
     const { player, option } = details;
-
     if (option === 'climb') {
         dialogueAction(player)
             .then(async d => d.options(
@@ -22,20 +21,18 @@ export const action: objectInteractionActionHandler = (details) => {
                 ]))
             .then(d => {
                 d.close();
-                switch (d.action) {
+                switch (d._action.data) {
                     case 1:
                     case 2:
-                        action({ ...details, option: `climb-${(d.action === 1 ? 'up' : 'down')}` });
+                        action({ ...details, option: `climb-${(d._action.data === 1 ? 'up' : 'down')}` });
                         return;
                 }
             });
         return;
     }
-
     const up = option === 'climb-up';
     const { position } = player;
     const level = position.level + (up ? 1 : -1);
-
     if (!validate(level)) return;
     if (!details.objectConfig.name.startsWith('Stair')) {
         player.playAnimation(up ? 828 : 827);