Opened 5 years ago
Closed 5 years ago
#6544 closed defect (bug) (fixed)
TinyMCE HTTPS handling
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Priority: | normal | Milestone: | 2.5.1 |
| Component: | TinyMCE | Version: | 2.5 |
| Severity: | normal | Keywords: | tinymce, https has-patch |
| Cc: |
Description
In /wp-includes/js/tiny_mce/tiny_mce_config.php, line 52 should read:
if ( $https ) $baseurl = str_replace('http://', 'https://', $baseurl);
and line 56 should read:
if ( $https ) $mce_css = str_replace('http://', 'https://', $mce_css);
At present (2.5) the replaced string is not being assigned, so when using HTTPS (with, for example, the admin-ssl plugin) the TinyMCE CSS files are being called using HTTP.
Attachments (2)
Change History (10)
- Keywords has-patch added
- Milestone set to 2.6
- Severity changed from blocker to normal
- Resolution set to fixed
- Status changed from new to closed
- Milestone changed from 2.5.1 to 2.6
It was fixed in trunk.
Maybe backport it to 2.5.1?
comment:7
craigandrews — 5 years ago
- Milestone changed from 2.6 to 2.5.1
- Resolution fixed deleted
- Status changed from closed to reopened
Yes - please backport to 2.5.1. My company requires the use of HTTPS for sites requiring login, so this a maintenance hassle for me as the admin :-/
(not sure if I'm breaking protocol by changing the milestone and re-opening - if so, my apologies!)
Thanks!
Patch attached, based on the given code snippets.
Also a candidate for 2.5.1.