I ran a code review using Coder and it turned up a handful of minor coding standards issues.

CommentFileSizeAuthor
#2 fix_minor_coding-2658598-2.patch3.55 KBrlhawk
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

rlhawk created an issue. See original summary.

rlhawk’s picture

rlhawk
Primary language English
Location Seattle, Washington, United States
CreditAttribution: rlhawk commented
FileSize
fix_minor_coding-2658598-2.patch3.55 KB

Here's a patch that addresses the items mentioned by the code review.

rlhawk’s picture

rlhawk
Primary language English
Location Seattle, Washington, United States
CreditAttribution: rlhawk commented
Status: Active » Needs review
rlhawk’s picture

rlhawk
Primary language English
Location Seattle, Washington, United States
CreditAttribution: rlhawk commented
Version: 7.x-2.x-dev » 7.x-3.x-dev

Fixing which version this patch is for.

  • rlhawk committed 31c56be on 
    Issue #2658598 by rlhawk: Fix minor coding standards issues
rlhawk’s picture

rlhawk
Primary language English
Location Seattle, Washington, United States
CreditAttribution: rlhawk commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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