Opened 18 years ago
Closed 18 years ago
#2017 closed New Feature (fixed)
Make FCKeditorAPI.__Instances public
| Reported by: | Paul Moers | Owned by: | Frederico Caldeira Knabben | 
|---|---|---|---|
| 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 (1)
Change History (6)
Changed 18 years ago by
| Attachment: | 2017.patch added | 
|---|
comment:1 Changed 18 years ago by
| Keywords: | Confirmed added; FCKeditorAPI __Instances removed | 
|---|---|
| Milestone: | → FCKeditor 2.6 | 
| Owner: | set to Frederico Caldeira Knabben | 
| Status: | new → assigned | 
| Type: | Bug → New Feature | 
comment:2 Changed 18 years ago by
| Keywords: | Review? added | 
|---|
comment:3 Changed 18 years ago by
| Keywords: | Review+ added; Review? removed | 
|---|
comment:4 Changed 18 years ago by
Small remark:
The check for the FCKeditorAPI existence in my example should be:
if (window.FCKeditorAPI)
comment:5 Changed 18 years ago by
| Resolution: | → fixed | 
|---|---|
| Status: | assigned → closed | 
Fixed with [1713]. Click here for more info about our SVN system.

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