It's possible to use the widget "Geocode from another field", it exposes an option to "Allow manual lat/long override". If we check this checkbox, the lat/lon field is exposed when editing a content without checking this checkbox.

It's related to the form structure that changed, the lat/lon fields are now a dedicated element type. Allowing the override should probably reworked to be passed as an attribute to the element type.
At least the code from geocoder_process_override() is invalid.

CommentFileSizeAuthor
#3 geocoder-3125359-lat_lon_checkbox_state_broken-3.patch1.51 KBartusamak
#2 geocoder-3125359-lat_lon_checkbox_state_broken-2.patch715 bytesartusamak

Comments

Artusamak created an issue. See original summary.

artusamak’s picture

artusamak
he/him
Primary language French
Location Bzh
commented
Status: Active » Needs review
StatusFileSize
new geocoder-3125359-lat_lon_checkbox_state_broken-2.patch715 bytes

Here is a patch for that.

artusamak’s picture

artusamak
he/him
Primary language French
Location Bzh
commented
StatusFileSize
new geocoder-3125359-lat_lon_checkbox_state_broken-3.patch1.51 KB

Here is an update, the widget form needs to have this update too or it generates a warning.

phily’s picture

phily
he/him
Primary language French
Location 🇪🇺🇫🇷 Paris, France
commented

Thanks Artusamak, the patch works.

denix’s picture

denix commented
Status: Needs review » Reviewed & tested by the community

Thanks @PhilY, for us is working too. I move to reviewed.

  • itamair committed f4f43c0 on 7.x-1.x authored by Artusamak 
    Issue #3125359 by Artusamak: The override lat/lon checkbox states API is...
itamair’s picture

itamair commented
Status: Reviewed & tested by the community » Fixed

committed into dev, will be part of the ext 7.x-1.x release

thegreatone’s picture

thegreatone commented

Just tested this in a clean simplytest.me site...and it didn't kept it.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.