UtilCompiler.cs setting Identifier to a row[] #5467

Open
nick2893 opened this Issue Jan 4, 2017 · 0 comments

Projects

None yet

3 participants

@nick2893
nick2893 commented Jan 4, 2017

Using Wix 4, September release; I'm trying to "candle" the following:

<util:ProductSearch Variable="VCRedistx86Installed" ProductCode="{F0C3E5D1-1ADE-321E-8167-68EF0DE699A5}" Result="state" />  <!-- http://wixtoolset.org/documentation/manual/v3/xsd/util/productsearch.html  GUID verified to be ProductCode after using DARK to decompile setup.exe -->
<util:ProductSearch Variable="VCRedistx64Installed" ProductCode="{1D8E6291-B0D5-35EC-8441-6616F567A0F7}" Result="state" />  <!-- http://wixtoolset.org/documentation/manual/v3/xsd/util/productsearch.html  GUID verified to be ProductCode after using DARK to decompile setup.exe -->

And I get an error:

candle.exe : error CNDL0001 : Cannot set string column 'WixSearch_' with a value
of type 'WixToolset.Extensibility.Identifier'.

Exception Type: System.InvalidOperationException

Stack Trace:
at WixToolset.Data.ColumnDefinition.ValidateValue(Object value)
at WixToolset.Data.Row.set_Item(Int32 field, Object value)
at WixToolset.Extensions.UtilCompiler.ParseProductSearchElement(XElement node
)
at WixToolset.Extensions.UtilCompiler.ParsePossibleKeyPathElement(XElement pa
rentElement, XElement element, IDictionary2 context) at WixToolset.Extensions.UtilCompiler.ParseElement(XElement parentElement, XE lement element, IDictionary2 context)
at WixToolset.CompilerCore.ParseExtensionElement(XElement parentElement, XEle
ment element, IDictionary`2 context)
at WixToolset.Compiler.ParseBundleElement(XElement node)
at WixToolset.Compiler.ParseWixElement(XElement node)
at WixToolset.Compiler.Compile(XDocument source)
at WixToolset.Tools.Candle.Run()
at WixToolset.Tools.Candle.Execute(String[] args)

I believe in UtilCompiler.cs, in private void ParseProductSearchElement(XElement node), the following line:

row[0] = id;

Should be:

row[0] = id.Id;

(similar code appears in multiple places)

@robmen robmen was assigned by barnson Jan 17, 2017
@barnson barnson added the bug label Jan 17, 2017
@barnson barnson added this to the v4.0 milestone Jan 17, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment