Make WordPress Core

Opened 5 years ago

Last modified 2 months ago

#51926 new defect (bug)

Network Site Search should search more than just wp_blogs.path

Reported by: madtownlems's profile MadtownLems Owned by:
Milestone: 7.0 Priority: normal
Severity: normal Version:
Component: Networks and Sites Keywords: has-patch has-unit-tests
Focuses: multisite Cc:

Description

From /wp-admin/network/sites.php, searching sites for 'foo' looks to only search the wp_blogs.path column. This was fine up until WordPress let you put sites any any domain.

Now searching for 'foo' would show me sites such as this:
https://mysite.com/foo
https://mysite.com/foodsite

but does NOT show me sites such as
https://foo.com
https://foo.mysite.com

I believe the search should include the domain for the site as well as the path.

Change History (8)

#2 @peterwilsoncc
4 years ago

  • Milestone changed from Awaiting Review to 6.1

I've linked a draft pull request to this ticket.

I'll put this on the WP 6.1 milestone although I think it will need a little more work and some unit tests.

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


3 years ago

#4 @audrasjb
3 years ago

  • Keywords needs-unit-tests added
  • Milestone changed from 6.1 to Future Release

Unfortunately, this ticket didn’t get more momentum in 6.1 than in 6.0… sorry about that, but the proposed patch still needs testing, so let's move it to Future Release milestone, at least until it's properly reviewed and tested.

Feel free to move it to 6.2 when it gets the needed unit tests and review :)

This ticket was mentioned in Slack in #core-multisite by realloc. View the logs.


8 months ago

This ticket was mentioned in Slack in #core-multisite by arunas. View the logs.


8 months ago

This ticket was mentioned in PR #8922 on WordPress/wordpress-develop by @Soean.


8 months ago
#7

  • Keywords has-unit-tests added; needs-unit-tests removed

#8 @SergeyBiryukov
2 months ago

  • Milestone changed from Future Release to 7.0
Note: See TracTickets for help on using tickets.