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.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

noslokire’s picture

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

jm.federico’s picture

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

tce’s picture

Can anyone confirm this patch works?

derjochenmeyer’s picture

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

Anyone else who tested this patch?

yanniboi’s picture

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

tce’s picture

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

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

interdruper’s picture

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

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

Status: Reviewed & tested by the community » Fixed

Committed. Thanks!

Status: Fixed » Closed (fixed)

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