Project: Drupal Version: cvs Component: other Category: bug reports Priority: normal Assigned to: Goba Reported by: Goba Updated by: Goba Status: patch Attachment: http://drupal.org/files/issues/Drupal-do-not-drop-accesslog.patch (1.02 KB) The introduction of the access log id was made into an update, which drops the acceslog, and then recreates the table just to add a primary key. This is perfectly possible without loosing any data, and is documented in the MySQL docs. Patch attached. BTW the 'path' introduction before instead of using 'nid' was also buggy in the update, since it does not fill the 'path' values of the items, so all previous log items are invisible. That also needs to be fixed, but before it is fixed, I need input on the intended value of the 'path' field. In HEAD, statistics.module gets the path alias if available and stores that in the 'path' (line 77 currently). But then node/%d is matched with the path (line 174 currently), when a node access tracker is printed, so those nodes with path aliases are not working with it. It seems to me that the storage should be modified to not store the alias, but let it be computed when needed (eg. when printing links). But I am unsure, why was the alias stored. If this is fixed, we can get back to fixing the update to actually provide the 'path' value, before removing the 'nid', so that old accesslog items are still fully powering the site admin after the update. Goba -- View: http://drupal.org/node/15962 Edit: http://drupal.org/project/comments/add/15962