C#•2mo 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

V0FBU1VMOP•2mo 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•2mo ago

Message Not Public

Anton•2mo ago

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

Unknown User•2mo ago

Message Not Public

Anton•2mo 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•2mo ago

Message Not Public

MODiX•2mo 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.

V0FBU1VMOP•2mo 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•2mo ago

Message Not Public

V0FBU1VMOP•2mo 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•2mo ago

Message Not Public

V0FBU1VMOP•2mo ago

I see what you mean. Very good point.

Unknown User•2mo ago

Message Not Public

V0FBU1VMOP•2mo 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•2mo ago

Message Not Public

V0FBU1VMOP•2mo ago

Not really no. 🤣

Unknown User•2mo ago

Message Not Public

Anton•2mo ago

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

Unknown User•2mo ago

Message Not Public

Anton•2mo ago

that's what people usually do

Unknown User•2mo ago

Message Not Public

V0FBU1VMOP•2mo 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•2mo ago

Message Not Public

V0FBU1VMOP•2mo ago

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

Anton•2mo ago

don't avoid learning SQL

V0FBU1VMOP•2mo ago

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

MODiX•2mo ago

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

Gaming

Programming

✅ Ef Core unexpected query behaviour