WordPress.org

Make WordPress Core

Opened 9 years ago

Closed 9 years ago

Last modified 9 years ago

#4118 closed defect (bug) (fixed)

tinymce language pack loading fails if no locale is defined

Reported by: m0n5t3r Owned by:
Milestone: Priority: high
Severity: major Version: 2.1
Component: TinyMCE Keywords: has-patch
Focuses: Cc:

Description

If there is no locale defined, the TinyMCE language setting is empty, which leads to incorrectly generated paths for plugin language packs. The attached patch sets it to "en", which is the TinyMCE default value.

I'm marking this as major because it affects to various degrees all TinyMCE plugins that use language packs (between localization not working and meltdown - in my case it disabled the post publishing i.e. meltdown).

Attachments (2)

mce_locale_fix.diff (876 bytes) - added by m0n5t3r 9 years ago.
tiny_mce_config.php.diff (512 bytes) - added by filosofo 9 years ago.

Download all attachments as: .zip

Change History (14)

#1 @rob1n
9 years ago

You're missing an echo.

#2 @rob1n
9 years ago

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

(In [5219]) Default to 'en' for TinyMCE language if get_locale() is empty, to avoid meltdowns. Props m0n5t3r. fixes #4118 for 2.1

#3 @rob1n
9 years ago

  • Milestone changed from 2.1.4 to 2.2
  • Resolution fixed deleted
  • Status changed from closed to reopened

now for 2.2

#4 @rob1n
9 years ago

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

(In [5220]) Default to 'en' for TinyMCE language if get_locale() is empty, to avoid meltdowns. Props m0n5t3r. fixes #4118

#5 @foolswisdom
9 years ago

  • Milestone changed from 2.2 to 2.1.4
  • Version set to 2.1

#6 @Viper007Bond
9 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

This is throwing a PHP error on my local install:

Fatal error: Can't use function return value in write context in ...\wp-includes\js\tinymce\tiny_mce_config.php on line 52

#7 @filosofo
9 years ago

I've attached a patch that should fix that problem: I think the argument passed to empty has to be a variable.

#8 @rob1n
9 years ago

  • Milestone changed from 2.1.4 to 2.2

Well, that's retarded ;). Fixing.

#9 @rob1n
9 years ago

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

(In [5224]) empty() only takes variables. Props filosofo. fixes #4118

#10 @rob1n
9 years ago

(In [5225]) empty() only takes variables. Props filosofo. fixes #4118 for 2.1

#11 @Nazgul
9 years ago

  • Milestone changed from 2.2 to 2.1.4

#12 @anonymous
9 years ago

  • Milestone 2.1.4 deleted

Milestone 2.1.4 deleted

Note: See TracTickets for help on using tickets.