Added option to choose if one ReverseGeoCodes invalid positions

1 files changed, 12 insertions, 5 deletions

diff --git a/src/org/traccar/ReverseGeocoderHandler.java b/src/org/traccar/ReverseGeocoderHandler.java
index 8bdd7ebfc..6d77ed253 100644
--- a/src/org/traccar/ReverseGeocoderHandler.java
+++ b/src/org/traccar/ReverseGeocoderHandler.java
@@ -26,9 +26,11 @@ import org.traccar.model.Position;
 public class ReverseGeocoderHandler extends OneToOneDecoder {
 
     private final ReverseGeocoder geocoder;
+    private final boolean processInvalidPositions;
 
-    public ReverseGeocoderHandler(ReverseGeocoder geocoder) {
+    public ReverseGeocoderHandler(ReverseGeocoder geocoder, boolean processInvalidPositions ) {
         this.geocoder = geocoder;
+        this.processInvalidPositions = processInvalidPositions;
     }
 
     @Override
@@ -39,13 +41,18 @@ public class ReverseGeocoderHandler extends OneToOneDecoder {
         if (geocoder != null) {
             if (msg instanceof Position) {
                 Position position = (Position) msg;
-                position.setAddress(geocoder.getAddress(
-                        position.getLatitude(), position.getLongitude()));
+                
+                if (processInvalidPositions || position.getValid()) {
+                    position.setAddress(geocoder.getAddress(
+                            position.getLatitude(), position.getLongitude()));
+                }
             } else if (msg instanceof List) {
                 List<Position> positions = (List<Position>) msg;
                 for (Position position : positions) {
-                    position.setAddress(geocoder.getAddress(
-                            position.getLatitude(), position.getLongitude()));
+                    if (processInvalidPositions || position.getValid()) {
+                        position.setAddress(geocoder.getAddress(
+                                position.getLatitude(), position.getLongitude()));
+                    }
                 }
             }
         }