From 75cc841a99721d6c658424b7af1432b0d020bb28 Mon Sep 17 00:00:00 2001
From: Bryan Berns <berns@uwalumni.com>
Date: Tue, 12 Nov 2024 06:38:34 -0500
Subject: [PATCH] Tweak WiX Installer For Conditionals

- Be more explicit in WiX variable conditional tests to prevent implicit evaluation from producing unexpected results.
---
 setup/WinDirStat.wxs | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/setup/WinDirStat.wxs b/setup/WinDirStat.wxs
index 680881c7..6fe5ea3c 100644
--- a/setup/WinDirStat.wxs
+++ b/setup/WinDirStat.wxs
@@ -40,12 +40,12 @@
       </DirectorySearch>
     </Property>
     <SetProperty Id="DESKTOP_SHORTCUT" After="AppSearch" Value="1" Sequence="first">
-    <![CDATA[((WIX_UPGRADE_DETECTED AND DESKTOP_SHORTCUT_LOCATE <> "") OR ENABLE_SHORTCUT)]]></SetProperty>
+    <![CDATA[((WIX_UPGRADE_DETECTED <> "" AND DESKTOP_SHORTCUT_LOCATE <> "") OR ENABLE_SHORTCUT = 1)]]></SetProperty>
     <Property Id="CONTEXT_MENU_LOCATE">
       <RegistrySearch Id='ContextMenuSearch' Type='raw' Root='HKCR' Key='Directory\shell\WinDirStat' Name="Icon" Win64="$(var.Win64)"/>
     </Property>
     <SetProperty Id="CONTEXTMENU_ENTRY" After="AppSearch" Value="1" Sequence="first">
-    <![CDATA[(WIX_UPGRADE_DETECTED AND CONTEXT_MENU_LOCATE <> "") OR (NOT WIX_UPGRADE_DETECTED AND NOT DISABLE_CONTEXTMENU)]]></SetProperty>
+    <![CDATA[((WIX_UPGRADE_DETECTED <> "" AND CONTEXT_MENU_LOCATE <> "") OR NOT WIX_UPGRADE_DETECTED) AND DISABLE_CONTEXTMENU <> 1]]></SetProperty>
     <UI Id="InstallerUI">
       <Dialog Id="OptionsDlg" Width="370" Height="270" Title="WinDirStat Options">
         <Control Id="BannerLine" Type="Line" X="0" Y="44" Width="370" Height="2"/>