Page 178 needs to be address

tedr · April 3rd, 2009, 08:49 AM

in the ENTUserAccountEO class there's code that I think should be change or needs to be address.

Code:

 
publicENTRoleCapabilityEO.CapabilityAccessFlagEnum GetCapabilityAccess(int capabilityId, ENTRoleEOList rolesWithCapabilities)
{
ENTRoleCapabilityEO.CapabilityAccessFlagEnum retVal = ENTRoleCapabilityEO.CapabilityAccessFlagEnum.None;
//the roles in the user object do not include the capabilities.
foreach (ENTRoleEO role in Roles)
ENTRoleEO roleWithCapabilities = rolesWithCapabilities.GetByRoleId(role.ID);

this line needs to be change from this

Code:

foreach (ENTRoleEO role in Roles)
{

to

Code:

foreach (ENTRoleEO role in rolesWithCapabilities)
{

varallov · April 4th, 2009, 12:14 PM

The code in the book is correct. The foreach loop is looping through the Roles because the Roles property has all of the roles this user is in. The rolesWithCapabilities is the master list of roles with all the associated capabilities.