Context only loads some of the block table fields, instead of the sel;ect * in block_load. Since there is not hook_block_load() functionility in drupal 7, modules that need fields in the block object need to append columns to block table. So context should load all the block table fields, and remove those it is overridding (status, visibility, and cache I believe).

(love this module!)

CommentFileSizeAuthor
#4 context-load-all-block-table-fields-1154908-4.patch1.3 KBtekante
#1 context-should-load-all-block-table-fields-1154908-1.patch3.31 KBjohnbarclay

Comments

johnbarclay’s picture

johnbarclay commented
StatusFileSize
new context-should-load-all-block-table-fields-1154908-1.patch3.31 KB

here's a patch for this.

bowersox’s picture

bowersox commented

+1 for getting this issue addressed.

If the current practice is for modules to add columns to the block table, then we should be loading them all.

tekante’s picture

tekante commented
Issue tags: +D7 stable release blocker

Adding tag for tracking

tekante’s picture

tekante commented
StatusFileSize
new context-load-all-block-table-fields-1154908-4.patch1.3 KB

Patch in 1 does not apply, attaching a version that should apply against the latest 7.x-3.x version. Needs feedback on whether this resolves the error that you are having (or information about how to replicate some of the inconsistent behavior). Could also use results of testing out this patch on your site.

mgifford’s picture

mgifford
he/him
Primary language English
commented
Status: Active » Needs review

Changing status for bot.

tekante’s picture

tekante commented
Status: Needs review » Fixed

Patch committed

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