X-Git-Url: https://git.mxchange.org/?a=blobdiff_plain;f=plugins%2FCacheLog%2FCacheLogPlugin.php;h=cf7e3a9884daa571f5618d546c5734b29c8a3a62;hb=14cb2d53980f27fb07f0327644a47e8722718a25;hp=5b0b43935458a0c0091cfa8d84d81037b7ca5b08;hpb=99194e03fa50b61f99164674afc949b4bbefd44a;p=quix0rs-gnu-social.git

diff --git a/plugins/CacheLog/CacheLogPlugin.php b/plugins/CacheLog/CacheLogPlugin.php
index 5b0b439354..cf7e3a9884 100644
--- a/plugins/CacheLog/CacheLogPlugin.php
+++ b/plugins/CacheLog/CacheLogPlugin.php
@@ -61,36 +61,26 @@ class CacheLogPlugin extends Plugin
     function onEndCacheGet($key, &$value)
     {
         if ($value === false) {
-            $this->log(LOG_INFO, "Cache MISS for key '$key'");
+            $this->log(LOG_INFO, sprintf('Cache MISS for key "%s"', $key));
         } else {
-            $this->log(LOG_INFO, "Cache HIT for key '$key'");
+            $this->log(LOG_INFO, sprintf('Cache HIT for key "%s": %s', $key, self::showValue($value)));
         }
         return true;
     }
 
     function onStartCacheSet(&$key, &$value, &$flag, &$expiry, &$success)
     {
-        if (empty($value)) {
-            if (is_array($value)) {
-                $this->log(LOG_INFO, "Setting empty array for key '$key'");
-            } else if (is_null($value)) {
-                $this->log(LOG_INFO, "Setting null value for key '$key'");
-            } else if (is_string($value)) {
-                $this->log(LOG_INFO, "Setting empty string for key '$key'");
-            } else if (is_integer($value)) {
-                $this->log(LOG_INFO, "Setting integer 0 for key '$key'");
-            } else {
-                $this->log(LOG_INFO, "Setting empty value '$value' for key '$key'");
-            }
-        } else {
-            $this->log(LOG_INFO, "Setting non-empty value for key '$key'");
-        }
+        $this->log(LOG_INFO, "Begin setting cache value for key '$key'");
         return true;
     }
 
     function onEndCacheSet($key, $value, $flag, $expiry)
     {
-        $this->log(LOG_INFO, "Done setting cache value for key '$key'");
+        $this->log(LOG_INFO, sprintf('Set cache value %s for key "%s" (flags: %d, expiry %d)',
+                                     self::showValue($value),
+                                     $key,
+                                     $flag,
+                                     $expiry));
         return true;
     }
 
@@ -106,14 +96,32 @@ class CacheLogPlugin extends Plugin
         return true;
     }
 
-    function onPluginVersion(&$versions)
+    function onPluginVersion(array &$versions)
     {
         $versions[] = array('name' => 'CacheLog',
-                            'version' => STATUSNET_VERSION,
+                            'version' => GNUSOCIAL_VERSION,
                             'author' => 'Evan Prodromou',
-                            'homepage' => 'http://status.net/wiki/Plugin:CacheLog',
+                            'homepage' => 'https://git.gnu.io/gnu/gnu-social/tree/master/plugins/CacheLog',
                             'description' =>
+                            // TRANS: Plugin description.
                             _m('Log reads and writes to the cache.'));
         return true;
     }
+
+    static function showValue($value)
+    {
+        if (is_object($value)) {
+            return sprintf('object of class %s', get_class($value));
+        } else if (is_array($value)) {
+            return sprintf('array of length %d', count($value));
+        } else if (is_string($value)) {
+            return sprintf('string "%s"', $value);
+        } else if (is_integer($value)) {
+            return sprintf('integer %d', $value);
+        } else if (is_null($value)) {
+            return 'null';
+        } else {
+            return 'unknown';
+        }
+    }
 }