Closed (fixed)
Project:
BigBlueButton
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
27 Nov 2012 at 11:12 UTC
Updated:
10 Jan 2013 at 12:58 UTC
Jump to comment: Most recent file
Comments
Comment #1
scottm316 commentedThis is done using suggestions from the coder module. Code changes based on notices and warnings found at or above "minor" severity level in the coder module. Mostly just a whitespace and commenting cleanup, but it would be nice to move forward writing patches against the cleaner code.
Comment #2
scottm316 commentedJust testing another patch.
Comment #4
scottm316 commented#1: bbb-coder-standards-1851622-1.patch queued for re-testing.
Comment #6
scottm316 commentedRe-rolled using the updated code from earlier. Mostly just spacing issues throughout.
Comment #7
Ash07 commentedTesting it with Coder only give me warnings just in bbb.module, here is my patch with corrections.
Comment #9
Ash07 commentedComment #10
Ash07 commentedSorry for the previous post
Comment #11
Ash07 commentedComment #12
scottm316 commentedI've applied the patch in #6 and it is now part of the latest 7.x-1.x-dev release. Any further patches written against *this* release (Jan 10/2013) are super appreciated. I'm going to close this issue. Feel free to re-open or (preferably) open a new issue for Drupal coding standards at a later time following some more dev.
Comment #12.0
scottm316 commentedAdded another filename.