Related to this issue, http://drupal.org/node/1414420 , but not entirely.

The permission for uploading files if wrapped by a check that you have CKFinder. This is good. However, it assumes CKFinder is in the CKEditor module.

    if (file_exists(drupal_get_path('module', 'ckeditor') . "/ckfinder")) {
        $arr['allow CKFinder file uploads'] = array(
            'title' => t('CKFinder access'),
            'description' => t('Allow users to use CKFinder.')
        );
    }

This is bad; it should be using the configurable CKFinder location path so people like me, who have CKEditor / CKFinder running on a multisite set up can keep the code stored in sites/all/libraries, and not have to re-download them with every CKEditor module release.

Patch to come next.

Comments

fakingfantastic’s picture

mkesicki’s picture

Status: Active » Needs review

@fakingfantastic thank you for patch.
We will check it as soon as possible.

dczepierga’s picture

Status: Needs review » Closed (fixed)

Thx for patch - this was fixed in last version.

Greetings