[D7] CKEditor LESS

We are currently quite busy with all the project applications and we prefer projects with a review bonus. Please help reviewing and put yourself on the high priority list, then we will take a look at your project right away :-)

Also, you should get your friends, colleagues or other community members involved to review this application. Let them go through the review checklist and post a comment that sets this issue to "needs work" (they found some problems with the project) or "reviewed & tested by the community" (they found no major flaws).

I'm a robot and this is an automated message from Project Applications Scraper.

Hi !

• So far there is no other project dealing with this subject, so you module is unique
• pareview shows no obvious errors: pareview
• I think you don't need the ckeditor_less.install file, you're not setting the less_ckeditor variable anywhere
• You should provide the public access clone command to your sandbox: git clone --branch 7.x-1.x http://git.drupal.org/sandbox/KyleTaylored/2306291.git ckeditor_less

1. You should edit module description and fix your clone command:

git clone --branch 7.x-1.x http://git.drupal.org/sandbox/KyleTaylored/2306291.git

2. Add more description to your module description and the readme file
3. Check if the variables exists or not (or return null) like less_dir, less module my not be setup at all, but enabled.

Repeating myself from https://www.drupal.org/node/2337931#comment-9176167 and removing bonus tag.

@bekirdag, you can also change the issue summary to correct clone command, as I did.

Thanks!

Closing due to lack of activity. If you are still working on this application, you should fix all known problems and then set the status to "Needs review". (See also the project application workflow).

I'm a robot and this is an automated message from Project Applications Scraper.

Installed the module and tried to set it up, but it didn't work. Debugged it a bit and on line 67 (ckeditor_less.module) seems to be a mess, the following modification worked:

// Build CSS path.
  $css_path = $less_path;

If there's no less folder on files directory, lines 38 and 39 throw some warnings, so check if path exists before using scandir.

Other than that, everything seems to be fine.

Closing due to lack of activity. If you are still working on this application, you should fix all known problems and then set the status to "Needs review". (See also the project application workflow).

I'm a robot and this is an automated message from Project Applications Scraper.