Make WordPress Core

Opened 20 months ago

Closed 17 months ago

Last modified 17 months ago

#56868 closed enhancement (fixed)

Add test coverage for _truncate_post_slug

Reported by: xknown's profile xknown Owned by: hellofromtonya's profile hellofromTonya
Milestone: 6.2 Priority: normal
Severity: normal Version:
Component: Posts, Post Types Keywords: has-patch has-unit-tests commit
Focuses: Cc:

Description

[52457] should have some unit tests for _truncate_post_slug() to confirm that post slugs are properly truncated when they contain URL encoded characters.

Change History (10)

This ticket was mentioned in PR #3505 on WordPress/wordpress-develop by xknown.


20 months ago
#1

  • Keywords has-patch has-unit-tests added

#2 @SergeyBiryukov
20 months ago

  • Milestone changed from Awaiting Review to 6.2

#3 @mukesh27
20 months ago

  • Keywords changes-requested added

@xknown left some review on PR.

This ticket was mentioned in Slack in #core by costdev. View the logs.


17 months ago

#5 @hellofromTonya
17 months ago

  • Owner set to hellofromTonya
  • Status changed from new to reviewing

Self-assigning ownership for code review and commit.

This ticket was mentioned in Slack in #core by mukeshpanchal27. View the logs.


17 months ago

#7 @mukesh27
17 months ago

  • Keywords changes-requested removed

#8 @mukesh27
17 months ago

  • Keywords commit added

This ticket was discussed in the recent bug scrub.

The PR got enough approval. Adding commit keyword for 6.2 consideration.

Props to @costdev

#9 @peterwilsoncc
17 months ago

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

In 55208:

Posts, Post Types: Add test coverage for _truncate_post_slug().

Props xknown, mukesh27, costdev, audrasjb.
Fixes #56868.
See #56793.

Note: See TracTickets for help on using tickets.