From 0abae87d85964f7b0afd3e2d128be7a211ae4715 Mon Sep 17 00:00:00 2001 From: Anton Tananaev Date: Tue, 27 Oct 2015 12:39:55 +1300 Subject: Extend Visiontek protocol decoder (fix #1475) --- src/org/traccar/helper/Parser.java | 5 ++++ .../traccar/protocol/VisiontekProtocolDecoder.java | 35 ++++++++++++++-------- 2 files changed, 27 insertions(+), 13 deletions(-) (limited to 'src') diff --git a/src/org/traccar/helper/Parser.java b/src/org/traccar/helper/Parser.java index f70fe2e89..bda7d6366 100644 --- a/src/org/traccar/helper/Parser.java +++ b/src/org/traccar/helper/Parser.java @@ -87,6 +87,7 @@ public class Parser { } public enum CoordinateFormat { + DEG_HEM, DEG_MIN_HEM, DEG_MIN_MIN_HEM, HEM_DEG, @@ -99,6 +100,10 @@ public class Parser { String hemisphere; switch (format) { + case DEG_HEM: + coordinate = nextDouble(); + hemisphere = next(); + break; case DEG_MIN_MIN_HEM: coordinate = nextInt(); coordinate += Double.parseDouble(next() + '.' + next()) / 60; diff --git a/src/org/traccar/protocol/VisiontekProtocolDecoder.java b/src/org/traccar/protocol/VisiontekProtocolDecoder.java index 0dce4954d..31e9ff069 100644 --- a/src/org/traccar/protocol/VisiontekProtocolDecoder.java +++ b/src/org/traccar/protocol/VisiontekProtocolDecoder.java @@ -22,10 +22,7 @@ import java.util.regex.Matcher; import java.util.regex.Pattern; import org.jboss.netty.channel.Channel; import org.traccar.BaseProtocolDecoder; -import org.traccar.helper.DateBuilder; -import org.traccar.helper.Parser; -import org.traccar.helper.PatternBuilder; -import org.traccar.helper.UnitsConverter; +import org.traccar.helper.*; import org.traccar.model.Event; import org.traccar.model.Position; @@ -41,13 +38,17 @@ public class VisiontekProtocolDecoder extends BaseProtocolDecoder { .number("(d+),").optional() // imei .number("(dd),(dd),(dd),") // date .number("(dd),(dd),(dd),") // time - .number("(dd)(dd).?(dddd)") // latitude - .expression("([NS]),") - .number("(ddd)(dd).?(dddd)") // longitude - .expression("([EW]),") + .groupBegin() + .number("(dd)(dd).?(d+)([NS]),") // latitude + .number("(ddd)(dd).?(d+)([EW]),") // longitude + .or() + .number("(dd.d+)([NS]),") // latitude + .number("(ddd.d+)([EW]),") // longitude + .groupEnd() .number("(d+.?d+),") // speed .number("(d+),") // course .groupBegin() + .groupBegin() .number("(d+),") // altitude .number("(d+),") // satellites .groupEnd("?") @@ -60,8 +61,10 @@ public class VisiontekProtocolDecoder extends BaseProtocolDecoder { .number("(d),") // external battery status .number("(d+),") // gsm .groupEnd("?") - .expression("([AV]),?") // validity - .number("(d+)?") // rfid + .groupEnd("?") + .any() + .expression("([AV])") // validity + .number(",(d{10})").optional() // rfid .any() .compile(); @@ -79,7 +82,7 @@ public class VisiontekProtocolDecoder extends BaseProtocolDecoder { String id = parser.next(); String imei = parser.next(); - if (!identify(id, channel, null, false) && !identify(imei, channel)) { + if (!identify(id, channel, null, false) && (imei == null || !identify(imei, channel))) { return null; } position.setDeviceId(getDeviceId()); @@ -89,8 +92,14 @@ public class VisiontekProtocolDecoder extends BaseProtocolDecoder { .setTime(parser.nextInt(), parser.nextInt(), parser.nextInt()); position.setTime(dateBuilder.getDate()); - position.setLatitude(parser.nextCoordinate(Parser.CoordinateFormat.DEG_MIN_MIN_HEM)); - position.setLongitude(parser.nextCoordinate(Parser.CoordinateFormat.DEG_MIN_MIN_HEM)); + if (parser.hasNext(8)) { + position.setLatitude(parser.nextCoordinate(Parser.CoordinateFormat.DEG_MIN_MIN_HEM)); + position.setLongitude(parser.nextCoordinate(Parser.CoordinateFormat.DEG_MIN_MIN_HEM)); + } + if (parser.hasNext(4)) { + position.setLatitude(parser.nextCoordinate(Parser.CoordinateFormat.DEG_HEM)); + position.setLongitude(parser.nextCoordinate(Parser.CoordinateFormat.DEG_HEM)); + } position.setSpeed(UnitsConverter.knotsFromKph(Double.parseDouble( parser.next().replace(".", "")) / 10)); -- cgit v1.2.3