Currently the module doesn't take in account that the entitycache could be flushed during execution and a new sync is necessary.
But the bigger problem is, that there's no way take this in account until this patch for entitycache is applied #1157430: Provide hook_entitycache_reset().
As soon as entitycache is patched the attached patch should do the trick.

CommentFileSizeAuthor
title-reset-cache-on-entitycache-reset.patch625 bytesdas-peter
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

plach’s picture

plach
he/him
Primary language Italian
Location Venezia
CreditAttribution: plach commented

Why don't we just do drupal_static_reset('title_entity_sync') here?

das-peter’s picture

das-peter CreditAttribution: das-peter commented

As far as I the code understand we only get the id's of the actually reset entities. This can be just one or multiple. Thus I think this loop is more efficient than reprocessing the sync of all entities.

plach’s picture

plach
he/him
Primary language Italian
Location Venezia
CreditAttribution: plach commented
Status: Needs review » Reviewed & tested by the community

Ok, thanks.

plach’s picture

plach
he/him
Primary language Italian
Location Venezia
CreditAttribution: plach commented
Status: Reviewed & tested by the community » Fixed

Committed to HEAD. Thanks!

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

  • Commit dea46f0 on 7.x-1.x, workbench by plach: 
    Issue #1157438 by das-peter: Fixed Reset sync cache on entitycache reset...