Make WordPress Core

Opened 9 years ago

Closed 9 years ago

Last modified 9 years ago

#16720 closed defect (bug) (fixed)

Network Admin - Site Search breaks Row Action Links

Reported by: Ipstenu Owned by:
Milestone: 3.1.1 Priority: normal
Severity: major Version:
Component: Multisite Keywords: has-patch commit
Focuses: Cc:




If you go to wp-admin/network/sites.php and try to mark a site as spam or anything on the hover-over links, it's fine.

BUT if you SEARCH for a site, it CHANGES those row action links from




Which blows up in your face.

Worse is if you use a wildcard, then you get SEARCHVALUE% as the URL.

Here's the best part. It only happens on subFOLDER multisite. Works perfectly fine on subdomain.

I was able to reproduce it on two separate subfolder installs, one with no plugins at all.

Attachments (1)

16720.diff (644 bytes) - added by greuben 9 years ago.

Download all attachments as: .zip

Change History (14)

#1 @Ipstenu
9 years ago

Forum post wherein we sorted this out: http://wordpress.org/support/topic/network-admin-user-and-site-admin-links-dont-work-31

(I can't repro the user issues he had, but I can the blog search)

#2 @firmdot
9 years ago

I'm having same/similar issue.

I go to Network Admin, Search sites. Search for a site with a wildcard i.e. photo* the results show up just fine, HOWEVER after clicking on any of the site options for the found site (i.e. Edit, Spam, Archive, Delete) the URL changes to:


#3 @nacin
9 years ago

  • Milestone changed from Awaiting Review to 3.1.1

#4 @nerx
9 years ago

  • Cc nerx added
  • Keywords needs-patch added
  • Severity changed from normal to major

Tested with Subdomain from [1und1.de] Result: Same as subFOLDER!

9 years ago

#5 @greuben
9 years ago

  • Keywords has-patch added; needs-patch removed

see patch!

#6 @Ipstenu
9 years ago

Patch works for me on subfolders (cannot repro on subdomains like nerx can, but it doesn't BREAK anythign I can see).

#7 @greuben
9 years ago

That part of code is only executed for directory installs so I am sure subdomain installs will not have this problem.

#8 @nacin
9 years ago

  • Keywords commit added

Typo introduced in [16161].

#9 @nacin
9 years ago

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

(In [17496]) Avoid mangling the current site path variable with the search query. props greuben, fixes #16720 for trunk.

#10 @nacin
9 years ago

(In [17497]) Avoid mangling the current site path variable with the search query. props greuben, fixes #16720 for the 3.1 branch.

#11 follow-up: @tmoorewp
9 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

Edit: Nevermind, I think I read Trac wrong. This patch will be out with 3.1.1?

This patch didn't make it into the 3.1 download on wordpress.org. I've got two MultiSite installations running upgraded 3.1 installations and this bug is still present. When I upgraded, I manually deleted the entire wp-admin folder and replaced it with the new copy from the wordpress.org download. If I apply this patch, they work fine.

Last edited 9 years ago by tmoorewp (previous) (diff)

#12 @tmoorewp
9 years ago

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

#13 in reply to: ↑ 11 @ocean90
9 years ago

Replying to tmoorewp:

This patch will be out with 3.1.1?

Yes. :)

Note: See TracTickets for help on using tickets.