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.