Skip to:
Content

Opened 4 years ago

Closed 4 years ago

Last modified 4 years ago

#2741 closed defect (fixed)

changing a reply from pending to public changes the IP address to the moderator's

Reported by: tharsheblows Owned by: johnjamesjacoby
Milestone: 2.6 Priority: normal
Severity: normal Version:
Component: Component - Replies Keywords: has-patch
Cc: jjjay@…

Description

From @jjj here: https://wordpress.slack.com/archives/bbpress/p1420902407000004

This is only happening for me in the admin section, not on the front end. I think we shouldn't change the IP address when changing the post status, except when it goes from a draft -> published.

Attachments (2)

2741.diff (643 bytes) - added by tharsheblows 4 years ago.
2741.2.diff (1.2 KB) - added by tharsheblows 4 years ago.
to include topics - just let me know the next error to fix. ;)

Download all attachments as: .zip

Change History (8)

@tharsheblows
4 years ago

#1 @tharsheblows
4 years ago

  • Keywords has-patch added

This ticket was mentioned in Slack in #bbpress by netweb. View the logs.


4 years ago

#3 @netweb
4 years ago

  • Keywords has-patch removed
  • Milestone changed from Awaiting Review to 2.6

Patch needs a refresh to also include topics ;)

@tharsheblows
4 years ago

to include topics - just let me know the next error to fix. ;)

#4 @tharsheblows
4 years ago

  • Keywords has-patch added

I forgot attachments don't comment related things -- this now has a patch for topics and replies.

#5 @johnjamesjacoby
4 years ago

  • Owner set to johnjamesjacoby
  • Resolution set to fixed
  • Status changed from new to closed

In 5850:

Admin: Add hidden_post_status field back to topic & reply metaboxes, and use draft or current post status as value.

This changeset fixes a bug that caused topics & replies to be marked as edited when moderated via wp-admin.

Hat-tip tharsheblows. Fixes #2741.

This ticket was mentioned in Slack in #bbpress by netweb. View the logs.


4 years ago

Note: See TracTickets for help on using tickets.