Closed (won't fix)
Project:
Drupal core
Version:
8.0.x-dev
Component:
block.module
Priority:
Normal
Category:
Bug report
Assigned:
Issue tags:
Reporter:
Created:
10 Sep 2010 at 20:54 UTC
Updated:
29 Jul 2014 at 19:01 UTC
This hook is totally missing.
| Comment | File | Size | Author |
|---|---|---|---|
| drupal.block-region-alter.0.patch | 756 bytes | sun |
Comments
Comment #1
webchickPlease explain why this is a bug, else move to D8.
Comment #2
webchickWell in fact. It is not a bug, since we never had this hook to begin with in any other Drupal release. Nice idea, though.
Comment #3
sunCan we revisit that decision? This hook is only new, because it didn't make sense before, as the collated results of that function were strings. Now that we have renderable arrays in regions, it makes much more sense, and modules should rightfully be able to alter the region contents that have been generated by Block module, before they are passed off to the page building process.
Also, drupal_alter() is very cheap in D7, and this API addition is not able to break anything, so there is absolutely no harm in providing a proper hook at the right location. If you look at it from the right perspective, then contributed module developers can rightfully state: This is a bug, because Block module is missing hook_block_region_alter(), and my use-case/module requires that.
I'm specifically able to think of plenty of use-cases in Panels, Context, and other projects.
Comment #4
webchickCopy/pasting from the other issue:
"It's not that it's a API breaking change. It's that it's a new feature. Feature freeze ended 12 months ago. The only patches we should be working on at this stage are those that lead directly to Drupal 7 getting released. Drupal 8 for this."
Also, needs work because you don't document this new hook addition. tsk tsk. :P
Comment #5
sun