Dapper-Extensions icon indicating copy to clipboard operation
Dapper-Extensions copied to clipboard

Custom mappers

Open jgwconsulting opened this issue 2 years ago • 2 comments

I've been looking at creating a custom mapper but the example seems off to me (although it's probably me misunderstanding it!).

public class CustomPluralizedMapper<T> : PluralizedAutoClassMapper<T> where T : class 
{
    public override void Table(string tableName)
    {
        if(tableName.Equals("Person", StringComparison.CurrentCultureIgnoreCase))
        {
            TableName = "Persons";
        }

        base.Table(tableName);
    }
}

The code above appears to set TableName to "Persons" if the original table name is "Person" but it then calls base.Table passing the original tableName.

public virtual void Table(string tableName)
        {
            TableName = tableName;
        }

...which will set TableName back to "Person", won't it? I guess this will still work if the table name is always referenced from the overridden Table method, but it seems a bit odd.

jgwconsulting avatar Oct 06 '22 08:10 jgwconsulting

Hi, I'll check this example but, just reading the code, it's seems wrong. tableName should be changed to Persons, not TableName

valfrid-ly avatar Oct 06 '22 09:10 valfrid-ly

Ah cool, that's what I thought but I'm no stranger to getting things wrong, so I figured I'd check :) Thanks for all your hard work on this super handy project!

jgwconsulting avatar Oct 06 '22 11:10 jgwconsulting