Opened 17 years ago
Closed 17 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 17 years ago by
Attachment: | 2017.patch added |
---|
comment:1 Changed 17 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 17 years ago by
Keywords: | Review? added |
---|
comment:3 Changed 17 years ago by
Keywords: | Review+ added; Review? removed |
---|
comment:4 Changed 17 years ago by
Small remark:
The check for the FCKeditorAPI existence in my example should be:
if (window.FCKeditorAPI)
comment:5 Changed 17 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.