Severity minor
sites/all/modules/jplayer/jplayer.module:
+90: [minor] There should be no trailing spaces
+101: [minor] There should be no trailing spaces
+107: [minor] There should be no trailing spaces
+117: [minor] There should be no trailing spaces
+119: [minor] There should be no trailing spaces
+120: [normal] Control statements should have one space between the control keyword and opening parenthesis
+130: [minor] There should be no trailing spaces
+140: [minor] There should be no trailing spaces
+150: [minor] There should be no trailing spaces
+161: [minor] There should be no trailing spaces
+171: [minor] There should be no trailing spaces
+181: [minor] There should be no trailing spaces
+192: [minor] There should be no trailing spaces
+202: [minor] There should be no trailing spaces
+211: [minor] There should be no trailing spaces
+223: [minor] There should be no trailing spaces
+225: [minor] There should be no trailing spaces
+234: [minor] There should be no trailing spaces
+263: [minor] There should be no trailing spaces
+267: [minor] There should be no trailing spaces
+277: [minor] There should be no trailing spaces
+280: [minor] There should be no trailing spaces
+284: [minor] There should be no trailing spaces
+286: [minor] There should be no trailing spaces
+310: [minor] There should be no trailing spacessites/all/modules/jplayer/includes/jplayer.admin.inc:
+12: [minor] There should be no trailing spaces
+14: [minor] There should be no trailing spaces
+15: [minor] There should be no trailing spaces
+16: [minor] There should be no trailing spaces
+17: [minor] There should be no trailing spaces
+20: [minor] There should be no trailing spaces
+27: [minor] There should be no trailing spaces
+29: [minor] There should be no trailing spaces
+30: [minor] There should be no trailing spaces
+31: [minor] There should be no trailing spaces
+32: [minor] There should be no trailing spaces
+35: [minor] There should be no trailing spaces
+45: [minor] There should be no trailing spaces
+55: [minor] There should be no trailing spaces
+65: [minor] There should be no trailing spaces
+75: [minor] There should be no trailing spaces
+85: [minor] There should be no trailing spaces
+95: [minor] There should be no trailing spaces
+102: [minor] There should be no trailing spaces
+109: [minor] There should be no trailing spaces
+116: [minor] There should be no trailing spaces
+119: [minor] There should be no trailing spacessites/all/modules/jplayer/includes/jplayer.theme.inc:
+12: [minor] There should be no trailing spaces
+14: [minor] There should be no trailing spaces
+17: [minor] There should be no trailing spaces
+35: [minor] There should be no trailing spaces
+99: [minor] There should be no trailing spaces
+104: [minor] There should be no trailing spaces
+108: [minor] There should be no trailing spaces
+110: [normal] Control statements should have one space between the control keyword and opening parenthesis
+115: [minor] There should be no trailing spaces
+121: [minor] There should be no trailing spaces
+131: [minor] There should be no trailing spaces
+137: [normal] Use "elseif" in place of "else if"
+141: [normal] Use "elseif" in place of "else if"
+145: [minor] There should be no trailing spaces
+148: [minor] There should be no trailing spaces
+157: [normal] Use "elseif" in place of "else if"
+163: [minor] There should be no trailing spaces
+170: [normal] Use "elseif" in place of "else if"
+180: [minor] There should be no trailing spaces
+186: [minor] There should be no trailing spaces
+198: [minor] There should be no trailing spaces
+200: [minor] There should be no trailing spaces
+203: [minor] There should be no trailing spaces
+221: [minor] There should be no trailing spacessites/all/modules/jplayer/includes/jplayer_style_plugin.inc:
+54: [critical] table names should be enclosed in {curly_brackets}
+68: [minor] There should be no trailing spacessites/all/modules/jplayer/jplayer.install:
+12: [minor] There should be no trailing spaces
+25: [minor] There should be no trailing spacesStatus Messages:
Coder found 1 projects, 8 files, 1 critical warnings, 6 normal warnings, 68 minor warnings, 0 warnings were flagged to be ignored
Comment | File | Size | Author |
---|---|---|---|
#1 | 1187020.1-coder-review.patch | 15.63 KB | deviantintegral |
Comments
Comment #1
deviantintegral CreditAttribution: deviantintegral commentedComment #2
deviantintegral CreditAttribution: deviantintegral commentedI rebased and committed this in bb824e3.