feature request #3133
security: preparation and first steps for Role, CdmAuthority and GrantedAuthorityImpl implement GrantedAuthority
0%
Description
There is no relation in the api between Role, CdmAuthority and GrantedAuthorityImpl they are connected only via the string representations. But all are GrantedAuthorities and thus should implement the interface and should provide conversion functionality which allows to convert from Role and CdmAuthority to GrantedAuthorityImpl and vice versa.
old subject:
security: merge Role enum and CdmAuthority into GrantedAuthorityImpl
old description:
There is now an enum for roles: @GrantedAuthorityImpl.Role@.
It would make more sense however to have all roles as final static instances of GrantedAuthorityImpl and it would also make sense to merge eu.etaxonomy.cdm.persistence.hibernate.permission.CdmAuthority into GrantedAuthorityImpl => 7099
Related issues
Associated revisions
fixing #3133
fixing #3133 (security: Role, CdmAuthority and GrantedAuthorityImpl implement GrantedAuthority)
History
#1 Updated by Andreas Kohlbecker over 8 years ago
- Status changed from New to In Progress
hi cherian I will start working on this today
#2 Updated by Andreas Kohlbecker over 8 years ago
- Subject changed from security: merge Role enum and CdmAuthority into GrantedAuthorityImpl to security: relate Role, CdmAuthority and GrantedAuthorityImpl to each other
completely merging all classes together seems not to make fully sense. So we decided to provide some conversion functionality which allows to convert from Role and CdmAuthority to GrantedAuthorityImpl and vice versa. Role and CdmAuthority should be in the cdmlib-persistence package.
I am also changing the subject of this ticket accordingly.
#3 Updated by Andreas Kohlbecker over 8 years ago
- Subject changed from security: relate Role, CdmAuthority and GrantedAuthorityImpl to each other to security: Role, CdmAuthority and GrantedAuthorityImpl implement GrantedAuthority
#4 Updated by Andreas Kohlbecker about 7 years ago
- Target version changed from cdmlib RELEASE 3.3.0 to cdmlib RELEASE 3.3.1
moving unresolved tickets to next release milestone due release of version 3.3
#5 Updated by Andreas Kohlbecker almost 6 years ago
- Target version changed from cdmlib RELEASE 3.5.0 to cdmlib RELEASE 3.5.1
moving tickets to next milestone
#6 Updated by Andreas Müller almost 6 years ago
- Target version changed from cdmlib RELEASE 3.5.1 to cdmlib RELEASE 3.5.2
move open 3.5.1 tickets to next milestone after release
#7 Updated by Katja Luther almost 6 years ago
- Keywords set to security,
- Target version changed from cdmlib RELEASE 3.5.2 to cdmlib - Reviewed Next Major Release
- Priority changed from Priority08 to Priority11
#8 Updated by Andreas Müller about 3 years ago
- Copied to bug #7099: Make CdmAuthority a persistable class added
#9 Updated by Andreas Müller about 3 years ago
- Description updated (diff)
Are there still open issues?
#10 Updated by Andreas Kohlbecker about 3 years ago
- Subject changed from security: Role, CdmAuthority and GrantedAuthorityImpl implement GrantedAuthority to security: preparation and first steps for Role, CdmAuthority and GrantedAuthorityImpl implement GrantedAuthority
- Status changed from In Progress to Closed
- Target version deleted (
Reviewed Next Major Release)
all requirements defined in this issue are now covered by #7099, so we can close this ticked
#11 Updated by Andreas Kohlbecker about 3 years ago
- Private changed from Yes to No
