C#•4w ago

✅ Ef Core unexpected query behaviour

        var memberships = await db.OrganizationMembers.AsTracking()
            .Include(om => om.Organization)
            .Where(om => om.UserId == user.Id && om.InvitationStatus == InvitationStatus.Approved &&
                         om.Role > MemberRole.None)
            .ToListAsync(cancellationToken);

        var memberships = await db.OrganizationMembers.AsTracking()
            .Include(om => om.Organization)
            .Where(om => om.UserId == user.Id && om.InvitationStatus == InvitationStatus.Approved &&
                         om.Role > MemberRole.None)
            .ToListAsync(cancellationToken);

/// <summary>
///     Represents an enumeration of member roles.
/// </summary>
public enum MemberRole
{
    /// <summary>
    ///     Represents the None role.
    /// </summary>
    None,

    /// <summary>
    ///     Represents the Read role.
    /// </summary>
    Read,

    /// <summary>
    ///     Represents the Write role.
    /// </summary>
    Write,

    /// <summary>
    ///     Represents the Administrator role.
    /// </summary>
    Admin
}

/// <summary>
///     Represents an enumeration of member roles.
/// </summary>
public enum MemberRole
{
    /// <summary>
    ///     Represents the None role.
    /// </summary>
    None,

    /// <summary>
    ///     Represents the Read role.
    /// </summary>
    Read,

    /// <summary>
    ///     Represents the Write role.
    /// </summary>
    Write,

    /// <summary>
    ///     Represents the Administrator role.
    /// </summary>
    Admin
}

For some reason it's not including MemberRole.admin. It's including MemberRole.Write?

27 Replies

V0FBU1VM•4w ago

After changing the comparison statement to om.Role != MemberRole.None all of a sudden it's working. I would still like to understand why though?

Unknown User•4w ago

Message Not Public

Anton•4w ago

That is how it works unless you copied MemberRole or the check wrong

Unknown User•4w ago

Message Not Public

Anton•4w ago

I kind of disagree. It makes sense if you can order the roles like this by the magnitude of privilege. It may make sense to wrap it in a struct to denote which operations are supposed to be used, because it may not be obvious that the members in the enum are ordered accordingly

Unknown User•4w ago

Message Not Public

MODiX•4w ago

TeBeCo

else nothing's wrong out of the ordinary you need to debug your data and see what's going on

Quoted by

<@689473681302224947> from #Ef Core unexpected query behaviour (click here)

React with ❌ to remove this embed.

V0FBU1VM•4w ago

I thought it would just compare the value assigned to the enum which is an int. So it sort of made sense. I guess om.Role != MemberRole.None makes more sense.

Unknown User•4w ago

Message Not Public

V0FBU1VM•4w ago

I do have a conversion though which might have led to the issue.

builder.Property(e => e.Role)
            .HasConversion(
                v => v.ToString(),
                v => Enum.Parse<MemberRole>(v))
            .IsRequired();

builder.Property(e => e.Role)
            .HasConversion(
                v => v.ToString(),
                v => Enum.Parse<MemberRole>(v))
            .IsRequired();

Unknown User•4w ago

Message Not Public

V0FBU1VM•4w ago

I see what you mean. Very good point.

Unknown User•4w ago

Message Not Public

V0FBU1VM•4w ago

It's a bit easier to read on the database side. Since I'm doing a lot of debuging at this stage.

Unknown User•4w ago

Message Not Public

V0FBU1VM•4w ago

Not really no. 🤣

Unknown User•4w ago

Message Not Public

Anton•4w ago

you can make a table for roles and join with it when you need a human readable output

Unknown User•4w ago

Message Not Public

Anton•4w ago

that's what people usually do

Unknown User•4w ago

Message Not Public

V0FBU1VM•4w ago

To be honest, I'm not very familiar with sql directly. I don't have a lot of knowledge in that field. That's why I'm using ef core.

Unknown User•4w ago

Message Not Public

V0FBU1VM•4w ago

Oh I see. Thank you guys very much for the input. I learned some new stuffs.

Anton•4w ago

don't avoid learning SQL

V0FBU1VM•4w ago

I have always wanted to learn. I just forget the syntax for some reason. 🤣

MODiX•4w ago

If you have no further questions, please use /close to mark the forum thread as answered

Gaming

Programming

✅ Ef Core unexpected query behaviour