If this is only a DragonflyCMS problem, feel free to "won't fix".

We have some banned users on our site, but all users are being imported as active. The code is checking the user_active field, which is always 1 (again, in my database). rather, user_level == 0 means a banned user. user_level looks like it's tied to roles, actually, since it's 1 for most users and 2-3 for users with elevated permissions.

Here's a patch that checks to see if user_level is greater than 0. If so, make the user active; else set them to blocked.

CommentFileSizeAuthor
user-status.patch738 byteswebchick

Comments

beginner’s picture

beginner commented
Title: Banned users imported as active » DragonflyCMS : Banned users imported as active
Status: Needs review » Needs work

The problem is that if I commit this patch, it's all those who use vanilla phpbb who will start complaining.

I am perfectly ok with committing some DragonflyCMS-specific patches, but obviously only as long as it does not break things for the majority of users.

One would need to make sure, but I don't think $user->user_level is set on a vanilla phpbb. You could use isset($user->user_level ) and if it is, override the status value.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented

It was in the table schema you posted over at the DragonflyCMS issue, but it might mean something totally different in the vanilla version.

I actually don't think DragonflyCMS-specific (or any non-vanilla-PHPBB-specific) patches should get committed to this module; when I'm done, I'll roll all of those specific changes up and post them over @ http://drupal.org/node/78509 as you asked me to back in, oh... August or something. ;) I was just reporting it separately in case it was a bug with vanilla PHPBB as well. If not, feel free to won't fix!

naheemsays’s picture

naheemsays commented
Status: Needs work » Closed (fixed)

closing old issues that may no longer be relevant.