Dapper error when deserializing to complex object

Question

I'm trying to convert a "flat" query into a complex object. The query returns custom_height and custom_width which I would like to be a child object off of the main Panel object being serialized. When I attempt to use splitOn to achieve this, I am getting a null ref exception from dapper but I'm not sure why.

My code looks like this

const string panelsSql = @"
SELECT p.panel_id, p.board_id, p.type_id, p.size_id, p.created_on, p.updated_on, p.custom_height, p.custom_width
FROM CAT.panels p
JOIN CAT.boards b ON b.board_id = p.board_id
WHERE p.board_id = @boardId
AND b.user_id = @userId;";
return await conn.Connection.QueryAsync<Panel, int, int, Panel>(
    panelsSql,
    (panel, width, height) => {
        panel.CustomSize = new Common.Objects.CustomPanelSize { Width = width, Height = height };
        return panel;
    },
    p,
    splitOn: "custom_height, custom_width").ConfigureAwait(false);

The exception I'm getting is

System.NullReferenceException: Object reference not set to an instance of an object.
   at Dapper.SqlMapper.<>c__DisplayClass158_0`8.<GenerateMapper>b__1(IDataReader r) in C:\projects\dapper\Dapper\SqlMapper.cs:line 1542
   at Dapper.SqlMapper.MultiMapImpl[TFirst,TSecond,TThird,TFourth,TFifth,TSixth,TSeventh,TReturn](IDbConnection cnn, CommandDefinition command, Delegate map, String splitOn, IDataReader reader, Identity identity, Boolean finalize)+MoveNext() in C:\projects\dapper\Dapper\SqlMapper.cs:line 1442
   at System.Collections.Generic.List`1..ctor(IEnumerable`1 collection)
   at System.Linq.Enumerable.ToList[TSource](IEnumerable`1 source)
   at Dapper.SqlMapper.MultiMapAsync[TFirst,TSecond,TThird,TFourth,TFifth,TSixth,TSeventh,TReturn](IDbConnection cnn, CommandDefinition command, Delegate map, String splitOn) in C:\projects\dapper\Dapper\SqlMapper.Async.cs:line 949
   at Thing.Common.Data.DbBase.GetBoardPanelsAsync(CgyDbType dbType, CurrentConfigs configs, Int32 userId, UInt32 boardId, Int32 maxRetries, Int32 retryDelay) in /Users/joephillips/repos/dotnet_e/Thing.Common.Data/DbBase.cs:line 6187

Answer 1

I don't know why you get the exception, but I would do like this:

const string panelsSql = @"
SELECT p.panel_id, p.board_id, p.type_id, p.size_id, p.created_on, p.updated_on, p.custom_height AS Height, p.custom_width AS Width
FROM CAT.panels p
JOIN CAT.boards b ON b.board_id = p.board_id
WHERE p.board_id = @boardId
AND b.user_id = @userId;";
return await conn.Connection.QueryAsync<Panel, Common.Objects.CustomPanelSize, Panel>(
    panelsSql,
    (panel, size) => {
        panel.CustomSize = size;
        return panel;
    },
    p,
    splitOn: "Width").ConfigureAwait(false);

I changed the SQL to return column names that are easier to map and simplified it to only two objects being returned. This requires CustomPanelSize to have a parameterless constructor.