diff options
| author | Christoph Krey <krey.christoph@gmail.com> | 2017-07-02 10:20:50 +0200 |
|---|---|---|
| committer | Christoph Krey <krey.christoph@gmail.com> | 2017-07-02 10:20:50 +0200 |
| commit | c67bf883bf0aa84293b94b000536c2061c3e00b2 (patch) | |
| tree | f2f478a2f9ac4b703f74c294d5a96a9842c405c1 | |
| parent | 4fce9334c72acf8e7ef78c7b2c7b60380e77e6cb (diff) | |
| download | trackermap-server-c67bf883bf0aa84293b94b000536c2061c3e00b2.tar.gz trackermap-server-c67bf883bf0aa84293b94b000536c2061c3e00b2.tar.bz2 trackermap-server-c67bf883bf0aa84293b94b000536c2061c3e00b2.zip | |
[FIX] permission check before getting device, eleminate unnecessary variable.
| -rw-r--r-- | src/org/traccar/api/resource/DeviceResource.java | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/src/org/traccar/api/resource/DeviceResource.java b/src/org/traccar/api/resource/DeviceResource.java index 0f75b5e33..ce46b4e29 100644 --- a/src/org/traccar/api/resource/DeviceResource.java +++ b/src/org/traccar/api/resource/DeviceResource.java @@ -64,9 +64,8 @@ public class DeviceResource extends BaseResource { } else { ArrayList<Device> devices = new ArrayList<>(); for (Long deviceId : deviceIds) { - Device device = Context.getDeviceManager().getDeviceById(deviceId); - Context.getPermissionsManager().checkDevice(getUserId(), device.getId()); - devices.add(device); + Context.getPermissionsManager().checkDevice(getUserId(), deviceId); + devices.add(Context.getDeviceManager().getDeviceById(deviceId)); } return devices; } |