Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
I can manually set a custom path for ckeditor just fine, but for ckfinder, the module is always looking for it in the module directory. Since all my module updates are automated, ckfinder will get deleted everytime I need to upgrade the ckeditor module. This is why I install ckeditor application under /sites/all/libraries/ and configure the module to look there. So instead of $module_full_path . "/ckfinder/..." should probably read $cke_path . "/ckeditor/..." or allow users to specify a custom path for ckfinder.
Comment | File | Size | Author |
---|---|---|---|
#24 | ckeditor_module.patch | 4.54 KB | jlea9378 |
#24 | ckeditor_install.patch | 1.18 KB | jlea9378 |
#8 | ckfinder_fix_888950.patch | 6.65 KB | Diogenes |
#3 | ckfinder-758936.patch | 17.34 KB | nicholas.alipaz |
Comments
Comment #1
nicholas.alipaz CreditAttribution: nicholas.alipaz commented+1 for this issue. But not necessarily a bug report, I think this is a feature request.
I would really like to see this fixed. Perhaps the ckeditor_path() function could be rewritten slightly to be used for locating ckfinder in the same way?
--
Los Angeles Web Design and development for Drupal.
Comment #2
nicholas.alipaz CreditAttribution: nicholas.alipaz commentedoops, change status.
Comment #3
nicholas.alipaz CreditAttribution: nicholas.alipaz commentedHere is a pretty extensive patch to get the work on this started. It currently breaks ckfinder, but it is a start. I don't know enough about the module to confidently say where it is going wrong, but I am sure this is the right direction to take in order to get this functionality in the module.
Please feel free to test this out on a copy and modify. DON'T USE ON LIVE SITE.
Patched against today's cvs checkout of the 1.x branch.
--
Los Angeles Web Design and development for Drupal.
Comment #4
mephir CreditAttribution: mephir commentedIt will be in 1.3 release.
Comment #5
anirvanroy CreditAttribution: anirvanroy commentedWhen i use CKFinder the following happens:
Click on image button > Browse Server > double click pic > at this point the URL text box shows //imagesimages/PIC_0111.JPG, where it should actually be /imagesimages/PIC_0111.JPG (only one slash)
how can i change this so it gets the right URL?
here's a test login:
username - test
password - test12345
www.reformpartyindia.org
I believe many ppl are facing this problem
thanks
Comment #6
nicholas.alipaz CreditAttribution: nicholas.alipaz commentedanirvanroy, please open your own issue, this is not related to this feature request. Do not change this one again, as what you did just causes work for more people.
Comment #7
j0rd CreditAttribution: j0rd commentedPatch does not work againsn't the stable version (1.1). It applies, but does not work.
I've hardcoded these values in to make it work.
$finder_path = '/sites/all/libraries/ckfinder'
$finder_local_path = '/sites/all/libraries/ckfinder'
Temp solution until I can dig deeper.
Comment #8
Diogenes CreditAttribution: Diogenes commented@j0rd
Could you try this patch and see if it works? I posted it here but no one seems interested. This patch is for ckeditor 6.x-1.1. Copy the patch to the ckeditor directory and apply it from there.
Any other reconfiguring that may be necessary is also explained.
Comment #9
dczepierga CreditAttribution: dczepierga commentedThis option will be added as a new feature in release 1.3
Comment #10
dczepierga CreditAttribution: dczepierga commentedThis option will be added as a new feature in release 1.3
Comment #11
jlea9378 CreditAttribution: jlea9378 commentedI installed 1.3 but can't figure out how to define the path to ckfinder.
Comment #12
nicholas.alipaz CreditAttribution: nicholas.alipaz commentedLooks to me as though the feature was never added. I did a quick search of the commit history and found nothing regarding this issue.
Comment #13
j0rd CreditAttribution: j0rd commentedI just updated my CKEditor to the latest version 1.3 from 1.1
I can not find the configurable CKFinder path in version 1.3.
With that said, I applied the patch in post #8 by @Diogenes . While some hunks fails against 1.3 I was able to manually merge them.
After patching ckfinder works fine.
Comment #14
dczepierga CreditAttribution: dczepierga commentedI make same changes to @Diogenes patch and commit it to GIT - @Diogenes really thx for patch.
Now CKEditor module automatically set the CKFinder path - it's related with path to CKEditor library, so CKFinder library should be near CKEditor lib - e.g.: if u had CKEditor library in libraries dir, your CKFinder library should be also in libraries dir.
Please check the latest dev release and let me know if you notice any poblems with it. Remember to clear Drupal cache and browsers cache before testing.
Greetings
Comment #15
j0rd CreditAttribution: j0rd commentedSuper. Thankx.
Comment #17
mkesicki CreditAttribution: mkesicki commentedComment #18
jlea9378 CreditAttribution: jlea9378 commentedNot sure what's going on... I upgraded ckeditor module from 1.5 to 1.6 and now I'm getting this:
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
And launching ckfinder from the editor no longer works (Page Not Found).
Comment #19
mkesicki CreditAttribution: mkesicki commentedTry run update.php (drupal script). Clear all drupal cache. Can you check if this file sites/all/modules/ckeditor/ckfinder/config.php exists ?
Comment #20
jlea9378 CreditAttribution: jlea9378 commentedI ran update.php and cleared the cache, but that didn't help.
My ckeditor and ckfinder libraries are both located in sites/all/libraries
sites/all/libraries/ckeditor
and sites/all/libraries/ckfinder
It's finding the ckeditor library fine, but for some reason it isn't finding ckfinder. It worked fine prior to upgrading to the newer module (worked on 6.x-1.5).
Comment #21
whiklojIt looks like ckeditor is back to looking under the ckeditor module. But I could be wrong.
6.x-1.6
6.x-1.5
and
6.x-1.6
6.x-1.5
I'm going back to 1.5 for now.
Comment #22
mkesicki CreditAttribution: mkesicki commented@whikloj
Yes, from latest version CKEditor module search CKFinder in ckeditor module directory. That was design decision. We will think about pathes to CKEditor and CKFinder in future versions.
Comment #23
jlea9378 CreditAttribution: jlea9378 commentedWhen will this be fixed? It looks like it's broken still in 6.x-1.7.
I'm still using 6.x-1.5 since I use ckfinder...
My ckeditor and ckfinder libraries are both located in sites/all/libraries
sites/all/libraries/ckeditor
and sites/all/libraries/ckfinder
Comment #24
jlea9378 CreditAttribution: jlea9378 commentedAttached are some diff outputs (based on Diogenes' patch) that you can use to patch version 6.x-1.7 or 6.x-1.8 to recognize ckfinder.
You can apply them by:
patch ckeditor.install ckeditor_install.patch
patch ckeditor.module ckeditor_module.patch
Sorry if I didn't create the patches per drupal standards, I have no clue how. I've never used CVS.
Comment #25
mkesicki CreditAttribution: mkesicki commentedThx for patches. We will check this ASAP. Please be patient.
Comment #26
jlea9378 CreditAttribution: jlea9378 commentedRTBC?
Comment #27
mkesicki CreditAttribution: mkesicki commented@guys , this bug was fixed in latest releases.
Please check :
http://drupalcode.org/project/ckeditor.git/commitdiff/4bc2427767ff9cc6ba9ce85f6f02a86545752fcd
http://drupalcode.org/project/ckeditor.git/commitdiff/072966868c5f5898a533fc907943b6f331fd0e80
I mark this ticket as closed.
Comment #28
jenlamptonLooks like this bug is back in 6.x-1.14
Comment #29
wwalc CreditAttribution: wwalc commented@jenlampton - from which version did you upgrade to 6.x-1.14?