Closed (fixed)
Project:
Block Cache Alter
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
3 Mar 2013 at 11:08 UTC
Updated:
22 Nov 2014 at 11:14 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
fabianx commentedClarifying title ...
Comment #2
iamEAP commentedRan into this months ago when initially upgrading from D6 to D7 and decided it wasn't a priority (so we just left it disabled). Finally found a need for block cache alter, and here I am.
Here's a patch that adds an update hook for D6->D7 upgrades. The basic logic:
I also wrapped the hook_block_info_alter() DB query in a try/catch so that no errors are thrown between enabling the module and running the update hook.
Comment #3
fabianx commentedI am okay with that, but isn't a db_query() update query to populate bid simpler to write and read?
Comment #4
iamEAP commentedI believe you're only allowed to use db_query() for SELECT queries.
Comment #5
fabianx commented**RTBM** - looks good to me.
Comment #6
fabianx commentedCommitted and pushed to 7.x-1.x. Thanks!