Move creation of hidden properties...

...for deferred CAs with HideTarget="yes" to the MSI backend where it belongs.

1 files changed, 12 insertions, 3 deletions

diff --git a/src/WixToolset.Core.WindowsInstaller/Bind/CreateSpecialPropertiesCommand.cs b/src/WixToolset.Core.WindowsInstaller/Bind/CreateSpecialPropertiesCommand.cs
index e6c089a1..8f769904 100644
--- a/src/WixToolset.Core.WindowsInstaller/Bind/CreateSpecialPropertiesCommand.cs
+++ b/src/WixToolset.Core.WindowsInstaller/Bind/CreateSpecialPropertiesCommand.cs
@@ -20,9 +20,9 @@ namespace WixToolset.Core.WindowsInstaller.Bind
         public void Execute()
         {
             // Create lists of the properties that contribute to the special lists of properties.
-            SortedSet<string> adminProperties = new SortedSet<string>();
-            SortedSet<string> secureProperties = new SortedSet<string>();
-            SortedSet<string> hiddenProperties = new SortedSet<string>();
+            var adminProperties = new SortedSet<string>();
+            var secureProperties = new SortedSet<string>();
+            var hiddenProperties = new SortedSet<string>();
 
             foreach (var wixPropertyRow in this.Section.Tuples.OfType<WixPropertyTuple>())
             {
@@ -42,6 +42,15 @@ namespace WixToolset.Core.WindowsInstaller.Bind
                 }
             }
 
+            // Hide properties for in-script custom actions that have HideTarget set.
+            var hideTargetCustomActions = this.Section.Tuples.OfType<CustomActionTuple>().Where(
+                ca => ca.Hidden
+                && (ca.ExecutionType == CustomActionExecutionType.Deferred
+                || ca.ExecutionType == CustomActionExecutionType.Commit
+                || ca.ExecutionType == CustomActionExecutionType.Rollback))
+                .Select(ca => ca.Id.Id);
+            hiddenProperties.UnionWith(hideTargetCustomActions);
+
             if (0 < adminProperties.Count)
             {
                 var tuple = new PropertyTuple(null, new Identifier(AccessModifier.Private, "AdminProperties"));