Index: zeitgeist.module
===================================================================
RCS file: /cvs/drupal-contrib/contributions/modules/zeitgeist/zeitgeist.module,v
retrieving revision 1.9.6.2
diff -u -p -r1.9.6.2 zeitgeist.module
--- zeitgeist.module	14 Oct 2007 16:52:53 -0000	1.9.6.2
+++ zeitgeist.module	22 Jan 2008 13:31:46 -0000
@@ -89,7 +89,7 @@ define('ZGBLOCKTOP',       1);
 
 // Module paths
 define('ZGPATHSETTINGS',   'admin/settings/zeitgeist');
-define('ZGPATHPAGE',       'admin/logs/zeitgeist');
+define('ZGPATHPAGE',       'admin/reports/zeitgeist');
 
 /**
  * implement hook_help
@@ -129,7 +129,7 @@ function zeitgeist_menu()
   {
   $items[ZGPATHSETTINGS] = array
     (
-    'title'            => 'Zeitgeist',
+    'title'            => 'Zeitgeist settings',
     'description'      => 'Define the various parameters used by the Zeitgeist module',
     'page callback'    => 'drupal_get_form',
     'page arguments'   => array('zeitgeist_admin_settings'),
@@ -139,7 +139,7 @@ function zeitgeist_menu()
 
   $items[ZGPATHPAGE] = array
     (
-    'title'              => t('Zeitgeist'),
+    'title'              => t('Zeitgeist report'),
     'description'        => t('Feel the Zeitgeist'),
     'page callback'      => 'zeitgeist_page',
     'type'               => MENU_NORMAL_ITEM,
Index: zeitgeist.install
===================================================================
RCS file: /cvs/drupal-contrib/contributions/modules/zeitgeist/Attic/zeitgeist.install,v
retrieving revision 1.1.6.3
diff -u -p -r1.1.6.3 zeitgeist.install
--- zeitgeist.install	14 Oct 2007 16:52:53 -0000	1.1.6.3
+++ zeitgeist.install	22 Jan 2008 13:31:46 -0000
@@ -21,13 +21,40 @@ function zeitgeist_schema()
     (
     'fields' => array
       (
-      'search'    => array('type' => 'varchar', 'length' => 80, 'not null' => true, 'default' => ''),
-      'category'  => array('type' => 'varchar', 'length' => 64, 'not null' => true, 'default' => 'node'),
-      'ts'        => array('type' => 'int', 'unsigned' => true, 'not null' => true, 'default' => 0),
-      ),
-    'primary key' => array(),
-    'unique keys' => array(),
-    'indexes' => array('search' => array('search')),
+      'search'      => array(
+        'type'        => 'varchar', 
+        'length'      => 80, 
+        'not null'    => true, 
+        'default'     => '',
+        'description' => t('The query string. Will statistically often be empty is ZG is allowed to log empty searches.'),
+        ),
+      'category'    => array(
+        'type'        => 'varchar', 
+        'length'      => 64, 
+        'not null'    => true, 
+        'default'     => 'node',
+        'description' => t('The search category. Core categories are "node" and "user".'), 
+        ),
+      'ts'          => array(
+        'type'        => 'int', 
+        'unsigned'    => true, 
+        'not null'    => true, 
+        'default'     => 0,
+        'description' => t('The timestamp of search submission. Duplicates <em>do</em> happen due to timer granularity.'),
+        ),
+      ),
+    /**
+     * ZG does not use a PK, because the timer granularity can actually cause duplicates,
+     * and there is no actual need to differentiate them. The alternatives would be heavier:
+     * adding an occurrence count on all rows, and update it instead of adding a row in the
+     * rare case when a duplicate occurs. This would be heavy on every request, with no obvious gain
+     * An additional solution would be to add an autoincrement column to differentiate otherwise
+     * identical rows, again to no actual benefit.
+     */    
+    // 'primary key' => array(),
+    'unique keys'   => array(),
+    'indexes'       => array('search' => array('search')),
+    'description'   => t('The search logging table for the Zeitgeist module.'), 
     );
 
   return $schema;