CodeSmith Community

I have a table with a column named Value.

The NetTiers generated code for set/get results in a bug.

if (value == value)
{  return value; }
else
{
    this.value = value;
    return value;
}

Should be:

if (value == this.Value)
{  return value; }
else
{
    this.value = value;
    return value;
}

This can be fixed in the Entities\Views\EntityViewBase.generated.cst file, on lines 152 and 156.

Line 152 looks like:

if (<%= GetFieldName(SourceView.Columns[x]) %> == value && <%=GetPropertyName(SourceView.Columns[x])%> != null )

Change it to:

if (this.<%= GetFieldName(SourceView.Columns[x]) %> == value && this.<%=GetPropertyName(SourceView.Columns[x])%> != null )

Similarly, line 156 looks like:

Change it to:

if (this.<%= GetFieldName(SourceView.Columns[x]) %> == value)

This would be a good change to make to the official template, IMHO.  With XProp approaches, "value" is a natural and common column name.