WordPress.org

Make WordPress Core

Opened 6 years ago

Closed 6 years ago

#6759 closed defect (bug) (fixed)

tinymce pop up functions fail under ssl

Reported by: kirblam Owned by:
Milestone: 2.6 Priority: normal
Severity: normal Version: 2.5
Component: TinyMCE Keywords:
Focuses: Cc:

Description

Using Wordpress 2.5 on a LAMP setup, applied changesets up to 7680 to see whether that helped, but still failed.

Tinymce will show up in WP 2.5 however, clicking the Image button, Charmap - anything that requires a popup - fails under ssl. From what I can tell, the baseURL isn't set properly in editor_template.js. At least, if I hard code the https address in editor_template in a particular mce function, the window renders properly.

For example:
url:'https://<server + directory>/wp-includes/js/tinymce/themes/advanced/charmap.htm'
as opposed to
url:tinymce.baseURL+'/themes/advanced/charmap.htm'

The former works, the latter produces bugs detailed below.

Screenshots should tell the whole story.

Tinymce in https environment:
http://www.kirbymixedmedia.com/images/Picture2.png

Charmap button clicked - popup blank. Firebug notes 2 errors in lower right
http://www.kirbymixedmedia.com/images/Picture3.png

Firebug's account of the problems:

  1. "uncaught exception: permission to denied get property Window.tinymce"
  2. This.params has no properties - tiny_mce_popup.js (line 47)

http://www.kirbymixedmedia.com/images/Picture4.png

If I right-click in the popup itself and say "Show only this frame", the URL isn't ssl based:
http://www.kirbymixedmedia.com/images/Picture5.png

Change History (4)

comment:1 DD326 years ago

  • Milestone set to 2.6

Has this been fixed by [8190] ?

comment:2 DD326 years ago

Has this been fixed by [8190] ?

Aparantly not, Must've been a different changeset, or still present.

comment:3 ryan6 years ago

This should already be fixed. It works for me. But, I'll update tiny_mce_config to the latest SSL-aware API for good measure.

comment:4 ryan6 years ago

  • Resolution set to fixed
  • Status changed from new to closed

(In [8193]) SSL touchups for tiny_mce_config. fixes #6759

Note: See TracTickets for help on using tickets.