Support dual-purpose packages in Burn.bob/ConfigurableScopeBundles

Fixes https://github.com/wixtoolset/issues/issues/8958

14 files changed, 168 insertions, 60 deletions

diff --git a/src/api/burn/WixToolset.BootstrapperApplicationApi/BootstrapperCommand.cs b/src/api/burn/WixToolset.BootstrapperApplicationApi/BootstrapperCommand.cs
index 612e8ce9..795a40d4 100644
--- a/src/api/burn/WixToolset.BootstrapperApplicationApi/BootstrapperCommand.cs
+++ b/src/api/burn/WixToolset.BootstrapperApplicationApi/BootstrapperCommand.cs
@@ -20,6 +20,7 @@ namespace WixToolset.BootstrapperApplicationApi
         [MarshalAs(UnmanagedType.I4)] internal int cbSize;
         [MarshalAs(UnmanagedType.U4)] private readonly LaunchAction action;
         [MarshalAs(UnmanagedType.U4)] private readonly Display display;
+        [MarshalAs(UnmanagedType.U4)] private readonly BundleScope scope;
         private readonly IntPtr wzCommandLine;
         [MarshalAs(UnmanagedType.I4)] private readonly int nCmdShow;
         [MarshalAs(UnmanagedType.U4)] private readonly ResumeType resume;
@@ -39,6 +40,7 @@ namespace WixToolset.BootstrapperApplicationApi
             return new BootstrapperCommand(
                 this.action,
                 this.display,
+                this.scope,
                 Marshal.PtrToStringUni(this.wzCommandLine),
                 this.nCmdShow,
                 this.resume,
@@ -62,6 +64,7 @@ namespace WixToolset.BootstrapperApplicationApi
         public BootstrapperCommand(
             LaunchAction action,
             Display display,
+            BundleScope scope,
             string commandLine,
             int cmdShow,
             ResumeType resume,
@@ -74,6 +77,7 @@ namespace WixToolset.BootstrapperApplicationApi
         {
             this.Action = action;
             this.Display = display;
+            this.Scope = scope;
             this.CommandLine = commandLine;
             this.CmdShow = cmdShow;
             this.Resume = resume;
@@ -92,6 +96,9 @@ namespace WixToolset.BootstrapperApplicationApi
         public Display Display { get; }
 
         /// <inheritdoc/>
+        public BundleScope Scope { get; }
+
+        /// <inheritdoc/>
         public string CommandLine { get; }
 
         /// <inheritdoc/>
diff --git a/src/api/burn/WixToolset.BootstrapperApplicationApi/Engine.cs b/src/api/burn/WixToolset.BootstrapperApplicationApi/Engine.cs
index 25413790..82978b7d 100644
--- a/src/api/burn/WixToolset.BootstrapperApplicationApi/Engine.cs
+++ b/src/api/burn/WixToolset.BootstrapperApplicationApi/Engine.cs
@@ -12,7 +12,7 @@ namespace WixToolset.BootstrapperApplicationApi
     /// </summary>
     public sealed class Engine : IEngine
     {
-        private IBootstrapperEngine engine;
+        private readonly IBootstrapperEngine engine;
 
         internal Engine(IBootstrapperEngine engine)
         {
@@ -24,8 +24,7 @@ namespace WixToolset.BootstrapperApplicationApi
         {
             get
             {
-                int count;
-                this.engine.GetPackageCount(out count);
+                this.engine.GetPackageCount(out var count);
 
                 return count;
             }
@@ -110,8 +109,7 @@ namespace WixToolset.BootstrapperApplicationApi
         /// <inheritdoc/>
         public bool EvaluateCondition(string condition)
         {
-            bool value;
-            this.engine.EvaluateCondition(condition, out value);
+            this.engine.EvaluateCondition(condition, out var value);
 
             return value;
         }
@@ -247,9 +245,9 @@ namespace WixToolset.BootstrapperApplicationApi
         }
 
         /// <inheritdoc/>
-        public void Plan(LaunchAction action)
+        public void Plan(LaunchAction action, BundleScope plannedScope)
         {
-            this.engine.Plan(action);
+            this.engine.Plan(action, plannedScope);
         }
 
         /// <inheritdoc/>
@@ -327,16 +325,16 @@ namespace WixToolset.BootstrapperApplicationApi
         /// <inheritdoc/>
         public int SendEmbeddedError(int errorCode, string message, int uiHint)
         {
-            int result = 0;
-            this.engine.SendEmbeddedError(errorCode, message, uiHint, out result);
+            this.engine.SendEmbeddedError(errorCode, message, uiHint, out var result);
+
             return result;
         }
 
         /// <inheritdoc/>
         public int SendEmbeddedProgress(int progressPercentage, int overallPercentage)
         {
-            int result = 0;
-            this.engine.SendEmbeddedProgress(progressPercentage, overallPercentage, out result);
+            this.engine.SendEmbeddedProgress(progressPercentage, overallPercentage, out var result);
+
             return result;
         }
 
diff --git a/src/api/burn/WixToolset.BootstrapperApplicationApi/IBootstrapperCommand.cs b/src/api/burn/WixToolset.BootstrapperApplicationApi/IBootstrapperCommand.cs
index a1f8bfe0..7a5a4cfd 100644
--- a/src/api/burn/WixToolset.BootstrapperApplicationApi/IBootstrapperCommand.cs
+++ b/src/api/burn/WixToolset.BootstrapperApplicationApi/IBootstrapperCommand.cs
@@ -20,6 +20,11 @@ namespace WixToolset.BootstrapperApplicationApi
         Display Display { get; }
 
         /// <summary>
+        /// Gets the bundle scope if set at the command line.
+        /// </summary>
+        BundleScope Scope { get; }
+
+        /// <summary>
         /// Gets the command line arguments.
         /// </summary>
         /// <returns>
diff --git a/src/api/burn/WixToolset.BootstrapperApplicationApi/IBootstrapperEngine.cs b/src/api/burn/WixToolset.BootstrapperApplicationApi/IBootstrapperEngine.cs
index 13702757..9df57cd3 100644
--- a/src/api/burn/WixToolset.BootstrapperApplicationApi/IBootstrapperEngine.cs
+++ b/src/api/burn/WixToolset.BootstrapperApplicationApi/IBootstrapperEngine.cs
@@ -178,10 +178,11 @@ namespace WixToolset.BootstrapperApplicationApi
             );
 
         /// <summary>
-        /// See <see cref="IEngine.Plan(LaunchAction)"/>.
+        /// See <see cref="IEngine.Plan(LaunchAction, BundleScope)"/>.
         /// </summary>
         void Plan(
-            [MarshalAs(UnmanagedType.U4)] LaunchAction action
+            [MarshalAs(UnmanagedType.U4)] LaunchAction action,
+            [MarshalAs(UnmanagedType.U4)] BundleScope plannedScope
             );
 
         /// <summary>
@@ -344,6 +345,28 @@ namespace WixToolset.BootstrapperApplicationApi
     }
 
     /// <summary>
+    /// The scope of the bundle when the chain contains per-user-or-machine or per-machone-or-user packages.
+    /// </summary>
+    public enum BundleScope
+    {
+        /// <summary>
+        /// Let Burn choose the scope. Per-user-or-machine packages will be
+        /// planned as per-machine packages.
+        /// </summary>
+        Default,
+
+        /// <summary>
+        /// Set per-machine scope for per-user-or-machine packages.
+        /// </summary>
+        PerMachine,
+
+        /// <summary>
+        /// Set per-user scope for per-user-or-machine packages.
+        /// </summary>
+        PerUser,
+    }
+
+    /// <summary>
     /// The message log level.
     /// </summary>
     public enum LogLevel
diff --git a/src/api/burn/WixToolset.BootstrapperApplicationApi/IEngine.cs b/src/api/burn/WixToolset.BootstrapperApplicationApi/IEngine.cs
index 03ceed06..a763d741 100644
--- a/src/api/burn/WixToolset.BootstrapperApplicationApi/IEngine.cs
+++ b/src/api/burn/WixToolset.BootstrapperApplicationApi/IEngine.cs
@@ -139,7 +139,8 @@ namespace WixToolset.BootstrapperApplicationApi
         /// Determine the installation sequencing and costing.
         /// </summary>
         /// <param name="action">The action to perform when planning.</param>
-        void Plan(LaunchAction action);
+        /// <param name="plannedScope">The bundle scope for per-user-or-machine packages.</param>
+        void Plan(LaunchAction action, BundleScope plannedScope);
 
         /// <summary>
         /// Set the update information for a bundle.
diff --git a/src/api/burn/balutil/BalBootstrapperEngine.cpp b/src/api/burn/balutil/BalBootstrapperEngine.cpp
index 61e7e31b..14aa5c25 100644
--- a/src/api/burn/balutil/BalBootstrapperEngine.cpp
+++ b/src/api/burn/balutil/BalBootstrapperEngine.cpp
@@ -1188,7 +1188,8 @@ public: // IBootstrapperEngine
     }
 
     virtual STDMETHODIMP Plan(
-        __in BOOTSTRAPPER_ACTION action
+        __in BOOTSTRAPPER_ACTION action,
+        __in BOOTSTRAPPER_SCOPE plannedScope
         )
     {
         HRESULT hr = S_OK;
@@ -1199,17 +1200,21 @@ public: // IBootstrapperEngine
         PIPE_RPC_RESULT rpc = { };
 
         // Init send structs.
-        args.dwApiVersion = WIX_5_BOOTSTRAPPER_APPLICATION_API_VERSION;
+        args.dwApiVersion = WIX_7_BOOTSTRAPPER_APPLICATION_API_VERSION;
         args.action = action;
+        args.plannedScope = plannedScope;
 
-        results.dwApiVersion = WIX_5_BOOTSTRAPPER_APPLICATION_API_VERSION;
+        results.dwApiVersion = WIX_7_BOOTSTRAPPER_APPLICATION_API_VERSION;
 
         // Send args.
         hr = BuffWriteNumberToBuffer(&bufferArgs, args.dwApiVersion);
         ExitOnFailure(hr, "Failed to write API version of Plan args.");
 
         hr = BuffWriteNumberToBuffer(&bufferArgs, static_cast<DWORD>(args.action));
-        ExitOnFailure(hr, "Failed to write parent window of Plan args.");
+        ExitOnFailure(hr, "Failed to write action of Plan args.");
+
+        hr = BuffWriteNumberToBuffer(&bufferArgs, static_cast<DWORD>(args.plannedScope));
+        ExitOnFailure(hr, "Failed to write planned scope of Plan args.");
 
         // Send results.
         hr = BuffWriteNumberToBuffer(&bufferResults, results.dwApiVersion);
diff --git a/src/api/burn/balutil/inc/BootstrapperApplicationBase.h b/src/api/burn/balutil/inc/BootstrapperApplicationBase.h
index 80bfd361..41c151d9 100644
--- a/src/api/burn/balutil/inc/BootstrapperApplicationBase.h
+++ b/src/api/burn/balutil/inc/BootstrapperApplicationBase.h
@@ -16,7 +16,7 @@ class CBootstrapperApplicationBase : public IBootstrapperApplication
 public: // IUnknown
     virtual STDMETHODIMP QueryInterface(
         __in REFIID riid,
-        __out LPVOID *ppvObject
+        __out LPVOID* ppvObject
         )
     {
         if (!ppvObject)
@@ -478,12 +478,14 @@ public: // IBootstrapperApplication
         __in DWORD dwCode,
         __in_z LPCWSTR wzError,
         __in DWORD dwUIHint,
-        __in DWORD /*cData*/,
+        __in DWORD cData,
         __in_ecount_z_opt(cData) LPCWSTR* /*rgwzData*/,
         __in int /*nRecommendation*/,
         __inout int* pResult
         )
     {
+        UNREFERENCED_PARAMETER(cData);
+
         BalRetryErrorOccurred(wzPackageId, dwCode);
 
         if (BOOTSTRAPPER_DISPLAY_EMBEDDED == m_commandDisplay)
@@ -781,12 +783,14 @@ public: // IBootstrapperApplication
         __in INSTALLMESSAGE /*messageType*/,
         __in DWORD /*dwUIHint*/,
         __in_z LPCWSTR /*wzMessage*/,
-        __in DWORD /*cData*/,
+        __in DWORD cData,
         __in_ecount_z_opt(cData) LPCWSTR* /*rgwzData*/,
         __in int /*nRecommendation*/,
         __inout int* pResult
         )
     {
+        UNREFERENCED_PARAMETER(cData);
+
         if (CheckCanceled())
         {
             *pResult = IDCANCEL;
@@ -797,13 +801,15 @@ public: // IBootstrapperApplication
 
     virtual STDMETHODIMP OnExecuteFilesInUse(
         __in_z LPCWSTR /*wzPackageId*/,
-        __in DWORD /*cFiles*/,
+        __in DWORD cFiles,
         __in_ecount_z(cFiles) LPCWSTR* /*rgwzFiles*/,
         __in int /*nRecommendation*/,
         __in BOOTSTRAPPER_FILES_IN_USE_TYPE /*source*/,
         __inout int* pResult
         )
     {
+        UNREFERENCED_PARAMETER(cFiles);
+
         if (CheckCanceled())
         {
             *pResult = IDCANCEL;
diff --git a/src/api/burn/balutil/inc/IBootstrapperEngine.h b/src/api/burn/balutil/inc/IBootstrapperEngine.h
index 57fc9be9..ad6e6042 100644
--- a/src/api/burn/balutil/inc/IBootstrapperEngine.h
+++ b/src/api/burn/balutil/inc/IBootstrapperEngine.h
@@ -108,7 +108,8 @@ DECLARE_INTERFACE_IID_(IBootstrapperEngine, IUnknown, "6480D616-27A0-44D7-905B-8
         ) = 0;
 
     STDMETHOD(Plan)(
-        __in BOOTSTRAPPER_ACTION action
+        __in BOOTSTRAPPER_ACTION action,
+        __in BOOTSTRAPPER_SCOPE plannedScope
         ) = 0;
 
     STDMETHOD(Elevate)(
diff --git a/src/api/burn/inc/BootstrapperApplicationTypes.h b/src/api/burn/inc/BootstrapperApplicationTypes.h
index 2ece1b7b..b430e04a 100644
--- a/src/api/burn/inc/BootstrapperApplicationTypes.h
+++ b/src/api/burn/inc/BootstrapperApplicationTypes.h
@@ -10,6 +10,7 @@ extern "C" {
 const LPCWSTR BOOTSTRAPPER_APPLICATION_COMMANDLINE_SWITCH_API_VERSION = L"burn.ba.apiver";
 const LPCWSTR BOOTSTRAPPER_APPLICATION_COMMANDLINE_SWITCH_PIPE_NAME = L"burn.ba.pipe";
 const DWORD WIX_5_BOOTSTRAPPER_APPLICATION_API_VERSION = 5;
+const DWORD WIX_7_BOOTSTRAPPER_APPLICATION_API_VERSION = 7;
 
 enum BOOTSTRAPPER_DISPLAY
 {
@@ -27,6 +28,15 @@ enum BOOTSTRAPPER_REGISTRATION_TYPE
     BOOTSTRAPPER_REGISTRATION_TYPE_FULL,
 };
 
+enum BOOTSTRAPPER_PACKAGE_SCOPE
+{
+    BOOTSTRAPPER_PACKAGE_SCOPE_INVALID,
+    BOOTSTRAPPER_PACKAGE_SCOPE_PER_MACHINE,
+    BOOTSTRAPPER_PACKAGE_SCOPE_PER_MACHINE_OR_PER_USER,
+    BOOTSTRAPPER_PACKAGE_SCOPE_PER_USER_OR_PER_MACHINE,
+    BOOTSTRAPPER_PACKAGE_SCOPE_PER_USER,
+};
+
 enum BOOTSTRAPPER_RESUME_TYPE
 {
     BOOTSTRAPPER_RESUME_TYPE_NONE,
@@ -353,6 +363,7 @@ struct BOOTSTRAPPER_COMMAND
     DWORD cbSize;
     BOOTSTRAPPER_ACTION action;
     BOOTSTRAPPER_DISPLAY display;
+    BOOTSTRAPPER_SCOPE commandLineScope;
 
     LPWSTR wzCommandLine;
     INT32 nCmdShow;
diff --git a/src/api/burn/inc/BootstrapperEngineTypes.h b/src/api/burn/inc/BootstrapperEngineTypes.h
index f89238c0..6dced349 100644
--- a/src/api/burn/inc/BootstrapperEngineTypes.h
+++ b/src/api/burn/inc/BootstrapperEngineTypes.h
@@ -30,6 +30,13 @@ enum BOOTSTRAPPER_ACTION
     BOOTSTRAPPER_ACTION_UPDATE_REPLACE_EMBEDDED,
 };
 
+enum BOOTSTRAPPER_SCOPE
+{
+    BOOTSTRAPPER_SCOPE_DEFAULT,
+    BOOTSTRAPPER_SCOPE_PER_MACHINE,
+    BOOTSTRAPPER_SCOPE_PER_USER,
+};
+
 enum BOOTSTRAPPER_ACTION_STATE
 {
     BOOTSTRAPPER_ACTION_STATE_NONE,
@@ -183,7 +190,7 @@ typedef struct _BAENGINE_ESCAPESTRING_RESULTS
 {
     DWORD dwApiVersion;
     LPWSTR wzOut;
-    // Should be initialized to the size of wzOut.
+    // Should be initialized to the count of wzOut.
     DWORD cchOut;
 } BAENGINE_ESCAPESTRING_RESULTS;
 
@@ -306,6 +313,7 @@ typedef struct _BAENGINE_PLAN_ARGS
 {
     DWORD dwApiVersion;
     BOOTSTRAPPER_ACTION action;
+    BOOTSTRAPPER_SCOPE plannedScope;
 } BAENGINE_PLAN_ARGS;
 
 typedef struct _BAENGINE_PLAN_RESULTS
diff --git a/src/api/wix/WixToolset.Data/Symbols/WixBundleHarvestedBundlePackageSymbol.cs b/src/api/wix/WixToolset.Data/Symbols/WixBundleHarvestedBundlePackageSymbol.cs
index 8c969e77..adec1d78 100644
--- a/src/api/wix/WixToolset.Data/Symbols/WixBundleHarvestedBundlePackageSymbol.cs
+++ b/src/api/wix/WixToolset.Data/Symbols/WixBundleHarvestedBundlePackageSymbol.cs
@@ -18,6 +18,7 @@ namespace WixToolset.Data
                 new IntermediateFieldDefinition(nameof(WixBundleHarvestedBundlePackageSymbolFields.Version), IntermediateFieldType.String),
                 new IntermediateFieldDefinition(nameof(WixBundleHarvestedBundlePackageSymbolFields.DisplayName), IntermediateFieldType.String),
                 new IntermediateFieldDefinition(nameof(WixBundleHarvestedBundlePackageSymbolFields.InstallSize), IntermediateFieldType.LargeNumber),
+                new IntermediateFieldDefinition(nameof(WixBundleHarvestedBundlePackageSymbolFields.Scope), IntermediateFieldType.Number),
             },
             typeof(WixBundleHarvestedBundlePackageSymbol));
     }
@@ -39,13 +40,13 @@ namespace WixToolset.Data.Symbols
         Version,
         DisplayName,
         InstallSize,
+        Scope,
     }
 
     [Flags]
     public enum WixBundleHarvestedBundlePackageAttributes
     {
         None = 0x0,
-        PerMachine = 0x1,
         Win64 = 0x2,
     }
 
@@ -116,20 +117,10 @@ namespace WixToolset.Data.Symbols
             set => this.Set((int)WixBundleHarvestedBundlePackageSymbolFields.InstallSize, value);
         }
 
-        public bool PerMachine
+        public WixBundleScopeType Scope
         {
-            get { return this.Attributes.HasFlag(WixBundleHarvestedBundlePackageAttributes.PerMachine); }
-            set
-            {
-                if (value)
-                {
-                    this.Attributes |= WixBundleHarvestedBundlePackageAttributes.PerMachine;
-                }
-                else
-                {
-                    this.Attributes &= ~WixBundleHarvestedBundlePackageAttributes.PerMachine;
-                }
-            }
+            get => Enum.TryParse((string)this.Fields[(int)WixBundleHarvestedBundlePackageSymbolFields.Scope], true, out WixBundleScopeType value) ? value : WixBundleScopeType.PerMachine;
+            set => this.Set((int)WixBundleHarvestedBundlePackageSymbolFields.Scope, (int)value);
         }
 
         public bool Win64
diff --git a/src/api/wix/WixToolset.Data/Symbols/WixBundleHarvestedMsiPackageSymbol.cs b/src/api/wix/WixToolset.Data/Symbols/WixBundleHarvestedMsiPackageSymbol.cs
index 8a274720..02cbc4bc 100644
--- a/src/api/wix/WixToolset.Data/Symbols/WixBundleHarvestedMsiPackageSymbol.cs
+++ b/src/api/wix/WixToolset.Data/Symbols/WixBundleHarvestedMsiPackageSymbol.cs
@@ -14,6 +14,7 @@ namespace WixToolset.Data
                 new IntermediateFieldDefinition(nameof(WixBundleHarvestedMsiPackageSymbolFields.ProductName), IntermediateFieldType.String),
                 new IntermediateFieldDefinition(nameof(WixBundleHarvestedMsiPackageSymbolFields.ArpComments), IntermediateFieldType.String),
                 new IntermediateFieldDefinition(nameof(WixBundleHarvestedMsiPackageSymbolFields.AllUsers), IntermediateFieldType.String),
+                new IntermediateFieldDefinition(nameof(WixBundleHarvestedMsiPackageSymbolFields.MsiInstallPerUser), IntermediateFieldType.String),
                 new IntermediateFieldDefinition(nameof(WixBundleHarvestedMsiPackageSymbolFields.MsiFastInstall), IntermediateFieldType.String),
                 new IntermediateFieldDefinition(nameof(WixBundleHarvestedMsiPackageSymbolFields.ArpSystemComponent), IntermediateFieldType.String),
                 new IntermediateFieldDefinition(nameof(WixBundleHarvestedMsiPackageSymbolFields.ProductCode), IntermediateFieldType.String),
@@ -37,6 +38,7 @@ namespace WixToolset.Data.Symbols
         ProductName,
         ArpComments,
         AllUsers,
+        MsiInstallPerUser,
         MsiFastInstall,
         ArpSystemComponent,
         ProductCode,
@@ -91,6 +93,12 @@ namespace WixToolset.Data.Symbols
             set => this.Set((int)WixBundleHarvestedMsiPackageSymbolFields.AllUsers, value);
         }
 
+        public string MsiInstallPerUser
+        {
+            get => this.Fields[(int)WixBundleHarvestedMsiPackageSymbolFields.MsiInstallPerUser].AsString();
+            set => this.Set((int)WixBundleHarvestedMsiPackageSymbolFields.MsiInstallPerUser, value);
+        }
+
         public string MsiFastInstall
         {
             get => this.Fields[(int)WixBundleHarvestedMsiPackageSymbolFields.MsiFastInstall].AsString();
diff --git a/src/api/wix/WixToolset.Data/Symbols/WixBundlePackageSymbol.cs b/src/api/wix/WixToolset.Data/Symbols/WixBundlePackageSymbol.cs
index 8d625664..e888856b 100644
--- a/src/api/wix/WixToolset.Data/Symbols/WixBundlePackageSymbol.cs
+++ b/src/api/wix/WixToolset.Data/Symbols/WixBundlePackageSymbol.cs
@@ -17,7 +17,7 @@ namespace WixToolset.Data
                 new IntermediateFieldDefinition(nameof(WixBundlePackageSymbolFields.RepairCondition), IntermediateFieldType.String),
                 new IntermediateFieldDefinition(nameof(WixBundlePackageSymbolFields.Cache), IntermediateFieldType.String),
                 new IntermediateFieldDefinition(nameof(WixBundlePackageSymbolFields.CacheId), IntermediateFieldType.String),
-                new IntermediateFieldDefinition(nameof(WixBundlePackageSymbolFields.PerMachine), IntermediateFieldType.Bool),
+                new IntermediateFieldDefinition(nameof(WixBundlePackageSymbolFields.Scope), IntermediateFieldType.Number),
                 new IntermediateFieldDefinition(nameof(WixBundlePackageSymbolFields.LogPathVariable), IntermediateFieldType.String),
                 new IntermediateFieldDefinition(nameof(WixBundlePackageSymbolFields.RollbackLogPathVariable), IntermediateFieldType.String),
                 new IntermediateFieldDefinition(nameof(WixBundlePackageSymbolFields.Size), IntermediateFieldType.LargeNumber),
@@ -46,7 +46,7 @@ namespace WixToolset.Data.Symbols
         RepairCondition,
         Cache,
         CacheId,
-        PerMachine,
+        Scope,
         LogPathVariable,
         RollbackLogPathVariable,
         Size,
@@ -144,10 +144,32 @@ namespace WixToolset.Data.Symbols
             set => this.Set((int)WixBundlePackageSymbolFields.CacheId, value);
         }
 
-        public bool? PerMachine
+        public WixBundleScopeType? Scope
         {
-            get => (bool?)this.Fields[(int)WixBundlePackageSymbolFields.PerMachine];
-            set => this.Set((int)WixBundlePackageSymbolFields.PerMachine, value);
+            get => (WixBundleScopeType?)this.Fields[(int)WixBundlePackageSymbolFields.Scope].AsNullableNumber();
+            set => this.Set((int)WixBundlePackageSymbolFields.Scope, (int)value);
+        }
+
+        public string ScopeAsString
+        {
+            get
+            {
+                var value = (WixBundleScopeType?)this.Fields[(int)WixBundlePackageSymbolFields.Scope].AsNullableNumber();
+
+                switch (value)
+                {
+                    case WixBundleScopeType.PerMachine:
+                        return "perMachine";
+                    case WixBundleScopeType.PerUser:
+                        return "perUser";
+                    case WixBundleScopeType.PerUserOrMachine:
+                        return "perUserOrMachine";
+                    case WixBundleScopeType.PerMachineOrUser:
+                        return "perMachineOrUser";
+                    default:
+                        return null;
+                }
+            }
         }
 
         public string LogPathVariable
diff --git a/src/api/wix/WixToolset.Data/Symbols/WixBundleSymbol.cs b/src/api/wix/WixToolset.Data/Symbols/WixBundleSymbol.cs
index f16cdbb8..a06a8610 100644
--- a/src/api/wix/WixToolset.Data/Symbols/WixBundleSymbol.cs
+++ b/src/api/wix/WixToolset.Data/Symbols/WixBundleSymbol.cs
@@ -16,6 +16,7 @@ namespace WixToolset.Data
                 new IntermediateFieldDefinition(nameof(WixBundleSymbolFields.Name), IntermediateFieldType.String),
                 new IntermediateFieldDefinition(nameof(WixBundleSymbolFields.Manufacturer), IntermediateFieldType.String),
                 new IntermediateFieldDefinition(nameof(WixBundleSymbolFields.Attributes), IntermediateFieldType.Number),
+                new IntermediateFieldDefinition(nameof(WixBundleSymbolFields.Scope), IntermediateFieldType.Number),
                 new IntermediateFieldDefinition(nameof(WixBundleSymbolFields.AboutUrl), IntermediateFieldType.String),
                 new IntermediateFieldDefinition(nameof(WixBundleSymbolFields.HelpUrl), IntermediateFieldType.String),
                 new IntermediateFieldDefinition(nameof(WixBundleSymbolFields.HelpTelephone), IntermediateFieldType.String),
@@ -42,6 +43,7 @@ namespace WixToolset.Data
 namespace WixToolset.Data.Symbols
 {
     using System;
+    using System.Xml.Linq;
 
     public enum WixBundleSymbolFields
     {
@@ -51,6 +53,7 @@ namespace WixToolset.Data.Symbols
         Name,
         Manufacturer,
         Attributes,
+        Scope,
         AboutUrl,
         HelpUrl,
         HelpTelephone,
@@ -78,7 +81,6 @@ namespace WixToolset.Data.Symbols
     {
         None = 0x0,
         DisableRemove = 0x1,
-        PerMachine = 0x2,
     }
 
     public enum WixBundleModifyType
@@ -88,6 +90,14 @@ namespace WixToolset.Data.Symbols
         SingleChangeUninstallButton = 2,
     }
 
+    public enum WixBundleScopeType
+    {
+        PerMachine,
+        PerMachineOrUser,
+        PerUserOrMachine,
+        PerUser,
+    }
+
     public class WixBundleSymbol : IntermediateSymbol
     {
         public WixBundleSymbol() : base(SymbolDefinitions.WixBundle, null, null)
@@ -136,6 +146,34 @@ namespace WixToolset.Data.Symbols
             set => this.Set((int)WixBundleSymbolFields.Attributes, (int)value);
         }
 
+        public WixBundleScopeType Scope
+        {
+            get => (WixBundleScopeType)this.Fields[(int)WixBundleSymbolFields.Scope].AsNumber();
+            set => this.Set((int)WixBundleSymbolFields.Scope, (int)value);
+        }
+
+        public string ScopeAsString
+        {
+            get
+            {
+                var value = (WixBundleScopeType)this.Fields[(int)WixBundleSymbolFields.Scope].AsNumber();
+
+                switch (value)
+                {
+                    case WixBundleScopeType.PerMachine:
+                        return "perMachine";
+                    case WixBundleScopeType.PerUser:
+                        return "perUser";
+                    case WixBundleScopeType.PerUserOrMachine:
+                        return "perUserOrMachine";
+                    case WixBundleScopeType.PerMachineOrUser:
+                        return "perMachineOrUser";
+                    default:
+                        return null;
+                }
+            }
+        }
+
         public string AboutUrl
         {
             get => (string)this.Fields[(int)WixBundleSymbolFields.AboutUrl];
@@ -276,21 +314,5 @@ namespace WixToolset.Data.Symbols
                 }
             }
         }
-
-        public bool PerMachine
-        {
-            get { return this.Attributes.HasFlag(WixBundleAttributes.PerMachine); }
-            set
-            {
-                if (value)
-                {
-                    this.Attributes |= WixBundleAttributes.PerMachine;
-                }
-                else
-                {
-                    this.Attributes &= ~WixBundleAttributes.PerMachine;
-                }
-            }
-        }
     }
 }