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

Add main admin access to forum without adding them as moderator #4659

Closed
Jimmi08 opened this issue Jan 15, 2022 · 1 comment · Fixed by #4730
Closed

Add main admin access to forum without adding them as moderator #4659

Jimmi08 opened this issue Jan 15, 2022 · 1 comment · Fixed by #4730
Assignees
Labels
type: enhancement An improvement or new feature request
Milestone

Comments

@Jimmi08
Copy link
Contributor

Jimmi08 commented Jan 15, 2022

Motivation

Main admin should have access to full site, but forum editing is limited by forum moderate class.
For example - class 248 Forum moderators
You can set this class to main admin too, but then is the main admin listed as a forum moderator, what is not correct.

Access to the forum (edit, moderate) should be for moderators, forum admin, and main admin.

For main admin just this is needed:

define('MODERATOR', (USER && in_array(USERID, $moderatorUserIds) || getperms('0') ));
in forum_viewforum.php
in forum_viewtopic.php
in forum_post.php

if(!in_array(USERID, $moderatorUserIds) && !getperms('0'))
in forum_class.php

I don't know how to add check if user is forum plugin admin P?) so... at least this should work for main admin.

Thanks

@Jimmi08 Jimmi08 added the type: enhancement An improvement or new feature request label Jan 15, 2022
@Jimmi08
Copy link
Contributor Author

Jimmi08 commented Mar 27, 2022

@CaMer0n Can I create PR for this one? Main admin as silent moderator of any forum . Thanks

@CaMer0n CaMer0n added this to the e107 2.3.2 milestone Apr 1, 2022
CaMer0n added a commit that referenced this issue Apr 1, 2022
Closes #4659 forum main admin as silent moderator
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: enhancement An improvement or new feature request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants