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.
 */
Files: 
CommentFileSizeAuthor
#1 1705536-1-block-typo-7.x.patch879 bytespfrenssen
PASSED: [[SimpleTest]]: [MySQL] 39,974 pass(es). View
#1 1705536-1-block-typo-8.x.patch904 bytespfrenssen
PASSED: [[SimpleTest]]: [MySQL] 39,743 pass(es). View

Comments

pfrenssen’s picture

Status: Active » Needs review
FileSize
904 bytes
PASSED: [[SimpleTest]]: [MySQL] 39,743 pass(es). View
879 bytes
PASSED: [[SimpleTest]]: [MySQL] 39,974 pass(es). View

Here are patches for both D7 and D8.

pfrenssen’s picture

Issue tags: +needs backport to D7

Adding tag.

pfrenssen’s picture

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

Status: Needs review » Reviewed & tested by the community

Seems RTBC

catch’s picture

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

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

Issue tags: +needs backport to D7

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

dcam’s picture

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

Status: Reviewed & tested by the community » Fixed

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