Skip to content

Commit

Permalink
Merge pull request #59 from snowflakedb/pr-55
Browse files Browse the repository at this point in the history
Pr 55
  • Loading branch information
howryu authored Aug 31, 2018
2 parents 8bc422e + aa23c01 commit 220a124
Showing 1 changed file with 21 additions and 20 deletions.
41 changes: 21 additions & 20 deletions Snowflake.Data/Client/SnowflakeDbParameter.cs
Original file line number Diff line number Diff line change
Expand Up @@ -54,14 +54,14 @@ public override ParameterDirection Direction

public override bool IsNullable
{
get
{
throw new NotImplementedException();
}
get { return false; }

set
{
throw new NotImplementedException();
if (value != false)
{
throw new SnowflakeDbException(SFError.UNSUPPORTED_FEATURE);
}
}
}

Expand All @@ -73,40 +73,41 @@ public override string ParameterName

public override int Size
{
get
{
throw new NotImplementedException();
}
get { return 0; }

set
{
throw new NotImplementedException();
if (value != 0)
{
throw new SnowflakeDbException(SFError.UNSUPPORTED_FEATURE);
}
}
}

public override string SourceColumn
{
get
{
throw new NotImplementedException();
}
get { return null; }

set
{
throw new NotImplementedException();
if (value != null)
{
throw new SnowflakeDbException(SFError.UNSUPPORTED_FEATURE);
}
}
}

public override bool SourceColumnNullMapping
{
get
{
throw new NotImplementedException();
}
get { return false; }

set
{
throw new NotImplementedException();
// ReSharper disable once RedundantBoolCompare (compare to false for clarity)
if (value != false)
{
throw new SnowflakeDbException(SFError.UNSUPPORTED_FEATURE);
}
}
}

Expand Down

0 comments on commit 220a124

Please sign in to comment.