Edit foci boundary enhancements

library/src/main/java/io/ona/kujaku/interfaces/IKujakuMapView.java

@@ -27,6 +27,7 @@
 import io.ona.kujaku.listeners.BoundsChangeListener;
 import io.ona.kujaku.listeners.LocationClientStartedCallback;
 import io.ona.kujaku.listeners.OnFeatureClickListener;
+import io.ona.kujaku.listeners.OnFeatureLongClickListener;
 import io.ona.kujaku.listeners.TrackingServiceListener;
 import io.ona.kujaku.location.KujakuLocation;
 import io.ona.kujaku.services.configurations.TrackingServiceUIConfiguration;
@@ -188,6 +189,25 @@ public interface IKujakuMapView extends IKujakuMapViewLowLevel {
      */
     void setOnFeatureClickListener(@NonNull OnFeatureClickListener onFeatureClickListener, @Nullable Expression expressionFilter, @Nullable String... layerIds);
 
+    /**
+     * Sets an {@link OnFeatureClickListener} which will be fired when a feature on the map in either of the {@code layerIds}
+     * is long touched/clicked
+     *
+     * @param onFeatureLongClickListener
+     * @param layerIds
+     */
+    void setOnFeatureLongClickListener(@NonNull OnFeatureLongClickListener onFeatureLongClickListener, @Nullable String... layerIds);
+
+    /**
+     * Sets an {@link OnFeatureClickListener} which will be fired when a feature on the map in either of the {@code layerIds}
+     * is long touched/clicked and/or fulfilling the filter defined in {@code filter}
+     *
+     * @param onFeatureLongClickListener
+     * @param expressionFilter
+     * @param layerIds
+     */
+    void setOnFeatureLongClickListener(@NonNull OnFeatureLongClickListener onFeatureLongClickListener, @Nullable Expression expressionFilter, @Nullable String... layerIds);
+
     /**
      * Checks if the map warms GPS(this just means the location service that is going to be used).
      * Warming the GPS in this case means that it starts the location services as soon as you open

library/src/main/java/io/ona/kujaku/listeners/OnFeatureLongClickListener.java

@@ -0,0 +1,21 @@
+package io.ona.kujaku.listeners;
+
+import com.mapbox.geojson.Feature;
+
+import java.util.List;
+
+/**
+ * Created by Richard Kareko on 5/18/20.
+ */
+
+public interface OnFeatureLongClickListener {
+
+    /**
+     * Called when a features(s) is long clicked on the map and adheres to params passed in
+     * {@link io.ona.kujaku.views.KujakuMapView#setOnFeatureClickListener(OnFeatureClickListener, String...)}
+     * or {@link io.ona.kujaku.views.KujakuMapView#setOnFeatureClickListener(OnFeatureClickListener, com.mapbox.mapboxsdk.style.expressions.Expression, String...)}
+     *
+     * @param features
+     */
+    void onFeatureLongClick(List<Feature> features);
+}

library/src/main/java/io/ona/kujaku/manager/DrawingManager.java

@@ -8,6 +8,7 @@
 import com.mapbox.geojson.Feature;
 import com.mapbox.geojson.FeatureCollection;
 import com.mapbox.geojson.Geometry;
+import com.mapbox.geojson.MultiPolygon;
 import com.mapbox.geojson.Point;
 import com.mapbox.geojson.Polygon;
 import com.mapbox.mapboxsdk.geometry.LatLng;
@@ -217,6 +218,16 @@ public boolean startDrawing(@Nullable FillBoundaryLayer fillBoundaryLayer) {
                 List<Point> points = polygon.coordinates().get(0);
                 this.startDrawingPoints(points);
                 return true;
+            } else if (geometry instanceof MultiPolygon) {
+                // hide layer
+                fillBoundaryLayer.disableLayerOnMap(mapboxMap);
+
+                MultiPolygon multiPolygon = (MultiPolygon) geometry;
+                for (List<List<Point>> polygonCoordinates : multiPolygon.coordinates()) {
+                    List<Point> points = polygonCoordinates.get(0);
+                    this.startDrawingPoints(points);
+                }
+                return true;
             }
         }
 

library/src/main/java/io/ona/kujaku/views/KujakuMapView.java

@@ -74,6 +74,7 @@
 import io.ona.kujaku.listeners.BoundsChangeListener;
 import io.ona.kujaku.listeners.LocationClientStartedCallback;
 import io.ona.kujaku.listeners.OnFeatureClickListener;
+import io.ona.kujaku.listeners.OnFeatureLongClickListener;
 import io.ona.kujaku.listeners.OnKujakuLayerClickListener;
 import io.ona.kujaku.listeners.OnKujakuLayerLongClickListener;
 import io.ona.kujaku.listeners.OnLocationChanged;
@@ -164,6 +165,7 @@ public class KujakuMapView extends MapView implements IKujakuMapView, MapboxMap.
     private BoundsChangeListener boundsChangeListener;
 
     private OnFeatureClickListener onFeatureClickListener;
+    private OnFeatureLongClickListener onFeatureLongClickListener;
     private String[] featureClickLayerIdFilters;
     private Expression featureClickExpressionFilter;
 
@@ -829,6 +831,18 @@ public void setOnFeatureClickListener(@NonNull OnFeatureClickListener onFeatureC
         this.featureClickExpressionFilter = expressionFilter;
     }
 
+    @Override
+    public void setOnFeatureLongClickListener(@NonNull OnFeatureLongClickListener onFeatureLongClickListener, @Nullable String... layerIds) {
+        this.setOnFeatureLongClickListener(onFeatureLongClickListener, null, layerIds);
+    }
+
+    @Override
+    public void setOnFeatureLongClickListener(@NonNull OnFeatureLongClickListener onFeatureLongClickListener, @Nullable Expression expressionFilter, @Nullable String... layerIds) {
+        this.onFeatureLongClickListener = onFeatureLongClickListener;
+        this.featureClickLayerIdFilters = layerIds;
+        this.featureClickExpressionFilter = expressionFilter;
+    }
+
     /**
      * Set listener when pressing a KujakuLayer
      *
@@ -1124,6 +1138,15 @@ public boolean onMapClick(@NonNull LatLng point) {
     public boolean onMapLongClick(@NonNull LatLng point) {
         PointF pixel = mapboxMap.getProjection().toScreenLocation(point);
 
+        if (onFeatureLongClickListener != null) {
+            List<com.mapbox.geojson.Feature> features = mapboxMap.queryRenderedFeatures(pixel, featureClickExpressionFilter, featureClickLayerIdFilters);
+
+            if (features.size() > 0) {
+                onFeatureLongClickListener.onFeatureLongClick(features);
+            }
+        }
+
+
         if (onKujakuLayerLongClickListener != null) {
             KujakuLayer layer = KujakuLayer.getKujakuLayerSelected(pixel, kujakuLayers, mapboxMap);
             if (layer != null) {

library/src/test/java/io/ona/kujaku/manager/DrawingManagerTest.java

@@ -2,6 +2,7 @@
 
 import com.mapbox.geojson.Feature;
 import com.mapbox.geojson.FeatureCollection;
+import com.mapbox.geojson.MultiPolygon;
 import com.mapbox.geojson.Point;
 import com.mapbox.geojson.Polygon;
 import com.mapbox.mapboxsdk.geometry.LatLng;
@@ -78,7 +79,26 @@ public void startDrawingWithPoints() {
 
     @Test
     public void startDrawingWithExistingLayer() {
-        manager.startDrawing(getFillBoundaryLayer());
+        manager.startDrawing(getFillBoundaryLayer(true));
+
+        Assert.assertTrue(manager.isDrawingEnabled());
+        Assert.assertNull(manager.getCurrentKujakuCircle());
+        // Middle Circles created between each point
+        Assert.assertEquals(8, manager.getKujakuCircles().size());
+
+        Assert.assertFalse(manager.getKujakuCircles().get(0).isMiddleCircle());
+        Assert.assertTrue(manager.getKujakuCircles().get(1).isMiddleCircle());
+        Assert.assertFalse(manager.getKujakuCircles().get(2).isMiddleCircle());
+        Assert.assertTrue(manager.getKujakuCircles().get(3).isMiddleCircle());
+        Assert.assertFalse(manager.getKujakuCircles().get(4).isMiddleCircle());
+        Assert.assertTrue(manager.getKujakuCircles().get(5).isMiddleCircle());
+        Assert.assertFalse(manager.getKujakuCircles().get(6).isMiddleCircle());
+        Assert.assertTrue(manager.getKujakuCircles().get(7).isMiddleCircle());
+    }
+
+    @Test
+    public void startDrawingWithExistingMultipolygonLayer() {
+        manager.startDrawing(getFillBoundaryLayer(false));
 
         Assert.assertTrue(manager.isDrawingEnabled());
         Assert.assertNull(manager.getCurrentKujakuCircle());
@@ -206,18 +226,24 @@ public void areMiddleCircles() {
         Assert.assertTrue(manager.getKujakuCircles().get(7).isMiddleCircle());
     }
 
-    private FillBoundaryLayer getFillBoundaryLayer() {
+    private FillBoundaryLayer getFillBoundaryLayer(boolean isPolygon) {
         List<Feature> features = new ArrayList<Feature>();
         List<List<Point>> lists = new ArrayList<>();
         List<Point> points = new ArrayList<>();
+        Polygon polygon = null;
 
         points.add(Point.fromLngLat(-11,15));
         points.add(Point.fromLngLat(-5,15));
         points.add(Point.fromLngLat(-5,11));
         points.add(Point.fromLngLat(-11,11));
         lists.add(points);
 
-        features.add(Feature.fromGeometry(Polygon.fromLngLats(lists)));
+        polygon = Polygon.fromLngLats(lists);
+        if (!isPolygon) {
+            features.add(Feature.fromGeometry(polygon));
+        } else {
+            features.add(Feature.fromGeometry(MultiPolygon.fromPolygon(polygon)));
+        }
 
         FeatureCollection featureCollection = FeatureCollection.fromFeatures(features);
         FillBoundaryLayer.Builder builder = new FillBoundaryLayer.Builder(featureCollection);

library/src/test/java/io/ona/kujaku/views/KujakuMapViewTest.java

@@ -24,12 +24,14 @@
 import io.ona.kujaku.BaseTest;
 import io.ona.kujaku.interfaces.ILocationClient;
 import io.ona.kujaku.listeners.LocationClientStartedCallback;
+import io.ona.kujaku.listeners.OnFeatureLongClickListener;
 import io.ona.kujaku.location.clients.GoogleLocationClient;
 import io.ona.kujaku.test.shadows.ShadowKujakuMapView;
 import io.ona.kujaku.test.shadows.ShadowMapView;
 
 import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertFalse;
+import static org.junit.Assert.assertNull;
 import static org.junit.Assert.assertTrue;
 
 /**
@@ -186,4 +188,16 @@ public void getLocationClientShouldCallCallbackWhenLocationClientIsNullAndWarmLo
                 .onStarted(Mockito.any(ILocationClient.class));
         assertEquals(0, callbacks.size());
     }
+
+    @Test
+    public void setOnFeatureLongClickListener() {
+        assertNull(ReflectionHelpers.getField(kujakuMapView,"onFeatureLongClickListener"));
+        assertNull(ReflectionHelpers.getField(kujakuMapView,"featureClickLayerIdFilters"));
+        OnFeatureLongClickListener onFeatureLongClickListenerMock = Mockito.mock(OnFeatureLongClickListener.class);
+        String[] layerIds =  {"id1"};
+
+        kujakuMapView.setOnFeatureLongClickListener(onFeatureLongClickListenerMock, layerIds);
+        assertEquals(onFeatureLongClickListenerMock, ReflectionHelpers.getField(kujakuMapView,"onFeatureLongClickListener"));
+        assertEquals(layerIds, ReflectionHelpers.getField(kujakuMapView,"featureClickLayerIdFilters"));
+    }
 }