NullReferenceException: When "Name" optionally not supplied to [Column] Attribute

[z0nekill]

Hello! I've been experimenting with your library in .NET 8 and I really like it - I'll cut straight to the chase:

SqlBulkHelpers/NetStandard.SqlBulkHelpers/SqlBulkHelper/QueryProcessing/SqlBulkHelpersProcessingDefinition.cs

Line 205 in 634737e

return attrAccessor[MappingAttributeNames.LinqToDbFieldMapAttributePropertyName].ToString();

Has a minor bug where it presumes that the Attribute name is required, it is not. When calling ToString on the Dictionary indexer throws the NullReferenceException

Example:

[Table("AppHit")]
public partial class AppHit
{
    [Key]
    [Column("HitID")]
    public Guid HitId { get; set; }

    [Column("SessionID")]
    public Guid SessionId { get; set; }

    // Issue => Name Optionally set
    [Column(TypeName = "datetime")]
    public DateTime Timestamp { get; set; }
}

In cases such as these I believe it would be acceptable to fall back to propInfo.Name.

Unfortunately, .NET developers using Database First projects cannot force the Name to be required, as the above file is generated automatically.

[cajuncoding]

Ah, ok. I don’t use that attribute much so didn’t realize the Name property was optional.

Yes then the default should definitely fall back as if no Column attribute was specified…because no name was specified.

Thanks for the heads up, lets get that stability issue resolved 👍

[cajuncoding]

Now resolved in v2.4.5 available in Nuget.

[z0nekill]

Now resolved in v2.4.5 available in Nuget.

Thanks for your help - the library is a beast, and is handling significant loads in our production environments very well.