If user is banned it shows the proper message on the page.

CommentFileSizeAuthor
ulogin.patch898 bytesFrizus
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

duozersk’s picture

duozersk
he/him
Primary language Russian
Location Moscow
CreditAttribution: duozersk commented
Assigned: Unassigned » duozersk
Category: support » task
Status: Patch (to be ported) » Fixed

Frizus,

Thanks for a patch... it didn't apply as it is really broken - there should have been "+" signs for added strings. Anyway, you are completely right that we should show an error message for blocked user accounts.

I have reworked your idea and committed to both 7.x and 6.x (6.x didn't really need it as it was handled quite nicely by the core function user_external_login() - I just think that the code in 7.x and 6.x branches should be similar to simplify the maintenance).

Thanks
AndyB

Frizus’s picture

Frizus CreditAttribution: Frizus commented
Status: Fixed » Closed (fixed)

Thanks for so quick response and reworking the idea into more clean code.