AaronVanGeffen / AwstatsParser

PHP script for parsing and merging awstats data files
BSD 2-Clause "Simplified" License
11 stars 6 forks source link

Fix 4 problems with current awparse.php #1

Closed DrDaveD closed 12 years ago

DrDaveD commented 12 years ago
  1. Make protected "$data" variable into public because otherwise php 5.1.6 complains access to protected data on line 161.
  2. Instead of only taking the first copy of every item in the GENERAL section, merge the FirstTime (min), LastTime (max), LastUpdate (max), and TotalVisits (sum) items since these values are displayed on the awstats web page.
  3. Fix bug that caused quite a bit of data to be incorrectly merged due to data references of [$key][$key] instead of [$section_name][$key]
  4. Rather than summing all indexes for VISITOR and EXTRA_1 sections, treat 3rd & 4th indexes as dates (take max) and treat 5th as a string (take the first one).
AaronVanGeffen commented 12 years ago

Thanks for the input! I love how making a script open source ends up in people picking it up, sooner or later.