Closed (fixed)
Project:
CKEditor 4 - WYSIWYG HTML editor
Version:
6.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
5 Jan 2010 at 03:44 UTC
Updated:
7 Oct 2011 at 12:44 UTC
Jump to comment: Most recent file
Comments
Comment #1
dave reidComment #2
wwalc commentedGreat, thanks for adding a patch. I didn't review it yet but looks good.
Just a note for me to not forget about it - the svn URL should be changed, http://svn.fckeditor.net/CKEditor/releases/stable will be valid since tomorrow or something like that (CKEditor 3.1 is coming! ;))
Comment #3
wwalc commentedComment #4
mephir commentedThanks
Comment #5
mephir commentedComment #6
jcisio commentedThis patch uses the default path sites/all/libraries/ckeditor instead of the one defined by the module sites/all/modules/ckeditor/ckeditor.
Comment #7
mephir commentedThe path of ckeditor isn't important, because ckeditor can find editor in libraries directory. Libraries directory is pretty good, because if you are using drush update, it delete ckeditor module directory and create new one, if we decided to download editor to ckeditor module directory, you would be forced to download editor again, after all ckeditor module updates.
Comment #8
jcisio commentedThanks for the explanation ;) Very helpful. I didn't know that before, so I created a symlink to libraries/ckeditor, but it's ok now.
Comment #10
robin van emden commentedThe "ckeditor download" command should be replaced by "ckeditor-download" due to changes in drush:
Notice: "ckeditor download" must be renamed to "ckeditor-download" in[error]
order to work with this version of drush. If you are the maintainer
for the module that defines this command, please rename it
immediately. If you are a user of this command, you may enable
spaces in commands for a while by setting "allow-spaces-in-commands"
in your drush configuration file. See example.drushrc.php.
Otherwise, great include!
Comment #11
mephir commentedChanges commited to CVS.
Comment #13
fuzzy76 commented