Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
I seem to have lost the ability for set moderators to lock topics by altering the comment settings. Edit and Delete options are available and work as expected, however the Vertical Tabs List with the comment settings options is not visible.
I've also installed Devel and enabled DNA but it's not outputting anything (and I have enabled the block for it). This is another issue though and may/may not be related.
Comments
Comment #1
KarinGWhen did this happen?
What configuration screen - exactly - are you looking at?
Comment #2
salvisPlease provide the path as well as before and after screenshots.
Comment #3
shane1090 CreditAttribution: shane1090 commentedAfter a bit of tinkering around I managed to get Devel working, and it was showing that the moderator had all the correct permissions to modify the node, however the vertical tabs with further options still were not visible. Looking into the code, I found it to be a problem with the _forum_access_node_form() function in forum_access.node.inc.
This may not be where the bug is originating from, but it's where I made a couple of tweaks to fix it on my installation.
In my case, the part of this function that applys modifications for moderators didn't seem to be adding options for the stickys, comment settings and revision. To make this available, I need to modify the switch statement to add in a few more variables:
Original Switch statement:
Modified Switch code:
Notice the additional 'case' additions for 'revision information' and 'comment_settings' and also the additional line after the foreach loop in the 'options' case. Making these additions enabled it to work as I would expect it to.
I've also tested this with a bare minimum installation as well and it's showing the same problem - no way to modify the options above as a moderator.
Comment #4
salvisSorry, we're not communicating. I still don't know what the problem is that you're trying to fix.
1. Provide the path (URL) where the problem occurs
2. Report according to
a) what you did
b) what you expected to happen
c) what happened instead
3. Provide screenshots of the faulty display and the healthy one.
4. Finally, please provide your changes as a patch and set the status to NR, so that the testbot will look at it.
Comment #5
salvisOk, found it now.
Let's try this...
Comment #7
salvisHmm, weird, no details link, and no retry link either.
Retry...
Comment #9
salvisThree times...
Comment #11
salvisUpdated patch.
Comment #13
salvisSame patch again after rfay has fixed the testbot: #1768442: Testbot fails to get dependency after adding a version number
Comment #15
salvisWe'll get there eventually...
Comment #16
salvisThere!
Committed to the -dev version.