From 3998f77f29b290ab97d86f88000bfea954bc1e32 Mon Sep 17 00:00:00 2001 From: Timm Fitschen <timm.fitschen@ds.mpg.de> Date: Mon, 10 Dec 2018 23:09:48 +0100 Subject: [PATCH] STY: formatting --- .../permissions/AbstractEntityACLFactory.java | 21 +++++++------------ .../caosdb/server/permissions/EntityACL.java | 10 +++++---- .../server/permissions/EntityACLFactory.java | 11 ++++++++++ .../java/caosdb/server/permissions/Role.java | 2 +- 4 files changed, 25 insertions(+), 19 deletions(-) create mode 100644 src/main/java/caosdb/server/permissions/EntityACLFactory.java diff --git a/src/main/java/caosdb/server/permissions/AbstractEntityACLFactory.java b/src/main/java/caosdb/server/permissions/AbstractEntityACLFactory.java index 6460cb34..5d817471 100644 --- a/src/main/java/caosdb/server/permissions/AbstractEntityACLFactory.java +++ b/src/main/java/caosdb/server/permissions/AbstractEntityACLFactory.java @@ -31,14 +31,10 @@ import java.util.Map.Entry; public abstract class AbstractEntityACLFactory<T extends EntityACL> { - private final Map<ResponsibleAgent, Long> normalGrants = - new HashMap<>(); - private final Map<ResponsibleAgent, Long> priorityGrants = - new HashMap<>(); - private final Map<ResponsibleAgent, Long> normalDenials = - new HashMap<>(); - private final Map<ResponsibleAgent, Long> priorityDenials = - new HashMap<>(); + private final Map<ResponsibleAgent, Long> normalGrants = new HashMap<>(); + private final Map<ResponsibleAgent, Long> priorityGrants = new HashMap<>(); + private final Map<ResponsibleAgent, Long> normalDenials = new HashMap<>(); + private final Map<ResponsibleAgent, Long> priorityDenials = new HashMap<>(); public void grant(final ResponsibleAgent role, final int... permissionBitNumber) { grant(role, false, permissionBitNumber); @@ -183,8 +179,7 @@ public abstract class AbstractEntityACLFactory<T extends EntityACL> { set.getKey(), this.normalDenials.get(set.getKey()) & ~set.getValue()); } if (this.normalGrants.containsKey(set.getKey())) { - this.normalGrants.put( - set.getKey(), this.normalGrants.get(set.getKey()) & ~set.getValue()); + this.normalGrants.put(set.getKey(), this.normalGrants.get(set.getKey()) & ~set.getValue()); } } for (final Entry<ResponsibleAgent, Long> set : this.priorityGrants.entrySet()) { @@ -193,14 +188,12 @@ public abstract class AbstractEntityACLFactory<T extends EntityACL> { set.getKey(), this.normalDenials.get(set.getKey()) & ~set.getValue()); } if (this.normalGrants.containsKey(set.getKey())) { - this.normalGrants.put( - set.getKey(), this.normalGrants.get(set.getKey()) & ~set.getValue()); + this.normalGrants.put(set.getKey(), this.normalGrants.get(set.getKey()) & ~set.getValue()); } } for (final Entry<ResponsibleAgent, Long> set : this.normalDenials.entrySet()) { if (this.normalGrants.containsKey(set.getKey())) { - this.normalGrants.put( - set.getKey(), this.normalGrants.get(set.getKey()) & ~set.getValue()); + this.normalGrants.put(set.getKey(), this.normalGrants.get(set.getKey()) & ~set.getValue()); } } } diff --git a/src/main/java/caosdb/server/permissions/EntityACL.java b/src/main/java/caosdb/server/permissions/EntityACL.java index 154c2bf7..86346bdf 100644 --- a/src/main/java/caosdb/server/permissions/EntityACL.java +++ b/src/main/java/caosdb/server/permissions/EntityACL.java @@ -24,6 +24,10 @@ package caosdb.server.permissions; import static caosdb.server.permissions.Role.OTHER_ROLE; import static caosdb.server.permissions.Role.OWNER_ROLE; + +import caosdb.server.accessControl.AuthenticationUtils; +import caosdb.server.accessControl.Principal; +import caosdb.server.database.exceptions.TransactionException; import java.util.ArrayList; import java.util.BitSet; import java.util.Collection; @@ -36,9 +40,6 @@ import org.apache.shiro.subject.Subject; import org.eclipse.jetty.util.ajax.JSON; import org.jdom2.DataConversionException; import org.jdom2.Element; -import caosdb.server.accessControl.AuthenticationUtils; -import caosdb.server.accessControl.Principal; -import caosdb.server.database.exceptions.TransactionException; public class EntityACL { @@ -153,7 +154,8 @@ public class EntityACL { private static boolean subjectIsOwner( final Subject subject, final List<ResponsibleAgent> owners) { for (final ResponsibleAgent owner : owners) { - if ((owner instanceof Role && subject.hasRole(owner.toString())) || (owner instanceof Principal && subject.getPrincipal().equals(owner))) { + if ((owner instanceof Role && subject.hasRole(owner.toString())) + || (owner instanceof Principal && subject.getPrincipal().equals(owner))) { return true; } } diff --git a/src/main/java/caosdb/server/permissions/EntityACLFactory.java b/src/main/java/caosdb/server/permissions/EntityACLFactory.java new file mode 100644 index 00000000..eeacb126 --- /dev/null +++ b/src/main/java/caosdb/server/permissions/EntityACLFactory.java @@ -0,0 +1,11 @@ +package caosdb.server.permissions; + +import java.util.Collection; + +public class EntityACLFactory extends AbstractEntityACLFactory<EntityACL> { + + @Override + protected EntityACL create(final Collection<EntityACI> acis) { + return new EntityACL(acis); + } +} diff --git a/src/main/java/caosdb/server/permissions/Role.java b/src/main/java/caosdb/server/permissions/Role.java index 95ce7289..70e1a61f 100644 --- a/src/main/java/caosdb/server/permissions/Role.java +++ b/src/main/java/caosdb/server/permissions/Role.java @@ -22,10 +22,10 @@ */ package caosdb.server.permissions; +import caosdb.server.accessControl.UserSources; import java.util.HashMap; import org.jdom2.Attribute; import org.jdom2.Element; -import caosdb.server.accessControl.UserSources; public class Role implements ResponsibleAgent { -- GitLab