From d33e126b2624c36501d3b380634040c660b26800 Mon Sep 17 00:00:00 2001
From: Wisdom Ebong <wisdom@ushahidi.com>
Date: Thu, 28 Jul 2022 11:29:51 +0100
Subject: [PATCH 1/6] fix: map jumps around when loading map view

---
 legacy/app/map/post-view-map.directive.js | 21 ++++++++++-----------
 1 file changed, 10 insertions(+), 11 deletions(-)

diff --git a/legacy/app/map/post-view-map.directive.js b/legacy/app/map/post-view-map.directive.js
index 325e5cdf80..a74137d1f7 100644
--- a/legacy/app/map/post-view-map.directive.js
+++ b/legacy/app/map/post-view-map.directive.js
@@ -77,32 +77,30 @@ function PostViewMap(PostEndpoint, Maps, _, PostFilters, L, $q, $rootScope, $com
             }
         }
 
-        function addPostsToMap(posts) {
+        function addPostsToMap(posts, isFirstOrLastPostChunk = false) {
             var geojson = L.geoJson(posts, {
                 pointToLayer: Maps.pointToLayer,
                 onEachFeature: onEachFeature
             });
 
             if (map.options.clustering) {
-
                 markers = markers ? markers : L.markerClusterGroup({
                     maxClusterRadius: map.options.cluster_radius
                 });
-                // This has to be done individually.
-                // Using clusterLayer.addLayers() breaks the clustering.
-                // Need to investigate as this should have been fixing in v1.0.0
-                angular.forEach(geojson.getLayers(), function (layer) {
-                    markers.addLayer(layer);
-                });
+
+                // Apparently cluster.addLayers doesn't break clustering anymore
+                markers.addLayers(geojson.getLayers());
             } else {
                 markers = geojson;
             }
             markers.addTo(map);
             geoJsonLayers.push(markers);
 
-            if (posts.features.length > 0) {
+            // Adjust map boundaries only when post fetching is first or last batch
+            if (posts.features.length > 0 && isFirstOrLastPostChunk) {
                 map.fitBounds(geojson.getBounds());
             }
+
             // Focus map on data points but..
             // Avoid zooming further than 15 (particularly when we just have a single point)
             if (map.getZoom() > 15) {
@@ -193,7 +191,7 @@ function PostViewMap(PostEndpoint, Maps, _, PostFilters, L, $q, $rootScope, $com
             return getFirstPostChunk.$promise.then(function (posts) {
                 // Adding the first 200 posts to map here and getting the totals
                 $scope.stats.filteredPosts = posts.total;
-                addPostsToMap(posts)
+                addPostsToMap(posts, true)
 
                 // Moving on to request rest of the posts
                 if (posts.total > limit) {
@@ -202,7 +200,8 @@ function PostViewMap(PostEndpoint, Maps, _, PostFilters, L, $q, $rootScope, $com
                         let request = PostEndpoint.geojson(conditions);
                         currentGeoJsonRequests.push(request);
                         request.$promise.then(result => {
-                            addPostsToMap(result)
+                            var isLastPostChunk = posts.total === i;
+                            addPostsToMap(result, isLastPostChunk)
                         });
 
                     }

From b2cea84a6487ec5eea31d6a5189f3a9df85738e7 Mon Sep 17 00:00:00 2001
From: Wisdom Ebong <wisdom@ushahidi.com>
Date: Thu, 28 Jul 2022 11:29:51 +0100
Subject: [PATCH 2/6] fix: map jumps around when loading map view

---
 legacy/app/map/post-view-map.directive.js | 20 +++++++++-----------
 1 file changed, 9 insertions(+), 11 deletions(-)

diff --git a/legacy/app/map/post-view-map.directive.js b/legacy/app/map/post-view-map.directive.js
index 325e5cdf80..a768977d40 100644
--- a/legacy/app/map/post-view-map.directive.js
+++ b/legacy/app/map/post-view-map.directive.js
@@ -77,32 +77,30 @@ function PostViewMap(PostEndpoint, Maps, _, PostFilters, L, $q, $rootScope, $com
             }
         }
 
-        function addPostsToMap(posts) {
+        function addPostsToMap(posts, shouldFitMapBoundary = true) {
             var geojson = L.geoJson(posts, {
                 pointToLayer: Maps.pointToLayer,
                 onEachFeature: onEachFeature
             });
 
             if (map.options.clustering) {
-
                 markers = markers ? markers : L.markerClusterGroup({
                     maxClusterRadius: map.options.cluster_radius
                 });
-                // This has to be done individually.
-                // Using clusterLayer.addLayers() breaks the clustering.
-                // Need to investigate as this should have been fixing in v1.0.0
-                angular.forEach(geojson.getLayers(), function (layer) {
-                    markers.addLayer(layer);
-                });
+
+                // Apparently cluster.addLayers doesn't break clustering anymore
+                markers.addLayers(geojson.getLayers());
             } else {
                 markers = geojson;
             }
             markers.addTo(map);
             geoJsonLayers.push(markers);
 
-            if (posts.features.length > 0) {
-                map.fitBounds(geojson.getBounds());
+            // Adjust map boundaries only when post fetching is first or last batch
+            if (posts.features.length > 0 && shouldFitMapBoundary) {
+                map.fitBounds(markers.getBounds());
             }
+
             // Focus map on data points but..
             // Avoid zooming further than 15 (particularly when we just have a single point)
             if (map.getZoom() > 15) {
@@ -202,9 +200,9 @@ function PostViewMap(PostEndpoint, Maps, _, PostFilters, L, $q, $rootScope, $com
                         let request = PostEndpoint.geojson(conditions);
                         currentGeoJsonRequests.push(request);
                         request.$promise.then(result => {
+                            // var isLastPostChunk = i + limit > posts.total;
                             addPostsToMap(result)
                         });
-
                     }
                 }
             });

From 4f58f71b1d6471f73ecf01a7e06b10a05122c3e5 Mon Sep 17 00:00:00 2001
From: Wisdom Ebong <30173059+webong@users.noreply.github.com>
Date: Mon, 8 Aug 2022 12:24:35 +0100
Subject: [PATCH 3/6] Update post-messages-reply.html

Increase the max length of the reply textbox to 160
---
 .../app/data/common/post-edit-detail/post-messages-reply.html   | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/legacy/app/data/common/post-edit-detail/post-messages-reply.html b/legacy/app/data/common/post-edit-detail/post-messages-reply.html
index 78373a2dab..5c0f131e78 100644
--- a/legacy/app/data/common/post-edit-detail/post-messages-reply.html
+++ b/legacy/app/data/common/post-edit-detail/post-messages-reply.html
@@ -2,7 +2,7 @@
   <form name="form" ng-submit="sendMessage()">
 
     <div class="form-field" ng-class="{ 'error': form.reply.$invalid && form.reply.$dirty }">
-      <textarea id="reply" name="reply" placeholder="Send message to {{ contact.contact }}" ng-model="message.reply_text" ng-minlength=2 ng-maxlength=140 required></textarea>
+      <textarea id="reply" name="reply" placeholder="Send message to {{ contact.contact }}" ng-model="message.reply_text" ng-minlength=2 ng-maxlength=160 required></textarea>
     </div>
     <div class="form-field">
       <button type="submit" class="button-alpha modal-trigger"

From d6b9f5210a93be3d98110f8df1ee1ac78a535cf3 Mon Sep 17 00:00:00 2001
From: David Losada Carballo <david@tuxpiper.com>
Date: Tue, 9 Aug 2022 22:54:08 +0300
Subject: [PATCH 4/6] fix: js unbalanced braces blocking Google Places loading

---
 root/src/index.ejs | 22 +++++++++++-----------
 1 file changed, 11 insertions(+), 11 deletions(-)

diff --git a/root/src/index.ejs b/root/src/index.ejs
index 120986dea1..68b06db09d 100644
--- a/root/src/index.ejs
+++ b/root/src/index.ejs
@@ -90,16 +90,17 @@
       a.async = 1;
       a.src = g;
       m.parentNode.insertBefore(a, m);
-  })(
-      window,
-      document,
-      "script",
-      "//www.google-analytics.com/analytics.js",
-      "ga"
-  );
-
-  ga("create", window.ushahidi.gaKey, "auto");
-  ga("send", "pageview");
+    })(
+        window,
+        document,
+        "script",
+        "//www.google-analytics.com/analytics.js",
+        "ga"
+    );
+
+    ga("create", window.ushahidi.gaKey, "auto");
+    ga("send", "pageview");
+  }
   
   // Loading Google-Maps to enable use of Places-API
   if (window.ushahidi && window.ushahidi.googleMapsApiKey) {
@@ -122,7 +123,6 @@
           window.googlePlaces = new google.maps.places.PlacesService(e);
       });
   }
-}
 </script>
 
 

From f917270329471bc5c13a6c89cde3e8a12c639a0f Mon Sep 17 00:00:00 2001
From: David Losada Carballo <david@tuxpiper.com>
Date: Tue, 9 Aug 2022 23:40:51 +0300
Subject: [PATCH 5/6] feat(docker): configure google maps api key via env
 variable

---
 app/config.js.j2 | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/app/config.js.j2 b/app/config.js.j2
index a7eebf3108..c1a5dd2a78 100644
--- a/app/config.js.j2
+++ b/app/config.js.j2
@@ -9,6 +9,9 @@ window.ushahidi = {
 {% else %}
   gaEnabled: false,
 {% endif %}
+{% if GOOGLE_MAPS_API_KEY %}
+  googleMapsApiKey: '{{ GOOGLE_MAPS_API_KEY }}',
+{% endif %}
 {% if GTM_CONTAINER_ID %}
   googleTagManager: '{{ GTM_CONTAINER_ID }}',
 {% endif %}

From 364aa34c406148a5dd97ca4df58bf105d383ed8f Mon Sep 17 00:00:00 2001
From: Ifycode <marychord@gmail.com>
Date: Thu, 11 Aug 2022 15:44:56 +0100
Subject: [PATCH 6/6] Add data-cy attribute to languages select

---
 legacy/app/common/directives/add-language.html | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/legacy/app/common/directives/add-language.html b/legacy/app/common/directives/add-language.html
index 28d82c6fbc..8e1afa5c1b 100644
--- a/legacy/app/common/directives/add-language.html
+++ b/legacy/app/common/directives/add-language.html
@@ -10,6 +10,7 @@
                                   ng-options="lang.code as lang.name for lang in languagesToSelect"
                                   ng-value="lang.code"
                                   ng-change="selectLanguage(selectedLanguage)"
+                                  data-cy="custom-select"
                               >
                                   <option value="" translate="app.select_language">Select language</option>
                               </select>
@@ -18,10 +19,10 @@
             </fieldset>
             <div ng-show="showLangError" class="alert error">
                 <svg class="iconic">
-                  <use xmlns:xlink="http://www.w3.org/1999/xlink" xlink:href="/img/iconic-sprite.svg#warning"></use>
+                  <use xmlns:xlink="http://www.w3.org/1999/xlink" xlink:href="#warning"></use>
                 </svg>
                 <p>{{langError}}</p>
-            </div>   
+            </div>
         </div>
         <div class="modal-actions">
             <div class="form-field">