geocode bug fixed, but it's not setting id to -1

MushroomObserver · Aug 3, 2024 · d80e74a · d80e74a
1 parent 71cfe0a
commit d80e74a
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 5 deletions.
diff --git a/app/javascript/controllers/geocode_controller.js b/app/javascript/controllers/geocode_controller.js
@@ -166,10 +166,10 @@ export default class extends Controller {
     const extents = results[0].geometry.bounds?.toJSON() // may not exist
     const center = results[0].geometry.location.toJSON()
 
-    if (viewport)
-      this.map.fitBounds(viewport)
-    if (this.map)
-      this.placeClosestRectangle(viewport, extents)
+    if (this.map) {
+      if (viewport) this.map.fitBounds(viewport)
+      this.placeClosestRectangle(viewport, extents) // viewport is optional
+    }
     this.updateFields(viewport, extents, center)
     // For non-autocompleted place input in the location form
     this.updatePlaceInputTarget(results[0])

diff --git a/app/views/controllers/herbaria/_form.erb b/app/views/controllers/herbaria/_form.erb
@@ -24,7 +24,11 @@ end
 
 <%= form_with(
   model: @herbarium, id: "herbarium_form", class: "geocode-outlet",
-  data: { controller: "geocode" }
+  data: {
+    controller: "geocode",
+    action:
+      "autocompleter:hiddenIdDataChanged@window->geocode#geolocatePlaceName"
+  }
 ) do |f| %>
 
   <%= submit_button(form: f, button: button_name, center: true) %>