In filemanager.config.php, Building $baseDir

Hi,

In 'ckeditor\includes\filemanager.config.php'. Line no 102.

 $baseUrl = strtr(base_path(), array(
        '/modules/ckeditor/ckfinder/core/connector/php' => '',
      )) . variable_get('file_private_path', conf_path() . '/files') . '/';
 $baseDir = resolveUrl($baseUrl);

Instead of having '/modules/ckeditor/ckfinder/core/connector/php', we can have the below code as per Drupal standards:

 $baseUrl = strtr(base_path(), array(
        drupal_get_path('module','ckeditor') . '/ckfinder/core/connector/php' => '',
      )) . variable_get('file_private_path', conf_path() . '/files') . '/';
 $baseDir = resolveUrl($baseUrl);

If this is not updated and you keep your 'ckfinder' library at 'modules/contrib/ckeditor/ckfinder' (or any other location except '/modules/ckeditor/ckfinder'), you may face issues related to ckfinder.

This issue exists in Drupal 7 and Drupal 6 as well.

Regards,
Neeraj Singh