Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Expand Forum Moderator permissions #688

Closed
Starpaul20 opened this issue Jun 2, 2014 · 30 comments
Closed

Expand Forum Moderator permissions #688

Starpaul20 opened this issue Jun 2, 2014 · 30 comments
Assignees
Labels
b:1.8 Branch: 1.8.x s:resolved Status: Resolved. Solution implemented or scheduled t:enhancement Type: Enhancement. Contains minor improvements
Milestone

Comments

@Starpaul20
Copy link
Member

Add more permissions to the forum moderator system to allow for more flexibility.

Also add group based permissions to the Mod CP

@Starpaul20 Starpaul20 added this to the 1.8 Beta 2 milestone Jun 2, 2014
@Starpaul20 Starpaul20 self-assigned this Jun 2, 2014
Starpaul20 added a commit that referenced this issue Jun 14, 2014
Adding the permissions themselves
@Starpaul20
Copy link
Member Author

This adds the permissions to the database and the Admin CP form.

Starpaul20 added a commit that referenced this issue Jun 14, 2014
Round 1 of updated Mod permissions
Starpaul20 added a commit that referenced this issue Jun 14, 2014
Round 2 of updated Mod permissions. Only showthread.php and Mod CP should be left
Starpaul20 added a commit that referenced this issue Jun 15, 2014
Finished Show Thread and Mod CP
@Starpaul20
Copy link
Member Author

That finishes everything

Starpaul20 added a commit that referenced this issue Jun 15, 2014
Forgot the Forum Display mod menu; improved on Show Thread menu
Starpaul20 added a commit that referenced this issue Jun 15, 2014
Starpaul20 added a commit that referenced this issue Jun 15, 2014
Permission scheme in Mod CP updated to work with new mod perms
Starpaul20 added a commit that referenced this issue Jun 16, 2014
Finished permission scheme in Mod CP updated to work with new mod perms
@Starpaul20
Copy link
Member Author

Okay, that finishes everything.

@Starpaul20 Starpaul20 added fixed and removed feedback labels Jun 16, 2014
@xykul
Copy link

xykul commented Jun 16, 2014

Can view unapproved threads and posts? and Can view deleted threads and posts? options do not work.

Starpaul20 added a commit that referenced this issue Jun 16, 2014
Fixed unapproved and deleted permissions
@Starpaul20
Copy link
Member Author

That should fix it.

@xykul
Copy link

xykul commented Jun 16, 2014

Thank you.
There is a problem.
Can view unapproved threads and posts?
Can view deleted threads and posts?
Two setting run at the same time, working separately.

@DiogoParrinha
Copy link
Contributor

@PaulBender

@Starpaul20
Copy link
Member Author

@xykul not sure what you mean. The settings work fine for me.

@xykul
Copy link

xykul commented Jun 20, 2014

Individual works..
Both work..
hatalar

hatalar2

@Starpaul20
Copy link
Member Author

Okay, I see what the problem is, fixing...

Starpaul20 added a commit that referenced this issue Jun 20, 2014
Fixed unapproved and deleted permissions with posts
Starpaul20 added a commit that referenced this issue Jun 20, 2014
Missed some permissions on the postbit(s)
@Sama34
Copy link
Contributor

Sama34 commented Jun 21, 2014

Did you only added per-moderator permissions or per-group and/or per-forum too?

@Starpaul20
Copy link
Member Author

It's just per moderator permissions I added

@Sama34
Copy link
Contributor

Sama34 commented Jun 22, 2014

Do you plan on adding per-group ones like "Can manage annoucements"? I thought you already did.

@Starpaul20
Copy link
Member Author

I added some Mod CP permissions in the moderator permissions (the ones that can be regulated per forum, like announcements, mod queue, reported posts and viewing the mod log). I will add per-group ones right now.

Starpaul20 added a commit that referenced this issue Jun 22, 2014
Added permissions to the Mod CP
@Starpaul20
Copy link
Member Author

That adds a group-based permission scheme to the Mod CP

Starpaul20 added a commit that referenced this issue Jun 22, 2014
Mod CP permissions for the profile Mod CP links
Added model pop-up for user notes if over cutoff
@Starpaul20
Copy link
Member Author

This commit adds the new Mod CP permissions for the profile Mod CP links (forgot about them)

It also adds model pop-up to view all user notes if notes are longer than the 100 characters cutoff.

Starpaul20 added a commit that referenced this issue Jun 22, 2014
Last remaining Mod CP locations now use Mod CP permissions
@xykul
Copy link

xykul commented Jun 22, 2014

Not displayed.. does not work.
modcp

modcp2

@Destroy666x
Copy link
Contributor

@xykul, does your account have group with Super Moderator permission? If not, you have to report/log/unapprove something in assigned forum(s) to see the menu. Not sure why, I don't think it should be like this.

EDIT: it's actually more complicated and messy than I thought - right now single forum user moderators lose their functionality because they have to be in a group which has these permissions.

@xykul
Copy link

xykul commented Jun 22, 2014

Moderators group
modcp.php
210, && $mybb->usergroup['canmanageannounce'] == 1)
215, && $mybb->usergroup['canmanagemodqueue'] == 1)
220, && $mybb->usergroup['canmanagereportedcontent'] == 1)
225, && $mybb->usergroup['canviewmodlogs'] == 1)
lines can not be valid.

@DiogoParrinha
Copy link
Contributor

@PaulBender this is urgent as you have committed directly.

@Starpaul20
Copy link
Member Author

Not quite sure what the problem is. If a user isn't a super moderator they must be assigned to at least one forum in order to see the options in the Forums & Posts menu. Otherwise I'd be showing them links that go nowhere.

I should add a notice in the Admin CP making it clear to Admins that it works that way.

Starpaul20 added a commit that referenced this issue Jun 29, 2014
Added notices in the Admin CP telling uses how the system works
@Destroy666x
Copy link
Contributor

  1. Assign moderator X to a forum, leave him in Registered group which doesn't have these permissions ticked.
  2. Single moderator can't see anything from his forum even though forum-specific "Moderator CP Permissions" allow him to do it. You have to move him to other group first.

Doesn't make any sense for me.

@Starpaul20
Copy link
Member Author

The Registered group shouldn't have access to the Mod CP at all. If a mod is being left in the Registered group, or any group that doesn't have permission to access the Mod CP, they shouldn't access anything in the Mod CP, period.

@Destroy666x
Copy link
Contributor

Well then, fair enough. I thought the moderator system was confusing already, now it's even more. But whatever, 'period'.

@Sama34
Copy link
Contributor

Sama34 commented Jun 29, 2014

@PaulBender Assigning an user as moderator to X forum doesn't behave the same that assigning a group to it. Probably it should be modified to work the same way. Currently single moderators are allowed to enter the ModCP regardless of their group permissions.

@Destroy666x, I think per-group permission should override per-moderator ones as it currently does (they may be moderate posts/content but cannot access the ModCP, which the permissions aim to control). But yeah, MyBB is confusing at some parts.

@DiogoParrinha
Copy link
Contributor

I agree with @Sama34

@Starpaul20
Copy link
Member Author

@Sama34 You sure? When I assigned a user to moderate a forum and made sure that they were in a group that can't access the Mod CP, I got a You do not have permission to access this page. error everywhere in the Mod CP.

modcp.php checks to see if you have the Can access Mod CP group permission first and throws an error if you don't have it (or you're a guest).

I agree that per-group permission should override per-moderator permission (the per-moderator permission was really intended to control ability to use that function for the forum the moderator is assigned to).

Starpaul20 added a commit that referenced this issue Jun 29, 2014
Updated descriptions for moderator permissions dealing with the Mod CP permissions.
@Starpaul20
Copy link
Member Author

I updated the descriptions for the moderator permissions dealing with the Mod CP permissions to make it clear that they only apply for that forum.

@Sama34
Copy link
Contributor

Sama34 commented Jun 29, 2014

Never mind, the reason is the user being moved to the "Moderators" groups when assigned as a single forum moderator. No sure why it works like this.

@DiogoParrinha
Copy link
Contributor

Closing this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
b:1.8 Branch: 1.8.x s:resolved Status: Resolved. Solution implemented or scheduled t:enhancement Type: Enhancement. Contains minor improvements
Projects
None yet
Development

No branches or pull requests

5 participants