Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
Hi,
Nice module. Been working on a patch for Search API Location and have enabled support for geolocation field. However, property info callbacks need to be added for it to work properly with Entity API. Patch attached.
Comment | File | Size | Author |
---|---|---|---|
#8 | geolocation-property-info-callback-1366642-8.patch | 3.01 KB | tce |
#6 | geolocation-property_info_callback-1366642-6.patch | 2.86 KB | yanniboi |
geolocation_property_info.patch | 2.84 KB | wesnick | |
Comments
Comment #1
noslokire CreditAttribution: noslokire commentedWe must be doing the same thing. I will try and apply get back
Comment #2
jm.federico CreditAttribution: jm.federico commentedHi Guys
Sorry for not checking this faster.
WOuld like to know if you have had success with this functionality and if there is a site where I can see it in action.
I'm really keen on having this committed, but don't have where to test it right now properly.
Feedback is appreciated.
Cheers
Comment #3
jm.federico CreditAttribution: jm.federico commentedMarking #1320564: Fix for Search API not picking up the entity to index it's fields as dup of this
Comment #4
tce CreditAttribution: tce commentedCan anyone confirm this patch works?
Comment #5
derjochenmeyer CreditAttribution: derjochenmeyer commentedHow about this: http://drupal.org/node/1435070#comment-6153698
Anyone else who tested this patch?
Comment #6
yanniboi CreditAttribution: yanniboi commentedJust rewrote the above patch to get rid of some indentation/white-space errors.
Comment #7
tce CreditAttribution: tce commentedI'm wondering if 'lat_sin' is supposed to be declared three times?
Great patch, works perfectly for me.
Comment #8
tce CreditAttribution: tce commentedAs mentioned above there were three elements in an array with the same key. I've update the patch.
Comment #9
interdruper CreditAttribution: interdruper commentedPatch #8 works fine for me.
For those who are also using the Geolocation Views module, I have provided a patch for extending its support to Search API too: #2306175: Search API support
Comment #10
rcodina CreditAttribution: rcodina commentedPatch on #8 works for me too! Many thanks, this saved me the day! I wish this gets commited sooner than later!
Comment #12
derjochenmeyer CreditAttribution: derjochenmeyer commentedCommitted. Thanks!