Context Navigation

← Previous Ticket
Next Ticket →

#33393 closed defect (bug) (fixed)

tinymce plugin wplink throw js TypeError: editor.wp undefined

Reported by:	hauvong	Owned by:	azaozz
Priority:	normal	Milestone:	4.3.1
Component:	TinyMCE	Version:	4.3
Severity:	normal	Keywords:	has-patch fixed-major
Cc:		Focuses:

Description

When TinyMCE initialize with plugin wplink enable but not wordpress, the wplink js throw TypeError: editor.wp undefined

To reproduce, initialize a new TinyMCE editor with wplink plugins enable.

tinyMCE.init({
	plugins: "wplink"
});
tinyMCE.execCommand( 'mceAddEditor', true, "textarea" )

To fix, the js should check for wp object exist before calling the function:

editor.on( 'preinit', function() {
	// editor may not have wp object if tinymce wordpress plugin isn't loaded.
	if ( editor.wp && editor.wp._createToolbar ) {
		toolbar = editor.wp._createToolbar( [
			'wp_link_preview',
			'wp_link_edit',
			'wp_link_remove'
		], true );
	}
} );

The patch file is attached.

Attachments (1)

tinymce-wp-link-plugin.js.patch (762 bytes ) - added by hauvong 11 years ago.: fix exception TypeError: editor.wp undefined

Download all attachments as: .zip

Change History (12)

@hauvong
11 years ago

Attachment tinymce-wp-link-plugin.js.patch added

fix exception TypeError: editor.wp undefined

#1 follow-up: ↓ 2 @helen
11 years ago

Keywords reporter-feedback added
Version trunk → 3.3

Does wplink work without the wordpress plugin?

#2 in reply to: ↑ 1 @hauvong
11 years ago

Replying to helen:

Does wplink work without the wordpress plugin?

Yes, wplink work without the wordpress plugin and there is no error prior to r32902, #32604

This ticket was mentioned in Slack in #core by lumberhack. View the logs.

11 years ago

#4 @LumberHack
11 years ago

Version 3.3 → 4.3

#5 @LumberHack
11 years ago

Tested the fix suggested by @hauvong and it works.

#6 @azaozz
11 years ago

Owner set to azaozz
Resolution → fixed
Status new → closed

In 33728:

TinyMCE: ensure the wordpress plugin is loaded before calling _createToolbar().
Props hauvong, azaozz. Fixes #33393.

#7 follow-up: ↓ 8 @azaozz
11 years ago

Milestone Awaiting Review → 4.3.1
Resolution fixed
Status closed → reopened

Reopen for 4.3.

Version 0, edited 11 years ago by azaozz (next)

#8 in reply to: ↑ 7 ; follow-up: ↓ 9 @LumberHack
11 years ago

Replying to azaozz:

Reopen for 4.3.

Any insights on why this got reopened ?

#9 in reply to: ↑ 8 @azaozz
11 years ago

Keywords has-patch added; reporter-feedback removed

Replying to LumberHack:

Disabling the wordpress plugin when using the other custom plugins for TinyMCE is a rare edge case that was fixed in trunk. It affects 4.3 so moved it to the 4.3.1 milestone for consideration.

#10 @samuelsidler
11 years ago

Keywords fixed-major added

#11 @azaozz
11 years ago

Resolution → fixed
Status reopened → closed

In 34032:

TinyMCE: ensure the wordpress plugin is loaded before calling _createToolbar().

Props hauvong, azaozz.
Fixes #33393 for 4.3.

Note: See TracTickets for help on using tickets.

Download in other formats:

Make WordPress Core

Context Navigation

#33393 closed defect (bug) (fixed)

tinymce plugin wplink throw js TypeError: editor.wp undefined

Description

Attachments (1)

Change History (12)

@hauvong 11 years ago

#1 follow-up: ↓ 2 @helen 11 years ago

#2 in reply to: ↑ 1 @hauvong 11 years ago

This ticket was mentioned in ​Slack in #core by lumberhack. ​View the logs. 11 years ago

#4 @LumberHack 11 years ago

#5 @LumberHack 11 years ago

#6 @azaozz 11 years ago

#7 follow-up: ↓ 8 @azaozz 11 years ago

#8 in reply to: ↑ 7 ; follow-up: ↓ 9 @LumberHack 11 years ago

#9 in reply to: ↑ 8 @azaozz 11 years ago

#10 @samuelsidler 11 years ago

#11 @azaozz 11 years ago