There is a typo in the documentation of block_block_list_alter(): 'visibilty' => 'visibility'.

/**
 * Implements hook_block_list_alter().
 *
 * Checks the page, user role, and user-specific visibilty settings.
 * Removes the block if the visibility conditions are not met.
 */
CommentFileSizeAuthor
#1 1705536-1-block-typo-7.x.patch879 bytespfrenssen
#1 1705536-1-block-typo-8.x.patch904 bytespfrenssen

Comments

pfrenssen’s picture

pfrenssen
Location Sofia
commented
Status: Active » Needs review
StatusFileSize
new 1705536-1-block-typo-8.x.patch904 bytes
new 1705536-1-block-typo-7.x.patch879 bytes

Here are patches for both D7 and D8.

pfrenssen’s picture

pfrenssen
Location Sofia
commented
Issue tags: +Needs backport to D7

Adding tag.

pfrenssen’s picture

pfrenssen
Location Sofia
commented

Just checked if this would be a candidate to be backported to D6 but this is not needed since this hook did not yet exist in D6.

steinmb’s picture

steinmb commented
Status: Needs review » Reviewed & tested by the community

Seems RTBC

catch’s picture

catch
he/him
Primary language English
commented
Version: 8.x-dev » 7.x-dev
Status: Reviewed & tested by the community » Patch (to be ported)

Thanks. Committed/pushed to 8.x, moving to 7.x for backport.

pfrenssen’s picture

pfrenssen
Location Sofia
commented
Status: Patch (to be ported) » Needs review
Issue tags: -Needs backport to D7

#1: 1705536-1-block-typo-7.x.patch queued for re-testing.

dcam’s picture

dcam commented
Issue tags: +Needs backport to D7

#1: 1705536-1-block-typo-7.x.patch queued for re-testing.

dcam’s picture

dcam commented
Status: Needs review » Reviewed & tested by the community

I checked out the D7 patch in #1. It's the same change that was made to D8 and it looks good to me. Marking it as RTBC.

David_Rothstein’s picture

David_Rothstein commented
Status: Reviewed & tested by the community » Fixed

Committed to 7.x - thanks! http://drupalcode.org/project/drupal.git/commit/e66de8b

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