Ticket #2017 (closed New Feature: fixed)

Opened 7 years ago

Last modified 7 years ago

Make FCKeditorAPI.__Instances public

Reported by: saul11 Owned by: fredck
Priority: Normal Milestone: FCKeditor 2.6
Component: General Version:
Keywords: Confirmed Review+ Cc:

Description

Put it in the docs and present it as a public thing so people can easily access every FCKeditor instance on the page.

E.g. (currenlty)

if (FCKeditorAPI)
{
	for (fckeditorName in FCKeditorAPI.__Instances)
	{
		if (FCKeditorAPI.GetInstance(fckeditorName).IsDirty())
		{
			changesMade = true;
			break;
		}
	}
}

Attachments

2017.patch (2.6 KB) - added by fredck 7 years ago.

Change History

Changed 7 years ago by fredck

comment:1 Changed 7 years ago by fredck

  • Keywords Confirmed added; FCKeditorAPI __Instances removed
  • Owner set to fredck
  • Status changed from new to assigned
  • Type changed from Bug to New Feature
  • Milestone set to FCKeditor 2.6

Even if __Instances is a private property, the proposed patch still defines it, as we know people is (wrongly) using it out there.

comment:2 Changed 7 years ago by fredck

  • Keywords Review? added

comment:3 Changed 7 years ago by martinkou

  • Keywords Review+ added; Review? removed

comment:4 Changed 7 years ago by saul11

Small remark:

The check for the FCKeditorAPI existence in my example should be:

if (window.FCKeditorAPI)

comment:5 Changed 7 years ago by fredck

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

Fixed with [1713]. Click here for more info about our SVN system.

Note: See TracTickets for help on using tickets.
© 2003 – 2012 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy