Add Support for money Store Type in Data Annotations

[dsum27]

Description

This pull request adds functionality to handle the money store type when generating data annotations for entity properties. The change ensures that money-typed columns in the database are correctly represented in the code with [Column(TypeName = "money")].

Changes Made

• Updated logic to check for the money store type when useDecimalDataAnnotation is enabled.
• Added an additional else if block to append the appropriate [Column] attribute for properties with money type.

Code Example

if (property.StoreType == "decimal" && useDecimalDataAnnotation)
{
    Sb.AppendLine($"[Column(\"{property.Name}\", TypeName = \"{property.StoreType}({property.Precision},{property.Scale})\")]");
}
else if (property.StoreType == "money" && useDecimalDataAnnotation)
{
    Sb.AppendLine($"[Column(\"{property.Name}\", TypeName = \"{property.StoreType}\")]");
}
else
{
    if (!string.IsNullOrEmpty(propertyNameAndAttribute.Item2))
    {
        Sb.AppendLine(propertyNameAndAttribute.Item2);
    }
}

Why This Change is Needed

• Improves support for database schemas that use money columns.
• Ensures that generated entity classes correctly reflect database column types, enhancing accuracy and maintainability.

Testing

• Verified that properties with StoreType = "money" generate [Column(TypeName = "money")] annotations when useDecimalDataAnnotation is enabled.
• Confirmed no regressions for other store types (decimal, etc.).

Impact

• This change is backward compatible and does not affect existing functionality for other store types.

Related Issues

If applicable, reference issue numbers or related discussions:

• Closes #Warning About Missing Store Type for Decimal Properties #2746

[ErikEJ]

Looks good, how did you verify the change works as expected?

[dsum27]

@ErikEJ I verified the change by running the project locally in a WSL environment on Windows to ensure it functions as expected.

[ErikEJ]

Sorry, which "project" - one way to verify is to launch the efcpt.8 Console app, adjust the command line (connection string) and the config to add changes that triggers your new feature.

[dsum27]

Yes, sorry. I ran the cfcpt.8 console app with adjustments to the args with my connection string and config values.

[ErikEJ]

@dsum27 Cool, thanks for confirming and thanks for your contribution!