The d8 version sort of honored blocked users by creating a session and never letting them get anywhere. This patch should fix that. see also https://www.drupal.org/node/2475257 for version 7x

CommentFileSizeAuthor
#6 honor_blocked_users_earlier-2914190-6.patch700 bytesel1_1el
#2 honor_blocked_users_earlier-2914190-2.patch815 bytesel1_1el
honor_blocked_users_earlier.patch799 bytesel1_1el
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

el1_1el created an issue. See original summary.

el1_1el’s picture

el1_1el CreditAttribution: el1_1el commented
FileSize
honor_blocked_users_earlier-2914190-2.patch815 bytes

missed a false check

  • mlhess committed 676ac66 on 8.x-1.x authored by el1_1el 
    Issue #2914190 by el1_1el: honor blocked users completely

mlhess credited mlhess.

mlhess’s picture

mlhess CreditAttribution: mlhess as a volunteer commented
Status: Needs review » Fixed

Thanks!

el1_1el’s picture

el1_1el CreditAttribution: el1_1el commented
Status: Fixed » Needs review
FileSize
honor_blocked_users_earlier-2914190-6.patch700 bytes

Sorry, I messed this up. I think returning the anonymous user when the user is blocked is preferable when they are allowed. Patch attached

SuperfluousApostrophe’s picture

SuperfluousApostrophe CreditAttribution: SuperfluousApostrophe at Pennsylvania State University commented
Assigned: Unassigned » SuperfluousApostrophe
Status: Needs review » Reviewed & tested by the community

Thanks for the followup patch. This has been added to the 8.x-1.1 branch, which will soon be released as the first stable Alpha.

SuperfluousApostrophe’s picture

SuperfluousApostrophe CreditAttribution: SuperfluousApostrophe at Pennsylvania State University commented
Version: 8.x-1.x-dev » 8.x-1.1-alpha1
Status: Reviewed & tested by the community » Fixed

This has been integrated into 8.x-1.1-alpha1

Status: Fixed » Closed (fixed)

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