In this class there is code that list like this

//Check if there were any validation errors
if (validationErrors.Count == 0)
{
if (ID <= 0)
{
//Add
ID = new ENTRoleCapabilityData().Insert(db, ENTRoleId, Capability.ID, Convert.ToByte(AccessFlag), userAccountId);
}
else
{
//Update
if (!new ENTRoleCapabilityData().Update(db, ID, ENTRoleId, Capability.ID, Convert.ToByte(AccessFlag), userAccountId, Version))
{
UpdateFailed(ref validationErrors);
returnfalse;
}
}

Shouldn't this section


ID = new ENTRoleCapabilityData().Insert(db, ENTRoleId, Capability.ID, Convert.ToByte(AccessFlag), userAccountId);
}

be replaced with this


ID = new ENTRoleCapabilityData().Insert(db, ENTRoleId, ENTCapabilityId, Convert.ToByte(AccessFlag), userAccountId);
}

in both the insert and update parts

Short answer, no :)

ENTRoleCapabilityEO has a property (Capability) that is of type ENTCapabilityBO. There is no ENTCapabilityId property in ENTRoleCapabilityEO - instead, the class loads the associated capability object into Capability.

You could, I suppose, add an ENTCapabilityId property, but that would be redundant since you can get the value from Capability.ID (unless you removed the Capability property. But then you'd lose the convenience of having the related Capability object information readily available within ENTRoleCapability.)

Tim

When I run this code the way it is, I only get one capabilityId, when ther counld be more than on for a EntRoleCapabilityEO.

So as the code is written I fail on insert, but the way wrote I get the capabilityId associated with a roleId, a roleId can have more than capabilityId.

So something I must be doing wrong. Because my Capability.ID is either 0 or stays the same as the first one selected.

So the second insert into the EntRoleCapabilityEO fails.