WordPress.org

Make WordPress Core

Opened 19 months ago

Closed 4 days ago

Last modified 4 days ago

#45398 closed defect (bug) (fixed)

Remove tinymce_version from _WP_Editors::editor_settings()

Reported by: ishitaka Owned by: whyisjake
Milestone: 5.5 Priority: normal
Severity: normal Version:
Component: Editor Keywords: has-patch good-first-bug
Focuses: Cc:

Description

$tinymce_version is not used but it is mentioned in the _WP_Editors::editor_settings() function so needs to remove that global variable declaration.

Attachments (2)

45398.patch (611 bytes) - added by ishitaka 19 months ago.
45398.2.diff (703 bytes) - added by Mista-Flo 5 weeks ago.
Fixed patch

Download all attachments as: .zip

Change History (9)

@ishitaka
19 months ago

#1 @ishitaka
19 months ago

  • Keywords has-patch added

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


2 months ago

#3 @noisysocks
2 months ago

  • Keywords needs-refresh good-first-bug added
  • Milestone changed from Awaiting Review to Future Release

Thanks @ishitaka. Looks good, but the patch needs to be updated as it no longer applies cleanly.

#4 @SergeyBiryukov
2 months ago

  • Milestone changed from Future Release to 5.5

@Mista-Flo
5 weeks ago

Fixed patch

#5 @Mista-Flo
5 weeks ago

  • Keywords needs-refresh removed

I have uploaded a new patch.

#6 @whyisjake
4 days ago

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

In 47860:

Editor: Remove TinyMCE global version.

The global variable is not being used in the editor_settings method.

Fixes #45398.
Props ishitaka, noisysocks, mista-flo.

#7 @SergeyBiryukov
4 days ago

In 47861:

Docs: Remove unused @global declarations from _WP_Editors::print_tinymce_scripts().

Follow-up to [47860].

See #45398.

Note: See TracTickets for help on using tickets.