diff options
-rw-r--r-- | src/WixToolset.Core.WindowsInstaller/Bind/BindDatabaseCommand.cs | 40 | ||||
-rw-r--r-- | src/WixToolset.Core.WindowsInstaller/Bind/CreateCabinetsCommand.cs | 12 | ||||
-rw-r--r-- | src/WixToolset.Core.WindowsInstaller/Bind/CreateWindowsInstallerDataFromIRCommand.cs (renamed from src/WixToolset.Core.WindowsInstaller/Bind/CreateOutputFromIRCommand.cs) | 50 | ||||
-rw-r--r-- | src/WixToolset.Core.WindowsInstaller/ExtensibilityServices/WindowsInstallerBackendHelper.cs | 8 |
4 files changed, 55 insertions, 55 deletions
diff --git a/src/WixToolset.Core.WindowsInstaller/Bind/BindDatabaseCommand.cs b/src/WixToolset.Core.WindowsInstaller/Bind/BindDatabaseCommand.cs index b3b9023c..7a64b777 100644 --- a/src/WixToolset.Core.WindowsInstaller/Bind/BindDatabaseCommand.cs +++ b/src/WixToolset.Core.WindowsInstaller/Bind/BindDatabaseCommand.cs | |||
@@ -402,31 +402,29 @@ namespace WixToolset.Core.WindowsInstaller.Bind | |||
402 | return null; | 402 | return null; |
403 | } | 403 | } |
404 | 404 | ||
405 | // Time to create the output object. Try to put as much above here as possible, updating the IR is better. | 405 | // Time to create the WindowsInstallerData object. Try to put as much above here as possible, updating the IR is better. |
406 | WindowsInstallerData output; | 406 | WindowsInstallerData data; |
407 | { | 407 | { |
408 | var command = new CreateOutputFromIRCommand(this.Messaging, section, tableDefinitions, this.BackendExtensions, this.WindowsInstallerBackendHelper); | 408 | var command = new CreateWindowsInstallerDataFromIRCommand(this.Messaging, section, tableDefinitions, this.BackendExtensions, this.WindowsInstallerBackendHelper); |
409 | command.Execute(); | 409 | data = command.Execute(); |
410 | |||
411 | output = command.Output; | ||
412 | } | 410 | } |
413 | 411 | ||
414 | IEnumerable<string> suppressedTableNames = null; | 412 | IEnumerable<string> suppressedTableNames = null; |
415 | if (output.Type == OutputType.Module) | 413 | if (data.Type == OutputType.Module) |
416 | { | 414 | { |
417 | // Modularize identifiers. | 415 | // Modularize identifiers. |
418 | var modularize = new ModularizeCommand(output, modularizationSuffix, section.Symbols.OfType<WixSuppressModularizationSymbol>()); | 416 | var modularize = new ModularizeCommand(data, modularizationSuffix, section.Symbols.OfType<WixSuppressModularizationSymbol>()); |
419 | modularize.Execute(); | 417 | modularize.Execute(); |
420 | 418 | ||
421 | // Ensure all sequence tables in place because, mergemod.dll requires them. | 419 | // Ensure all sequence tables in place because, mergemod.dll requires them. |
422 | var unsuppress = new AddBackSuppressedSequenceTablesCommand(output, tableDefinitions); | 420 | var unsuppress = new AddBackSuppressedSequenceTablesCommand(data, tableDefinitions); |
423 | suppressedTableNames = unsuppress.Execute(); | 421 | suppressedTableNames = unsuppress.Execute(); |
424 | } | 422 | } |
425 | else if (output.Type == OutputType.Patch) | 423 | else if (data.Type == OutputType.Patch) |
426 | { | 424 | { |
427 | foreach (var storage in this.SubStorages) | 425 | foreach (var storage in this.SubStorages) |
428 | { | 426 | { |
429 | output.SubStorages.Add(storage); | 427 | data.SubStorages.Add(storage); |
430 | } | 428 | } |
431 | } | 429 | } |
432 | 430 | ||
@@ -448,7 +446,7 @@ namespace WixToolset.Core.WindowsInstaller.Bind | |||
448 | 446 | ||
449 | // create cabinet files and process uncompressed files | 447 | // create cabinet files and process uncompressed files |
450 | var layoutDirectory = Path.GetDirectoryName(this.OutputPath); | 448 | var layoutDirectory = Path.GetDirectoryName(this.OutputPath); |
451 | if (!this.SuppressLayout || OutputType.Module == output.Type) | 449 | if (!this.SuppressLayout || OutputType.Module == data.Type) |
452 | { | 450 | { |
453 | this.Messaging.Write(VerboseMessages.CreatingCabinetFiles()); | 451 | this.Messaging.Write(VerboseMessages.CreatingCabinetFiles()); |
454 | 452 | ||
@@ -458,7 +456,7 @@ namespace WixToolset.Core.WindowsInstaller.Bind | |||
458 | command.CabbingThreadCount = this.CabbingThreadCount; | 456 | command.CabbingThreadCount = this.CabbingThreadCount; |
459 | command.CabCachePath = this.CabCachePath; | 457 | command.CabCachePath = this.CabCachePath; |
460 | command.DefaultCompressionLevel = this.DefaultCompressionLevel; | 458 | command.DefaultCompressionLevel = this.DefaultCompressionLevel; |
461 | command.Output = output; | 459 | command.Data = data; |
462 | command.Messaging = this.Messaging; | 460 | command.Messaging = this.Messaging; |
463 | command.BackendExtensions = this.BackendExtensions; | 461 | command.BackendExtensions = this.BackendExtensions; |
464 | command.LayoutDirectory = layoutDirectory; | 462 | command.LayoutDirectory = layoutDirectory; |
@@ -481,15 +479,15 @@ namespace WixToolset.Core.WindowsInstaller.Bind | |||
481 | } | 479 | } |
482 | 480 | ||
483 | // We can create instance transforms since Component Guids and Outputs are created. | 481 | // We can create instance transforms since Component Guids and Outputs are created. |
484 | if (output.Type == OutputType.Product) | 482 | if (data.Type == OutputType.Product) |
485 | { | 483 | { |
486 | var command = new CreateInstanceTransformsCommand(section, output, tableDefinitions, this.WindowsInstallerBackendHelper); | 484 | var command = new CreateInstanceTransformsCommand(section, data, tableDefinitions, this.WindowsInstallerBackendHelper); |
487 | command.Execute(); | 485 | command.Execute(); |
488 | } | 486 | } |
489 | else if (output.Type == OutputType.Patch) | 487 | else if (data.Type == OutputType.Patch) |
490 | { | 488 | { |
491 | // Copy output data back into the transforms. | 489 | // Copy output data back into the transforms. |
492 | var command = new UpdateTransformsWithFileFacades(this.Messaging, output, this.SubStorages, tableDefinitions, fileFacades); | 490 | var command = new UpdateTransformsWithFileFacades(this.Messaging, data, this.SubStorages, tableDefinitions, fileFacades); |
493 | command.Execute(); | 491 | command.Execute(); |
494 | } | 492 | } |
495 | 493 | ||
@@ -500,7 +498,7 @@ namespace WixToolset.Core.WindowsInstaller.Bind | |||
500 | var trackMsi = this.WindowsInstallerBackendHelper.TrackFile(this.OutputPath, TrackedFileType.Final); | 498 | var trackMsi = this.WindowsInstallerBackendHelper.TrackFile(this.OutputPath, TrackedFileType.Final); |
501 | trackedFiles.Add(trackMsi); | 499 | trackedFiles.Add(trackMsi); |
502 | 500 | ||
503 | var command = new GenerateDatabaseCommand(this.Messaging, this.WindowsInstallerBackendHelper, this.FileSystemManager, output, trackMsi.Path, tableDefinitions, this.IntermediateFolder, this.Codepage, keepAddedColumns: false, this.SuppressAddingValidationRows, useSubdirectory: false); | 501 | var command = new GenerateDatabaseCommand(this.Messaging, this.WindowsInstallerBackendHelper, this.FileSystemManager, data, trackMsi.Path, tableDefinitions, this.IntermediateFolder, this.Codepage, keepAddedColumns: false, this.SuppressAddingValidationRows, useSubdirectory: false); |
504 | command.Execute(); | 502 | command.Execute(); |
505 | 503 | ||
506 | trackedFiles.AddRange(command.GeneratedTemporaryFiles); | 504 | trackedFiles.AddRange(command.GeneratedTemporaryFiles); |
@@ -572,12 +570,12 @@ namespace WixToolset.Core.WindowsInstaller.Bind | |||
572 | var result = this.ServiceProvider.GetService<IBindResult>(); | 570 | var result = this.ServiceProvider.GetService<IBindResult>(); |
573 | result.FileTransfers = fileTransfers; | 571 | result.FileTransfers = fileTransfers; |
574 | result.TrackedFiles = trackedFiles; | 572 | result.TrackedFiles = trackedFiles; |
575 | result.Wixout = this.CreateWixout(trackedFiles, this.Intermediate, output); | 573 | result.Wixout = this.CreateWixout(trackedFiles, this.Intermediate, data); |
576 | 574 | ||
577 | return result; | 575 | return result; |
578 | } | 576 | } |
579 | 577 | ||
580 | private WixOutput CreateWixout(List<ITrackedFile> trackedFiles, Intermediate intermediate, WindowsInstallerData output) | 578 | private WixOutput CreateWixout(List<ITrackedFile> trackedFiles, Intermediate intermediate, WindowsInstallerData data) |
581 | { | 579 | { |
582 | WixOutput wixout; | 580 | WixOutput wixout; |
583 | 581 | ||
@@ -595,7 +593,7 @@ namespace WixToolset.Core.WindowsInstaller.Bind | |||
595 | 593 | ||
596 | intermediate.Save(wixout); | 594 | intermediate.Save(wixout); |
597 | 595 | ||
598 | output.Save(wixout); | 596 | data.Save(wixout); |
599 | 597 | ||
600 | wixout.Reopen(); | 598 | wixout.Reopen(); |
601 | 599 | ||
diff --git a/src/WixToolset.Core.WindowsInstaller/Bind/CreateCabinetsCommand.cs b/src/WixToolset.Core.WindowsInstaller/Bind/CreateCabinetsCommand.cs index f0acd3d4..9f94b2c7 100644 --- a/src/WixToolset.Core.WindowsInstaller/Bind/CreateCabinetsCommand.cs +++ b/src/WixToolset.Core.WindowsInstaller/Bind/CreateCabinetsCommand.cs | |||
@@ -72,7 +72,7 @@ namespace WixToolset.Core.WindowsInstaller.Bind | |||
72 | 72 | ||
73 | public IEnumerable<IWindowsInstallerBackendBinderExtension> BackendExtensions { private get; set; } | 73 | public IEnumerable<IWindowsInstallerBackendBinderExtension> BackendExtensions { private get; set; } |
74 | 74 | ||
75 | public WindowsInstallerData Output { private get; set; } | 75 | public WindowsInstallerData Data { private get; set; } |
76 | 76 | ||
77 | public string LayoutDirectory { private get; set; } | 77 | public string LayoutDirectory { private get; set; } |
78 | 78 | ||
@@ -115,7 +115,7 @@ namespace WixToolset.Core.WindowsInstaller.Bind | |||
115 | var compressionLevel = mediaSymbol.CompressionLevel ?? this.DefaultCompressionLevel ?? CompressionLevel.Medium; | 115 | var compressionLevel = mediaSymbol.CompressionLevel ?? this.DefaultCompressionLevel ?? CompressionLevel.Medium; |
116 | var cabinetDir = this.ResolveMedia(mediaSymbol, mediaSymbol.Layout, this.LayoutDirectory); | 116 | var cabinetDir = this.ResolveMedia(mediaSymbol, mediaSymbol.Layout, this.LayoutDirectory); |
117 | 117 | ||
118 | var cabinetWorkItem = this.CreateCabinetWorkItem(this.Output, cabinetDir, mediaSymbol, compressionLevel, files); | 118 | var cabinetWorkItem = this.CreateCabinetWorkItem(this.Data, cabinetDir, mediaSymbol, compressionLevel, files); |
119 | if (null != cabinetWorkItem) | 119 | if (null != cabinetWorkItem) |
120 | { | 120 | { |
121 | cabinetBuilder.Enqueue(cabinetWorkItem); | 121 | cabinetBuilder.Enqueue(cabinetWorkItem); |
@@ -171,13 +171,13 @@ namespace WixToolset.Core.WindowsInstaller.Bind | |||
171 | /// <summary> | 171 | /// <summary> |
172 | /// Creates a work item to create a cabinet. | 172 | /// Creates a work item to create a cabinet. |
173 | /// </summary> | 173 | /// </summary> |
174 | /// <param name="output">Output for the current database.</param> | 174 | /// <param name="data">Windows Installer data for the current database.</param> |
175 | /// <param name="cabinetDir">Directory to create cabinet in.</param> | 175 | /// <param name="cabinetDir">Directory to create cabinet in.</param> |
176 | /// <param name="mediaSymbol">Media symbol containing information about the cabinet.</param> | 176 | /// <param name="mediaSymbol">Media symbol containing information about the cabinet.</param> |
177 | /// <param name="compressionLevel">Desired compression level.</param> | 177 | /// <param name="compressionLevel">Desired compression level.</param> |
178 | /// <param name="fileFacades">Collection of files in this cabinet.</param> | 178 | /// <param name="fileFacades">Collection of files in this cabinet.</param> |
179 | /// <returns>created CabinetWorkItem object</returns> | 179 | /// <returns>created CabinetWorkItem object</returns> |
180 | private CabinetWorkItem CreateCabinetWorkItem(WindowsInstallerData output, string cabinetDir, MediaSymbol mediaSymbol, CompressionLevel compressionLevel, IEnumerable<FileFacade> fileFacades) | 180 | private CabinetWorkItem CreateCabinetWorkItem(WindowsInstallerData data, string cabinetDir, MediaSymbol mediaSymbol, CompressionLevel compressionLevel, IEnumerable<FileFacade> fileFacades) |
181 | { | 181 | { |
182 | CabinetWorkItem cabinetWorkItem = null; | 182 | CabinetWorkItem cabinetWorkItem = null; |
183 | var tempCabinetFileX = Path.Combine(this.IntermediateFolder, mediaSymbol.Cabinet); | 183 | var tempCabinetFileX = Path.Combine(this.IntermediateFolder, mediaSymbol.Cabinet); |
@@ -189,7 +189,7 @@ namespace WixToolset.Core.WindowsInstaller.Bind | |||
189 | var cabinetName = mediaSymbol.Cabinet.TrimStart('#'); | 189 | var cabinetName = mediaSymbol.Cabinet.TrimStart('#'); |
190 | 190 | ||
191 | // If building a patch, remind them to run -p for torch. | 191 | // If building a patch, remind them to run -p for torch. |
192 | if (OutputType.Patch == output.Type) | 192 | if (OutputType.Patch == data.Type) |
193 | { | 193 | { |
194 | this.Messaging.Write(WarningMessages.EmptyCabinet(mediaSymbol.SourceLineNumbers, cabinetName, true)); | 194 | this.Messaging.Write(WarningMessages.EmptyCabinet(mediaSymbol.SourceLineNumbers, cabinetName, true)); |
195 | } | 195 | } |
@@ -234,7 +234,7 @@ namespace WixToolset.Core.WindowsInstaller.Bind | |||
234 | 234 | ||
235 | if (mediaSymbol.Cabinet.StartsWith("#", StringComparison.Ordinal)) | 235 | if (mediaSymbol.Cabinet.StartsWith("#", StringComparison.Ordinal)) |
236 | { | 236 | { |
237 | var streamsTable = output.EnsureTable(this.TableDefinitions["_Streams"]); | 237 | var streamsTable = data.EnsureTable(this.TableDefinitions["_Streams"]); |
238 | 238 | ||
239 | var streamRow = streamsTable.CreateRow(mediaSymbol.SourceLineNumbers); | 239 | var streamRow = streamsTable.CreateRow(mediaSymbol.SourceLineNumbers); |
240 | streamRow[0] = mediaSymbol.Cabinet.Substring(1); | 240 | streamRow[0] = mediaSymbol.Cabinet.Substring(1); |
diff --git a/src/WixToolset.Core.WindowsInstaller/Bind/CreateOutputFromIRCommand.cs b/src/WixToolset.Core.WindowsInstaller/Bind/CreateWindowsInstallerDataFromIRCommand.cs index 37383caa..dc60a9ac 100644 --- a/src/WixToolset.Core.WindowsInstaller/Bind/CreateOutputFromIRCommand.cs +++ b/src/WixToolset.Core.WindowsInstaller/Bind/CreateWindowsInstallerDataFromIRCommand.cs | |||
@@ -16,9 +16,9 @@ namespace WixToolset.Core.WindowsInstaller.Bind | |||
16 | using WixToolset.Extensibility; | 16 | using WixToolset.Extensibility; |
17 | using WixToolset.Extensibility.Services; | 17 | using WixToolset.Extensibility.Services; |
18 | 18 | ||
19 | internal class CreateOutputFromIRCommand | 19 | internal class CreateWindowsInstallerDataFromIRCommand |
20 | { | 20 | { |
21 | public CreateOutputFromIRCommand(IMessaging messaging, IntermediateSection section, TableDefinitionCollection tableDefinitions, IEnumerable<IWindowsInstallerBackendBinderExtension> backendExtensions, IWindowsInstallerBackendHelper backendHelper) | 21 | public CreateWindowsInstallerDataFromIRCommand(IMessaging messaging, IntermediateSection section, TableDefinitionCollection tableDefinitions, IEnumerable<IWindowsInstallerBackendBinderExtension> backendExtensions, IWindowsInstallerBackendHelper backendHelper) |
22 | { | 22 | { |
23 | this.Messaging = messaging; | 23 | this.Messaging = messaging; |
24 | this.Section = section; | 24 | this.Section = section; |
@@ -37,20 +37,22 @@ namespace WixToolset.Core.WindowsInstaller.Bind | |||
37 | 37 | ||
38 | private IntermediateSection Section { get; } | 38 | private IntermediateSection Section { get; } |
39 | 39 | ||
40 | public WindowsInstallerData Output { get; private set; } | 40 | public WindowsInstallerData Data { get; private set; } |
41 | 41 | ||
42 | public void Execute() | 42 | public WindowsInstallerData Execute() |
43 | { | 43 | { |
44 | this.Output = new WindowsInstallerData(this.Section.Symbols.First().SourceLineNumbers) | 44 | this.Data = new WindowsInstallerData(this.Section.Symbols.First().SourceLineNumbers) |
45 | { | 45 | { |
46 | Codepage = this.Section.Codepage, | 46 | Codepage = this.Section.Codepage, |
47 | Type = SectionTypeToOutputType(this.Section.Type) | 47 | Type = SectionTypeToOutputType(this.Section.Type) |
48 | }; | 48 | }; |
49 | 49 | ||
50 | this.AddSectionToOutput(); | 50 | this.AddSectionToData(); |
51 | |||
52 | return this.Data; | ||
51 | } | 53 | } |
52 | 54 | ||
53 | private void AddSectionToOutput() | 55 | private void AddSectionToData() |
54 | { | 56 | { |
55 | var cellsByTableAndRowId = new Dictionary<string, List<WixCustomTableCellSymbol>>(); | 57 | var cellsByTableAndRowId = new Dictionary<string, List<WixCustomTableCellSymbol>>(); |
56 | 58 | ||
@@ -61,7 +63,7 @@ namespace WixToolset.Core.WindowsInstaller.Bind | |||
61 | { | 63 | { |
62 | case SymbolDefinitionType.AppSearch: | 64 | case SymbolDefinitionType.AppSearch: |
63 | this.AddSymbolDefaultly(symbol); | 65 | this.AddSymbolDefaultly(symbol); |
64 | this.Output.EnsureTable(this.TableDefinitions["Signature"]); | 66 | this.Data.EnsureTable(this.TableDefinitions["Signature"]); |
65 | break; | 67 | break; |
66 | 68 | ||
67 | case SymbolDefinitionType.Assembly: | 69 | case SymbolDefinitionType.Assembly: |
@@ -154,7 +156,7 @@ namespace WixToolset.Core.WindowsInstaller.Bind | |||
154 | 156 | ||
155 | case SymbolDefinitionType.ProgId: | 157 | case SymbolDefinitionType.ProgId: |
156 | this.AddSymbolDefaultly(symbol); | 158 | this.AddSymbolDefaultly(symbol); |
157 | this.Output.EnsureTable(this.TableDefinitions["Extension"]); | 159 | this.Data.EnsureTable(this.TableDefinitions["Extension"]); |
158 | break; | 160 | break; |
159 | 161 | ||
160 | case SymbolDefinitionType.Property: | 162 | case SymbolDefinitionType.Property: |
@@ -452,7 +454,7 @@ namespace WixToolset.Core.WindowsInstaller.Bind | |||
452 | row[8] = symbol.DefaultControlRef; | 454 | row[8] = symbol.DefaultControlRef; |
453 | row[9] = symbol.CancelControlRef; | 455 | row[9] = symbol.CancelControlRef; |
454 | 456 | ||
455 | this.Output.EnsureTable(this.TableDefinitions["ListBox"]); | 457 | this.Data.EnsureTable(this.TableDefinitions["ListBox"]); |
456 | } | 458 | } |
457 | 459 | ||
458 | private void AddDirectorySymbol(DirectorySymbol symbol) | 460 | private void AddDirectorySymbol(DirectorySymbol symbol) |
@@ -976,9 +978,9 @@ namespace WixToolset.Core.WindowsInstaller.Bind | |||
976 | switch (symbol.SequenceTable) | 978 | switch (symbol.SequenceTable) |
977 | { | 979 | { |
978 | case SequenceTable.AdminExecuteSequence: | 980 | case SequenceTable.AdminExecuteSequence: |
979 | if (OutputType.Module == this.Output.Type) | 981 | if (OutputType.Module == this.Data.Type) |
980 | { | 982 | { |
981 | this.Output.EnsureTable(this.TableDefinitions["AdminExecuteSequence"]); | 983 | this.Data.EnsureTable(this.TableDefinitions["AdminExecuteSequence"]); |
982 | sequenceTableName = "ModuleAdminExecuteSequence"; | 984 | sequenceTableName = "ModuleAdminExecuteSequence"; |
983 | } | 985 | } |
984 | else | 986 | else |
@@ -987,9 +989,9 @@ namespace WixToolset.Core.WindowsInstaller.Bind | |||
987 | } | 989 | } |
988 | break; | 990 | break; |
989 | case SequenceTable.AdminUISequence: | 991 | case SequenceTable.AdminUISequence: |
990 | if (OutputType.Module == this.Output.Type) | 992 | if (OutputType.Module == this.Data.Type) |
991 | { | 993 | { |
992 | this.Output.EnsureTable(this.TableDefinitions["AdminUISequence"]); | 994 | this.Data.EnsureTable(this.TableDefinitions["AdminUISequence"]); |
993 | sequenceTableName = "ModuleAdminUISequence"; | 995 | sequenceTableName = "ModuleAdminUISequence"; |
994 | } | 996 | } |
995 | else | 997 | else |
@@ -998,9 +1000,9 @@ namespace WixToolset.Core.WindowsInstaller.Bind | |||
998 | } | 1000 | } |
999 | break; | 1001 | break; |
1000 | case SequenceTable.AdvertiseExecuteSequence: | 1002 | case SequenceTable.AdvertiseExecuteSequence: |
1001 | if (OutputType.Module == this.Output.Type) | 1003 | if (OutputType.Module == this.Data.Type) |
1002 | { | 1004 | { |
1003 | this.Output.EnsureTable(this.TableDefinitions["AdvtExecuteSequence"]); | 1005 | this.Data.EnsureTable(this.TableDefinitions["AdvtExecuteSequence"]); |
1004 | sequenceTableName = "ModuleAdvtExecuteSequence"; | 1006 | sequenceTableName = "ModuleAdvtExecuteSequence"; |
1005 | } | 1007 | } |
1006 | else | 1008 | else |
@@ -1009,9 +1011,9 @@ namespace WixToolset.Core.WindowsInstaller.Bind | |||
1009 | } | 1011 | } |
1010 | break; | 1012 | break; |
1011 | case SequenceTable.InstallExecuteSequence: | 1013 | case SequenceTable.InstallExecuteSequence: |
1012 | if (OutputType.Module == this.Output.Type) | 1014 | if (OutputType.Module == this.Data.Type) |
1013 | { | 1015 | { |
1014 | this.Output.EnsureTable(this.TableDefinitions["InstallExecuteSequence"]); | 1016 | this.Data.EnsureTable(this.TableDefinitions["InstallExecuteSequence"]); |
1015 | sequenceTableName = "ModuleInstallExecuteSequence"; | 1017 | sequenceTableName = "ModuleInstallExecuteSequence"; |
1016 | } | 1018 | } |
1017 | else | 1019 | else |
@@ -1020,9 +1022,9 @@ namespace WixToolset.Core.WindowsInstaller.Bind | |||
1020 | } | 1022 | } |
1021 | break; | 1023 | break; |
1022 | case SequenceTable.InstallUISequence: | 1024 | case SequenceTable.InstallUISequence: |
1023 | if (OutputType.Module == this.Output.Type) | 1025 | if (OutputType.Module == this.Data.Type) |
1024 | { | 1026 | { |
1025 | this.Output.EnsureTable(this.TableDefinitions["InstallUISequence"]); | 1027 | this.Data.EnsureTable(this.TableDefinitions["InstallUISequence"]); |
1026 | sequenceTableName = "ModuleInstallUISequence"; | 1028 | sequenceTableName = "ModuleInstallUISequence"; |
1027 | } | 1029 | } |
1028 | else | 1030 | else |
@@ -1147,14 +1149,14 @@ namespace WixToolset.Core.WindowsInstaller.Bind | |||
1147 | private void AddWixEnsureTableSymbol(WixEnsureTableSymbol symbol) | 1149 | private void AddWixEnsureTableSymbol(WixEnsureTableSymbol symbol) |
1148 | { | 1150 | { |
1149 | var tableDefinition = this.TableDefinitions[symbol.Table]; | 1151 | var tableDefinition = this.TableDefinitions[symbol.Table]; |
1150 | this.Output.EnsureTable(tableDefinition); | 1152 | this.Data.EnsureTable(tableDefinition); |
1151 | } | 1153 | } |
1152 | 1154 | ||
1153 | private bool AddSymbolFromExtension(IntermediateSymbol symbol) | 1155 | private bool AddSymbolFromExtension(IntermediateSymbol symbol) |
1154 | { | 1156 | { |
1155 | foreach (var extension in this.BackendExtensions) | 1157 | foreach (var extension in this.BackendExtensions) |
1156 | { | 1158 | { |
1157 | if (extension.TryProcessSymbol(this.Section, symbol, this.Output, this.TableDefinitions)) | 1159 | if (extension.TryProcessSymbol(this.Section, symbol, this.Data, this.TableDefinitions)) |
1158 | { | 1160 | { |
1159 | return true; | 1161 | return true; |
1160 | } | 1162 | } |
@@ -1164,7 +1166,7 @@ namespace WixToolset.Core.WindowsInstaller.Bind | |||
1164 | } | 1166 | } |
1165 | 1167 | ||
1166 | private bool AddSymbolDefaultly(IntermediateSymbol symbol) => | 1168 | private bool AddSymbolDefaultly(IntermediateSymbol symbol) => |
1167 | this.BackendHelper.TryAddSymbolToOutputMatchingTableDefinitions(this.Section, symbol, this.Output, this.TableDefinitions); | 1169 | this.BackendHelper.TryAddSymbolToMatchingTableDefinitions(this.Section, symbol, this.Data, this.TableDefinitions); |
1168 | 1170 | ||
1169 | private static OutputType SectionTypeToOutputType(SectionType type) | 1171 | private static OutputType SectionTypeToOutputType(SectionType type) |
1170 | { | 1172 | { |
@@ -1190,7 +1192,7 @@ namespace WixToolset.Core.WindowsInstaller.Bind | |||
1190 | this.CreateRow(symbol, this.TableDefinitions[tableDefinitionName]); | 1192 | this.CreateRow(symbol, this.TableDefinitions[tableDefinitionName]); |
1191 | 1193 | ||
1192 | private Row CreateRow(IntermediateSymbol symbol, TableDefinition tableDefinition) => | 1194 | private Row CreateRow(IntermediateSymbol symbol, TableDefinition tableDefinition) => |
1193 | this.BackendHelper.CreateRow(this.Section, symbol, this.Output, tableDefinition); | 1195 | this.BackendHelper.CreateRow(this.Section, symbol, this.Data, tableDefinition); |
1194 | 1196 | ||
1195 | private static string GetMsiFilenameValue(string shortName, string longName) | 1197 | private static string GetMsiFilenameValue(string shortName, string longName) |
1196 | { | 1198 | { |
diff --git a/src/WixToolset.Core.WindowsInstaller/ExtensibilityServices/WindowsInstallerBackendHelper.cs b/src/WixToolset.Core.WindowsInstaller/ExtensibilityServices/WindowsInstallerBackendHelper.cs index 248dea9b..09194d4e 100644 --- a/src/WixToolset.Core.WindowsInstaller/ExtensibilityServices/WindowsInstallerBackendHelper.cs +++ b/src/WixToolset.Core.WindowsInstaller/ExtensibilityServices/WindowsInstallerBackendHelper.cs | |||
@@ -34,9 +34,9 @@ namespace WixToolset.Core.WindowsInstaller.ExtensibilityServices | |||
34 | 34 | ||
35 | #region IWindowsInstallerBackendHelper interfaces | 35 | #region IWindowsInstallerBackendHelper interfaces |
36 | 36 | ||
37 | public Row CreateRow(IntermediateSection section, IntermediateSymbol symbol, WindowsInstallerData output, TableDefinition tableDefinition) | 37 | public Row CreateRow(IntermediateSection section, IntermediateSymbol symbol, WindowsInstallerData data, TableDefinition tableDefinition) |
38 | { | 38 | { |
39 | var table = output.EnsureTable(tableDefinition); | 39 | var table = data.EnsureTable(tableDefinition); |
40 | 40 | ||
41 | var row = table.CreateRow(symbol.SourceLineNumbers); | 41 | var row = table.CreateRow(symbol.SourceLineNumbers); |
42 | row.SectionId = section.Id; | 42 | row.SectionId = section.Id; |
@@ -44,7 +44,7 @@ namespace WixToolset.Core.WindowsInstaller.ExtensibilityServices | |||
44 | return row; | 44 | return row; |
45 | } | 45 | } |
46 | 46 | ||
47 | public bool TryAddSymbolToOutputMatchingTableDefinitions(IntermediateSection section, IntermediateSymbol symbol, WindowsInstallerData output, TableDefinitionCollection tableDefinitions) | 47 | public bool TryAddSymbolToMatchingTableDefinitions(IntermediateSection section, IntermediateSymbol symbol, WindowsInstallerData data, TableDefinitionCollection tableDefinitions) |
48 | { | 48 | { |
49 | var tableDefinition = tableDefinitions.FirstOrDefault(t => t.SymbolDefinition?.Name == symbol.Definition.Name); | 49 | var tableDefinition = tableDefinitions.FirstOrDefault(t => t.SymbolDefinition?.Name == symbol.Definition.Name); |
50 | if (tableDefinition == null) | 50 | if (tableDefinition == null) |
@@ -52,7 +52,7 @@ namespace WixToolset.Core.WindowsInstaller.ExtensibilityServices | |||
52 | return false; | 52 | return false; |
53 | } | 53 | } |
54 | 54 | ||
55 | var row = this.CreateRow(section, symbol, output, tableDefinition); | 55 | var row = this.CreateRow(section, symbol, data, tableDefinition); |
56 | var rowOffset = 0; | 56 | var rowOffset = 0; |
57 | 57 | ||
58 | if (tableDefinition.SymbolIdIsPrimaryKey) | 58 | if (tableDefinition.SymbolIdIsPrimaryKey) |