Good day
I downloaded CKEdtitor 3.3.1 the other day and found a small bug in the PHP helper script (most specifically in the php5 one -- I don't know if the same problem occurs in the php4 one). The issue is minor and trivial to fix -- I just wanted to contribute the fix back again.
At line 505 in ckeditor_php5.php we have the code:
The problem is that the dirname for the file "/foo.php" should be "" but is instead reported as / or \, meaning that the string manipulations counting on string lengths around lines 509-512 are out by one and the resultant path to the ckeditor.js file is erroneous. The simple fix for this original problem is the addition of a new line after the one mentioned above:
Also, this lead me to inspect the lines below:
This is obviously "failsafe"/"fallback code, but doesn't cover the situation where ckeditor is not installed into a directory of that name, of course. With the above fix in place, however, I found it quite adequate to commend out those three lines. Both the editor example I'm playing with in my server / and in another dir (/ck) work just fine with these additions.
Thanks
I downloaded CKEdtitor 3.3.1 the other day and found a small bug in the PHP helper script (most specifically in the php5 one -- I don't know if the same problem occurs in the php4 one). The issue is minor and trivial to fix -- I just wanted to contribute the fix back again.
At line 505 in ckeditor_php5.php we have the code:
$selfPath = dirname($_SERVER["PHP_SELF"]);
The problem is that the dirname for the file "/foo.php" should be "" but is instead reported as / or \, meaning that the string manipulations counting on string lengths around lines 509-512 are out by one and the resultant path to the ckeditor.js file is erroneous. The simple fix for this original problem is the addition of a new line after the one mentioned above:
if (($selfPath == "/") || ($selfPath == "\\")) $selfPath = "";
Also, this lead me to inspect the lines below:
if (!$selfPath || !$realPath || !$file) { return "/ckeditor/"; }
This is obviously "failsafe"/"fallback code, but doesn't cover the situation where ckeditor is not installed into a directory of that name, of course. With the above fix in place, however, I found it quite adequate to commend out those three lines. Both the editor example I'm playing with in my server / and in another dir (/ck) work just fine with these additions.
Thanks