Download & Extend

Block list should also consider the context region disabled reaction

Project:Fusion
Version:7.x-1.x-dev
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:needs review

Issue Summary

When Fusion is calculating grid widths on regions, it loads the blocks in each region from core block placement and blocks placed with the context module.

The context module has a region reaction which disables a region by removing blocks from the page array. Since Fusion counts the blocks by directly accessing them from core and context the region disable reaction doesn't notify Fusion that it is disabled. A solution for this is to alter the fusion_core_block_list() function to also check the region reaction.

I have included a patch needs testing.

AttachmentSizeStatusTest resultOperations
fusion_block_list_region_reaction.patch875 bytesIdlePASSED: [[SimpleTest]]: [MySQL] 0 pass(es).View details | Re-test

Comments

#1

Actually, here is a patch for the current state of 1.x-dev

AttachmentSizeStatusTest resultOperations
fusion_block_list_region_reaction-1298916-1.patch880 bytesIdleFAILED: [[SimpleTest]]: [MySQL] Unable to apply patch fusion_block_list_region_reaction-1298916-1.patch. Unable to apply patch. See the log in the details link for more information.View details | Re-test

#2

Assigned to:Anonymous» aquariumtap

#3

Thanks for the patch. The patch from comment #1 worked for me.

#4

Status:needs review» needs work

The last submitted patch, fusion_block_list_region_reaction-1298916-1.patch, failed testing.

#5

Assigned to:aquariumtap» Anonymous

#6

Status:needs work» needs review

Hi, had time. Patch attached.

AttachmentSizeStatusTest resultOperations
context-region-disabled-reaction-1298916-6.patch760 bytesIdleFAILED: [[SimpleTest]]: [MySQL] Unable to apply patch context-region-disabled-reaction-1298916-6.patch. Unable to apply patch. See the log in the details link for more information.View details | Re-test

#7

Status:needs review» needs work

The last submitted patch, context-region-disabled-reaction-1298916-6.patch, failed testing.

#8

Status:needs work» needs review

Hm, weird.

AttachmentSizeStatusTest resultOperations
context-region-disabled-reaction-1298916-7.patch735 bytesIdleFAILED: [[SimpleTest]]: [MySQL] Unable to apply patch context-region-disabled-reaction-1298916-7.patch. Unable to apply patch. See the log in the details link for more information.View details | Re-test

#9

Status:needs review» needs work

The last submitted patch, context-region-disabled-reaction-1298916-7.patch, failed testing.

#10

Status:needs work» needs review

One last try. I have no idea which branch the bot is patching the file.. :(

AttachmentSizeStatusTest resultOperations
context-region-disabled-reaction-1298916-8.patch773 bytesIdlePASSED: [[SimpleTest]]: [MySQL] 0 pass(es).View details | Re-test

#11

My bad, didn't notice 'region' in context_get_plugin('reaction', 'region').

AttachmentSizeStatusTest resultOperations
context-region-disabled-reaction-1298916-9.patch843 bytesIdlePASSED: [[SimpleTest]]: [MySQL] 0 pass(es).View details | Re-test
nobody click here