I know that this module is still in active development, but I thought a nice round of code cleanup per-

http://drupal.org/node/1354

was in order. There's definitely more that can be done, so this can just be a starting point.

CommentFileSizeAuthor
commerce_stock-clean_up.patch34.47 KBalbert volkman

Comments

guy_schneerson’s picture

Status: Active » Needs review

Hi Albert Volkman,
Yes I am aware of the issue but didn't have the time (Going to review my sandbox modules soon) I got the Drupal CodeSniffer integrated into my dev environment so all future code should be clean :)

This is very much appreciated and the patch looks great.

guy_schneerson’s picture

Status: Needs review » Fixed

went over all changes and all look to be comments and code formatting and change no code. also gave a quick test and all looks good.
But this is a big patch so lets keep an eye.

committed http://drupalcode.org/project/commerce_stock.git/commit/3bee6f1

Status: Fixed » Closed (fixed)

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

Anonymous’s picture

Issue summary: View changes

Link didn't get created