ExtendedFileManager masks ImageManager (Trac #1021)

[sleemanj]

(Tracking ticket about to commit).

If you use the ExtendedFileManager plugin and the ImageManager plugin, if ExtendedFileManager is loaded after ImageManager it will mask ImageManager (make it not work), because they both override Xinha.prototype._insertImage.

Change to be commmitted will correct this so that is ExtendedFileManager is loaded after ImageManager then ImageManager will continue to be used for insertImage (and ExtendedFileManager will continue to be used for it's own button).

One day the two plugins should be merged into one which does all the same functions as both, ExtendedFileManager used to be like that originally I think but it didn't keep up with things added to ImageManager in the time before ExtendedFileManager came over to Xinha. I suspect the easiest way would be to add the missing features from ImageManager to ExtendedFileManager.

Reported by @sleemanj, migrated from http://trac.xinha.org/ticket/1021

[sleemanj]

• @sleemanj commented:

changeset:828

• @sleemanj changed resolution to fixed
• @sleemanj changed status from new to closed

[sleemanj]

koto commented:

I am just browsing history logs for both plugins. When, back in June 2006 I modified EFM at it was introduced in Xinha SVN it was up to date with ImageManager functionality at that time (IIRC).

Current changesets are usually applied to both plugins by ray, so I guess it's OK to replace IM with EFM.

The only differences I can spot right now are:

• 829 small bugfix
• 831,832 generic input field Image URL picking functionality
• 833 small bugfix (Opera)

And there is this new IM configuration option 'show_full_options' - but in EFM there already are options accounting for it.

Could someone (ray?) confirm on this? Dropping ImageManager would simplify future changes as we would not be maintaining both plugins that perform identical tasks.