WordPress.org

Make WordPress Core

Opened 11 years ago

Closed 11 years ago

Last modified 11 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 11 years ago.
tiny_mce_config.php.diff (512 bytes) - added by filosofo 11 years ago.

Download all attachments as: .zip

Change History (14)

#1 @rob1n
11 years ago

You're missing an echo.

#2 @rob1n
11 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
11 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
11 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
11 years ago

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

#6 @Viper007Bond
11 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
11 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
11 years ago

  • Milestone changed from 2.1.4 to 2.2

Well, that's retarded ;). Fixing.

#9 @rob1n
11 years ago

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

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

#10 @rob1n
11 years ago

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

#11 @Nazgul
11 years ago

  • Milestone changed from 2.2 to 2.1.4

#12 @anonymous
11 years ago

  • Milestone 2.1.4 deleted

Milestone 2.1.4 deleted

Note: See TracTickets for help on using tickets.