Clean-up the upgrade path: Aggregator.

From: Damien Tournoud <damien@commerceguys.com>


---
 aggregator/aggregator.install |   29 ++++++++++++++---------------
 1 files changed, 14 insertions(+), 15 deletions(-)

diff --git modules/aggregator/aggregator.install modules/aggregator/aggregator.install
index 8cc421b..33ed444 100644
--- modules/aggregator/aggregator.install
+++ modules/aggregator/aggregator.install
@@ -281,23 +281,16 @@ function aggregator_schema() {
 }
 
 /**
- * Add hash column to aggregator_feed table.
+ * Update the {aggregator_feed} table.
  */
 function aggregator_update_7000() {
-  db_add_field('aggregator_feed', 'hash', array('type' => 'varchar', 'length' => 64, 'not null' => TRUE, 'default' => ''));
-}
-
-/**
- * Add aggregator teaser length to settings from old global default teaser length
- */
-function aggregator_update_7001() {
-  variable_set('aggregator_teaser_length', variable_get('teaser_length'));
-}
-
-/**
- * Add queued timestamp.
- */
-function aggregator_update_7002() {
+  db_add_field('aggregator_feed', 'hash', array(
+    'type' => 'varchar',
+    'length' => 64,
+    'not null' => TRUE,
+    'default' => '',
+    'description' => 'Calculated hash of the feed data, used for validating cache.',
+  ));
   db_add_field('aggregator_feed', 'queued', array(
     'type' => 'int',
     'not null' => TRUE,
@@ -307,3 +300,9 @@ function aggregator_update_7002() {
   db_add_index('aggregator_feed', 'queued', array('queued'));
 }
 
+/**
+ * Add aggregator teaser length to settings from old global default teaser length.
+ */
+function aggregator_update_7001() {
+  variable_set('aggregator_teaser_length', variable_get('teaser_length'));
+}