C#•4y ago

❔ Unable to get automapper's projectTo() to work properly.

I need to use ProjectTo() to properly cast a query into a DTO, but in this case my user can only contains pictures which have

Deleted = false

Deleted = false

Deleted = false

Deleted = false. The code below works, but all photo's are returned, including deleted ones. The commented

Include()

Include()

Include()

Include() works fine, but I need the mapping to happen on query level, and not after, because of the creation of a paginatedList. If I map later, the PaginatedList misses crucial information, and I do not want to change its behaviour.

public async Task<PaginatedList<DTODataUser>> GetAllDTOPaginatedAsync(PaginatedQuery pagination, bool includeDeleted, CancellationToken cancellationToken)
    {
        var userQuery = this._databaseContext
            .Users
            .Where(x => !includeDeleted ? !x.Deleted : true)
            .ProjectTo<DTODataUser>(this._mapper.ConfigurationProvider, null, (user) => user.Photos.Where(y => !includeDeleted ? !y.Deleted : true));
        //.Include(x => x.Photos.Where(y => !includeDeleted ? !y.Deleted : true))

        var users = await PaginatedList<DTODataUser>.CreateAsync(userQuery, pagination, cancellationToken);
        return users;
    }

public async Task<PaginatedList<DTODataUser>> GetAllDTOPaginatedAsync(PaginatedQuery pagination, bool includeDeleted, CancellationToken cancellationToken)
    {
        var userQuery = this._databaseContext
            .Users
            .Where(x => !includeDeleted ? !x.Deleted : true)
            .ProjectTo<DTODataUser>(this._mapper.ConfigurationProvider, null, (user) => user.Photos.Where(y => !includeDeleted ? !y.Deleted : true));
        //.Include(x => x.Photos.Where(y => !includeDeleted ? !y.Deleted : true))

        var users = await PaginatedList<DTODataUser>.CreateAsync(userQuery, pagination, cancellationToken);
        return users;
    }

public async Task<PaginatedList<DTODataUser>> GetAllDTOPaginatedAsync(PaginatedQuery pagination, bool includeDeleted, CancellationToken cancellationToken)
    {
        var userQuery = this._databaseContext
            .Users
            .Where(x => !includeDeleted ? !x.Deleted : true)
            .ProjectTo<DTODataUser>(this._mapper.ConfigurationProvider, null, (user) => user.Photos.Where(y => !includeDeleted ? !y.Deleted : true));
        //.Include(x => x.Photos.Where(y => !includeDeleted ? !y.Deleted : true))

        var users = await PaginatedList<DTODataUser>.CreateAsync(userQuery, pagination, cancellationToken);
        return users;
    }

public async Task<PaginatedList<DTODataUser>> GetAllDTOPaginatedAsync(PaginatedQuery pagination, bool includeDeleted, CancellationToken cancellationToken)
    {
        var userQuery = this._databaseContext
            .Users
            .Where(x => !includeDeleted ? !x.Deleted : true)
            .ProjectTo<DTODataUser>(this._mapper.ConfigurationProvider, null, (user) => user.Photos.Where(y => !includeDeleted ? !y.Deleted : true));
        //.Include(x => x.Photos.Where(y => !includeDeleted ? !y.Deleted : true))

        var users = await PaginatedList<DTODataUser>.CreateAsync(userQuery, pagination, cancellationToken);
        return users;
    }

How can I replicate

Include

Include

to work with projectTo? As said above, the current version is allowed, but deleted photo's are returned too.

Angius•12/3/22, 4:01 PM

You do it in automapper's config

AAngius You do it in automapper's config

FusedQyouOP•12/3/22, 4:06 PM

I just added .ExplicitExpansion().ExplicitExpansion(), but this did not change its behaviour

FusedQyouOP•12/3/22, 4:07 PM

FusedQyouOP•12/3/22, 4:07 PM

The conversion is DataUser -> DTODataUser

FusedQyouOP•12/3/22, 4:07 PM

Am I missing something?

Angius•12/3/22, 4:09 PM

Huh

Angius•12/3/22, 4:09 PM

Honestly, that's why I dropped AutoMapper, way too much fuss

Angius•12/3/22, 4:10 PM

But I think that you can have a .Where().Where() instead of that .ExplicitExpansion().ExplicitExpansion()..?

AAngius But I think that you can have a `.Where()` instead of that `.ExplicitExpansion()...

FusedQyouOP•12/3/22, 4:10 PM

Yeah, but then I need a way to determine if the includeDeletedincludeDeleted boolean is set

Angius•12/3/22, 4:11 PM

You can pass that to the mapper config

Angius•12/3/22, 4:11 PM

The way to do it is quite ugly, but there is a way

Angius•12/3/22, 4:11 PM

Again, one of the reasons I dropped it lol

FusedQyouOP•12/3/22, 4:11 PM

Do/Can can you suggest an alternative?

Angius•12/3/22, 4:12 PM

Just a .Select().Select()

Angius•12/3/22, 4:12 PM

If you need to reuse some mapping, make a class with static Expression<Func<TSource, TTarget>>Expression<Func<TSource, TTarget>> properties and use those

FusedQyouOP•12/3/22, 4:14 PM

Makes sense

FusedQyouOP•12/3/22, 4:14 PM

After all it's just a lazy way of using it

FusedQyouOP•12/3/22, 4:15 PM

But the reason why I need this is because PagedList inherits from List, and any properties in there are not serialized properly, and neither are they converted if I map later. I can also just fix this by not making it inherit from List

FusedQyouOP•12/3/22, 4:15 PM

I do kinda want ProjectTo to just work properly because otherwise I get too much data, but oh well

FusedQyouOP•12/3/22, 4:15 PM

I suppose I can still use a mapper outside of this service

Angius•12/3/22, 4:15 PM

static class UserMappings
{
    public static Expression<Func<User, UserDto>> ToDto = user => new UserDto {
        Name = user.UserName,
        Age = user.Age,
        Comments = user.Comments.Where(c => !c.IsDeleted)
    }

    
    public static Expression<Func<User, UserDto>> ToDtoWithParam(int param) => user => new UserDto {
        Name = user.UserName,
        Age = user.Age,
        Whatever = param > 10 ? user.Foo : user.Bae
    }
}

static class UserMappings
{
    public static Expression<Func<User, UserDto>> ToDto = user => new UserDto {
        Name = user.UserName,
        Age = user.Age,
        Comments = user.Comments.Where(c => !c.IsDeleted)
    }

    
    public static Expression<Func<User, UserDto>> ToDtoWithParam(int param) => user => new UserDto {
        Name = user.UserName,
        Age = user.Age,
        Whatever = param > 10 ? user.Foo : user.Bae
    }
}

Angius•12/3/22, 4:16 PM

An example of doing the mappings with Expression<Func<,>>Expression<Func<,>>

Angius•12/3/22, 4:16 PM

Then just .Select(UserMappings.ToDto).Select(UserMappings.ToDto)

Angius•12/3/22, 4:17 PM

Not sure how that paginated list of yours works, so can't speak on that

Angius•12/3/22, 4:17 PM

Me, I just have a simple extension method for IQueryableIQueryable

Angius•12/3/22, 4:19 PM

public static IQueryable<T> Paginate<T>(this IQueryable<T> query, int page, int perPage)
{
    if (page < 1)
        throw new ArgumentOutOfRangeException(nameof(page), "Page has to be greater than 0");
    if (perPage < 1)
        throw new ArgumentOutOfRangeException(nameof(perPage), "PerPage has to be greater than 0");

    return query
        .Skip(Math.Max(0, page - 1) * perPage)
        .Take(perPage);
}

public static IQueryable<T> Paginate<T>(this IQueryable<T> query, int page, int perPage)
{
    if (page < 1)
        throw new ArgumentOutOfRangeException(nameof(page), "Page has to be greater than 0");
    if (perPage < 1)
        throw new ArgumentOutOfRangeException(nameof(perPage), "PerPage has to be greater than 0");

    return query
        .Skip(Math.Max(0, page - 1) * perPage)
        .Take(perPage);
}

Accord•12/4/22, 4:47 PM

Was this issue resolved? If so, run

/close

/close

otherwise I will mark this as stale and this post will be archived until there is new activity.

❔ Unable to get automapper's projectTo() to work properly.

Similar Threads

Similar Threads

Similar Threads