#57174 closed defect (bug) (fixed)
space showing as as URL encoded character after searching for plugin
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 6.2 | Priority: | normal |
Severity: | normal | Version: | 6.1 |
Component: | Plugins | Keywords: | has-patch has-screenshots |
Focuses: | javascript, administration | Cc: |
Description
Screenshot here https://ibb.co/GnVj1kW
Attachments (6)
Change History (19)
#1
@
2 years ago
- Component changed from Query to Plugins
- Focuses administration added
- Keywords has-patch added; needs-patch removed
@254volkan Thank you for the ticket.
I Have reviewed this issue and fixed it with the attached patch.
It seems the issue created by the previous ticket: https://core.trac.wordpress.org/ticket/56339
Thanks
#2
@
2 years ago
- Owner set to audrasjb
- Status changed from new to reviewing
- Version changed from 6.1.1 to 6.1
Introduced in [53844].
#3
@
2 years ago
- Focuses javascript added; accessibility removed
- Keywords changes-requested added
- Severity changed from trivial to normal
Hello @254volkan, welcome to WordPress Core Trac, and thanks for the ticket!
@dilipbheda thanks for the patch too, but this would revert [53844] which fixed an issue with some characters (see the related ticket). We need an implementation that doesn't revert that fix :)
#5
@
2 years ago
To clarify @254volkan I moved the Version
information from 6.1.1 to 6.1 because this metadata is about the version that introduced the issue, not the version you reproduced the issue ;-)
#7
@
2 years ago
The above patch will show the url decoded string on plugin install page.
Also this #56339 issue is not coming back after applying the above patch.
@
2 years ago
Also the pagination with special characters working fine as expected. #56339 issue is not getting reverted after applying the above patch.
#8
@
2 years ago
- Keywords has-screenshots added; changes-requested removed
This new patch looks good to me (see above screenshot). I'll add a PR just to make sure tests are passing well, then commit
. Thanks.
This ticket was mentioned in PR #3705 on WordPress/wordpress-develop by @audrasjb.
2 years ago
#10
@audrasjb commented on PR #3705:
2 years ago
#12
Committed in https://core.trac.wordpress.org/changeset/54904
Here is the patch to fix this issue.