WordPress.org

Make WordPress Core

Opened 7 years ago

Closed 5 years ago

Last modified 5 years ago

#16943 closed enhancement (maybelater)

Code Cleanup add_query_arg() + friends

Reported by: hakre Owned by:
Milestone: Priority: normal
Severity: normal Version: 3.1
Component: General Keywords: has-patch needs-refresh needs-unit-tests
Focuses: Cc:

Description

This should reduce the complexity of add_query_arg() a bit.

This is partially related to PHP 5 stuff (#16918,#16920) we can make use of now.

Originated in #16932

Related: #16942

Attachments (1)

16943.patch (5.8 KB) - added by hakre 7 years ago.
add_query_arg() + friends

Download all attachments as: .zip

Change History (8)

@hakre
7 years ago

add_query_arg() + friends

#1 @hakre
7 years ago

Tested and Benchmarked. Looks good from my point of view for a first patch.

#2 @dd32
7 years ago

Could we get some unit tests for those changes?

#3 @dd32
7 years ago

  • Keywords needs-unit-tests added

#4 @hakre
7 years ago

You can find some functional tests here: https://gist.github.com/882734

#5 @c3mdigital
5 years ago

  • Keywords close added; has-patch needs-unit-tests removed
  • Resolution set to maybelater
  • Status changed from new to closed

#6 @ocean90
5 years ago

  • Keywords needs-refresh added; close removed
  • Milestone Awaiting Review deleted

#7 @SergeyBiryukov
5 years ago

  • Keywords has-patch needs-unit-tests added
Note: See TracTickets for help on using tickets.