Provide a rendered cache version of field_view_field().

This is great! Thanks so much.

The cleanup makes sense to me and the code re-use is nice.

I will probably get to a more detailed review in the next few days.

Committed and pushed to 7.x-1.x. Thanks!

I added the following interdiff:

diff --git a/render_cache.module b/render_cache.module
index 9fbe9c2..b015cb5 100644
--- a/render_cache.module
+++ b/render_cache.module
@@ -45,6 +45,7 @@ function render_cache_entity_view_callback($entities, $view_mode, $langcode = NU
   // Setup drupal_render style cache array.
   $cache_info = render_cache_cache_info_defaults();
   $cache_info['keys'] = array(
+    'render_cache',
     'entity',
     $entity_type,
     $view_mode,

instead of using just 'entity'.

If we need to change this, lets please discuss this in another issue.

Thanks for your great contribution.

This still needs a forward port to 7.x-2.x.

Nvm, I reverted the adding of render_cache: as prefix as its not really needed being in cache_render bin ...

Instead I added the following:

diff --git a/render_cache.module b/render_cache.module
index 9fbe9c2..5f3aee3 100644
--- a/render_cache.module
+++ b/render_cache.module
@@ -486,7 +486,11 @@ function render_cache_view_field($entity_type, $entity, $field_name, $display =
     'bundle' => $bundle,
   );
   $cid = render_cache_get_entity_cid($entity, $cache_info, $entity_context);
-  $cache = cache_get($cid, 'cache_render');
+
+  $cache = NULL;
+  if (isset($cache_info['granularity']) && $cache_info['granularity'] != DRUPAL_NO_CACHE) {
+    $cache = cache_get($cid, 'cache_render');
+  }
 
   if (!$cache) {
     $field_render = field_view_field($entity_type, $entity, $field_name, $display, $langcode);

to be consistent with the setting for entities.

So this is fixed?

#10 Yes, but not yet in 7.x-2.x it seems, so needs to still be ported ... I think ...