From 9b46ec1a123a6e19313924130ac9171a7dc43077 Mon Sep 17 00:00:00 2001
From: Marterich <m-wiethan@gmx.de>
Date: Sun, 19 May 2024 14:46:14 +0200
Subject: [PATCH] Extract WPFNamecreation to function, fix hide all if none
 match

---
 functions/private/Get-TabXaml.ps1       |  7 +++++--
 functions/private/Get-WPFObjectName.ps1 | 27 +++++++++++++++++++++++++
 scripts/main.ps1                        | 14 +++++++------
 3 files changed, 40 insertions(+), 8 deletions(-)
 create mode 100644 functions/private/Get-WPFObjectName.ps1

diff --git a/functions/private/Get-TabXaml.ps1 b/functions/private/Get-TabXaml.ps1
index bda4d836..b70363c1 100644
--- a/functions/private/Get-TabXaml.ps1
+++ b/functions/private/Get-TabXaml.ps1
@@ -77,9 +77,12 @@ function Get-TabXaml {
                     $panelcount++
                 }
             }
+
+            # Dot-source the Get-WPFObjectName function
+            . .\functions\private\Get-WPFObjectName
+            
             $categorycontent = $($category -replace '^.__', '')
-            # Remove all special Characters from the name because it will later be used as a Powershell Variable
-            $categoryname = "WPFLabel"+$($categorycontent -replace '[^a-zA-Z0-9]','')
+            $categoryname = Get-WPFObjectName -type "Label" -name $categorycontent
             $blockXml += "<Label Name=""$categoryname"" Content=""$categorycontent"" FontSize=""16""/>`n"
             $sortedApps = $organizedData[$panel][$category].Keys | Sort-Object
             foreach ($appName in $sortedApps) {
diff --git a/functions/private/Get-WPFObjectName.ps1 b/functions/private/Get-WPFObjectName.ps1
new file mode 100644
index 00000000..654c564b
--- /dev/null
+++ b/functions/private/Get-WPFObjectName.ps1
@@ -0,0 +1,27 @@
+function Get-WPFObjectName {
+        <#
+    .SYNOPSIS
+        This is a helper function that generates an objectname with the prefix WPF that can be used as a Powershell Variable after compilation.
+        To achieve this, all characters that are not a-z, A-Z or 0-9 are simply removed from the name.
+    .PARAMETER type
+        The type of object for which the name should be generated. (e.g. Label, Button, CheckBox...)
+    .PARAMETER name
+        The name or description to be used for the object. (invalid characters are removed)
+    .OUTPUTS
+        A string that can be used as a object/variable name in powershell.
+        For example: WPFLabelMicrosoftTools
+        
+    .EXAMPLE
+        Get-WPFObjectName -type Label -name "Microsoft Tools"
+    #>
+
+    param( [Parameter(Mandatory=$true)] 
+    $type, 
+    $name
+)
+
+$Output = $("WPF"+$type+$name) -replace '[^a-zA-Z0-9]', '' 
+
+return $Output
+
+}
\ No newline at end of file
diff --git a/scripts/main.ps1 b/scripts/main.ps1
index 1be1d7f9..5f58000a 100644
--- a/scripts/main.ps1
+++ b/scripts/main.ps1
@@ -416,16 +416,18 @@ $sync["CheckboxFilter"].Add_TextChanged({
     $activeCategories = $activeApplications | Select-Object -ExpandProperty category -Unique
 
     foreach ($category in $activeCategories){
-        $label = $labels["WPFLabel"+$($category -replace '[^a-zA-Z0-9]')]
+        $label = $labels[$(Get-WPFObjectName -type "Label" -name $category)]
         $label.Visibility = "Visible"
     }
-    if ($activeCategories -ne $null){
+    if ($activeCategories){
         $inactiveCategories = Compare-Object -ReferenceObject $allCategories -DifferenceObject $activeCategories -PassThru
-        foreach ($category in $inactiveCategories){
-            $label = $labels["WPFLabel"+$($category -replace '[^a-zA-Z0-9]')]
-            $label.Visibility = "Collapsed"}
     }
-
+    else{
+        $inactiveCategories = $allCategories
+    }
+    foreach ($category in $inactiveCategories){
+        $label = $labels[$(Get-WPFObjectName -type "Label" -name $category)]
+        $label.Visibility = "Collapsed"}
 })
 
 # Define event handler for button click