7.3 beta: Upload image from CKEditor gives error

1. Quickcreate> New Document
2. Under the Notes field, click on the image option of CKeditor
3. Click on upload tab in uploader popup
4. Select a file and try to upload:

Added enhancement label

@uma.s why is this marked as enhancement? It was working back on vtiger 7.1 (not sure if on 7.2 too).

The error is: Fatal error: Call to undefined function vtlib_purify() in /var/www/html/vt/kcfinder/core/uploader.php on line 50

@ruben.estrada Solving this inclusion, in turn accepts for further dependencies to be included. So here the solution is to upgrade the kcfinder file. Hence this has been marked for enhancement.

Well, this was a working feature on 7.1. There must have been something between 7.1 and 7.3 that broke it. Could it be related to this: #1254 (closed) ?

By the way, the kcfinder library seems to be abandoned since 2014: https://github.com/sunhater/kcfinder/commits/master

And you started modifying the code of the library in the above mentioned commit. what is the rational behind that? to keep maintaining it onwards?

Could it be that the problem is that you started introducing vtiger functions inside of the library? that's why we get the "undefined function" errors. I suspect, the code broke from this commmit: b577028a

one solution would be to move the /kcfinder/upload.php and /kcfinder/browse.php to vtiger's root and include the necessary files so that the used functions are available.

Another solution would be to replace the library. I'm not sure if we could use cfkinder instead: https://github.com/ckfinder

@ruben.estrada Thanks! for the inputs, Let me review this and check if we can get it solved through any of these ways.

Did you got a chance to try with any of these methods? I would curious to known about the output.

What I did in the end was simply replacing the vtigercrm/kcfinder/core/uploader.php with the original one from the library used in vtiger 7.1. Thst's how I got it working.

As a long term solution I see 3 options:

• use the lastest version of kcfinder which is 3.12 and keep on developing it.
• replace it with an alternative library that is still alive (I wouldn't know which one though)
• create an inhouse custom light version which only does the basics.

IMHO we should add the bug label to this issue.

To solve the error, the following lines must be added before the require "core/autoload.php" line, in the files: kcfinder/browser.php, kcfinder/css.php and kcfinder/upload.php:

$root_directory = dirname( getcwd () ) . "/"; require_once $root_directory . "include/utils/VtlibUtils.php"; require_once $root_directory . "vtlib/Vtiger/Functions.php";

in the file include/utils/VtlibUtils.php on line 698 you must modify the line:

include_once ('libraries/htmlpurifier410/library/HTMLPurifier.auto.php ');

for

include_once ($ use_root_directory.'libraries/htmlpurifier410/library/HTMLPurifier.auto.php ');

@felipe.camacho I think this has already been solved (Maybe it's a similar solution to the one you propose). But the issue hasn't been closed.

@uma.s maybe you guys can close it and relate it to the relevant commit. ThankS.

@ruben.estrada Thanks! for the notification.

@felipe.camacho Yes! This issue has been addressed,Please do refere here - Ref

Status changed to closed