Opened 2 years ago
Closed 20 months ago
#56964 closed defect (bug) (fixed)
Automatic update emails have HTML entities in plugin/theme names
Reported by: | paulschreiber | Owned by: | audrasjb |
---|---|---|---|
Milestone: | 6.2 | Priority: | normal |
Severity: | normal | Version: | 6.0.3 |
Component: | Keywords: | has-patch has-screenshots | |
Focuses: | administration | Cc: |
Description
Automatic update emails have HTML entities in plugin/theme names.
Example:
These plugins are now up to date: - Health Check & Troubleshooting (from version 1.5.0 to 1.5.1) - WP Super Cache (from version 1.9 to 1.9.1)
Attachments (3)
Change History (14)
#1
@
2 years ago
- Milestone changed from Awaiting Review to 6.2
Thanks for spotting this!
Moving for 6.1.1 consideration.
If we have a patch quickly, I think we could even move it to the next minor.
#2
@
2 years ago
Hello there !
Maybe a simple html_entity_decode can do the job ?
If it's the right way to fix it, I can do it this PM.
Have a great day !
#3
@
21 months ago
- Keywords needs-patch added
Hey @benjgrolleau, sorry for the delay!
html_entity_decode
may do the job, let's put together a patch and test it :)
@
21 months ago
Added html_entity_decode()
to all required places where plugin/theme name is used for email.
#5
@
20 months ago
In wordpress-develop/src/wp-admin/includes/class-wp-automatic-updater.php
Found few functions containing mail feature:
- send_debug_email
- send_plugin_theme_email
If we consider send_plugin_theme_email only for this ticket then:
✅ all occurrences (plugin and theme name) are addressed properly with html_entity_decode
.
Note: I couldn't test the email.
This ticket was mentioned in Slack in #core-test by robinwpdeveloper. View the logs.
20 months ago
#7
@
20 months ago
I have searched with Plugin name and found 5 occurrences. 4 of them are in the send_plugin_theme_email method.
I have searched with Theme name and found 5 occurrences. 4 of them are in the send_plugin_theme_email method.
✅ After applying the patch all 4 occurrences are addressed.
I couldn’t test the auto-generated email.
#8
@
20 months ago
Thanks for the testing reports, but we still need to ensure the actual email is ok.
update email screen shot