Skip to:
Content

Opened 4 years ago

Closed 4 years ago

Last modified 4 years ago

#2843 closed defect (fixed)

Include replies menu order to `bbp_insert_reply()` improving reply unit tests

Reported by: netweb Owned by: netweb
Milestone: 2.6 Priority: normal
Severity: normal Version: 2.1
Component: Component - Replies Keywords: commit
Cc:

Description

We need to include the replies menu_order when creating replies with bbp_insert_reply() for accurate unit testing

Change History (3)

#1 @netweb
4 years ago

In 5889:

Replies: Include replies menu order when creating replies with bbp_insert_reply() improving reply unit tests

See #2843

#2 @netweb
4 years ago

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

In 5890:

Tests: Update tests to always include a parent topic for replies

This changeset ensures replies always have a parent topic which in turn ensures each reply is created with an accurate menu_order position.

Fixes #2843

#3 @netweb
4 years ago

In 5891:

Replies: Remove previous menu_order default from bbp_insert_reply() missed in r5889

Antiprops netweb. See #2843.

Note: See TracTickets for help on using tickets.