fix(LegacyAPI): avoid calling null handlers

Closes #117
cooldogepm · Dec 23, 2023 · a89f3fc · a89f3fc
1 parent 76fd70f
commit a89f3fc
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/src/cooldogedev/BedrockEconomy/api/type/LegacyAPI.php b/src/cooldogedev/BedrockEconomy/api/type/LegacyAPI.php
@@ -52,26 +52,31 @@ public function getVersion(): string
 
     public function setPlayerBalance(string $username, int $amount, ?Closure $handler = null): void
     {
+        $handler ??= fn () => null;
         BedrockEconomyAPI::CLOSURE()->set(Search::EMPTY, $username, $amount, 0, fn () => $handler(true), fn () => $handler(false));
     }
 
     public function getPlayerBalance(string $username, ?Closure $handler = null): void
     {
+        $handler ??= fn () => null;
         BedrockEconomyAPI::CLOSURE()->get(Search::EMPTY, $username, fn (int $amount) => $handler($amount), fn () => $handler(null));
     }
 
     public function addToPlayerBalance(string $username, int $amount, ?Closure $handler = null): void
     {
+        $handler ??= fn () => null;
         BedrockEconomyAPI::CLOSURE()->add(Search::EMPTY, $username, $amount, 0, fn () => $handler(true), fn () => $handler(false));
     }
 
     public function subtractFromPlayerBalance(string $username, int $amount, ?Closure $handler = null): void
     {
+        $handler ??= fn () => null;
         BedrockEconomyAPI::CLOSURE()->subtract(Search::EMPTY, $username, $amount, 0, fn () => $handler(true), fn () => $handler(false));
     }
 
     public function transferFromPlayerBalance(string $source, string $target, int $amount, ?Closure $handler = null): void
     {
+        $handler ??= fn () => null;
         BedrockEconomyAPI::CLOSURE()->transfer(
             source: [
                 "xuid" => Search::EMPTY,