Making WordPress.org

Opened 3 years ago

Last modified 2 years ago

#5708 new defect (bug)

Allow only moderators to remove modlook tag

Reported by: vladytimy's profile vladytimy Owned by:
Milestone: Priority: high
Component: Support Forums Keywords: has-patch needs-testing
Cc:

Description

Currently, if someone reports a topic, the tag modlook is added. As far as I understand, the author or anyone replying on that topic can remove modlook tag from the list, before a moderator has the chance to review that report, so unless the topic is reported again, it stays in the wild as it is.

The ability to remove moodlook tag should only be available to moderators.
Maybe an extra check user_can( $user_id, 'moderate' ) when modlook is removed from the list of tags would do the trick.

Attachments (1)

5708.diff (1.2 KB) - added by alexstine 2 years ago.

Download all attachments as: .zip

Change History (13)

#1 @vladytimy
3 years ago

  • Summary changed from Allow only moderators to remove moodlook tag to Allow only moderators to remove modlook tag

#2 follow-up: @sterndata
3 years ago

Perhaps it would be better to remove this as an allowable tag and allow it to be set/reset only via the "report topic" button. Anyone could report a topic, only a mod or above could click the removal button.

#3 in reply to: ↑ 2 @vladytimy
3 years ago

Replying to sterndata:

That is even better. Marius & Dion had the same idea. (see chat)

To keep the implementation of the Modlook Tag View, maybe it has to stay a tag, but don't show it in the tags list anymore.

This ticket was mentioned in Slack in #meta by vladytimy. View the logs.


3 years ago

This ticket was mentioned in Slack in #forums by yui. View the logs.


3 years ago

#6 @fierevere
3 years ago

Implementing related ticket #5717
will obsolete this one.

This ticket was mentioned in Slack in #meta by vladytimy. View the logs.


3 years ago

This ticket was mentioned in Slack in #forums by yui. View the logs.


3 years ago

This ticket was mentioned in Slack in #meta by tellyworth. View the logs.


2 years ago

@alexstine
2 years ago

#10 @alexstine
2 years ago

  • Keywords has-patch needs-testing added

I made an attempt at blocking this in my patch. It isn't pretty, but it may work for now.

#11 @sterndata
2 years ago

Just occurs to me: Whoever added the modlook should also be able to remove it -- maybe it was a mistake or they re-thought it.

#12 @alexstine
2 years ago

I am not sure that info is tracked. It looks like there is some level of topic and tag relationship but I never saw a good way for user associations. Would probably need a way to track that in some form of extra meta info.

Note: See TracTickets for help on using tickets.