refactored actions and hooks to not contain underscores

Author: Twistt

src/game-engine/world/action/magic-on-npc.action.ts

@@ -8,7 +8,7 @@ import { Npc } from '../actor/npc/npc';
 /**
  * Defines a button action hook.
  */
-export interface Magic_On_NPCActionHook extends ActionHook<Magic_On_NPCAction, magic_on_npcActionHandler> {
+export interface MagicOnNPCActionHook extends ActionHook<MagicOnNPCAction, magiconnpcActionHandler> {
     // The npc world id that was clicked on after choosing the spell
     npcworldId?: number;
     // The IDs of the UI widgets that the buttons are on.
@@ -23,13 +23,13 @@ export interface Magic_On_NPCActionHook extends ActionHook<Magic_On_NPCAction, m
 /**
  * The button action hook handler function to be called when the hook's conditions are met.
  */
-export type magic_on_npcActionHandler = (buttonAction: Magic_On_NPCAction) => void | Promise<void>;
+export type magiconnpcActionHandler = (buttonAction: MagicOnNPCAction) => void | Promise<void>;
 
 
 /**
  * Details about a button action being performed.
  */
-export interface Magic_On_NPCAction {
+export interface MagicOnNPCAction {
     // The npc world id that was clicked on after choosing the spell
     npc: Npc;
     // The player performing the action.
@@ -48,11 +48,11 @@ export interface Magic_On_NPCAction {
  * @param widgetId
  * @param buttonId
  */
-const buttonActionPipe = (npc:Npc, player: Player, widgetId: number, buttonId: number): RunnableHooks<Magic_On_NPCAction> => {
+const buttonActionPipe = (npc:Npc, player: Player, widgetId: number, buttonId: number): RunnableHooks<MagicOnNPCAction> => {
     //console.info(`pew pew you use magic on ${npc.name}!`);
 
     // Find all object action plugins that reference this location object
-    const matchingHooks = getActionHooks<Magic_On_NPCActionHook>('magic_on_npc');
+    const matchingHooks = getActionHooks<MagicOnNPCActionHook>('magic_on_npc');
 
 
     return {

src/plugins/buttons/magic-attack.plugin.ts

@@ -7,7 +7,7 @@ import { gfxIds } from '@engine/world/config/gfx-ids';
 import { loopingEvent } from '@engine/game-server';
 import { TaskExecutor } from '@engine/world/action';
 import { widgetButtonIds } from '../skills/smithing/smelting-constants';
-import { magic_on_npcActionHandler, Magic_On_NPCActionHook, Magic_On_NPCAction } from '../../game-engine/world/action/magic-on-npc.action';
+import { magiconnpcActionHandler, MagicOnNPCActionHook, MagicOnNPCAction } from '../../game-engine/world/action/magic-on-npc.action';
 import { logger } from '@runejs/core';
 
 const buttonIds: number[] = [
@@ -20,7 +20,7 @@ function attack_target(player: Player, elapsedTicks: number): boolean {
 }
 
 const spells = ['Wind Strike','Confuse', 'Water Strike','unknown?', 'Earth Strike'];
-export const activate = (task: TaskExecutor<Magic_On_NPCAction>, elapsedTicks: number = 0) => {
+export const activate = (task: TaskExecutor<MagicOnNPCAction>, elapsedTicks: number = 0) => {
     const {
         npc,
         player,
@@ -42,6 +42,6 @@ export default {
                 activate,
                 interval: 0
             }
-        } as Magic_On_NPCActionHook
+        } as MagicOnNPCActionHook
 
 };