Code style cleanup

deekayen - August 24, 2009 - 23:03
Project:Scheduler
Version:6.x-1.x-dev
Component:Code
Category:bug report
Priority:minor
Assigned:Unassigned
Status:closed
Description

I processed the source with Coder and updated the layout of the .install file.

The highlights: It removes the obsolete hook_help() that was used before .info files provided the module description on the modules page, switches update functions to use schema api, removes trailing whitespaces, and adds @file docblocks according to webchick's guidelines in #263956: Ye Grande Olde Drupal 7 .install File Cleanup.

AttachmentSize
scheduler_code_style.patch18.02 KB

#1

Eric Schaefer - August 25, 2009 - 19:36

Looks good to me. Did you test the update functions?

#2

deekayen - August 25, 2009 - 20:03

no

#3

Eric Schaefer - August 26, 2009 - 17:42
Status:needs review» fixed

http://drupal.org/cvs?commit=256408

Thanks.

#4

Eric Schaefer - August 26, 2009 - 17:42

BTW: The update works...

#5

System Message - September 9, 2009 - 17:50
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.