aboutsummaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorBob Arnson <bob@firegiant.com>2020-01-06 14:54:31 -0500
committerBob Arnson <bob@firegiant.com>2020-01-06 15:04:11 -0500
commit69651a71ff2fdc1e9897b878782d79dcc1f9b896 (patch)
tree103699cd0d8a3f8887c3f0b04f33f411fde62789 /src
parent427e1a8c65c038e63674503f4f56e53fe00ac7f0 (diff)
downloadwix-69651a71ff2fdc1e9897b878782d79dcc1f9b896.tar.gz
wix-69651a71ff2fdc1e9897b878782d79dcc1f9b896.tar.bz2
wix-69651a71ff2fdc1e9897b878782d79dcc1f9b896.zip
Ensure Errors have ids so they can be referenced.
Diffstat (limited to 'src')
-rw-r--r--src/WixToolset.Core/Compiler.cs2
-rw-r--r--src/test/WixToolsetTest.CoreIntegration/MsiFixture.cs6
-rw-r--r--src/test/WixToolsetTest.CoreIntegration/TestData/ErrorsInUI/Package.wxs3
3 files changed, 10 insertions, 1 deletions
diff --git a/src/WixToolset.Core/Compiler.cs b/src/WixToolset.Core/Compiler.cs
index 2a020ad5..6b9fe59e 100644
--- a/src/WixToolset.Core/Compiler.cs
+++ b/src/WixToolset.Core/Compiler.cs
@@ -5297,7 +5297,7 @@ namespace WixToolset.Core
5297 5297
5298 if (!this.Core.EncounteredError) 5298 if (!this.Core.EncounteredError)
5299 { 5299 {
5300 var tuple = new ErrorTuple(sourceLineNumbers) 5300 var tuple = new ErrorTuple(sourceLineNumbers, new Identifier(AccessModifier.Public, id))
5301 { 5301 {
5302 Error = id, 5302 Error = id,
5303 Message = Common.GetInnerText(node) 5303 Message = Common.GetInnerText(node)
diff --git a/src/test/WixToolsetTest.CoreIntegration/MsiFixture.cs b/src/test/WixToolsetTest.CoreIntegration/MsiFixture.cs
index 81289e5a..a12ad469 100644
--- a/src/test/WixToolsetTest.CoreIntegration/MsiFixture.cs
+++ b/src/test/WixToolsetTest.CoreIntegration/MsiFixture.cs
@@ -280,6 +280,12 @@ namespace WixToolsetTest.CoreIntegration
280 var errors = section.Tuples.OfType<ErrorTuple>().ToDictionary(t => t.Error); 280 var errors = section.Tuples.OfType<ErrorTuple>().ToDictionary(t => t.Error);
281 Assert.Equal("Category 55 Emergency Doomsday Crisis", errors[1234].Message.Trim()); 281 Assert.Equal("Category 55 Emergency Doomsday Crisis", errors[1234].Message.Trim());
282 Assert.Equal(" ", errors[5678].Message); 282 Assert.Equal(" ", errors[5678].Message);
283
284 var customAction1 = section.Tuples.OfType<CustomActionTuple>().Where(t => t.Id.Id == "CanWeReferenceAnError_YesWeCan").Single();
285 Assert.Equal("1234", customAction1.Target);
286
287 var customAction2 = section.Tuples.OfType<CustomActionTuple>().Where(t => t.Id.Id == "TextErrorsWorkOKToo").Single();
288 Assert.Equal("If you see this, something went wrong.", customAction2.Target);
283 } 289 }
284 } 290 }
285 291
diff --git a/src/test/WixToolsetTest.CoreIntegration/TestData/ErrorsInUI/Package.wxs b/src/test/WixToolsetTest.CoreIntegration/TestData/ErrorsInUI/Package.wxs
index 6da3dcbe..a4c01d7e 100644
--- a/src/test/WixToolsetTest.CoreIntegration/TestData/ErrorsInUI/Package.wxs
+++ b/src/test/WixToolsetTest.CoreIntegration/TestData/ErrorsInUI/Package.wxs
@@ -6,6 +6,9 @@
6 <MajorUpgrade DowngradeErrorMessage="!(loc.DowngradeError)" /> 6 <MajorUpgrade DowngradeErrorMessage="!(loc.DowngradeError)" />
7 <MediaTemplate /> 7 <MediaTemplate />
8 8
9 <CustomAction Id="CanWeReferenceAnError_YesWeCan" Error="1234" />
10 <CustomAction Id="TextErrorsWorkOKToo" Error="If you see this, something went wrong." />
11
9 <Feature Id="ProductFeature" Title="!(loc.FeatureTitle)"> 12 <Feature Id="ProductFeature" Title="!(loc.FeatureTitle)">
10 <ComponentGroupRef Id="ProductComponents" /> 13 <ComponentGroupRef Id="ProductComponents" />
11 </Feature> 14 </Feature>