WordPress.org
Get WordPress

Make WordPress Core

Opened 4 years ago

Closed 4 years ago

Last modified 4 years ago

#56868 closed enhancement (fixed)

Add test coverage for _truncate_post_slug

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

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.
4 years ago #1

  • Keywords has-patch has-unit-tests added

Trac ticket: https://core.trac.wordpress.org/ticket/56868

#2 @SergeyBiryukov
4 years ago

  • Milestone Awaiting Review → 6.2

#3 @mukesh27
4 years ago

  • Keywords changes-requested added

@xknown left some review on PR.

This ticket was mentioned in ​Slack in #core by costdev. ​View the logs.
4 years ago

#5 @hellofromTonya
4 years ago

  • Owner set to hellofromTonya
  • Status new → reviewing

Self-assigning ownership for code review and commit.

This ticket was mentioned in ​Slack in #core by mukeshpanchal27. ​View the logs.
4 years ago

#7 @mukesh27
4 years ago

  • Keywords changes-requested removed

#8 @mukesh27
4 years 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
4 years ago

  • Resolution → fixed
  • Status reviewing → 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.