Feature Requests item #1968090, was opened at 2008-05-20 10:07
Message generated for change (Settings changed) made by jasongin
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=642717&aid=1968090&group_id=105970
Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
>Category: dtf
Group: None
Status: Open
Priority: 5
Private: No
Submitted By: Christopher Painter (chrpai)
>Assigned to: Jason Ginchereau (jasongin)
Summary: Evaluating Non-Defined Conditions
Initial Comment:
Session.Evaluatecondition throws an exception if passed a null. This can be
prevented by checking for string.IsNullOrEmpty prior to calling the method but
I'm wondering that since an empty condition in Windows Installer evaluates to
TRUE that perhaps it might be simpler for developers consuming the method if
the method simply returned true if passed an empty argument.
Just a thought.
System.ArgumentNullException: Value cannot be null.
Parameter name: condition
at Microsoft.Deployment.WindowsInstaller.Session.EvaluateCondition(String
condition)
if (
string.IsNullOrEmpty( condition ) || session.EvaluateCondition(condition)
== true)
----------------------------------------------------------------------
Comment By: Jason Ginchereau (jasongin)
Date: 2008-05-20 10:17
Message:
Logged In: YES
user_id=2086430
Originator: NO
There is a Session.EvaluateCondition(string condition, bool defaultValue)
overload. However I agree it would make more sense to just default to true
when the condition is empty.
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=642717&aid=1968090&group_id=105970
-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
WiX-devs mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/wix-devs
