Ye Grande Olde Drupal 7 .install File Cleanup

Point 1 is pending here : #320011: Add doxygen @file for all .install

I'm in a mood to work on this, so I'll just leave out @file part.

I added a couple defgroups and moved hooks around.

I mark issue #320011: Add doxygen @file for all .install as duplicated.

Add tag.

@webchick: Less phone calls, less crazy ideas. :P

To get this done, I'd propose to do one file at a time. Equaling: One phone call of webchick.

.

I'll try to look into this over the next couple of days

Ok. From running through the files:

• Block module already conforms
• Blog module has no install file
• Color module already conforms (Only has color_requirements)
• Of the Field submodules, only field_sql_storage has an install file
• Filter module has no install() or uninstall()
• Help module has no install file
• Node module has no install() or uninstall()
• Path module has no install file
• PHP module already conforms
• Syslog module has no install file
• Tracker module has no install file
• Translation module has no install file
• User module already conforms

I will attach individual patches - I can't attach multiple files to one comment!

Blog API module

Book Module

Comment Module

Contact Module

DB Log Module

Field Module

Field SQL Storage Module

Forum Module

Locale Module

Menu Module

Open ID Module

Poll Module

Profile Module

Search Module

SimpleTest Module

Statistics Module

System Module

Taxonomy Module

Trigger Module

Update Module

Upload Module

Ye Grand Olde Massive rollup patch. Includes all my patches above, plus suns patch to Aggregator in #9

Wooohooo! :)

@wretched sinner: Unfortunately, there is one thing missing in your patches - the leading @file and the @ingroup PHPDoc statements (both and after a set of updates - have a look at my Aggregator patch to see what I mean).

Another important thing is that we also want to move hook_requirements from the .module into the .install file, if existent.

@webchick: By the way, what about hook_enable()/_disable() ?

@sun patch : don't change @file description in aggregator module (see #320011: Add doxygen @file for all .install, http://drupal.org/node/224333#afile and coding standard)

 /**
  * @file
- * Install, update and uninstall functions for the aggregator module.
+ * Installation functions for Aggregator module.
  */

If webchick can chime in on whether we want to have the hook_enable() and hook_disable() merged in, then I will look to re-roll with the comments above.

There are several other issues in the queue that tackle this now: http://drupal.org/project/issues/search/drupal?issue_tags=coding%20stand...