with #1662584: Add geohash support? in place a next step would be providing a widget, so users are able to input location data using geohashes

CommentFileSizeAuthor
#1 1772192_geofield_geohash_widget_1.patch1.54 KBdasjo

Comments

dasjo’s picture

dasjo
he/him
Primary language German
Location Zurich
commented
Status: Active » Needs review
StatusFileSize
new 1772192_geofield_geohash_widget_1.patch1.54 KB

find a first, minimalistic patch attached. it allows me to add/edit geofield data using geohashes.

a related question: how should we deal with granularity? should this be part of the widget settings?

edit: i would also like to extend the geofield schema in order to store geohashes in the database and access it in views. this might be out-of-scope for the geofield module, but could live in geocluster. as of #691932: Add hook_field_schema_alter(), i'm wondering what would be the best way to doing so.

phayes’s picture

phayes commented

Hi dasjo,

Thanks for this, but I think i'm gonna wait for a bit on this one. Now that geoPHP has really good format detection, I'd like to see if we can simplify and roll all the textual input widgets into a single all-encompassing widget that will take any geospatial input. What do you think?

phayes’s picture

phayes commented

As for a geohash addition to the geometry columns, I actually would support adding it.

I'd like to remove SRID, accuracy, and source columns - then add a geohash column. Especially nice about using geohash is that the length of the geohash is a good indicator of accuracy.

dasjo’s picture

dasjo
he/him
Primary language German
Location Zurich
commented

hi phayes, i would be pleased to see input widgets being simplified code-wise.

i'm not a geofield expert, so i wouldn't know where to start but i could help with testing a patch or writing one given some exemplary code snippets.

adding the link here to related discussion for adding a geohash column: #1772416: Remixing geofield columns

Brandonian’s picture

Brandonian commented
Status: Needs review » Needs work

Marking as 'needs work' to better reflect actual state of ticket.