Skip to content

Commit

Permalink
TODO comment update - confirmed System.DBNull is not available in PCL.
Browse files Browse the repository at this point in the history
  • Loading branch information
Travis@tillig-win8-vm committed Dec 14, 2012
1 parent 4cde665 commit bf188bd
Showing 1 changed file with 4 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -53,7 +53,10 @@ public override bool CanSupplyValue(ParameterInfo pi, IComponentContext context,
throw new ArgumentNullException("pi");
}
// System.DBNull is not included in PCL even though it seems to be available in the selected targets.
// TODO:Make sure the documentation for Metro Style apps is correct and that System.DBNull is available.
// Verified through experimentation 12/14/2012 - PCL initial release in VS 2012 does not support System.DBNull
// even though the documentation claims it's available. It doesn't appear to matter what the target
// framework combination is - .NET for Windows Store apps, Windows Phone, Silverlight... it's never
// available.
// http://msdn.microsoft.com/en-us/library/windows/apps/system.dbnull(v=vs.110).aspx

var hasDefaultValue = pi.DefaultValue == null || pi.DefaultValue.GetType().FullName != "System.DBNull";
Expand Down

0 comments on commit bf188bd

Please sign in to comment.