Fixed: #2384 Vanishing pins on AS3 code editing save, on script delet…

…ion and few other cases

CHANGELOG.md

@@ -11,7 +11,7 @@ All notable changes to this project will be documented in this file.
 - [#2375] Added limit of simultaneously played sounds
 - AS1/2 - Push action hilighting, GetProperty, Call action hilighting
 - [#2381] Font color values with alpha allowed in html edittext, but alpha ignored
-- [#2384] Vanishing pins on AS3 code editing save
+- [#2384] Vanishing pins on AS3 code editing save, on script deletion and few other cases
 
 ## [22.0.1] - 2024-11-20
 ### Added

src/com/jpexs/decompiler/flash/gui/MainPanel.java

@@ -500,6 +500,10 @@ public boolean isPinned(TreeItem item) {
     public void replaceItemPin(TreeItem oldItem, TreeItem newItem) {
         pinsPanel.replaceItem(oldItem, newItem);
     }
+
+    public void refreshPinnedScriptPacks() {
+        pinsPanel.refreshScriptPacks();
+    }
 
     private void handleKeyReleased(KeyEvent e) {
         if (checkEdited()) {
@@ -1878,6 +1882,7 @@ public void updateClassesList() {
         if (tp != null) {
             getCurrentTree().setSelectionPath(tp);
         }
+        refreshPinnedScriptPacks();
     }
 
     private boolean isFilterEmpty(String filter) {

src/com/jpexs/decompiler/flash/gui/PinsPanel.java

@@ -16,6 +16,7 @@
  */
 package com.jpexs.decompiler.flash.gui;
 
+import com.jpexs.decompiler.flash.abc.ScriptPack;
 import com.jpexs.decompiler.flash.configuration.Configuration;
 import com.jpexs.decompiler.flash.gui.tagtree.TreeRoot;
 import com.jpexs.decompiler.flash.timeline.TagScript;
@@ -25,6 +26,7 @@
 import java.awt.event.ActionListener;
 import java.util.ArrayList;
 import java.util.List;
+import java.util.Objects;
 import java.util.regex.Pattern;
 import javax.swing.BorderFactory;
 import javax.swing.JPanel;
@@ -448,4 +450,20 @@ public void refresh() {
             currentUnpinnedButton.refresh();
         }
     }
+
+    public void refreshScriptPacks() {
+        for (int b = 0; b < buttons.size(); b++) {
+            PinButton button = buttons.get(b);
+            if (button.getItem() instanceof ScriptPack) {
+                ScriptPack sp = (ScriptPack) button.getItem();                
+                List<ScriptPack> packs = sp.abc.getScriptPacks(null, sp.allABCs);
+                for (ScriptPack sp2 : packs) {
+                    if (Objects.equals(sp.getClassPath(), sp2.getClassPath())) {
+                        replaceItem(sp, sp2);
+                        break;
+                    }
+                }
+            }
+        }
+    }
 }

src/com/jpexs/decompiler/flash/gui/abc/ABCPanel.java

@@ -1737,7 +1737,7 @@ private void saveDecompiled(boolean refreshTree) {
                 TreeItem sel = mainPanel.getCurrentTree().getCurrentTreeItem();
                 if ((sel instanceof ScriptPack) && (sel != pack)) {
                     mainPanel.replaceItemPin(pack, sel);
-                }
+                }               
             }
             reload();
             mainPanel.clearEditingStatus();

src/com/jpexs/decompiler/flash/gui/tagtree/TagTreeContextMenu.java

@@ -4221,6 +4221,7 @@ public void tagRemoved(Tag tag) {
                     mainPanel.refreshTree();
                 }
                 mainPanel.savePins();
+                mainPanel.updateClassesList();
             }
         }
     }