- Remove "version" from the ./views_slideshow_j360.info file, it will be added by drupal.org packaging automatically.
- Coder Sniffer has found some issues with your code (please check the Drupal coding standards). See attachment.
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: ...BeansProjects/Drupal7/sites/all/modules/views_slideshow_j360/README.txt
--------------------------------------------------------------------------------
FOUND 1 ERROR(S) AFFECTING 1 LINE(S)
--------------------------------------------------------------------------------
168 | ERROR | Files must end in a single new line character
--------------------------------------------------------------------------------
FILE: ...ll/modules/views_slideshow_j360/views-slideshow-j360-main-frame.tpl.php
--------------------------------------------------------------------------------
FOUND 26 ERROR(S) AND 2 WARNING(S) AFFECTING 23 LINE(S)
--------------------------------------------------------------------------------
17 | ERROR | The control statement should use the ":" alternative syntax
| | instead of curly braces in template files
24 | ERROR | Spaces must be used to indent lines; tabs are not allowed
25 | ERROR | Spaces must be used to indent lines; tabs are not allowed
25 | ERROR | The control statement should use the ":" alternative syntax
| | instead of curly braces in template files
26 | ERROR | Spaces must be used to indent lines; tabs are not allowed
26 | ERROR | The control statement should use the ":" alternative syntax
| | instead of curly braces in template files
27 | ERROR | Spaces must be used to indent lines; tabs are not allowed
27 | ERROR | The control statement should use the ":" alternative syntax
| | instead of curly braces in template files
28 | ERROR | Spaces must be used to indent lines; tabs are not allowed
29 | ERROR | Spaces must be used to indent lines; tabs are not allowed
30 | WARNING | Line exceeds 80 characters; contains 86 characters
31 | WARNING | Line exceeds 80 characters; contains 85 characters
33 | ERROR | Spaces must be used to indent lines; tabs are not allowed
33 | ERROR | The control statement should use the ":" alternative syntax
| | instead of curly braces in template files
34 | ERROR | Spaces must be used to indent lines; tabs are not allowed
35 | ERROR | Spaces must be used to indent lines; tabs are not allowed
36 | ERROR | Spaces must be used to indent lines; tabs are not allowed
37 | ERROR | Spaces must be used to indent lines; tabs are not allowed
38 | ERROR | Spaces must be used to indent lines; tabs are not allowed
39 | ERROR | Spaces must be used to indent lines; tabs are not allowed
40 | ERROR | Whitespace found at end of line
41 | ERROR | Spaces must be used to indent lines; tabs are not allowed
41 | ERROR | The control statement should use the ":" alternative syntax
| | instead of curly braces in template files
42 | ERROR | Spaces must be used to indent lines; tabs are not allowed
43 | ERROR | Spaces must be used to indent lines; tabs are not allowed
44 | ERROR | Spaces must be used to indent lines; tabs are not allowed
45 | ERROR | Spaces must be used to indent lines; tabs are not allowed
46 | ERROR | Spaces must be used to indent lines; tabs are not allowed
--------------------------------------------------------------------------------
FILE: ...Drupal7/sites/all/modules/views_slideshow_j360/views_slideshow_j360.css
--------------------------------------------------------------------------------
FOUND 4 ERROR(S) AFFECTING 2 LINE(S)
--------------------------------------------------------------------------------
5 | ERROR | Line indented incorrectly; expected 2 spaces, found 1
5 | ERROR | Spaces must be used to indent lines; tabs are not allowed
8 | ERROR | Line indented incorrectly; expected 2 spaces, found 1
8 | ERROR | Spaces must be used to indent lines; tabs are not allowed
--------------------------------------------------------------------------------
FILE: ...rupal7/sites/all/modules/views_slideshow_j360/views_slideshow_j360.info
--------------------------------------------------------------------------------
FOUND 1 ERROR(S) AFFECTING 1 LINE(S)
--------------------------------------------------------------------------------
6 | ERROR | Files must end in a single new line character
--------------------------------------------------------------------------------
FILE: .../Drupal7/sites/all/modules/views_slideshow_j360/views_slideshow_j360.js
--------------------------------------------------------------------------------
FOUND 117 ERROR(S) AFFECTING 92 LINE(S)
--------------------------------------------------------------------------------
10 | ERROR | Spaces must be used to indent lines; tabs are not allowed
11 | ERROR | Spaces must be used to indent lines; tabs are not allowed
12 | ERROR | Spaces must be used to indent lines; tabs are not allowed
13 | ERROR | Spaces must be used to indent lines; tabs are not allowed
14 | ERROR | Spaces must be used to indent lines; tabs are not allowed
15 | ERROR | Whitespace found at end of line
15 | ERROR | Spaces must be used to indent lines; tabs are not allowed
16 | ERROR | Spaces must be used to indent lines; tabs are not allowed
17 | ERROR | Spaces must be used to indent lines; tabs are not allowed
18 | ERROR | Spaces must be used to indent lines; tabs are not allowed
19 | ERROR | Whitespace found at end of line
19 | ERROR | Spaces must be used to indent lines; tabs are not allowed
20 | ERROR | Spaces must be used to indent lines; tabs are not allowed
21 | ERROR | Spaces must be used to indent lines; tabs are not allowed
22 | ERROR | Spaces must be used to indent lines; tabs are not allowed
23 | ERROR | Spaces must be used to indent lines; tabs are not allowed
24 | ERROR | Spaces must be used to indent lines; tabs are not allowed
25 | ERROR | Spaces must be used to indent lines; tabs are not allowed
26 | ERROR | Spaces must be used to indent lines; tabs are not allowed
26 | ERROR | There should be no white space after an opening "("
26 | ERROR | There should be no white space before a closing ")"
27 | ERROR | Spaces must be used to indent lines; tabs are not allowed
28 | ERROR | Spaces must be used to indent lines; tabs are not allowed
29 | ERROR | Whitespace found at end of line
29 | ERROR | Spaces must be used to indent lines; tabs are not allowed
30 | ERROR | Spaces must be used to indent lines; tabs are not allowed
31 | ERROR | Spaces must be used to indent lines; tabs are not allowed
32 | ERROR | Spaces must be used to indent lines; tabs are not allowed
33 | ERROR | Spaces must be used to indent lines; tabs are not allowed
34 | ERROR | Spaces must be used to indent lines; tabs are not allowed
34 | ERROR | Comments may not appear after statements.
35 | ERROR | Spaces must be used to indent lines; tabs are not allowed
35 | ERROR | Comments may not appear after statements.
36 | ERROR | Spaces must be used to indent lines; tabs are not allowed
37 | ERROR | Spaces must be used to indent lines; tabs are not allowed
38 | ERROR | Whitespace found at end of line
38 | ERROR | Spaces must be used to indent lines; tabs are not allowed
39 | ERROR | Spaces must be used to indent lines; tabs are not allowed
40 | ERROR | Spaces must be used to indent lines; tabs are not allowed
41 | ERROR | Spaces must be used to indent lines; tabs are not allowed
42 | ERROR | Spaces must be used to indent lines; tabs are not allowed
43 | ERROR | Spaces must be used to indent lines; tabs are not allowed
44 | ERROR | Spaces must be used to indent lines; tabs are not allowed
44 | ERROR | Comments may not appear after statements.
45 | ERROR | Whitespace found at end of line
45 | ERROR | Spaces must be used to indent lines; tabs are not allowed
46 | ERROR | Spaces must be used to indent lines; tabs are not allowed
47 | ERROR | Spaces must be used to indent lines; tabs are not allowed
48 | ERROR | Spaces must be used to indent lines; tabs are not allowed
49 | ERROR | Spaces must be used to indent lines; tabs are not allowed
50 | ERROR | Spaces must be used to indent lines; tabs are not allowed
51 | ERROR | Spaces must be used to indent lines; tabs are not allowed
52 | ERROR | Whitespace found at end of line
52 | ERROR | Spaces must be used to indent lines; tabs are not allowed
53 | ERROR | Spaces must be used to indent lines; tabs are not allowed
54 | ERROR | Whitespace found at end of line
54 | ERROR | Spaces must be used to indent lines; tabs are not allowed
55 | ERROR | Spaces must be used to indent lines; tabs are not allowed
56 | ERROR | Spaces must be used to indent lines; tabs are not allowed
57 | ERROR | Spaces must be used to indent lines; tabs are not allowed
58 | ERROR | Whitespace found at end of line
58 | ERROR | Spaces must be used to indent lines; tabs are not allowed
59 | ERROR | Spaces must be used to indent lines; tabs are not allowed
59 | ERROR | Case breaking statements must be followed by a single blank line
60 | ERROR | Spaces must be used to indent lines; tabs are not allowed
61 | ERROR | Spaces must be used to indent lines; tabs are not allowed
62 | ERROR | Spaces must be used to indent lines; tabs are not allowed
63 | ERROR | Spaces must be used to indent lines; tabs are not allowed
64 | ERROR | Spaces must be used to indent lines; tabs are not allowed
65 | ERROR | Spaces must be used to indent lines; tabs are not allowed
65 | ERROR | Comments may not appear after statements.
66 | ERROR | Whitespace found at end of line
66 | ERROR | Spaces must be used to indent lines; tabs are not allowed
67 | ERROR | Spaces must be used to indent lines; tabs are not allowed
68 | ERROR | Whitespace found at end of line
68 | ERROR | Spaces must be used to indent lines; tabs are not allowed
69 | ERROR | Spaces must be used to indent lines; tabs are not allowed
69 | ERROR | Case breaking statements must be followed by a single blank line
70 | ERROR | Spaces must be used to indent lines; tabs are not allowed
71 | ERROR | Spaces must be used to indent lines; tabs are not allowed
72 | ERROR | Spaces must be used to indent lines; tabs are not allowed
73 | ERROR | Whitespace found at end of line
73 | ERROR | Spaces must be used to indent lines; tabs are not allowed
74 | ERROR | Spaces must be used to indent lines; tabs are not allowed
75 | ERROR | Spaces must be used to indent lines; tabs are not allowed
75 | ERROR | Comments may not appear after statements.
76 | ERROR | Spaces must be used to indent lines; tabs are not allowed
77 | ERROR | Spaces must be used to indent lines; tabs are not allowed
78 | ERROR | Spaces must be used to indent lines; tabs are not allowed
78 | ERROR | Comments may not appear after statements.
79 | ERROR | Spaces must be used to indent lines; tabs are not allowed
80 | ERROR | Whitespace found at end of line
80 | ERROR | Spaces must be used to indent lines; tabs are not allowed
81 | ERROR | Spaces must be used to indent lines; tabs are not allowed
82 | ERROR | Spaces must be used to indent lines; tabs are not allowed
83 | ERROR | Spaces must be used to indent lines; tabs are not allowed
84 | ERROR | Whitespace found at end of line
84 | ERROR | Spaces must be used to indent lines; tabs are not allowed
85 | ERROR | Spaces must be used to indent lines; tabs are not allowed
86 | ERROR | Spaces must be used to indent lines; tabs are not allowed
87 | ERROR | Spaces must be used to indent lines; tabs are not allowed
88 | ERROR | Spaces must be used to indent lines; tabs are not allowed
89 | ERROR | Spaces must be used to indent lines; tabs are not allowed
90 | ERROR | Spaces must be used to indent lines; tabs are not allowed
91 | ERROR | Spaces must be used to indent lines; tabs are not allowed
92 | ERROR | Spaces must be used to indent lines; tabs are not allowed
93 | ERROR | Whitespace found at end of line
93 | ERROR | Spaces must be used to indent lines; tabs are not allowed
94 | ERROR | Spaces must be used to indent lines; tabs are not allowed
96 | ERROR | Whitespace found at end of line
104 | ERROR | Whitespace found at end of line
104 | ERROR | Spaces must be used to indent lines; tabs are not allowed
105 | ERROR | Spaces must be used to indent lines; tabs are not allowed
106 | ERROR | Spaces must be used to indent lines; tabs are not allowed
107 | ERROR | Spaces must be used to indent lines; tabs are not allowed
108 | ERROR | Spaces must be used to indent lines; tabs are not allowed
109 | ERROR | Spaces must be used to indent lines; tabs are not allowed
--------------------------------------------------------------------------------
FILE: ...pal7/sites/all/modules/views_slideshow_j360/views_slideshow_j360.module
--------------------------------------------------------------------------------
FOUND 1 ERROR(S) AND 1 WARNING(S) AFFECTING 2 LINE(S)
--------------------------------------------------------------------------------
18 | WARNING | A comma should follow the last multiline array item. Found:
| | NULL
29 | ERROR | Data type of return value is missing
--------------------------------------------------------------------------------
FILE: ...7/sites/all/modules/views_slideshow_j360/views_slideshow_j360.theme.inc
--------------------------------------------------------------------------------
FOUND 0 ERROR(S) AND 1 WARNING(S) AFFECTING 1 LINE(S)
--------------------------------------------------------------------------------
35 | WARNING | A comma should follow the last multiline array item. Found: )
--------------------------------------------------------------------------------
FILE: ...l/modules/views_slideshow_j360/views_slideshow_j360.views_slideshow.inc
--------------------------------------------------------------------------------
FOUND 1 ERROR(S) AND 5 WARNING(S) AFFECTING 5 LINE(S)
--------------------------------------------------------------------------------
38 | ERROR | The $text argument to l() should be enclosed within t() so
| | that it is translatable
50 | WARNING | Line exceeds 80 characters; contains 88 characters
50 | WARNING | There must be no blank line following an inline comment
69 | WARNING | A comma should follow the last multiline array item. Found: )
70 | WARNING | A comma should follow the last multiline array item. Found: )
140 | WARNING | A comma should follow the last multiline array item. Found: )
--------------------------------------------------------------------------------