Opened 8 years ago

Closed 8 years ago

#2017 closed New Feature (fixed)

Make FCKeditorAPI.__Instances public

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


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;

Attachments (1)

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

Download all attachments as: .zip

Change History (6)

Changed 8 years ago by fredck

comment:1 Changed 8 years ago by fredck

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

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 8 years ago by fredck

  • Keywords Review? added

comment:3 Changed 8 years ago by martinkou

  • Keywords Review+ added; Review? removed

comment:4 Changed 8 years ago by saul11

Small remark:

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

if (window.FCKeditorAPI)

comment:5 Changed 8 years ago by fredck

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

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

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