The way the module is currently written it is using hook_init for what is traditionally performed via hook_update_N. The downside to that is that even once the update is performed, the module continues to check the schema on every Drupal bootstrap. The attached patch moves this code into event_views.install and uses hook_update_N.

CommentFileSizeAuthor
event_views.hook_update.patch1.98 KBjhedstrom
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

crystaldawn’s picture

Yes this is leftover code from those that were attempting to update older installs (the old 6.x version didnt work and people tried to update the older one that was completely broken). See, previously I did use hook_update_N but it was not affecting the older installs for some unknown reason. So I whipped up a quick hack to force it to work. I think it's been long enough that we can probably revert this back to hook_update_N since the old version is no longer supported anyways.

pwolanin’s picture

Patch looks sensible - running this on hook_itit() is really going to kill performance.