From aea184d4744cb605477e995f3864da201047510f Mon Sep 17 00:00:00 2001
From: Manuel Pol <90402724+Manuel-Pol@users.noreply.github.com>
Date: Thu, 14 Nov 2024 12:05:25 -0300
Subject: [PATCH] fix: finishing packet sending when packet is selected (#48)
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

An error is thrown when a packet’s sending process ends with the packet
being selected. The error was resolved by deselecting the packet once
its sending process is complete.
---
 src/types/packet.ts          | 5 ++++-
 src/types/viewportManager.ts | 4 ++++
 2 files changed, 8 insertions(+), 1 deletion(-)

diff --git a/src/types/packet.ts b/src/types/packet.ts
index 5101acf..c5584ea 100644
--- a/src/types/packet.ts
+++ b/src/types/packet.ts
@@ -5,7 +5,7 @@ import {
   Ticker,
 } from "pixi.js";
 import { Edge, Position } from "./edge";
-import { selectElement } from "./viewportManager";
+import { deselectElement, isSelected, selectElement } from "./viewportManager";
 import { circleGraphicsContext, Colors, ZIndexLevels } from "../utils";
 import { RightBar } from "../index";
 
@@ -116,6 +116,9 @@ export class Packet extends Graphics {
       this.removeFromParent();
       if (this.currentPath.length == 0) {
         ticker.remove(this.animationTick, this);
+        if (isSelected(this)) {
+          deselectElement();
+        }
         this.destroy();
         return;
       }
diff --git a/src/types/viewportManager.ts b/src/types/viewportManager.ts
index 56ab99d..06b795c 100644
--- a/src/types/viewportManager.ts
+++ b/src/types/viewportManager.ts
@@ -35,6 +35,10 @@ export function refreshElement() {
   }
 }
 
+export function isSelected(element: Device | Edge | Packet) {
+  return element === selectedElement;
+}
+
 document.addEventListener("keydown", (event) => {
   if (event.key === "Delete" || event.key === "Backspace") {
     if (selectedElement) {