Search code examples
c#entity-framework-4.1ef-code-first

FirstOrDefault throws 'Sequence contains more than one matching element'


I have seen all over that most people are getting this error when using SingleOrDefault. I, however, am using FirstOrDefault. Has anyone seen this anomoly before? I am using the Repository Pattern in order to use Dependency Injection.

return context.Users.FirstOrDefault(p => p.Username.ToLower() == username.ToLower());

EDIT

See below: The error comes from internal code to the EntityFramework from what I can tell.

[InvalidOperationException: Sequence contains more than one matching element]
   System.Linq.Enumerable.SingleOrDefault(IEnumerable`1 source, Func`2 predicate) +2668318
   System.Data.Entity.ModelConfiguration.Conventions.IdKeyDiscoveryConventionImpl.MatchKeyProperty(EdmEntityType entityType, IEnumerable`1 primitiveProperties) +121
   System.Data.Entity.ModelConfiguration.Conventions.KeyDiscoveryConvention.System.Data.Entity.ModelConfiguration.Conventions.IEdmConvention<System.Data.Edm.EdmEntityType>.Apply(EdmEntityType entityType, EdmModel model) +72
   System.Data.Entity.ModelConfiguration.Conventions.IdKeyDiscoveryConvention.System.Data.Entity.ModelConfiguration.Conventions.IEdmConvention<System.Data.Edm.EdmEntityType>.Apply(EdmEntityType entityType, EdmModel model) +17
   System.Data.Entity.ModelConfiguration.Configuration.EdmConventionDispatcher.Dispatch(TEdmDataModelItem item) +100
   System.Data.Entity.ModelConfiguration.Configuration.EdmConventionDispatcher.VisitEdmEntityType(EdmEntityType item) +22
   System.Data.Edm.Internal.DataModelItemVisitor.VisitCollection(IEnumerable`1 collection, Action`1 visitMethod) +138
   System.Data.Edm.Internal.EdmModelVisitor.VisitEntityTypes(EdmNamespace edmNamespace, IEnumerable`1 entityTypes) +75
   System.Data.Edm.Internal.EdmModelVisitor.VisitEdmNamespace(EdmNamespace item) +88
   System.Data.Entity.ModelConfiguration.Configuration.EdmConventionDispatcher.VisitEdmNamespace(EdmNamespace item) +31
   System.Data.Edm.Internal.DataModelItemVisitor.VisitCollection(IEnumerable`1 collection, Action`1 visitMethod) +138
   System.Data.Edm.Internal.EdmModelVisitor.VisitNamespaces(EdmModel model, IEnumerable`1 namespaces) +75
   System.Data.Edm.Internal.EdmModelVisitor.VisitEdmModel(EdmModel item) +56
   System.Data.Entity.ModelConfiguration.Configuration.EdmConventionDispatcher.VisitEdmModel(EdmModel item) +44
   System.Data.Entity.ModelConfiguration.Configuration.ConventionsConfiguration.ApplyModel(EdmModel model) +126
   System.Data.Entity.DbModelBuilder.Build(DbProviderManifest providerManifest, DbProviderInfo providerInfo) +125
   System.Data.Entity.DbModelBuilder.Build(DbConnection providerConnection) +165
   System.Data.Entity.Internal.LazyInternalContext.CreateModel(LazyInternalContext internalContext) +61
   System.Data.Entity.Internal.RetryLazy`2.GetValue(TInput input) +111
   System.Data.Entity.Internal.LazyInternalContext.InitializeContext() +417
   System.Data.Entity.Internal.InternalContext.GetEntitySetAndBaseTypeForType(Type entityType) +18
   System.Data.Entity.Internal.Linq.InternalSet`1.Initialize() +63
   System.Data.Entity.Internal.Linq.InternalSet`1.get_InternalContext() +15
   System.Data.Entity.Infrastructure.DbQuery`1.System.Linq.IQueryable.get_Provider() +37
   System.Linq.Queryable.FirstOrDefault(IQueryable`1 source, Expression`1 predicate) +63
   Entities.User.GetCurrentPerson(String username, KmManagerDbContext context) in C:\Users\user\Documents\Visual Studio 2010\Projects\KmManager\Entities\User.cs:85

User.cs

public class User
{
    public long Id { get; set; }

    public string FirstName { get; set; }

    public string LastName { get; set; }

    public string Username { get; set; }

    // Custom Propreties
    public string FullName
    {
        get
        {
            return FirstName + " " + LastName;
        }
    }

    public string LastNameFirst
    {
        get
        {
            return LastName + ", " + FirstName;
        }
    }

    public static string TableName
    {
        get
        {
            return "Users";
        }
    }

    public static User GetCurrentPerson(string username, KmManagerDbContext context)
    {
        try
        {
            // find the person who has the ad name = username
            return context.Users.FirstOrDefault(p => p.Username.ToLower() == username.ToLower());
        }
        catch (Exception ex)
        {
            throw new ApplicationException("There was an error retrieving the user from the database.", ex);
        }
    }
}

UserConfiguration.cs

public UserConfiguration()
{
    this.ToTable(User.TableName);

    this.HasKey(x => x.Id);

    this.Property(x => x.Id).HasDatabaseGeneratedOption(DatabaseGeneratedOption.Identity);
    this.Property(x => x.FirstName).IsRequired();
    this.Property(x => x.LastName).IsRequired();
    this.Property(x => x.Username).IsRequired();
}

Solution

  • If anyone is worried about the answer...

    I split some commonalities amongst my POCOs using BaseEntity.cs

    BaseEntity.cs

    public class BaseEntity<T> where T : BaseEntity<T>
    {
        public long Id { get; set; }
    
        public class Comparer : IEqualityComparer<T>
        {
            public bool Equals(T x, T y)
            {
                if (x.Id == y.Id)
                {
                    return true;
                }
    
                return false;
            }
    
            public int GetHashCode(T obj)
            {
                return (int)obj.Id;
            }
        }
    }
    

    This caused the configuration to have wierd behavior. I changed all the POCOs to the previous state, and everything works as expected. Sorry for the waste of time.

    And User POCO looked like this...

    User.cs

    public class User : BaseEntity<User>
    {
        public string FirstName { get; set; } 
    
        public string LastName { get; set; } 
    
        public string Username { get; set; } 
    
        // Custom Propreties 
        public string FullName 
        { 
            get 
            { 
                return FirstName + " " + LastName; 
            } 
        } 
    
        public string LastNameFirst 
        { 
            get 
            { 
                return LastName + ", " + FirstName; 
            } 
        } 
    
        public static string TableName 
        { 
            get 
            { 
                return "Users"; 
            } 
        } 
    
        public static User GetCurrentPerson(string username, KmManagerDbContext context) 
        { 
            try 
            { 
                // find the person who has the ad name = username 
                return context.Users.FirstOrDefault(p => p.Username.ToLower() == username.ToLower()); 
            } 
            catch (Exception ex) 
            { 
                throw new ApplicationException("There was an error retrieving the user from the database.", ex); 
            } 
        } 
    }