Support hook_entity_property_info() [#1489214]

For proper entity module support we need a property_type in hook_field_info() and a hook_entity_property_info().
This would make fields accessible to modules using entity wrapper functionality etc. like Rules and Search API.

For more info see #1393452-4: Getting fields to appear on the 'Select fields to index' page

I plan to do this very soon (hopefully by tomorrow) and will post a patch.

Comment	File	Size	Author
#4	phone_number-property_info-1489214-4.patch	2.94 KB	grasmash
#3	phone_number-property_info-1489214-3.patch	2.96 KB	grasmash
#2	phone_number-property_info-1489214-2.patch	2.96 KB	grasmash

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

grasmash CreditAttribution: grasmash commented 21 May 2012 at 16:38

Did you ever make this change? I would love a patch!

Comment #2

grasmash CreditAttribution: grasmash commented 21 May 2012 at 17:23

Status:

Active

» Needs review

File	Size
phone_number-property_info-1489214-2.patch	2.96 KB

Here's a first attempt. It seems to be working so far! Thanks to @rooby for posting helpful links.

Comment #3

grasmash CreditAttribution: grasmash commented 21 May 2012 at 17:28

File	Size
phone_number-property_info-1489214-3.patch	2.96 KB

Just realized I used the wrong function naming convention for this module. Updated patch attached.

Comment #4

grasmash CreditAttribution: grasmash commented 21 May 2012 at 17:31

File	Size
phone_number-property_info-1489214-4.patch	2.94 KB

Alright. Last try. I rolled that last patch incorrectly. Monday mornings are killers.

Comment #5

rooby CreditAttribution: rooby commented 21 May 2012 at 23:11

Wow I never got back to this one.

Thanks for the patch, I will try to test it soon.

Comment #6

rodmaz CreditAttribution: rodmaz commented 6 February 2013 at 21:53

Tried to use this patch. Although no complains I don't see the fields visible.

Comment #7

rodmaz CreditAttribution: rodmaz commented 7 February 2013 at 00:50

Priority:

Normal

» Major

Can anyone please check this out? I could not find yet the reason but calls to entity_get_all_property_info() still do not show cck_phone fields, even after this patch.