diff options
author | Abyss777 <abyss@fox5.ru> | 2016-09-22 10:44:29 +0500 |
---|---|---|
committer | Abyss777 <abyss@fox5.ru> | 2016-09-22 10:44:29 +0500 |
commit | c8816bdd85d62fb767795b2dc4d31326fdad9cd5 (patch) | |
tree | cc1422319766a1ffe0688642830e2bdd535314b4 /src/org/traccar/api/resource/AttributeAliasResource.java | |
parent | baad910016f2f17e6bf89f9d5db17349c4a9a62a (diff) | |
download | trackermap-server-c8816bdd85d62fb767795b2dc4d31326fdad9cd5.tar.gz trackermap-server-c8816bdd85d62fb767795b2dc4d31326fdad9cd5.tar.bz2 trackermap-server-c8816bdd85d62fb767795b2dc4d31326fdad9cd5.zip |
- rename table and unique constraint
- rename some functions
- optimization and simplification
Diffstat (limited to 'src/org/traccar/api/resource/AttributeAliasResource.java')
-rw-r--r-- | src/org/traccar/api/resource/AttributeAliasResource.java | 15 |
1 files changed, 6 insertions, 9 deletions
diff --git a/src/org/traccar/api/resource/AttributeAliasResource.java b/src/org/traccar/api/resource/AttributeAliasResource.java index 827b50c38..6dbcf6ce8 100644 --- a/src/org/traccar/api/resource/AttributeAliasResource.java +++ b/src/org/traccar/api/resource/AttributeAliasResource.java @@ -46,9 +46,9 @@ public class AttributeAliasResource extends BaseResource { if (!Context.getPermissionsManager().isAdmin(getUserId())) { Context.getPermissionsManager().checkDevice(getUserId(), deviceId); } - return Context.getAliasesManager().getDeviceAttributeAliases(deviceId); + return Context.getAliasesManager().getAttributeAliases(deviceId); } else { - return Context.getAliasesManager().getUserDevicesAttributeAliases(getUserId()); + return Context.getAliasesManager().getAllAttributeAliases(getUserId()); } } @@ -67,10 +67,8 @@ public class AttributeAliasResource extends BaseResource { public Response update(@PathParam("id") long id, AttributeAlias entity) throws SQLException { Context.getPermissionsManager().checkReadonly(getUserId()); if (!Context.getPermissionsManager().isAdmin(getUserId())) { - AttributeAlias oldAttrbuteAlias = Context.getAliasesManager().getAttributeAliasById(entity.getId()); - if (oldAttrbuteAlias != null && oldAttrbuteAlias.getDeviceId() != entity.getDeviceId()) { - Context.getPermissionsManager().checkDevice(getUserId(), oldAttrbuteAlias.getDeviceId()); - } + AttributeAlias oldEntity = Context.getAliasesManager().getAttributeAlias(entity.getId()); + Context.getPermissionsManager().checkDevice(getUserId(), oldEntity.getDeviceId()); Context.getPermissionsManager().checkDevice(getUserId(), entity.getDeviceId()); } Context.getAliasesManager().updateAttributeAlias(entity); @@ -82,9 +80,8 @@ public class AttributeAliasResource extends BaseResource { public Response remove(@PathParam("id") long id) throws SQLException { Context.getPermissionsManager().checkReadonly(getUserId()); if (!Context.getPermissionsManager().isAdmin(getUserId())) { - AttributeAlias attrbuteAlias = Context.getAliasesManager().getAttributeAliasById(id); - Context.getPermissionsManager().checkDevice(getUserId(), - attrbuteAlias != null ? attrbuteAlias.getDeviceId() : 0); + AttributeAlias entity = Context.getAliasesManager().getAttributeAlias(id); + Context.getPermissionsManager().checkDevice(getUserId(), entity.getDeviceId()); } Context.getAliasesManager().removeArrtibuteAlias(id); return Response.noContent().build(); |