Opened 7 months ago
Closed 6 months ago
#61578 closed defect (bug) (fixed)
Issue with search functionality in installed themes section
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 6.6.2 | Priority: | normal |
Severity: | normal | Version: | 6.6 |
Component: | Themes | Keywords: | has-patch has-testing-info commit fixed-major dev-reviewed |
Focuses: | accessibility, administration | Cc: |
Description
If we navigate to Appearance > Themes and use the search box to check if a theme is already installed, it works as expected as we are entering the search term and don't press the enter key. In this instance, it will display "No themes found. Try a different search." However, if we enter the name of a theme that is not installed and press the enter key, the page will refresh and show the list of installed themes instead of displaying "No themes found. Try a different search.".
Environment
- WordPress: 6.6-RC2
- PHP: 8.2.0
- Server: Apache/2.4.54 (Unix) OpenSSL/1.0.2u PHP/8.2.0 mod_wsgi/3.5 Python/2.7.18 mod_fastcgi/mod_fastcgi-SNAP-0910052141 mod_perl/2.0.11 Perl/v5.30.1
- Database: mysqli (Server: 5.7.39 / Client: mysqlnd 8.2.0)
- Browser: Firefox 127.0 (macOS)
- Theme: Twenty Twenty-Four 1.1
- MU-Plugins: None activated
- Plugins:
- WordPress Beta Tester 3.5.5
Steps to Reproduce
- Navigate to Appearance > Themes.
- Enter the name of a theme that is not installed in the search box.
- Press the enter key.
- Observe that the page refreshes and shows the list of installed themes instead of displaying the "No themes found. Try a different search" message.
Expected Results
- When pressing the enter key after entering a search term for a theme that is not installed, it should display "No themes found. Try a different search."
Actual Results
- When pressing the enter key after entering a search term for a theme then the page refreshes and shows the list of installed themes instead of the "No themes found. Try a different search" message.
Attachments (6)
Change History (34)
#2
@
7 months ago
Hi,
This bug exists in Playground. However, if we download 6.6-RC2 now, it does not show that bug anymore.
Edit- Yes I can see the bug when press Enter.
This ticket was mentioned in PR #6976 on WordPress/wordpress-develop by @the ank.
7 months ago
#3
- Keywords has-patch added
This ticket was mentioned in Slack in #core-test by monikarao. View the logs.
7 months ago
#8
@
7 months ago
- Milestone changed from 6.6 to 6.6.1
Wish I'd seen this sooner; it could have made 6.6. At this point, however, I think it'll have to go into 6.6.1.
This ticket was mentioned in Slack in #core by hellofromtonya. View the logs.
7 months ago
#10
@
7 months ago
6.6.1 RC will likely happen late tomorrow, July 18th (no time is set yet). Discussed this ticket in today's scrub. As the patch isn't ready yet, will move this ticket to 6.6.2 (once that milestone is available in Trac).
#16
follow-up:
↓ 18
@
6 months ago
#61788 was closed as duplicate, although it described a different location where the bug appears (wp-admin/theme-install.php
). That's for installing new themes. But it's probably the same issue.
I've applied the changes on wp-admin/js/theme.js
(it's probably me, because I don't understand the paths in the PR, but I guess that's the one that needs the change), but that doesn't seem to work.
Via Playground I can't test this, because it's not possible to install new theme's so the search doesn't work there.
Can somebody please test the PR by going to Appearance - Themes - Install new theme and do a search there followed by pressing enter?
#17
@
6 months ago
- Keywords has-testing-info added
[58405] appears to be the changeset that introduced the bug. Noting here for context and commit follow-up history.
#18
in reply to:
↑ 16
;
follow-up:
↓ 19
@
6 months ago
Replying to josklever:
#61788 was closed as duplicate, although it described a different location where the bug appears (
wp-admin/theme-install.php
). That's for installing new themes. But it's probably the same issue.
I've applied the changes on
wp-admin/js/theme.js
(it's probably me, because I don't understand the paths in the PR, but I guess that's the one that needs the change), but that doesn't seem to work.
Via Playground I can't test this, because it's not possible to install new theme's so the search doesn't work there.
Can somebody please test the PR by going to Appearance - Themes - Install new theme and do a search there followed by pressing enter?
Thanks for alerting. I'll reopen that Trac ticket as I just tested it on 6.5 and it doesn't work there either for me. Seems unrelated to the bug introduced in 6.6.
#19
in reply to:
↑ 18
@
6 months ago
Replying to hellofromTonya:
Replying to josklever:
#61788 was closed as duplicate, although it described a different location where the bug appears (
wp-admin/theme-install.php
). That's for installing new themes. But it's probably the same issue.
I've applied the changes on
wp-admin/js/theme.js
(it's probably me, because I don't understand the paths in the PR, but I guess that's the one that needs the change), but that doesn't seem to work.
Via Playground I can't test this, because it's not possible to install new theme's so the search doesn't work there.
Can somebody please test the PR by going to Appearance - Themes - Install new theme and do a search there followed by pressing enter?
Thanks for alerting. I'll reopen that Trac ticket as I just tested it on 6.5 and it doesn't work there either for me. Seems unrelated to the bug introduced in 6.6.
Wait, I was wrong. I now understand. Entering a theme name in the Add Themes > search field should automatically trigger a search. That worked in 6.5.5, but doesn't in 6.6.0.
Applying the patch resolves the issue for me, showing the issue is related to the [58405] change. Thus, #61788 is a duplicate of this ticket but with a different set of testing steps.
I'll add a test report shortly.
#20
@
6 months ago
Test Report for Appearance > Themes > Add Themes
Patch tested: https://github.com/WordPress/wordpress-develop/pull/6976
Steps to Reproduce or Test
- Navigate to Appearance > Themes > Add Theme.
- In the search box, enter a valid theme's name or part of a name, e.g. "News".
- Expected: Page should refresh with a list of themes with "News" in their name.
- In the search box, press return or enter key. 🐞 Bug occurs.
- Expected: Nothing should happen.
- In the search box, enter an invalid theme name, e.g. "sometheme".
- Expected: Page should refresh with no themes listed and a "No themes found. Try a different search." message in the search results.
- In the search box, press return or enter key. 🐞 Bug occurs.
- Expected: Nothing should happen.
Expected Results
When testing if the patch resolves the issue:
- Step 2. In the search box, enter a valid theme's name or part of a name, e.g. "News".
- ✅ Expected: Page should refresh with a list of themes with "News" in their name.
- Step 3. In the search box, press return or enter key.
- ✅ Expected: Nothing should happen.
- Step 4: In the search box, enter an invalid theme name, e.g. "sometheme".
- ✅ Expected: Page should refresh with no themes listed and a "No themes found. Try a different search." message in the search results.
- Step 5. In the search box, press return or enter key.
- ✅ Expected: Nothing should happen.
When reproducing a bug:
- Step 3. In the search box, press return or enter key. 🐞 Bug occurs.
- ❌ page refreshes to
/wp-admin/theme-install.php?browse=popular
, search box is cleared, and a list of the popular themes is shown in the search results.
- ❌ page refreshes to
- Step 5. In the search box, press return or enter key. 🐞 Bug occurs.
- ❌ page refreshes to
/wp-admin/theme-install.php?browse=popular
, search box is cleared, and a list of the popular themes is shown in the search results.
- ❌ page refreshes to
Environment
- OS: macOS
- Web Server: Nginx (using Local for 6.5.5 and 6.6.1 testing) and Docker (for trunk with patch testing)
- PHP: 8.3.8
- WordPress: 6.5.5, 6.6.1, and trunk
- Browser: Firefox 129.0.1
- Theme: Twenty Twenty-Four
- Active Plugins: None
Actual Results
When reproducing a bug/defect:
Can reproduce the issues of pressing enter/return key as shown in AddThemeSearch-6.6.1.gif:
- Step 3. In the search box, press return or enter key. 🐞 Bug occurs.
- ❌ page refreshes to
/wp-admin/theme-install.php?browse=popular
, search box is cleared, and a list of the popular themes is shown in the search results.
- ❌ page refreshes to
- Step 5. In the search box, press return or enter key. 🐞 Bug occurs.
- ❌ page refreshes to
/wp-admin/theme-install.php?browse=popular
, search box is cleared, and a list of the popular themes is shown in the search results.
- ❌ page refreshes to
When testing the bugfix patch:
AddThemeSearch-withPR6976.gif shows the results of Steps 4 and 5.
- Step 2. In the search box, enter a valid theme's name or part of a name, e.g. "News".
- ✅ Page refreshes and displays a list of themes with "News" in their name.
- Step 3. In the search box, press return or enter key.
- ✅ Nothing happens.
- Step 4: In the search box, enter an invalid theme name, e.g. "sometheme".
- ✅ Page refreshes with no themes listed and a "No themes found. Try a different search." message in the search results.
- Step 5. In the search box, press return or enter key.
- ✅ Nothing happens.
Summary
I can reproduce the issues and with the patch, the issues are resolved.
@
6 months ago
Appearance > Themes > Search on WP trunk with PR 6976 applied : shows bugs are resolved and works as expected ✅
#21
@
6 months ago
Test Report for Appearance > Themes
Patch tested: https://github.com/WordPress/wordpress-develop/pull/6976
Steps to Reproduce or Test
- Navigate to Appearance > Themes.
- In the search box, enter a valid theme's name or part of a name, e.g. "Three".
- Expected: Page should refresh with a list of themes with "Three" in their name.
- In the search box, press return or enter key. 🐞 Bug occurs.
- Expected: Nothing should happen.
- In the search box, enter an invalid theme name, e.g. "sometheme".
- Expected: Page should refresh with no themes listed and a "No themes found. Try a different search." message in the search results.
- In the search box, press return or enter key. 🐞 Bug occurs.
- Expected: Nothing should happen.
Expected Results
ThemesSearch-6.5.5.gif shows what used to happen before 6.6.0.
When testing if the patch resolves the issue:
- Step 2. In the search box, enter a valid theme's name or part of a name, e.g. "Three".
- ✅ Expected: Page should refresh with a list of installed themes with "Three" in their name.
- Step 3. In the search box, press return or enter key.
- ✅ Expected: Nothing should happen.
- Step 4: In the search box, enter an invalid theme name, e.g. "sometheme".
- ✅ Expected: Page should refresh with no themes listed and a "No themes found. Try a different search." message in the search results.
- Step 5. In the search box, press return or enter key.
- ✅ Expected: Nothing should happen.
When reproducing a bug:
- Step 3. In the search box, press return or enter key. 🐞 Bug occurs.
- ❌ page refreshes to
/wp-admin/themes.php
, search box is cleared, and a list of installed themes is shown.
- ❌ page refreshes to
- Step 5. In the search box, press return or enter key. 🐞 Bug occurs.
- ❌ page refreshes to
/wp-admin/themes.php
, search box is cleared, and a list of installed themes is shown.
- ❌ page refreshes to
Environment
- OS: macOS
- Web Server: Nginx (using Local for 6.5.5 and 6.6.1 testing) and Docker (for trunk with patch testing)
- PHP: 8.3.8
- WordPress: 6.5.5, 6.6.1, and trunk
- Browser: Firefox 129.0.1
- Theme: Twenty Twenty-Four
- Active Plugins: None
Actual Results
When reproducing a bug/defect:
Can reproduce the issues of pressing enter/return key as shown in ThemesSearch-6.6.1.gif:
- Step 3. In the search box, press return or enter key. 🐞 Bug occurs.
- ❌ page refreshed to
/wp-admin/themes.php
, search box cleared, and the installed themes are shown.
- ❌ page refreshed to
- Step 5. In the search box, press return or enter key. 🐞 Bug occurs.
- ❌ page refreshed to
/wp-admin/themes.php
, search box cleared, and the installed themes are shown.
- ❌ page refreshed to
When testing the bugfix patch:
ThemesSearch-withPatch.gif shows the results of Steps 4 and 5.
- Step 2. Entered "Twenty" into the search box.
- ✅ Page refreshed to
/wp-admin/themes.php?search=Twenty
and all themes with "Twenty" in their name were shown.
- ✅ Page refreshed to
- Step 3. Clicked the search box field. Pressed return/enter key.
- ✅ Nothing happened.
- Step 4: Entered "SomeTheme" into the search box.
- ✅ Page refreshed to
/wp-admin/themes.php?search=SomeTheme
, no themes were listed, and a "No themes found. Try a different search." message was shown.
- ✅ Page refreshed to
- Step 5. Clicked the search box field. Pressed return/enter key.
- ✅ Nothing happened.
Summary
I can reproduce the issues and with the patch, the issues are resolved.
#22
@
6 months ago
- Keywords commit added
Patch: https://github.com/WordPress/wordpress-develop/pull/6976
With multiple test reports confirming the patch resolves the issue and PR approval, marking the patch for commit
.
@joedolson do you have any concerns with patch?
#25
@
6 months ago
- Keywords dev-feedback added
- Resolution fixed deleted
- Status changed from closed to reopened
Re-opening and marking dev-feedback for backport to 6.6.2.
@hellofromTonya commented on PR #6976:
6 months ago
#26
Committed via https://core.trac.wordpress.org/changeset/58914.
#27
@
6 months ago
- Keywords fixed-major dev-reviewed added; dev-feedback removed
Tested [58914] on 6.6 branch for both Appearance > Themes > Add Themes and comment:21 Appearance > Themes]. Results are same as both test reports ✅, i.e. resolving the issues.
LGTM. I'll backport it the 6.6 branch.
Attaching video for reference.
https://www.loom.com/share/0c124a609b2c4e1985e5379691819579?sid=89894226-ffde-4f92-88b5-814216524ff8