Skip to:
Content

Opened 5 years ago

Closed 4 years ago

#2624 closed defect (fixed)

Split topics function updates including reply position and reply title

Reported by: netweb Owned by: johnjamesjacoby
Milestone: 2.6 Priority: normal
Severity: normal Version: trunk
Component: Component - Topics Keywords: needs-testing has-patch
Cc:

Description

bbp_split_topic_handler() needs updating similarly to the recent changes in bbp_merge_topic_handler()

  • Replies should not have a post title or post name
  • Reply menu_order needs to be recalculated after splitting

Related: #2528

Attachments (1)

2624.1.diff (1023 bytes) - added by netweb 5 years ago.

Download all attachments as: .zip

Change History (6)

@netweb
5 years ago

#1 @netweb
5 years ago

  • Keywords needs-testing added

#2 @netweb
5 years ago

Related: #2174

#3 @netweb
5 years ago

  • Keywords has-patch added; needs-patch removed

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


4 years ago

#5 @johnjamesjacoby
4 years ago

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

In 5491:

In bbp_split_topic_handler() remove $reply_position cheater code, and rely on bbp_update_reply_position() to recalculate the correct position in the hierarchy after wp_update_post() has finished. Props netweb. Fixes #2624.

Note: See TracTickets for help on using tickets.