#2645 closed defect (bug) (fixed)
Add support to approve and unapprove pending topics and replies
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 2.6 | Priority: | normal |
Severity: | normal | Version: | |
Component: | API - Moderation | Keywords: | has-patch commit |
Cc: |
Description
Add support to approve and unapprove pending topics and replies
Related:
- #459 Consider making moderation forum specific
- #2546 bbpress skips check for number of links if blacklist is empty
- #2589 Pending Moderation template notice
- #2593 Allow Moderators and Keymasters to see private topic content from users
- #2644 Add editable reply-status dropdown in form-reply.php for moderators when editing replies
Attachments (6)
Change History (34)
This ticket was mentioned in IRC in #bbpress-dev by netweb. View the logs.
11 years ago
This ticket was mentioned in IRC in #bbpress-dev by netweb. View the logs.
11 years ago
#4
@
11 years ago
Latest 2645.2.diff
patch is a refresh only of 2645.diff
to apply cleanly against trunk.
#5
@
11 years ago
Patch 2645.3.diff adds 'pending' support to topic/reply 'hidden' counts and some minor iterations on previous patch:
- Include 'pending' topics in bbp_get_forum_topic_count_hidden() Added in 2645.3.diff
- Include 'pending' replies in bbp_get_topic_reply_count_hidden() Added in 2645.3.diff
#6
@
10 years ago
- Keywords needs-refresh added; dev-feedback 2nd-opinion removed
Patch no longer applies clean, sadly. Mind refreshing this? It's a huge effort, and would be nice to get into 2.6 since it's already patched.
#17
@
10 years ago
All that's left is admin/topics.php
I believe, needing manual patch-merging to work around conflicts introduced in r5496.
This ticket was mentioned in IRC in #bbpress-dev by netweb. View the logs.
10 years ago
#19
follow-up:
↓ 21
@
10 years ago
- Keywords has-patch added; needs-refresh removed
This should finish the ticket. Attached is a clean patch for admin/topics.php.
#21
in reply to:
↑ 19
@
10 years ago
- Keywords commit added
Replying to thebrandonallen:
This should finish the ticket. Attached is a clean patch for admin/topics.php.
Thanks, 2645.4.diff is good, just needs committing by @jjj for sanity ;)
One last thing and I think we can resolve this as fixed, in 2645.5.diff we need to update, the topic bbp_single_topic_description()
and forum bbp_get_single_forum_description()
when a topic or reply is approved or unapproved with the topic walker bbp_update_topic_walker
or reply walker bbp_update_reply_walker
.
This ticket was mentioned in Slack in #bbpress by netweb. View the logs.
10 years ago
#24
@
10 years ago
Updated patch 2645.6.diff replaces 2645.4.diff and 2645.5.diff and also add's spam
row action for replies with pending
status.
This ticket was mentioned in Slack in #bbpress by netweb. View the logs.
10 years ago
#26
@
10 years ago
- Owner set to johnjamesjacoby
- Resolution set to fixed
- Status changed from new to closed
In 5583:
In 2645.diff first pass:
Approve
/Unapprove
topic and reply toggles in the backendApprove
/Unapprove
topic and reply admin links in the frontend.status-pending
CSSbackground-color: #fef7f1;
for 'pending' topics/replies (via WP Core)bbp_approve_reply()
/bbp_unapprove_reply()
functionsbbp_approve_topic()
/bbp_unapprove_topic()
functionsbbp_is_reply_pending()/
bbp_is_topic_pending()` helper functionsbbp_is_reply_private()/
bbp_is_topic_private()` helper functionsbbp_reply_approve_link()
/bbp_topic_approve_link()
functions$success = wp_update_post( $post_data );
from reply admintoggle_reply()
bbp_unspam_reply()
/bbp_spam_reply()
andbbp_approve_reply()
/bbp_unapprove_reply()
functions.toggle_topic()
does not firewp_update_post()
after topiic actionsToDo:
bbp_check_for_moderation()
andbbp_check_for_blacklist()
published
/pending
verbiage topic status viabbp_get_form_topic_status_dropdown()
in frontend and backendpublished
/pending
verbiage reply status viabbp_get_form_reply_status_dropdown()
in frontend and backendbbp_get_forum_topic_count_hidden()
bbp_get_topic_reply_count_hidden()