Fix for incorrect 'config' permissions related to automoderator config #1405
Conversation
Mods with 'wiki' and not 'config' permissions will be sent to a viewable version of automoderator config instead of a forbidden page. Also, a mod has 'config' permissions will now be able see the button in moderation tools clinging with prior reddit documentation.
Like I said in #1404 adding the wiki mods the shortcut is both inconsistent and makes no sense. Wiki mods do not have the shortcut for anything else; so why should tehy have it for the automod config? They also would have no use for it. Given the inconsistency it's not robust at all; just weird. |
I don't understand why you are so adamant against this. AutoMod is a huge part of moderating and having a shortcut to view AutoMod's config is extremely helpful. This commit doesn't change or add anything new, it just fixes a bug. |
I'm "adamant" about this because 1, inconsistencies are never a good thing, and 2, even if it is a huge part of moderating (which it is), a wiki mod gains absolutely nothing from having a quick and easy shortcut, except that shortcut, which is not useful to him or her, since that page is not useful to him/her _whatsoever_. Edit: Combining that, with the fact that only after I had found and fixed the bug and mentioned it on /r/bugs, you deleted your /r/bugs post, and came here to make an originally semi broken #1404 as it is; and were unable to do the action of rebasing, gives off the vibe that you don't really know what you are doing. |
'Wiki' mods already have access to this config, this doesn't change that. As a mod of a 3.1+ million subscriber subreddit where I have 'wiki' perms without 'config' let me say the shortcut is extremely valuable. I would not be able to moderate without viewing that config a couple of times per day to see what domains we screen, or other settings we have dialed in. |
...the comment you made ends in the middle of a sentence. Edit: Even with that you said, yes, wiki mods have access to the config. So why is there not a link to /wiki/config/sidebar? or /wiki/config/stylesheet? Those can be useful to a wiki mod, using images / special css based hidden info. But nowhere near to a proper extent, so it is not. I don't personally see how it could be useful still to use the automod config to see your domains or "other settings". You can not affect them. You can not use them for something else, other than writing to the wiki, which since you are already on it, you could just view /pages. |
Update: it no longer ends in the middle of a sentence; continue your peace after reading mine. |
Because anyone can view sidebar or stylesheet of any subreddit. You don't have to be a mod for that. Look I know you don't understand and I can't explain it to you. Your largest sub is 17,000 and that's fine. This is not a big deal and I am moving on. If the dev team wants this commit then wonderful. All I am doing here is fixing a bug. /endreplies |
"Your largest subreddit is 17,000 and that's fine." Firstly, that's not the point. You're acting like that gives you some kind of privilege. You have been a mod of that sub (which I will keep the name of private) for 12 damn days. Less than two damn weeks. Don't act high and mighty. Even if you were a mod of it for much much longer; it still gives you no privilege. You also are considering that I have no alts; and am not a mod of a larger sub. For the sake of telling, no, I do not mod a larger sub; but this matter is of principle (which I also used on you). It's not that I don't understand; it's that you don't understand the matter of the fact that it's highly inconsistent, and has no actual use unless you can prove a use; which you have just now given up in doing. Yes, it isn't a big deal, but it is still a matter of consistency and principle. You are not "fixing a bug". You are correcting a bug (which if I may remind you, I'd bet if I had not gone and done it for you first and notify you, you would either take forever to find it or wouldn't find it at all); while also adding some absolutely useless functionality of your own; that is also inconsistent with previous matter, and originally your code had a duplication issue, and now it isn't structured in the best of ways (an elif block would be more easy to comprehend to an outsider). Additional functionality should not really be in this repo according to the README and CONTRIBUTING notes (though surprisingly it doesn't apply to API AFAIK); but technically no one is stopping it so meh. Even so, that's for /r/ideasfortheadmins / a message to /r/reddit.com. This repo should be for reporting confirmed issues, and or fixing them. "/endreplies" What, is this some kind of grade schooler chatroom? |
Thanks for the contribution, I think we're going to pass on this for now. Also as an aside (to both of you), arguing isn't tolerated in pull requests here, as it can be a chilling effect to other folks who may want to contribute but fear the discussion around it. Feel free to flame each other via reddit PM next time. |
Grants mods with 'config' permissions the ability to see "automoderator config" in the moderation tools sidebar and link to the editor. Previously this button only showed for 'wiki' mods which did not match with prior documentation that allows this action for 'config' mods. Furthermore this resolves #1402 by linking wiki mods to the viewable URL instead of the edit one to which they do not have permission.
A more robust solution than #1403 as it seamlessly retains 'wiki' mods access to that "automoderator config" button and instead links them to a viewable page instead of just simply giving the access to 'config' mods and leaving 'wiki' mods with a shortcut.