f

JaylyDev · Jun 18, 2023 · e49738b · e49738b
1 parent 43bf610
commit e49738b
Show file tree

Hide file tree

Showing 5 changed files with 26 additions and 22 deletions.
diff --git a/scripts/cps-counter/index.ts b/scripts/cps-counter/index.ts
@@ -3,24 +3,28 @@
 // Project: https://github.com/JaylyDev/ScriptAPI
 import { Player, world } from "@minecraft/server";
 
-declare module "@minecraft/server" {
-  interface ClickInfo { timestamp: number }
-  interface Player {
-    clicks?: ClickInfo[];
-  }
+interface ClickInfo {
+  readonly timestamp: number
 };
 
+// Using map because typescript doesn't support prototype property declaration properly
+const clicks = new Map<Player, ClickInfo[]>();
+
 world.afterEvents.entityHitBlock.subscribe(function ({ damagingEntity }) {
   if (damagingEntity instanceof Player) {
-    damagingEntity.clicks ||= [];
-    damagingEntity.clicks.push({ timestamp: Date.now() });
+    const clickInfo = { timestamp: Date.now() };
+    const playerClicks = clicks.get(damagingEntity) || [];
+    playerClicks.push(clickInfo);
+    clicks.set(damagingEntity, playerClicks);
   }
 });
 
 world.afterEvents.entityHitEntity.subscribe(function ({ damagingEntity }) {
   if (damagingEntity instanceof Player) {
-    damagingEntity.clicks ||= [];
-    damagingEntity.clicks.push({ timestamp: Date.now() });
+    const clickInfo = { timestamp: Date.now() };
+    const playerClicks = clicks.get(damagingEntity) || [];
+    playerClicks.push(clickInfo);
+    clicks.set(damagingEntity, playerClicks);
   }
 });
 
@@ -31,9 +35,9 @@ world.afterEvents.entityHitEntity.subscribe(function ({ damagingEntity }) {
  */
 export function getPlayerCPS(player: Player) {
   const currentTime = Date.now();
-  player.clicks ||= [];
-  const recentClicks = player.clicks.filter(({ timestamp }) => currentTime - 1000 < timestamp);
-  player.clicks = recentClicks;
+  const playerClicks = clicks.get(player) || [];
+  const recentClicks = playerClicks.filter(({ timestamp }) => currentTime - 1000 < timestamp);
+  clicks.set(player, recentClicks);
 
   return recentClicks.length;
 }
diff --git a/scripts/editor-extensions/Extensions/Actions/NightVision/Start.js b/scripts/editor-extensions/Extensions/Actions/NightVision/Start.js
@@ -1,5 +1,6 @@
 import * as Server from "@minecraft/server";
 import * as Editor from "@minecraft/server-editor";
+import { MinecraftEffectTypes } from "@minecraft/vanilla-data";
 /**
  * @param {import("@minecraft/server-editor").IPlayerUISession} uiSession 
  */
@@ -13,7 +14,7 @@ export const Start = (uiSession) => {
         uiSession.actionManager.createAction(
             {
                 actionType: Editor.ActionTypes.NoArgsAction,
-                onExecute: () => uiSession.extensionContext.player.addEffect( Server.MinecraftEffectTypes.nightVision, 20000000, { amplifier: 1, showParticles: false } ),
+                onExecute: () => uiSession.extensionContext.player.addEffect( MinecraftEffectTypes.NightVision, 20000000, { amplifier: 1, showParticles: false } ),
             },
         ),
     );
@@ -23,7 +24,7 @@ export const Start = (uiSession) => {
         uiSession.actionManager.createAction(
             {
                 actionType: Editor.ActionTypes.NoArgsAction,
-                onExecute: () => uiSession.extensionContext.player.removeEffect( Server.MinecraftEffectTypes.nightVision ),
+                onExecute: () => uiSession.extensionContext.player.removeEffect( MinecraftEffectTypes.NightVision ),
             },
         ),
     );

diff --git a/scripts/editor-fullbright/index.ts b/scripts/editor-fullbright/index.ts
@@ -2,7 +2,7 @@
 // Author: Jayly <https://github.com/JaylyDev>
 // Project: https://github.com/JaylyDev/ScriptAPI
 import { ActionTypes, EditorInputContext, IMenu, InputModifier, IPlayerUISession, KeyboardKey } from "@minecraft/server-editor";
-import { MinecraftEffectTypes } from "@minecraft/server";
+import { MinecraftEffectTypes } from "@minecraft/vanilla-data";
 
 /**
  * Class to enable toggles for full bright in menu in editor mode
@@ -14,13 +14,13 @@ export class FullbrightToggle {
     const enableAction = uiSession.actionManager.createAction({
         actionType: ActionTypes.NoArgsAction,
         onExecute: () => {
-            player.addEffect(MinecraftEffectTypes.nightVision, 20000000, { amplifier: 1, showParticles: false });
+            player.addEffect(MinecraftEffectTypes.NightVision, 20000000, { amplifier: 1, showParticles: false });
         },
     });
     const disableAction = uiSession.actionManager.createAction({
         actionType: ActionTypes.NoArgsAction,
         onExecute: () => {
-            player.runCommand("effect @s " + MinecraftEffectTypes.nightVision.getName() + " 0");
+          player.removeEffect(MinecraftEffectTypes.NightVision);
         },
     });
     // Add actions to menu

diff --git a/scripts/get-effects/index.js b/scripts/get-effects/index.js
@@ -3,13 +3,12 @@
 // Project: https://github.com/JaylyDev/ScriptAPI
 
 import {
-  world,
-  MinecraftEffectTypes,
   /* Typings only */
   Entity,
   Player,
   Effect,
-  EffectType
+  EffectType,
+  EffectTypes
 } from "@minecraft/server";
 
 /**
@@ -22,7 +21,7 @@ import {
  */
 export function getEffects(entity) {
   const effectList = [];
-  for (const eff of Object.values(MinecraftEffectTypes)) {
+  for (const eff of EffectTypes.getAll()) {
     const effect = entity.getEffect(eff);
     if (!effect) continue;
     effectList.push({

diff --git a/scripts/get-scores/index.js b/scripts/get-scores/index.js
@@ -11,7 +11,7 @@ export function getScores (target) {
   let targetScoreboard = {};
 
   if (!(target.scoreboardIdentity instanceof ScoreboardIdentity)) return targetScoreboard;
-  for (const objective of objectives) targetScoreboard[objective.id] = objective.getScores().find((score) => score.participant.type !== ScoreboardIdentityType.fakePlayer ? score.participant.getEntity() === target : false)?.score;
+  for (const objective of objectives) targetScoreboard[objective.id] = objective.getScores().find((score) => score.participant.type !== ScoreboardIdentityType.FakePlayer ? score.participant.getEntity() === target : false)?.score;
   return targetScoreboard;
 };