Revert "Save centroid of geometry when the geometry is not a point (k…

…artoza#3422)" This reverts commit 918ed19.
dimasciput · Jul 30, 2023 · 7c604df · 7c604df
1 parent b736e08
commit 7c604df
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 17 deletions.
diff --git a/bims/models/location_site.py b/bims/models/location_site.py
@@ -185,10 +185,7 @@ def data_name(self):
     def get_centroid(self):
         """ Getting centroid of location site """
 
-        if (
-            self.geometry_point and
-            self.location_type.allowed_geometry == 'POINT'
-        ):
+        if self.geometry_point:
             return self.geometry_point
         else:
             if self.get_geometry():
@@ -201,11 +198,10 @@ def get_geometry(self):
         geometry = None
         validation_error = ValidationError('Only one geometry allowed.')
 
-        if (
-            self.geometry_point and
-            self.location_type.allowed_geometry == 'POINT'
-        ):
-            return self.geometry_point
+        if self.geometry_point:
+            if geometry:
+                raise validation_error
+            geometry = self.geometry_point
 
         if self.geometry_polygon:
             if geometry:
@@ -407,9 +403,6 @@ def save(self, *args, **kwargs):
                     if not lon_lat_changed:
                         self.geometry_point.x = self.longitude
                         self.geometry_point.y = self.latitude
-                if self.location_type.allowed_geometry != 'POINT':
-                    self.geometry_point = self.get_centroid()
-
                 super(LocationSite, self).save(*args, **kwargs)
                 self.__original_centroid = self.get_centroid()
             else:

diff --git a/bims/static/js/forms/location-site-admin-form.js b/bims/static/js/forms/location-site-admin-form.js
@@ -1,9 +1,10 @@
 (function ($) {
     $(document).ready(function () {
-        var $geometryInput = $('[class*="geometry_"]').not(".geometry_point");
+        var $geometryInput = $('[class*="geometry_"]');
         var init = true;
         $geometryInput.hide();
         function resetAllGeometries() {
+            geodjango_geometry_point.clearFeatures();
             geodjango_geometry_line.clearFeatures();
             geodjango_geometry_polygon.clearFeatures();
             geodjango_geometry_multipolygon.clearFeatures();
@@ -17,14 +18,14 @@
             if ($(this).val()) {
                 $.get("/api/location-type/" + $(this).val() + '/allowed-geometry/', function (data) {
                     var className = 'geometry_' + data.toLowerCase();
-                    $geometryInput.not('.' + className).not(".geometry_point").hide();
+                    $geometryInput.not('.' + className).hide();
                     if (!$('.' + className).is(":visible")) {
                         $('.' + className).show();
                     }
                 })
-                .fail(function () {
-                    console.log('fail');
-                })
+                    .fail(function () {
+                        console.log('fail');
+                    })
             } else {
                 $geometryInput.hide();
             }