Removing unnecessary try-catch and adding more commands to Totem protocol

3 files changed, 13 insertions, 12 deletions

diff --git a/src/org/traccar/protocol/TotemFrameDecoder.java b/src/org/traccar/protocol/TotemFrameDecoder.java
index 00eb5ec82..6f7c3a37c 100644
--- a/src/org/traccar/protocol/TotemFrameDecoder.java
+++ b/src/org/traccar/protocol/TotemFrameDecoder.java
@@ -19,8 +19,7 @@ import java.nio.charset.Charset;
 import org.jboss.netty.buffer.ChannelBuffer;
 import org.jboss.netty.channel.Channel;
 import org.jboss.netty.channel.ChannelHandlerContext;
-import org.jboss.netty.handler.codec.frame.FrameDecoder;
-import org.traccar.helper.Log;
+import org.jboss.netty.handler.codec.frame.FrameDecoder; 
 
 public class TotemFrameDecoder extends FrameDecoder {
 
@@ -40,17 +39,14 @@ public class TotemFrameDecoder extends FrameDecoder {
             buf.skipBytes(2);
         }
         
-        try{
-            // Read message
+        if(buf.toString(buf.readerIndex(), 2, Charset.defaultCharset()).equals("$$")){
             int length = Integer.parseInt(buf.toString(buf.readerIndex() + 2, 2, Charset.defaultCharset()), 16);
             if (length <= buf.readableBytes()) {
                 return buf.readBytes(length);
             }
-        }catch(Exception e){
+        }else{
             //TODO: notify to user the GPS response
-            
-            Log.debug("GPS Response: " + buf.toString(buf.readerIndex(), buf.readableBytes(), Charset.defaultCharset()));
-            //return buf.readBytes(buf.readableBytes());
+            return buf.readBytes(buf.readableBytes());
         }
         
         return null;
diff --git a/src/org/traccar/protocol/TotemProtocol.java b/src/org/traccar/protocol/TotemProtocol.java
index 8445f04b6..097f6a593 100644
--- a/src/org/traccar/protocol/TotemProtocol.java
+++ b/src/org/traccar/protocol/TotemProtocol.java
@@ -30,7 +30,9 @@ public class TotemProtocol extends BaseProtocol {
     public TotemProtocol() {
         super("totem");
         setSupportedCommands(
-                Command.TYPE_IMEI);
+                Command.TYPE_ENGINE_RESUME,
+                Command.TYPE_ENGINE_STOP
+        );
     }
 
     @Override
diff --git a/src/org/traccar/protocol/TotemProtocolEncoder.java b/src/org/traccar/protocol/TotemProtocolEncoder.java
index 02dabc5fb..d236a7e2c 100644
--- a/src/org/traccar/protocol/TotemProtocolEncoder.java
+++ b/src/org/traccar/protocol/TotemProtocolEncoder.java
@@ -15,7 +15,7 @@
  */
 package org.traccar.protocol;
  
-import org.traccar.StringProtocolEncoder;
+import org.traccar.StringProtocolEncoder; 
 import org.traccar.model.Command;
 
 /**
@@ -28,8 +28,11 @@ public class TotemProtocolEncoder extends StringProtocolEncoder{
     protected Object encodeCommand(Command command) {
         
         switch (command.getType()) {
-            case Command.TYPE_IMEI:
-                return "*000000,801#";
+            //Assuming PIN 8 (Output C) is the power wire, like manual says but it can be PIN 5,7,8
+            case Command.TYPE_ENGINE_STOP:
+                return formatCommand(command, "*{%s},025,C,1#", Command.KEY_GPS_PASSWORD);
+            case Command.TYPE_ENGINE_RESUME:
+                return formatCommand(command, "*{%s},025,C,0#", Command.KEY_GPS_PASSWORD);
         }
         
         return null;