Merge pull request #34 from target/nonScale_changes

Check Scale connect status before acquiring lock

Author: arpal7

src/main/java/com/target/devicemanager/components/scale/ScaleDevice.java

@@ -203,9 +203,9 @@ void startStableWeightRead(int timeout) {
                         return;
                 } catch (JposException jposException) {
                     if(isConnected()) {
-                        LOGGER.error(MARKER, "Scale Failed to Read Stable Weight: " + jposException.getErrorCode() + ", " + jposException.getErrorCodeExtended());
+                        LOGGER.trace("Scale Failed to Read Stable Weight: " + jposException.getErrorCode() + ", " + jposException.getErrorCodeExtended());
                     } else {
-                        LOGGER.error(MARKER, "Scale not connected in Read Stable Weight: " + jposException.getErrorCode() + ", " + jposException.getErrorCodeExtended());
+                        LOGGER.trace("Scale not connected in Read Stable Weight: " + jposException.getErrorCode() + ", " + jposException.getErrorCodeExtended());
                     }
                     if(jposException.getErrorCode() != JposConst.JPOS_E_TIMEOUT) {
                         fireScaleWeightErrorEvent(jposException);

src/main/java/com/target/devicemanager/components/scale/ScaleManager.java

@@ -113,7 +113,7 @@ void subscribeToLiveWeight(SseEmitter liveWeightEmitter) throws IOException {
     }
 
     public FormattedWeight getStableWeight(CompletableFuture<FormattedWeight> stableWeightClient) throws ScaleException {
-        if (scaleDevice.tryLock() && isScaleReady()) {
+        if (isScaleReady() && scaleDevice.tryLock()) {
             //Create new future and add it to the list
             stableWeightClients.add(stableWeightClient);
             scaleDevice.startStableWeightRead(STABLE_WEIGHT_TIMEOUT_MSEC);