In the README.txt it still says to install CKFinder at ./sites/all/modules/ckeditor/ckfinder.

When updating to ckeditor 1.5, and still having CKFinder at ./sites/all/modules/ckeditor/ckfinder (while the CKEditor library is at ./sites/all/libraries/ckeditor) things seem to become confusing.

For example

function ckeditor_perm() {
  $arr = array('administer ckeditor', 'access ckeditor');
  if (file_exists(dirname(ckeditor_path(TRUE)) . "/ckfinder")) {
    $arr[] = 'allow CKFinder file uploads';
  }
  return $arr;
}

is looking for CKFinder under ./sites/all/libraries, because dirname(ckeditor_path(TRUE)) returns ./sites/all/libraries.

I **guess**, that when having the ckeditor library under ./sites/all/libraries, one is expected to also have ckfinder under ./sites/all/libraries?

Furthermore, when CKFinder is at ./sites/all/libraries/ckfinder, the "require_once '../../../../../includes/filemanager.config.php'" command in ./ckfinder/config.php does not work anymore.
I specified the full path like "require_once '/var/www/....../ckeditor/includes/filemanager.config.php'" to make it work.

Where is all this documented, if anywhere?

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mkesicki’s picture

Thx for notice this , we will check this. I think there is some bug in documentation. CKEditor looks CKFinder in module's directory.

nelslynn’s picture

Any other hints? Does CKFinder go in libraries or in the ckeditor module folder? Wouldn't it be best in libraries so the module can be updated easier??

I tried to modify includes/filemanager.config.php line and still doesn't work.

mkesicki’s picture

CKFinder should be subdirectory in CKEditor's directory. So if you have CKEditor module in sites/all/libraries/ckeditor it should be sites/all/libraries/ckeditor/ckfinder or if you have CKEditor module in sites/all/modules/ckeditor put CKFinder in sites/all/modules/ckeditor/ckfinder.
Integration of CKFinder and CKEditor module is described in readme file attached to CKEditor module.

nelslynn’s picture

Have you tried this, because it does not work.

Here's my config straight from the README:
- CKFinder (downloaded from the http://ckfinder.com/download) is in the /sites/all/modules/ckeditor folder.
- CKEditor (downloaded from http://ckeditor.com/download) is in my /sites/all/libraries
- ckedior module from Drupal.org is in /sites/all/modules folder
- Add file edits have been done.

When I try to upload an image, or browse the server for an image via the ckeditor WYSIWYG, I get "Page Not Found", and the URL is: http://mysite.com/sites/all/libraries/ckfinder/ckfinder.html?Type=Images...

SO, why is the finder trying to look in /sites/all/libraries when ckfinder folder is in /sites/all/modules/ckeditor? Where can I tell ckfinder to look in the /sites/all/modules/ckeditor folder? Where is it grabbing this WRONG URL? What might I check??

See the first comment.... it's not working for either of us when following the README.

Thanks for any direction.

dczepierga’s picture

Title: Is it now required to have CKFinder at ./sites/all/libraries/ckfinder ? » Bug with CKFinder path when CKEditor is in libraries dir
Status: Active » Fixed

Ok, i can confirm that is bug in CKEditor module...
I fix it and commit to GIT. Pls update to last DEV and report here does help or not...

Greetings

mkesicki’s picture

@nelslynn
I tried you description and I can confirm there was a bug. The latest DEV fix it.

nelslynn’s picture

Latest Dev works... thanks!! BUT I still get an error message in Status (see screenshot).

My CKFinder is in sites/all/modules/ckeditor/ckfinder, NOT libraries.
My CKEditor download is in sites/all/libraries folder.

mkesicki’s picture

FileSize
163.07 KB

I can confirm this bug and probably I found another one : I can't see CKEditor module on "available updates" site.

dczepierga’s picture

@nelslynn I fix it and commit to GIT.

Greetings

hd’s picture

When I use CKEditor 6.x-1.x-dev module from 2011-Jul-19 AND when having the CKEditor library under ./sites/all/libraries/ckeditor (not under ./sites/all/modules/ckeditor/ckeditor), THEN I have to put CKFinder at ./sites/all/libraries/ckfinder (not under ./sites/all/libraries/ckeditor/ckfinder as suggested at http://drupal.org/node/1211554#comment-4727802) to make it become visible and to produce the permission "allow CKFinder file uploads" on the permissions page. But still, CKFinder is looked for at ./sites/all/modules/ckeditor/ckfinder when it comes to be called.

And I do get this error on admin/reports/status page: CKEditor 3.5.3. CKFinder is not installed correctly: sites/all/modules/ckeditor/ckfinder/config.php not found. Make sure that you have uploaded all files or didn't remove that file accidentally.

Strangely I can get CKFinder to work while the CKEditor library is under ./sites/all/libraries/ckeditor and CKFinder at ./sites/all/modules/ckeditor/ckfinder, while the permission "allow CKFinder file uploads" is NOT shown on the permissions page. I guess CKFinder works because I had this permission still defined from previous scenarios. But after a server reboot things changed again. However, when creating a symbolic link in ./sites/all/libraries like ckfinder -> ../modules/ckeditor/ckfinder, I do get the "allow CKFinder file uploads" shown again on the permissions page and things working.

Therefore, the only way to get it all (CKEditor + CKFinder) to work **reliably and without tricks** is when I bring both "home" under ./sites/all/modules/ckeditor: the CKEditor library to ./sites/all/modules/ckeditor/ckeditor and the CKFinder to ./sites/all/modules/ckeditor/ckfinder.

It would, however, be better to keep stuff under ./sites/all/libraries/ so that they are out of the way when doing the Drupal module update.

BTW: When changing and moving things around, it is important to go to admin/settings/ckeditor and go into edit mode for all profiles (those used for roles and the one used for global) and, without even changing a thing, click on save or update. And then refresh the permission page admin/user/permissions to hopefully see permission "allow CKFinder file uploads" advertised. It is also a good idea to clear the Drupal cache and the cache of the web browser.

Bottom line: with CKEditor 6.x-1.x-dev module from 2011-Jul-19 it seems as everything got to be under ./sites/all/modules/ckeditor to also make CKFinder work properly. For me things seem to be pretty much where I was when I opened this issue on July 7. I cannot say it has been fixed.

mkesicki’s picture

Status: Fixed » Needs work
jay-dee-ess’s picture

subscribe

dczepierga’s picture

Status: Needs work » Fixed

I commit next patch and now everything should works.

In summary:
1) CKEditor library could be placed in libraries or ckeditor module directory
2) CKFinder must be in ckeditor module directory

If is still problem with it pls write here...

Greetings

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

pog21’s picture

Status: Closed (fixed) » Active

1) CKEditor library could be placed in libraries or ckeditor module directory
2) CKFinder must be in ckeditor module directory

Let me know if this should be a new thread, but wouldn't it make more sense if ckeditor *and* ckfinder could be in 'sites/all/libraries'?

mkesicki’s picture

Status: Active » Closed (fixed)

@pog21,
there is a topic about this #758936: Add automatic setting of the CKFinder library path based on CKEditor library path. Please don't reopen this ticket.