Comments for blog.kungf.eu

Comment on Colour Gradient PHP Class by Todd Cesere

Todd Cesere — Wed, 03 Mar 2010 20:07:55 +0000

I got (and fixed – so that I could use it) a bunch of parse errors because of spaces in less than or equal to tokens. In other words, “< =" instead of "<=".

Comment on Colour Gradient PHP Class by Skapa en ‘heatmap’ för fastighetspriser med Google maps | Payema

Thu, 04 Feb 2010 12:27:44 +0000

[...] hittade en php-class som heter Colour gradient class som på ett enkelt sätt skapar en färggradient array i hundra steg, baserat på vilka färger man [...]

Comment on Wordpress 2.1 yay… oh wait… by borg

borg — Tue, 12 Jan 2010 11:58:26 +0000

mmm….thanks for info

Comment on Moved To A New Host by borg

borg — Tue, 12 Jan 2010 11:57:46 +0000

hm///useful..

Comment on Mac OS X Activity Monitor – why have you forsaken me? by borg

borg — Tue, 12 Jan 2010 11:55:58 +0000

as for me I use ActyMac DutyWatch

Comment on Apache Log Parser Class by hamish

hamish — Wed, 01 Apr 2009 13:43:05 +0000

hamish : - $quotes = preg_match(’/^\\\”/’, $element) ? true : false; + $quotes = preg_match(’/^\\”/’, $element) ? true : false; - array(’/^\\\”/’, ‘/\\\”$/’), + array(’/^\\”/’, ‘/\\”$/’), - if(!preg_match($time_format, $time, $matches) + if(!preg_match($time_format, $time, $m)

Of the changes suggested above, I agree with the last. There is clearly a mistake here that will stop the time stamps from being parsed. The other two however, I am not so sure. In both cases it is matching the quotes on the element string. Eg: \"%{User-Agent}i\" But the quotes must be escaped in the httpd.conf so we must match \". The \ (backslash) is the escape character for both PHP strings and regular expressions, so it must be escaped twice. Therefore I think the correct value here is ’/^\\\\”/ with four slashes.

Comment on Colour Gradient PHP Class by hamish

hamish — Mon, 30 Mar 2009 02:34:26 +0000

Okay I fixed the file containing the wrong class problem… and it only took me a year. :p

Comment on Apache Log Parser Class by hamish

hamish — Mon, 30 Mar 2009 02:19:54 +0000

Justin Randel; suggested to following patch, which I may get around to merging some time:

--- src/ApacheLogParser.php	2007-05-31 23:03:27.000000000 +1000
+++ ApacheLogParser.php	2009-01-08 20:26:48.000000000 +1100
@@ -137,12 +137,12 @@ class ApacheLogRegex {
 
         foreach(explode(' ', $this->_format) as $element)
         {
-            $quotes = preg_match('/^\\\"/', $element) ? true : false;
+            $quotes = preg_match('/^\\"/', $element) ? true : false;
 
             if($quotes)
             {
                 $element = preg_replace(
-                    array('/^\\\"/', '/\\\"$/'),
+                    array('/^\\"/', '/\\"$/'),
                     '',
                     $element
                 );
@@ -333,7 +333,7 @@ class ApacheLogRegex {
             .':([\d]{2}):([\d]{2}) ([\+\-])([\d]{2})([\d]{2})\]/';
 
         $m = array();    //matches
-        if(!preg_match($time_format, $time, $matches)
+        if(!preg_match($time_format, $time, $m)
                 || count($m) != 10)
             return null;
 
@@ -347,5 +347,3 @@ class ApacheLogRegex {
 
 } // end class ApacheLogRegex
 
-
-?>
\ No newline at end of file

Comment on Apache Log Parser Class by Adrien

Adrien — Fri, 06 Mar 2009 14:44:47 +0000

Hi, thanks for this usefull class, i use it for a statistics.

However there is an syntax error in your script that disable the logtime_to_timestamp method:
At line 384 (in version 1.2.1) you must set “$m” as parameter of “preg_match” instead of “$matches”.

So the code will look like (at line 384):
if(!preg_match($time_format, $time, $m)

Have a nice day!

Comment on Apache Log Parser Class by Lubo

Lubo — Sun, 18 Jan 2009 13:53:14 +0000

find line 383 -
if(!preg_match($time_format, $time, $matches)

and replace it with

if(!preg_match($time_format, $time, $m)

- so $matches should be $m and it’ll work fine

chers