Make WordPress Core

Opened 16 years ago

Closed 16 years ago

Last modified 16 years ago

#14995 closed defect (bug) (fixed)

minor typing mistake results in ignoring parameters for the pack() function call

Reported by: jakub.misek Owned by:
Priority: normal Milestone: 3.1
Component: TinyMCE Version: 3.1
Severity: trivial Keywords: has-patch
Cc: Focuses:

Description

/wp-includes/js/tinymce/plugins/spellchecker/classes/utils/JSON.php

line 535, 539, 543

wrong order of parenthesis and commas results in calling ord() function with more than 1 parameter.

Warning: ord() expects exactly 1 parameter, 2 given

Attachments (1)

14995.diff (1.5 KB ) - added by duck_ 16 years ago.

Download all attachments as: .zip

Change History (6)

#1 @duck_
16 years ago

This is an upstream bug in the Spellchecker plugin for TinyMCE with a PHP backend. Never the less, patch attached and I guess we should report to Moxiecode (quick search I couldn't find anything about this).

Problem is hard to come across since I believe it will only occur when the spellchecking service returns a result containing a UTF-8 character represented by either a 4, 5 or 6 byte sequence (aka uncommon ones).

@duck_
16 years ago

#2 @duck_
16 years ago

  • Keywords has-patch added
  • Milestone Awaiting Review3.1

#3 @nacin
16 years ago

Sent a note to the TinyMCE guys.

#4 @nacin
16 years ago

  • Resolutionfixed
  • Status newclosed

(In [16929]) Fix nesting of parentheses. fixes #14995, props duck_, jakub.misek.

#5 @nacin
16 years ago

Note came back from upstream, they'll be including it soon. Figured it was safe to go in here now.

Note: See TracTickets for help on using tickets.