Feedback on PageStatsPlugin

"inspired" by BBClone, a clone of Big Brother Webstats (aka BBStats)

simple implementation ( it uses grep to search the log files, so it's not exactly a speed demon )

i would prefer create links at the bottom of pages, which would lead to another page with the list of views and saves, but i don't know how to set that up...

-- WillNorris - 25 Oct 2003

I did some editing of the Plugins topic, feel free to take this into the next release.

Feedback from a quick look at the code:

• the Plugin uses undocumented TWiki functions like TWiki::extractNameValuePair. Use the TWiki::Func equivalents
• grep is called directly with backticks. Not all systems have grep in the path. You could use $TWiki::egrepCmd
• the Plugin assumes $TWiki::dataDir/log*.txt for the log files. The latest TWiki version has a log dir setting, the default is the data dir, but an admin might change that. Better to parse $TWiki::logFilename", whcih is ="$logDir/log07 Sep 2008.txt" by default

You could create a generic TWiki04x02.WebTopicStatsDisplay topic that contains a %PAGESTATS{ web="%URLPARAM{web}%" topic="%URLPARAM{topic}%" }% line, then add a [[%SCRIPTURL%/view%SCRIPTSUFFIX%/%TWIKIWEB%/WebTopicStatsDisplay?web=%WEB%&topic=%TOPIC%][page stats]] link in your view template (or skin)

-- PeterThoeny - 25 Oct 2003

Oh.

"grep is called directly with backticks"

-- AntonAylward - 25 Nov 2004

of course we do

-- WillNorris - 25 Nov 2004

checked .zip into CVS

-- WillNorris - 19 Jul 2005