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 spaces

sites/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 spaces

sites/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 spaces

sites/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 spaces

sites/all/modules/jplayer/jplayer.install:
+12: [minor] There should be no trailing spaces
+25: [minor] There should be no trailing spaces

Status Messages:
Coder found 1 projects, 8 files, 1 critical warnings, 6 normal warnings, 68 minor warnings, 0 warnings were flagged to be ignored

CommentFileSizeAuthor
#1 1187020.1-coder-review.patch15.63 KBdeviantintegral
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

deviantintegral’s picture

Status: Active » Needs review
FileSize
15.63 KB
deviantintegral’s picture

Status: Needs review » Fixed

I rebased and committed this in bb824e3.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.