Make WordPress Core

Opened 6 years ago

Closed 5 years ago

Last modified 4 years ago

#47437 closed defect (bug) (fixed)

Update tinymce 4.x

Reported by: pross's profile pross Owned by: azaozz's profile azaozz
Milestone: 5.3 Priority: normal
Severity: normal Version:
Component: TinyMCE Keywords: has-patch
Focuses: Cc:

Description

Couple of days ago tinymce finally fixed the firefox Error: Permission denied to access property "nodeType": bug.

See:
https://github.com/WordPress/gutenberg/issues/6703
https://github.com/WordPress/gutenberg/issues/4744

Tinymce bug:
https://github.com/tinymce/tinymce/issues/2194

The fix was in the 5.x version of tinymce but I requested that it be backported to 4.x so as soon as they build a release it can be updated in WordPress.

Attachments (1)

47437.patch (735.9 KB) - added by pross 6 years ago.

Download all attachments as: .zip

Change History (8)

#1 @SergeyBiryukov
6 years ago

  • Component changed from Editor to TinyMCE
  • Milestone changed from Awaiting Review to 5.3

@pross
6 years ago

#3 @Presskopp
5 years ago

  • Keywords has-patch added

#4 follow-up: @azaozz
5 years ago

@pross thanks for the patch but it has to include the default TinyMCE plugins used in WP and the theme. Generally not worth it making a patch as it is mostly copy/paste of the new files, and the patch gets far too big :)

#5 @azaozz
5 years ago

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

In 46173:

TinyMCE: update to version 4.9.6. Changelog: https://github.com/tinymce/tinymce/blob/4.x/changelog.txt. Includes fix for the Permission denied to access property "nodeType" error.

Fixes #47437.

#6 in reply to: ↑ 4 @pross
5 years ago

Replying to azaozz:

@pross thanks for the patch but it has to include the default TinyMCE plugins used in WP and the theme. Generally not worth it making a patch as it is mostly copy/paste of the new files, and the patch gets far too big :)

Thanks for including the fix.

Note: See TracTickets for help on using tickets.