Making WordPress.org

Opened 2 years ago

Last modified 14 months 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 14 months ago.

Download all attachments as: .zip

Change History (13)

#1 @vladytimy
2 years ago

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

#2 follow-up: @sterndata
2 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
2 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.


2 years ago

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


2 years ago

#6 @fierevere
2 years ago

Implementing related ticket #5717
will obsolete this one.

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


2 years ago

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


22 months ago

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


14 months ago

@alexstine
14 months ago

#10 @alexstine
14 months 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
14 months 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
14 months 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.