Closed (won't fix)
Project:
Drupal.org security advisory coverage applications
Component:
module
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
4 Dec 2012 at 22:59 UTC
Updated:
20 Feb 2013 at 18:32 UTC
The "StackEdit" module provides a WYSIWYM text editor inspired by WMD editor used in StackOverflow site.
StackEdit sandbox project
This module is for Drupal 7.
Clone from git repository:
git clone --recursive --branch 7.x-1.x http://git.drupal.org/sandbox/alessandro.pezzato/1835986.git stackedit
You also need external libraries:
Download `google-code-prettify` from
and extract the content of `distrib` directory into `libraries` directory.
Download http://google-code-prettify.googlecode.com/files/prettify-small-1-Jun-20...
tar jxvf prettify-small-1-Jun-2011.tar.bz2 -C libraries
rm -r prettify-small-1-Jun-2011.tar.bz2
Clone also submodule (jquery.hotkeys)
git submodule status # show repository's submodules
git submodule init # initialize local configuration
git submodule update # fetch all the data from submodules

| Comment | File | Size | Author |
|---|---|---|---|
| Screenshot | 72.55 KB | Alessandro Pezzato |
Comments
Comment #0.0
Alessandro Pezzato commentedfix screenshot url
Comment #1
vineet.osscube commentedHi,
Firstly there is lots of issue regarding line spaces, indentation & comments.
Look at this !
http://ventral.org/pareview/httpgitdrupalorgsandboxalessandropezzato1835...
Here you can check source code whether it meets drupal coding standards or not, and advise you what to change in your code. You can repeat review after your commits, and can fix those errors.
Comment #2
vineet.osscube commentedManual Review :
1) Please don't add LICENSE.txt, it will be added by drupal community after completing review process and full project release.
2) Use L() function to make your links in line no 19, 20 in stackedit.help.inc & line 67 in stackedit.text_filter.inc
3) Do not add "?>" PHP delimiter at the end of files is discouraged in ./libraries/markdown/markdown.php.
Comment #3
Alessandro Pezzato commentedI've done some fixes in my code, but external code in the libraries directory is out of my control. It's here because there is not a simple way to download it all. Is there any way to make code reviewer ingnoring all "libraries" directory?
Comment #3.0
Alessandro Pezzato commentedfix git command, screenshot to the end, external libraries installation
Comment #4
pere orgaI don't know for sure if the module is ready to be a full project, but after reviewing it I don't see any problems with it.
Just one thing, it's better to not include any javascript library, you should encourage users to use the common libraries directory instead. That way the files can be shared across modules.
Comment #5
mohs3n71 commentedHi
first ) see this automatic review : http://ventral.org/pareview/httpgitdrupalorgsandboxalessandropezzato1835...
there are many erorrs ...
you must see drupal coding standards : http://drupal.org/node/318
second ) if you are using external libraries you must use libraries module : https://drupal.org/project/libraries and add this module as a dependency in your .info file and in your project page give a link to users to download the libraries and put them in the libraries folder this is very better way
Comment #6
klausiClosing due to lack of activity. Feel free to reopen if you are still working on this application.
If you reopen this please keep in mind that we are currently quite busy with all the project applications and I can only review projects with a review bonus. Please help me reviewing and I'll take a look at your project right away :-)
Comment #6.0
klausichanged git clone command