Fluid Infusion

The jquery.tinymce plugin assumes that TinyMCE is present and invokes a method on it immediately upon load without checking first.

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Blocker Blocker
  • Resolution: Fixed
  • Affects Version/s: 0.6, 0.7, 0.8, 1.0
  • Fix Version/s: 1.1
  • Component/s: Inline Edit
  • Labels:
    None

Description

At the moment, the jquery.tinymce plugin invokes the following code immediately upon being parsed:

    // Invoke this immediately to prime TinyMCE.
    tinyMCE.init({
        mode: "none",
        theme: "simple"
    });

However, to properly be included in Fluid-all.js, the plugin should not assume that TinyMCE is present, in case a user is using Fluid-all.js without being interested in TinyMCE-related code.

The correct implementation should be:

if (typeof(tinyMCE) !== "undefined") {
    // Invoke this immediately to prime TinyMCE.
    tinyMCE.init({
        mode: "none",
        theme: "simple"
    });
}

Activity

Hide
Colin Clark added a comment -
I fixed this issue in r7160.
Show
Colin Clark added a comment - I fixed this issue in r7160.
Hide
Michelle D'Souza added a comment -
Closing issues that were fixed in 1.1
Show
Michelle D'Souza added a comment - Closing issues that were fixed in 1.1

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: