Skip to:
Content

bbPress.org

Opened 10 years ago

Last modified 7 years ago

#2587 reopened defect (bug)

Moving a topic in the backend the forum meta is not updated

Reported by: tsabar's profile tsabar Owned by: johnjamesjacoby's profile johnjamesjacoby
Milestone: 2.7 Priority: normal
Severity: normal Version: 2.5.3
Component: Component - Topics Keywords: needs-patch
Cc:

Description

after moving a post from one forum category to another, i noticed the category did not update in the main forum listing, only within the forum post itself.

also, when i tried deleting the forum post, it still shows up in the main forum listing even though the category itself is now empty

as you can see in the forum main listing, the post still shows up and in the original category (OCD) instead of the new category (הפרעות אכילה) even though it has been deleted:
​http://cbtisrael.co.il/forums/

wordpress v3.9
buddypress v2.0
bbpress v2.5.3

Attachments (2)

2587.diff​ (842 bytes) - added by netweb 10 years ago.
2587.2.diff​ (753 bytes) - added by netweb 8 years ago.

Download all attachments as: .zip

Change History (14)

#1 @netweb
10 years ago

Did you move the topic from the front end or by using the 'Topics' admin panel in the WP Dashboard?

@netweb
10 years ago

#2 @netweb
10 years ago

  • Component changed from Forums to Topics
  • Keywords has-patch added
  • Milestone changed from Awaiting Review to 2.6
  • Summary changed from possible bug? moving forum post to different forum or deleting doesn't update main listing to Moving a topic in the backend the forum meta is not updated

In 2587.diff​ - If the new forum id is not equal to the old forum id run bbp_move_topic_handler on update

This ticket was mentioned in IRC in #bbpress-dev by netweb. ​View the logs.
10 years ago

#4 @netweb
10 years ago

  • Owner set to netweb

This ticket was mentioned in ​Slack in #bbpress by netweb. ​View the logs.
9 years ago

#6 @johnjamesjacoby
9 years ago

  • Owner changed from netweb to johnjamesjacoby

#7 @johnjamesjacoby
9 years ago

In 5930:

Topics: sanity check $_REQUEST and introduce list-table methods for reply table-navigation.

This changeset fixes a potential debug notice introduced in r5886, and removes unused functionality inherited from WP_List_Table.

Fixes #2842. See #2587.

@netweb
8 years ago

#8 @netweb
8 years ago

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

In 6098:

Topics: Call bbp_move_topic_handler() when moving a topic to a new forum in the back end.

This changeset ensures that both the source and destination forums meta is updated when a topic is moved.

Fixes #2587.

#9 @netweb
8 years ago

  • Keywords needs-patch added; has-patch removed
  • Resolution fixed deleted
  • Status changed from closed to reopened

#10 @netweb
8 years ago

In 6099:

Topics: Revert [6098]

This doesn't work, on many levels, when testing a patch make sure the patch your testing matches the same site you're testing with.

Antiprops netweb.
See #2587.

#11 @johnjamesjacoby
8 years ago

  • Milestone changed from 2.6 to 2.7

I'm unable to duplicate this, but it's plausible, so I'm moving to 2.7.

#12 @johnjamesjacoby
7 years ago

In 6546:

Replies: Add single_row() method to replies list table when editing a topic.

This lets us use get_post_class() on the table rows, so statuses like pending or spam can be properly in the list for users who have the capability to see them.

See #2587.

Note: See TracTickets for help on using tickets.