WordPress.org

Make WordPress Core

Opened 15 years ago

Closed 15 years ago

Last modified 14 years ago

#1844 closed defect (bug) (fixed)

Missing "ob_start" in "tiny_mce_gzip.php"

Reported by: IoZ Owned by: ryan
Milestone: Priority: normal
Severity: normal Version: 1.6
Component: Administration Keywords: tinymce bg|has-patch
Focuses: Cc:

Description

In the latest TinyMCE version (2.0RC4), it only gzip the contents (ob_start("ob_gzhandler");) if clients and server support it, but it doesn't start normal cache (ob_start();) in ther other case.

Attachments (1)

tiny_mce_gzip_ob_start_bug.diff (703 bytes) - added by IoZ 15 years ago.
Fix missing "ob_start"

Download all attachments as: .zip

Change History (6)

@IoZ
15 years ago

Fix missing "ob_start"

#1 @IoZ
15 years ago

  • Keywords bg|has-patch added

#2 @skeltoac
15 years ago

  • Milestone set to 1.6
  • Owner changed from anonymous to ryan

Sorry about this... on non-gz browsers, the script outputs "Warning: cannot modify header informaion..." in the middle of the js because I accidentally left a header() call in the wp_translate_tinymce_lang() function. If you delete that line, everything works normally.

I see no reason to apply the ob_start() diff after this is fixed.

#4 @skeltoac
15 years ago

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

(In [3003]) Resolved by removing extra header() call.

#5 @(none)
14 years ago

  • Milestone 2.0 deleted

Milestone 2.0 deleted

Note: See TracTickets for help on using tickets.