Make WordPress Core

Opened 2 months ago

Closed 2 months ago

Last modified 2 months ago

#62659 closed defect (bug) (duplicate)

Users list omitting @ symbol in search after clicking "Go to Users" link

Reported by: stromhalm's profile stromhalm Owned by:
Milestone: Priority: normal
Severity: normal Version: trunk
Component: Administration Keywords: has-patch
Focuses: administration Cc:

Description

Hi there, I came across this bug in the admin's users list. To reproduce:

  1. Create a new user with a valid email address, e.g. user@…
  2. Search for the user by that email address
  3. Click on the username in the result list
  4. Click the "Update User" button
  5. Click the "← Go to Users" link

The users.php is now back on the search result page, but the search is for "userlocalhost.com", omitting the @ symbol, therefore showing an empty list.

This behavior can be reproduced in a new install without any active plugins, using the nightly build and PHP 8.3.

Attachments (1)

62659 wordpress bug video.gif (2.8 MB) - added by stromhalm 2 months ago.
Video showing the steps to reproduce this bug

Change History (7)

@stromhalm
2 months ago

Video showing the steps to reproduce this bug

#1 @sainathpoojary
2 months ago

Reproduction Report

Description

This report validates whether the issue can be reproduced.

Environment

  • WordPress: 6.8-alpha-59274-src
  • PHP: 8.2.26
  • Server: nginx/1.27.3
  • Database: mysqli (Server: 8.0.40 / Client: mysqlnd 8.2.26)
  • Browser: Chrome 131.0.0.0
  • OS: macOS
  • Theme: Twenty Sixteen 3.4
  • MU Plugins: None activated
  • Plugins: Test Reports 1.2.0

Actual Results

✅ Error condition occurs.

Supplemental Artifacts

Video: https://utfs.io/f/PL8E4NiPUWyOhwh3Wa006Edx2tfphuaYK5oLsA9IcTvyjC8N

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


2 months ago
#2

  • Keywords has-patch added; needs-patch removed

Replaced sanitize_text_field() with sanitize_url() for the wp_http_referer variable in user-edit.php.

Trac ticket: #62659

#3 @siliconforks
2 months ago

This appears to be a duplicate of #62551?

#4 @stromhalm
2 months ago

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

Duplicate of #62551.

Thanks @sainathpoojary for providing the fix and @siliconforks for pointing to the existing ticket! I wasn't able to to find that previously.

#5 @swissspidy
2 months ago

  • Milestone Awaiting Review deleted

@SergeyBiryukov commented on PR #7964:


2 months ago
#6

Thanks for the PR! Merged in r59496.

Note: See TracTickets for help on using tickets.