Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.
A problem with the patch is that bitcache.inc should not contain any Drupal-specific code, in order that it can be reused in other non-Drupal PHP projects.
I will refactor the code to provide event triggering in the repository class, which the Drupal module can then subclass implementing module_invoke_all().
I thought that bitcache.inc should not contain any Drupal-specific code, but put it there just to show what is expected. Your patch should fix the issue. So far I don't have any specific use case, just was playing around and noticed that the cron was not triggered even was configured so.
Comments
Comment #1
miglius commentedComment #2
Arto commentedComment #3
miglius commentedTypo - casting should be to object type in the following line:
+ $object = (array) array('id' => $id);
Comment #4
Arto commentedComment #5
Arto commentedA problem with the patch is that
bitcache.incshould not contain any Drupal-specific code, in order that it can be reused in other non-Drupal PHP projects.I will refactor the code to provide event triggering in the repository class, which the Drupal module can then subclass implementing
module_invoke_all().Comment #6
Arto commentedChangeset [180380] implements this in a way that preserves
bitcache.inc's suitability for non-Drupal use.Please test and report any problems... did you have a File Framework use case for this functionality, already?
Comment #7
miglius commentedI thought that bitcache.inc should not contain any Drupal-specific code, but put it there just to show what is expected. Your patch should fix the issue. So far I don't have any specific use case, just was playing around and noticed that the cron was not triggered even was configured so.