"Call to undefined function" with PHP4

[Frederico Caldeira Knabben]

The PHP4 integration is not working. It throws the following error:

Fatal error: Call to undefined function FCKeditor_IsCompatibleBrowser() in D:\FCKeditor\SVN\FCKeditor\local\fckeditor_php4.php on line 167

To test it in a PHP5 environment, it is enough to change the include at the top of sample01.php to:

include("../../fckeditor_php4.php") ;

[Frederico Caldeira Knabben]

This bug has been originally reported at our forums (​here).

[Wiktor Walc]

Documentation states that fckeditor.php should be included: ​http://docs.fckeditor.net/FCKeditor_2.x/Developers_Guide/Integration/PHP

Anyway, it may be a good idea to move fckeditor_php4.php and fckeditor_php5.php somewhere deeper into "editor" directory and hide it from other users to avoid such errors.

[Alfonso Martínez de Lizarrondo]

or maybe rename them fckutils_php4.php and fckutils_php5.php ?
(to avoid having them placed in a different folder that the rest of integration files, and it also express correctly what they are)

[Wiktor Walc]

Nice idea. Anyway, there is another thing to consider: some people might use include/require calls instead of include_once/require_once. This may ocassionally be causing problems when calling include "fckeditor.php"; twice in the same script. If we move FCKeditor_IsCompatibleBrowser() into fckeditor_php4.php and fckeditor_php5.php, we will get rid of this error and issue described in this ticket as well.

The only disadvantage is that this way we will break DRY rule... but that's not a big deal and we can live with it, at least until V3.

I'm attaching a patch that moves 'FCKeditor_IsCompatibleBrowser() function. It's not an ideal solution, but at least it solves most problems that php beginners may encounter. I believe that it makes no sense to put this into separate file, because we already have plenty of files in the root directory.

[Frederico Caldeira Knabben]

It would be a nice idea moving fckeditor_php4.php and fckeditor_php5.php inside the "editor" folder, but it is already too late for that. It may break installations where people is pointing to those files.

So, for now, the proposed patch is ok.

[Wiktor Walc]

Fixed with [2008].