The module Carousel Nanofaz is based on Jquery and enables you to check a list of objects according to a horizontal or vertical position. Content can be dynamically inserted via the control panel. The 'look and feel is simple and nice.

Project Page:
https://drupal.org/sandbox/fziviello/1994592

GIT link:
git clone --branch 7.x-1.x fziviello@git.drupal.org:sandbox/fziviello/1994592.git carousel_nanofaz

http://drupalcode.org/sandbox/fziviello/1994592.git

Comments

alexmoreno’s picture

alexmoreno CreditAttribution: alexmoreno commented

Hi fzviello,

you have a lot of coding standard issues. Correct them first please (use this tool for help): http://pareview.sh/pareview/httpgitdrupalorgsandboxfziviello1994592git

Remove all .DS_Store files from your repository.
Bad line endings were found, always use unix style terminators. See http://drupal.org/coding-standards#indenting
carousel_nanofaz.install
carousel_nanofaz.module

Coder Sniffer has found some issues with your code (please check the Drupal coding standards). See attachment.
DrupalPractice has found some issues with your code, but could be false positives.
FILE: /var/www/drupal-7-pareview/pareview_temp/carousel_nanofaz.module
--------------------------------------------------------------------------------
FOUND 0 ERROR(S) AND 14 WARNING(S) AFFECTING 14 LINE(S)
--------------------------------------------------------------------------------
75 | WARNING | All variables defined by your module must be prefixed with
| | your module's name to avoid name collisions with others.
| | Expected start with "carousel_nanofaz" but found
| | "carousel_width"
82 | WARNING | All variables defined by your module must be prefixed with
| | your module's name to avoid name collisions with others.
| | Expected start with "carousel_nanofaz" but found
| | "carousel_height"
89 | WARNING | All variables defined by your module must be prefixed with
| | your module's name to avoid name collisions with others.
| | Expected start with "carousel_nanofaz" but found
| | "carousel_background"
105 | WARNING | All variables defined by your module must be prefixed with
| | your module's name to avoid name collisions with others.
| | Expected start with "carousel_nanofaz" but found
| | "carousel_navigator"
113 | WARNING | All variables defined by your module must be prefixed with
| | your module's name to avoid name collisions with others.
| | Expected start with "carousel_nanofaz" but found
| | "carousel_autoplay"
120 | WARNING | All variables defined by your module must be prefixed with
| | your module's name to avoid name collisions with others.
| | Expected start with "carousel_nanofaz" but found
| | "carousel_prev"
127 | WARNING | All variables defined by your module must be prefixed with
| | your module's name to avoid name collisions with others.
| | Expected start with "carousel_nanofaz" but found
| | "carousel_succ"
134 | WARNING | All variables defined by your module must be prefixed with
| | your module's name to avoid name collisions with others.
| | Expected start with "carousel_nanofaz" but found
| | "carousel_play"
141 | WARNING | All variables defined by your module must be prefixed with
| | your module's name to avoid name collisions with others.
| | Expected start with "carousel_nanofaz" but found
| | "carousel_stop"
148 | WARNING | All variables defined by your module must be prefixed with
| | your module's name to avoid name collisions with others.
| | Expected start with "carousel_nanofaz" but found
| | "carousel_speed"
158 | WARNING | All variables defined by your module must be prefixed with
| | your module's name to avoid name collisions with others.
| | Expected start with "carousel_nanofaz" but found
| | "carousel_image-"
234 | WARNING | Variable $i is undefined.
287 | WARNING | Unused variable $image.
291 | WARNING | Variable $validators is undefined.
--------------------------------------------------------------------------------

DrupalSecure has found some issues with your code (please check the Writing secure core handbook).
FILE: /var/www/drupal-7-pareview/pareview_temp/carousel_nanofaz.module
--------------------------------------------------------------------------------
FOUND 0 ERROR(S) AND 1 WARNING(S) AFFECTING 1 LINE(S)
--------------------------------------------------------------------------------
269 | WARNING | Possible SQL injection in db_query() via variable $fid
--------------------------------------------------------------------------------

This automated report was generated with PAReview.sh, your friendly project application review script. You can also use the online version to check your project. You have to get a review bonus to get a review from me.


FILE: /var/www/drupal-7-pareview/pareview_temp/README.txt
--------------------------------------------------------------------------------
FOUND 0 ERROR(S) AND 1 WARNING(S) AFFECTING 1 LINE(S)
--------------------------------------------------------------------------------
3 | WARNING | Line exceeds 80 characters; contains 235 characters
--------------------------------------------------------------------------------

FILE: /var/www/drupal-7-pareview/pareview_temp/carousel_nanofaz.info
--------------------------------------------------------------------------------
FOUND 1 ERROR(S) AFFECTING 1 LINE(S)
--------------------------------------------------------------------------------
5 | ERROR | Files must end in a single new line character
--------------------------------------------------------------------------------

FILE: /var/www/drupal-7-pareview/pareview_temp/carousel_nanofaz.install
--------------------------------------------------------------------------------
FOUND 14 ERROR(S) AND 2 WARNING(S) AFFECTING 9 LINE(S)
--------------------------------------------------------------------------------
1 | ERROR | End of line character is invalid; expected "\n" but found
| | "\r\n"
3 | ERROR | The second line in the file doc comment must be " * @file"
3 | ERROR | Expected 1 space(s) before asterisk; 0 found
4 | ERROR | The third line in the file doc comment must contain a
| | description and must not be indented
4 | ERROR | Expected 1 space(s) before asterisk; 0 found
5 | ERROR | The last line in the file doc comment must be " */"
5 | ERROR | Expected 1 space(s) before asterisk; 0 found
8 | WARNING | Format should be "* Implements hook_foo().", "* Implements
| | hook_foo_BAR_ID_bar() for xyz_bar().", or "* Implements
| | hook_foo_BAR_ID_bar() for xyz_bar.tpl.php.".
8 | ERROR | Expected 1 space(s) before asterisk; 0 found
8 | ERROR | Function comment short description must end with a full stop
9 | ERROR | Expected 1 space(s) before asterisk; 0 found
15 | WARNING | Format should be "* Implements hook_foo().", "* Implements
| | hook_foo_BAR_ID_bar() for xyz_bar().", or "* Implements
| | hook_foo_BAR_ID_bar() for xyz_bar.tpl.php.".
15 | ERROR | Expected 1 space(s) before asterisk; 0 found
15 | ERROR | Function comment short description must end with a full stop
16 | ERROR | Expected 1 space(s) before asterisk; 0 found
19 | ERROR | Files must end in a single new line character
--------------------------------------------------------------------------------

FILE: /var/www/drupal-7-pareview/pareview_temp/carousel_nanofaz.module
--------------------------------------------------------------------------------
FOUND 202 ERROR(S) AND 3 WARNING(S) AFFECTING 158 LINE(S)
--------------------------------------------------------------------------------
1 | ERROR | End of line character is invalid; expected "\n" but found
| | "\r\n"
4 | ERROR | The second line in the file doc comment must be " * @file"
4 | ERROR | Expected 1 space between asterisk and tag; 0 found
5 | WARNING | Line exceeds 80 characters; contains 84 characters
6 | WARNING | Line exceeds 80 characters; contains 85 characters
14 | ERROR | Space found before comma in function call
18 | ERROR | Space found before comma in function call
18 | ERROR | There should be no white space before a closing ")"
27 | ERROR | Array indentation error, expected 4 spaces but found 3
28 | ERROR | Array indentation error, expected 4 spaces but found 3
29 | ERROR | Array indentation error, expected 4 spaces but found 3
30 | ERROR | Array indentation error, expected 4 spaces but found 3
31 | ERROR | Array indentation error, expected 4 spaces but found 3
32 | ERROR | Array indentation error, expected 4 spaces but found 3
33 | ERROR | Array indentation error, expected 4 spaces but found 3
34 | ERROR | Array indentation error, expected 4 spaces but found 3
37 | ERROR | Array indentation error, expected 4 spaces but found 3
38 | ERROR | Array indentation error, expected 4 spaces but found 3
39 | ERROR | Array indentation error, expected 4 spaces but found 3
40 | ERROR | Array indentation error, expected 4 spaces but found 3
41 | ERROR | Array indentation error, expected 4 spaces but found 3
53 | ERROR | Array indentation error, expected 4 spaces but found 3
54 | ERROR | Array indentation error, expected 4 spaces but found 3
57 | ERROR | Array indentation error, expected 4 spaces but found 3
58 | ERROR | Array indentation error, expected 4 spaces but found 3
59 | ERROR | Array indentation error, expected 4 spaces but found 3
60 | ERROR | Array indentation error, expected 4 spaces but found 3
61 | ERROR | Array indentation error, expected 4 spaces but found 3
62 | ERROR | Array indentation error, expected 4 spaces but found 3
63 | ERROR | Array indentation error, expected 4 spaces but found 3
66 | ERROR | Array indentation error, expected 4 spaces but found 3
67 | ERROR | Array indentation error, expected 4 spaces but found 3
68 | ERROR | Array indentation error, expected 4 spaces but found 3
69 | ERROR | Array indentation error, expected 4 spaces but found 3
70 | ERROR | Array indentation error, expected 4 spaces but found 3
73 | ERROR | Array indentation error, expected 4 spaces but found 3
74 | ERROR | Array indentation error, expected 4 spaces but found 3
75 | ERROR | Array indentation error, expected 4 spaces but found 3
76 | ERROR | Array indentation error, expected 4 spaces but found 3
77 | ERROR | Array indentation error, expected 4 spaces but found 3
80 | ERROR | Array indentation error, expected 4 spaces but found 3
81 | ERROR | Array indentation error, expected 4 spaces but found 3
82 | ERROR | Array indentation error, expected 4 spaces but found 3
83 | ERROR | Array indentation error, expected 4 spaces but found 3
84 | ERROR | Array indentation error, expected 4 spaces but found 3
87 | ERROR | Array indentation error, expected 4 spaces but found 3
88 | ERROR | Array indentation error, expected 4 spaces but found 3
89 | ERROR | Array indentation error, expected 4 spaces but found 3
90 | ERROR | Array indentation error, expected 4 spaces but found 3
91 | ERROR | Array indentation error, expected 4 spaces but found 3
92 | ERROR | Array indentation error, expected 4 spaces but found 3
95 | ERROR | Array indentation error, expected 4 spaces but found 3
96 | ERROR | Array indentation error, expected 4 spaces but found 3
97 | ERROR | Array indentation error, expected 4 spaces but found 3
98 | ERROR | Array indentation error, expected 4 spaces but found 3
99 | ERROR | Array indentation error, expected 4 spaces but found 3
102 | ERROR | Array indentation error, expected 4 spaces but found 3
103 | ERROR | Array indentation error, expected 4 spaces but found 3
104 | ERROR | Array indentation error, expected 4 spaces but found 3
105 | ERROR | Array indentation error, expected 4 spaces but found 3
106 | ERROR | Array indentation error, expected 4 spaces but found 3
107 | ERROR | Array indentation error, expected 4 spaces but found 3
110 | ERROR | Array indentation error, expected 4 spaces but found 3
111 | ERROR | Array indentation error, expected 4 spaces but found 3
112 | ERROR | Array indentation error, expected 4 spaces but found 3
113 | ERROR | Array indentation error, expected 4 spaces but found 3
114 | ERROR | Array indentation error, expected 4 spaces but found 3
115 | ERROR | Array indentation error, expected 4 spaces but found 3
118 | ERROR | Array indentation error, expected 4 spaces but found 3
119 | ERROR | Array indentation error, expected 4 spaces but found 3
120 | ERROR | Array indentation error, expected 4 spaces but found 3
120 | ERROR | Space found before comma in function call
121 | ERROR | Array indentation error, expected 4 spaces but found 3
122 | ERROR | Array indentation error, expected 4 spaces but found 3
125 | ERROR | Array indentation error, expected 4 spaces but found 3
126 | ERROR | Array indentation error, expected 4 spaces but found 3
127 | ERROR | Array indentation error, expected 4 spaces but found 3
127 | ERROR | Space found before comma in function call
128 | ERROR | Array indentation error, expected 4 spaces but found 3
129 | ERROR | Array indentation error, expected 4 spaces but found 3
132 | ERROR | Array indentation error, expected 4 spaces but found 3
133 | ERROR | Array indentation error, expected 4 spaces but found 3
134 | ERROR | Array indentation error, expected 4 spaces but found 3
134 | ERROR | Space found before comma in function call
135 | ERROR | Array indentation error, expected 4 spaces but found 3
136 | ERROR | Array indentation error, expected 4 spaces but found 3
139 | ERROR | Array indentation error, expected 4 spaces but found 3
140 | ERROR | Array indentation error, expected 4 spaces but found 3
141 | ERROR | Array indentation error, expected 4 spaces but found 3
141 | ERROR | Space found before comma in function call
142 | ERROR | Array indentation error, expected 4 spaces but found 3
143 | ERROR | Array indentation error, expected 4 spaces but found 3
146 | ERROR | Array indentation error, expected 4 spaces but found 3
147 | ERROR | Array indentation error, expected 4 spaces but found 3
148 | ERROR | Array indentation error, expected 4 spaces but found 3
148 | ERROR | Space found before comma in function call
149 | ERROR | Array indentation error, expected 4 spaces but found 3
150 | ERROR | Array indentation error, expected 4 spaces but found 3
153 | ERROR | Line indented incorrectly; expected 4 spaces, found 2
153 | ERROR | Expected 1 space before "="; 0 found
153 | ERROR | Expected 1 space after "="; 0 found
154 | ERROR | Line indented incorrectly; expected 6 spaces, found 2
155 | ERROR | Line indented incorrectly; expected 6 spaces, found 3
155 | ERROR | Array indentation error, expected 4 spaces but found 3
156 | ERROR | Line indented incorrectly; expected 6 spaces, found 3
156 | ERROR | Array indentation error, expected 4 spaces but found 3
157 | ERROR | Line indented incorrectly; expected 6 spaces, found 3
157 | ERROR | Array indentation error, expected 4 spaces but found 3
158 | ERROR | Line indented incorrectly; expected 6 spaces, found 3
158 | ERROR | Array indentation error, expected 4 spaces but found 3
159 | ERROR | Line indented incorrectly; expected 6 spaces, found 3
159 | ERROR | Array indentation error, expected 4 spaces but found 3
159 | ERROR | Whitespace found at end of line
160 | ERROR | Line indented incorrectly; expected 6 spaces, found 3
160 | ERROR | Array indentation error, expected 4 spaces but found 3
161 | ERROR | Line indented incorrectly; expected 6 spaces, found 3
161 | ERROR | Array indentation error, expected 4 spaces but found 3
162 | ERROR | Line indented incorrectly; expected 6 spaces, found 3
162 | ERROR | Array indentation error, expected 4 spaces but found 3
163 | ERROR | Line indented incorrectly; expected 6 spaces, found 3
163 | ERROR | Array indentation error, expected 4 spaces but found 3
163 | WARNING | Last item of an inline array must not be followed by a comma
163 | ERROR | There should be no white space before a closing ")"
164 | ERROR | Line indented incorrectly; expected 6 spaces, found 3
164 | ERROR | Array indentation error, expected 4 spaces but found 3
165 | ERROR | Line indented incorrectly; expected 6 spaces, found 2
188 | ERROR | There should be no white space before a closing ")"
189 | ERROR | Line indented incorrectly; expected 4 spaces, found 2
190 | ERROR | Line indented incorrectly; expected 4 spaces, found 2
191 | ERROR | Line indented incorrectly; expected 4 spaces, found 3
192 | ERROR | Line indented incorrectly; expected 4 spaces, found 3
193 | ERROR | Line indented incorrectly; expected 4 spaces, found 3
194 | ERROR | Line indented incorrectly; expected 4 spaces, found 2
195 | ERROR | Line indented incorrectly; expected 4 spaces, found 2
215 | ERROR | Incorrect spacing between argument "$delta" and equals sign;
| | expected 1 but found 0
215 | ERROR | Incorrect spacing between default value and equals sign for
| | argument "$delta"; expected 1 but found 0
218 | ERROR | There must be no space before the colon in a CASE statement
218 | ERROR | Line indented incorrectly; expected 4 spaces, found 2
219 | ERROR | Line indented incorrectly; expected 6 spaces, found 2
220 | ERROR | Line indented incorrectly; expected 6 spaces, found 2
221 | ERROR | Case breaking statement indented incorrectly; expected 4
| | spaces, found 2
229 | ERROR | Space found before comma in function call
230 | ERROR | Expected 1 space before "="; 0 found
230 | ERROR | Space found before comma in function call
231 | ERROR | Expected 1 space before "="; 0 found
231 | ERROR | Space found before comma in function call
232 | ERROR | Expected 1 space before "="; 0 found
232 | ERROR | Expected 1 space after "="; 0 found
232 | ERROR | Expected 1 space before "+"; 0 found
232 | ERROR | Expected 1 space after "+"; 0 found
233 | ERROR | Expected 1 space before "="; 0 found
233 | ERROR | Expected 1 space after "="; 0 found
235 | ERROR | Array indentation error, expected 4 spaces but found 3
236 | ERROR | Array indentation error, expected 4 spaces but found 3
237 | ERROR | Whitespace found at end of line
238 | ERROR | Expected 1 space before "="; 0 found
238 | ERROR | Expected 1 space after "="; 0 found
238 | ERROR | Expected 1 space before "<="; 0 found
238 | ERROR | Expected 1 space after "<="; 0 found
239 | ERROR | Line indented incorrectly; expected 4 spaces, found 2
240 | ERROR | Line indented incorrectly; expected 4 spaces, found 3
240 | ERROR | Array indentation error, expected 4 spaces but found 3
241 | ERROR | Line indented incorrectly; expected 4 spaces, found 3
241 | ERROR | Array indentation error, expected 4 spaces but found 3
242 | ERROR | Line indented incorrectly; expected 4 spaces, found 2
242 | ERROR | Whitespace found at end of line
245 | ERROR | Line indented incorrectly; expected 4 spaces, found 2
246 | ERROR | Line indented incorrectly; expected 4 spaces, found 2
246 | ERROR | Array indentation error, expected 4 spaces but found 2
247 | ERROR | Line indented incorrectly; expected 4 spaces, found 2
247 | ERROR | Array indentation error, expected 4 spaces but found 2
248 | ERROR | Line indented incorrectly; expected 4 spaces, found 2
248 | ERROR | Whitespace found at end of line
251 | ERROR | Line indented incorrectly; expected 4 spaces, found 2
252 | ERROR | Line indented incorrectly; expected 4 spaces, found 3
252 | ERROR | Array indentation error, expected 4 spaces but found 3
253 | ERROR | Line indented incorrectly; expected 4 spaces, found 3
253 | ERROR | Array indentation error, expected 4 spaces but found 3
254 | ERROR | Line indented incorrectly; expected 4 spaces, found 2
257 | ERROR | Line indented incorrectly; expected 4 spaces, found 2
258 | ERROR | Line indented incorrectly; expected 4 spaces, found 3
258 | ERROR | Array indentation error, expected 4 spaces but found 3
259 | ERROR | Line indented incorrectly; expected 4 spaces, found 3
259 | ERROR | Array indentation error, expected 4 spaces but found 3
260 | ERROR | Line indented incorrectly; expected 4 spaces, found 2
269 | ERROR | Line indented incorrectly; expected 4 spaces, found 2
270 | ERROR | Line indented incorrectly; expected 4 spaces, found 2
271 | ERROR | Line indented incorrectly; expected 6 spaces, found 2
272 | ERROR | Whitespace found at end of line
273 | ERROR | Line indented incorrectly; expected 4 spaces, found 2
274 | ERROR | Line indented incorrectly; expected 6 spaces, found 2
278 | ERROR | Line indented incorrectly; expected 4 spaces, found 2
285 | ERROR | Expected 1 space after "="; 0 found
286 | ERROR | Expected 1 space before "="; 0 found
286 | ERROR | Expected 1 space after "="; 0 found
287 | ERROR | Expected 1 space before "="; 0 found
287 | ERROR | Expected 1 space after "="; 0 found
289 | ERROR | Line indented incorrectly; expected 4 spaces, found 2
289 | ERROR | Expected 1 space before "="; 0 found
289 | ERROR | Expected 1 space after "="; 0 found
290 | ERROR | Line indented incorrectly; expected 6 spaces, found 2
291 | ERROR | Line indented incorrectly; expected 6 spaces, found 2
292 | ERROR | Line indented incorrectly; expected 6 spaces, found 2
293 | ERROR | Line indented incorrectly; expected 6 spaces, found 2
296 | ERROR | Files must end in a single new line character
--------------------------------------------------------------------------------

FILE: /var/www/drupal-7-pareview/pareview_temp/css/carousel.css
--------------------------------------------------------------------------------
FOUND 29 ERROR(S) AFFECTING 28 LINE(S)
--------------------------------------------------------------------------------
7 | ERROR | Expected 1 space after colon in style definition; 0 found
8 | ERROR | Expected 1 space after colon in style definition; 0 found
12 | ERROR | Expected 1 space after colon in style definition; 0 found
13 | ERROR | Expected 1 space after colon in style definition; 0 found
14 | ERROR | Expected 1 space after colon in style definition; 0 found
15 | ERROR | Expected 1 space after colon in style definition; 0 found
16 | ERROR | Expected 1 space after colon in style definition; 0 found
17 | ERROR | Expected 1 space after colon in style definition; 0 found
21 | ERROR | Expected 1 space after colon in style definition; 0 found
22 | ERROR | Expected 1 space after colon in style definition; 0 found
26 | ERROR | Expected 1 space after colon in style definition; 0 found
27 | ERROR | Expected 1 space after colon in style definition; 0 found
31 | ERROR | Expected 1 space after colon in style definition; 0 found
32 | ERROR | Expected 1 space after colon in style definition; 0 found
33 | ERROR | Expected 1 space after colon in style definition; 0 found
34 | ERROR | Whitespace found at end of line
35 | ERROR | Line indented incorrectly; expected 0 spaces, found 2
35 | ERROR | Whitespace found at end of line
37 | ERROR | Expected 1 space after colon in style definition; 0 found
38 | ERROR | Expected 1 space after colon in style definition; 0 found
42 | ERROR | Expected 1 space after colon in style definition; 0 found
46 | ERROR | Expected 1 space after colon in style definition; 0 found
47 | ERROR | Expected 1 space after colon in style definition; 0 found
54 | ERROR | Expected 1 space after colon in style definition; 0 found
55 | ERROR | Expected 1 space after colon in style definition; 0 found
59 | ERROR | Expected 1 space after colon in style definition; 0 found
60 | ERROR | Expected 1 space after colon in style definition; 0 found
61 | ERROR | Additional whitespace found at end of file
62 | ERROR | Files must end in a single new line character
--------------------------------------------------------------------------------

FILE: /var/www/drupal-7-pareview/pareview_temp/js/Carousel.js
--------------------------------------------------------------------------------
FOUND 3 ERROR(S) AFFECTING 3 LINE(S)
--------------------------------------------------------------------------------
1 | ERROR | Additional whitespace found at start of file
6 | ERROR | Whitespace found at end of line
52 | ERROR | Files must end in a single new line character
--------------------------------------------------------------------------------

FILE: ...var/www/drupal-7-pareview/pareview_temp/js/jquery.waterwheelCarousel.js
--------------------------------------------------------------------------------
FOUND 57 ERROR(S) AFFECTING 45 LINE(S)
--------------------------------------------------------------------------------
1 | ERROR | Additional whitespace found at start of file
6 | ERROR | Whitespace found at end of line
136 | ERROR | Expected 1 space before "-"; 0 found
136 | ERROR | Expected 1 space after "-"; 0 found
137 | ERROR | Expected 1 space before "-"; 0 found
137 | ERROR | Expected 1 space after "-"; 0 found
138 | ERROR | Expected 1 space before "-"; 0 found
138 | ERROR | Expected 1 space after "-"; 0 found
142 | ERROR | Expected 1 space before "+"; 0 found
142 | ERROR | Expected 1 space after "+"; 0 found
144 | ERROR | Expected 1 space before "+"; 0 found
144 | ERROR | Expected 1 space after "+"; 0 found
207 | ERROR | Expected 1 space before "-"; 0 found
207 | ERROR | Expected 1 space after "-"; 0 found
208 | ERROR | Expected 1 space before "-"; 0 found
208 | ERROR | Expected 1 space after "-"; 0 found
210 | ERROR | Expected 1 space before "-"; 0 found
210 | ERROR | Expected 1 space after "-"; 0 found
211 | ERROR | Expected 1 space before "-"; 0 found
211 | ERROR | Expected 1 space after "-"; 0 found
219 | ERROR | Expected 1 space before "*"; 0 found
219 | ERROR | Expected 1 space after "*"; 0 found
362 | ERROR | Whitespace found at end of line
411 | ERROR | Comments may not appear after statements.
412 | ERROR | Comments may not appear after statements.
507 | ERROR | Spaces must be used to indent lines; tabs are not allowed
511 | ERROR | Spaces must be used to indent lines; tabs are not allowed
518 | ERROR | Comments may not appear after statements.
519 | ERROR | Comments may not appear after statements.
520 | ERROR | Comments may not appear after statements.
521 | ERROR | Comments may not appear after statements.
522 | ERROR | Comments may not appear after statements.
523 | ERROR | Comments may not appear after statements.
524 | ERROR | Comments may not appear after statements.
525 | ERROR | Comments may not appear after statements.
526 | ERROR | Comments may not appear after statements.
526 | ERROR | Whitespace found at end of line
527 | ERROR | Comments may not appear after statements.
528 | ERROR | Comments may not appear after statements.
529 | ERROR | Comments may not appear after statements.
530 | ERROR | Comments may not appear after statements.
531 | ERROR | Comments may not appear after statements.
532 | ERROR | Comments may not appear after statements.
533 | ERROR | Comments may not appear after statements.
534 | ERROR | Comments may not appear after statements.
535 | ERROR | Comments may not appear after statements.
536 | ERROR | Comments may not appear after statements.
537 | ERROR | Comments may not appear after statements.
538 | ERROR | Comments may not appear after statements.
538 | ERROR | Whitespace found at end of line
539 | ERROR | Comments may not appear after statements.
540 | ERROR | Comments may not appear after statements.
541 | ERROR | Comments may not appear after statements.
542 | ERROR | Comments may not appear after statements.
543 | ERROR | Comments may not appear after statements.
544 | ERROR | Comments may not appear after statements.
545 | ERROR | Comments may not appear after statements.
--------------------------------------------------------------------------------
alexmoreno’s picture

alexmoreno CreditAttribution: alexmoreno commented

also help to review other modules, like myself :-): https://drupal.org/node/2083527

and add your reviews in your module's description.

PA robot’s picture

PA robot CreditAttribution: PA robot commented
Multiple Applications
It appears that there have been multiple project applications opened under your username:

Project 1: https://drupal.org/node/2083347

Project 2: https://drupal.org/node/2081439

Project 3: https://drupal.org/node/2045689

As successful completion of the project application process results in the applicant being granted the 'Create Full Projects' permission, there is no need to take multiple applications through the process. Once the first application has been successfully approved, then the applicant can promote other projects without review. Because of this, posting multiple applications is not necessary, and results in additional workload for reviewers ... which in turn results in longer wait times for everyone in the queue. With this in mind, your secondary applications have been marked as 'closed(duplicate)', with only one application left open (chosen at random).

If you prefer that we proceed through this review process with a different application than the one which was left open, then feel free to close the 'open' application as a duplicate, and re-open one of the project applications which had been closed.

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

tibezh’s picture

tibezh CreditAttribution: tibezh commented
Status: Needs review » Needs work

Ciao fziviello!

README.txt
Please take a moment to make your README.txt follow the guidelines for in-project documentation.
Master Branch
It appears you are working in the "master" branch in git. You should really be working in a version specific branch. The most direct documentation on this is Moving from a master branch to a version branch. For additional resources please see the documentation about release naming conventions and creating a branch in git.

Other:

  1. Project page is too short, see https://drupal.org/node/997024
  2. Your .info file does not correct (https://drupal.org/node/542202) please check again.
  3. It should be used only English in module.

Kind regards

PA robot’s picture

PA robot CreditAttribution: PA robot commented
Issue summary: View changes
Status: Needs work » Closed (won't fix)

Closing due to lack of activity. Feel free to reopen if you are still working on this application (see also the project application workflow).

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

apaderno’s picture

apaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
CreditAttribution: apaderno as a volunteer commented
Assigned: fziviello » Unassigned