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.

CommentFileSizeAuthor
#8 geolocation-property-info-callback-1366642-8.patch3.01 KBtce
#6 geolocation-property_info_callback-1366642-6.patch2.86 KByanniboi
geolocation_property_info.patch2.84 KBwesnick

Comments

noslokire’s picture

noslokire commented

We must be doing the same thing. I will try and apply get back

jm.federico’s picture

jm.federico commented

Hi 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

jm.federico’s picture

jm.federico commented

Marking #1320564: Fix for Search API not picking up the entity to index it's fields as dup of this

tce’s picture

tce commented

Can anyone confirm this patch works?

derjochenmeyer’s picture

derjochenmeyer commented

How about this: http://drupal.org/node/1435070#comment-6153698

Anyone else who tested this patch?

yanniboi’s picture

yanniboi commented
StatusFileSize
new geolocation-property_info_callback-1366642-6.patch2.86 KB

Just rewrote the above patch to get rid of some indentation/white-space errors.

tce’s picture

tce commented

I'm wondering if 'lat_sin' is supposed to be declared three times?

<?php
+    'lat_sin' => array(
+      'label' => 'Left Latitude',
+      'description' => '',
+      'type' => 'decimal',
+      'getter callback' => 'entity_property_verbatim_get',
+    ),
+    'lat_sin' => array(
+      'label' =>  'the sine of latitude',
+      'description' => '',
+      'type' => 'decimal',
+      'getter callback' => 'entity_property_verbatim_get',
+    ),
+    'lat_sin' => array(
+      'label' =>'the sine of latitude',
+      'description' => '',
+      'type' => 'decimal',
+      'getter callback' => 'entity_property_verbatim_get',
+    ),
?>

Great patch, works perfectly for me.

tce’s picture

tce commented
Issue summary: View changes
StatusFileSize
new geolocation-property-info-callback-1366642-8.patch3.01 KB

As mentioned above there were three elements in an array with the same key. I've update the patch.

interdruper’s picture

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

Patch #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

rcodina’s picture

rcodina
Primary language Catalan
Location Barcelona
commented

Patch on #8 works for me too! Many thanks, this saved me the day! I wish this gets commited sooner than later!

  • derjochenmeyer committed 9e41e24 on 7.x-1.x 
    Issue #1366642 by tce, yanniboi, wesnick: Added Property Info callback...
derjochenmeyer’s picture

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

Committed. Thanks!

Status: Fixed » Closed (fixed)

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