eclipse-sailing-analytics/sailing-analytics
1
0
Fork 0
mirror of https://github.com/eclipse-sailing-analytics/sailing-analytics.git synced 2026-09-22 21:55:39 +00:00
Code Activity

bug5230: Introduced transitive flag to equals/hashcode functions of

Browse Source 
AbstractRole
This commit is contained in:
Dennis Aulenbacher
2021-03-08 14:30:12 +01:00
parent 2220d53c11
commit d353a0495a
1 changed files with 11 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
java/com.sap.sse.security.common/src/com/sap/sse/security/shared/AbstractRole.java
+11 -5
View File
@@ -98,8 +98,8 @@ public abstract class AbstractRole<RD extends RoleDefinition, G extends Security
/**
* For hashing, the {@link RoleDefinition#getId() role definition ID}, the {@link Tenant#getId() tenant ID} of a
* possible tenant qualifier as well as the {@link SecurityUser#getName() user name} of a possible user qualifier
* are hashed.
* possible tenant qualifier, the {@link SecurityUser#getName() user name} of a possible user qualifier, as well as
* the {@link AbstractRole#transitive transitive flag} are hashed.
*/
@Override
public int hashCode() {
@@ -108,13 +108,14 @@ public abstract class AbstractRole<RD extends RoleDefinition, G extends Security
result = prime * result + ((qualifiedForTenant == null) ? 0 : qualifiedForTenant.getId().hashCode());
result = prime * result + ((qualifiedForUser == null) ? 0 : qualifiedForUser.getName().hashCode());
result = prime * result + ((roleDefinition == null) ? 0 : roleDefinition.hashCode());
result = prime * result + ((transitive == null) ? 0 : transitive.hashCode());
return result;
}
/**
* For equality, the {@link RoleDefinition#getId() role definition ID}, the {@link Tenant#getId() tenant ID} of a
* possible tenant qualifier as well as the {@link SecurityUser#getName() user name} of a possible user qualifier
* are compared.
* For equality, the {@link RoleDefinition role definition}, the {@link Tenant#getId() tenant ID} of a possible
* tenant qualifier, the {@link SecurityUser#getName() user name} of a possible user qualifier, aswell as the
* {@link AbstractRole#transitive transitive flag} are compared.
*/
@SuppressWarnings("rawtypes")
@Override
@@ -147,6 +148,11 @@ public abstract class AbstractRole<RD extends RoleDefinition, G extends Security
return false;
} else if (!roleDefinition.equals(other.roleDefinition))
return false;
if (transitive == null) {
if (other.transitive != null)
return false;
} else if (!transitive.equals(other.transitive))
return false;
return true;
}
}