Ticket #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
-
6544.diff
(898 bytes) -
added by Nazgul 4 years ago.
-
6544.2.diff
(1.0 KB) -
added by azaozz 4 years ago.
- Includes the above patch plus one more place that needed fixing.
Change History
- Keywords has-patch added
- Severity changed from blocker to normal
- Milestone set to 2.6
Patch attached, based on the given code snippets.
Also a candidate for 2.5.1.
-
attachment
6544.2.diff
added
Includes the above patch plus one more place that needed fixing.
- Status changed from new to closed
- Resolution set to fixed
- Milestone changed from 2.5.1 to 2.6
It was fixed in trunk.
Maybe backport it to 2.5.1?
comment:7
craigandrews — 4 years ago
- Status changed from closed to reopened
- Resolution fixed deleted
- Milestone changed from 2.6 to 2.5.1
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!
