Code style issues

chx - June 7, 2008 - 04:54
Project:Plugin Manager
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed
Description

At a cursory glance, i saw } else { this needs to be on separate lines.

#1

cwgordon7 - June 11, 2008 - 17:35
Status:active» patch (code needs review)

Here's a patch.

#2

cwgordon7 - June 11, 2008 - 17:37

Er... there was a patch at least. :P Since comment upload seems to be malfunctioning, here's an off-site patch: http://cwgordon.com/sites/default/files/plugin_manager_code_style_01.pat...

#3

JoshuaRogers - June 20, 2008 - 01:28
Status:patch (code needs review)» fixed

I applied your patch. It looks great. I will try to spend a little bit more time familiarizing myself with Drupal's code styling.

Thank you for the patch. (And sorry about my long delay.)

#4

Anonymous (not verified) - July 4, 2008 - 01:32
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.