Local cache is broken

The option to cache urchin.js locally has a few problems.

1. For sites that run cron more than once per day, the local cache file is never updated.

function googleanalytics_cron() {
  // Regenerate the google analytics urchin.js every day.
  if (time() - variable_get('cron_last', 0) >= 86400) {
    file_delete(file_directory_path() .'/googleanalytics/urchin.js');
  }
}

The if statement is always false if cron runs more than once per day. It should be:

  function googleanalytics_cron() {
    // Regenerate the google analytics urchin.js every day.
!   if (time() - variable_get('googleanalytics_last_cache', 0) >= 86400) {
      file_delete(file_directory_path() .'/googleanalytics/urchin.js');
+     variable_set('googleanalytics_last_cache', time());
    }
  }
 

2. If local caching is enabled, downloadtracker.js is also linked from the cache but it is never downloaded (and never cleaned from the cache).