Improve logging (#129)

* Updated change log

* Improve logging

Author: Nyholm

AbstractCachePool.php

@@ -220,7 +220,8 @@ public function deleteItems(array $keys)
     public function save(CacheItemInterface $item)
     {
         if (!$item instanceof PhpCacheItem) {
-            throw new InvalidArgumentException('Cache items are not transferable between pools. Item MUST implement PhpCacheItem.');
+            $e = new InvalidArgumentException('Cache items are not transferable between pools. Item MUST implement PhpCacheItem.');
+            $this->handleException($e, __FUNCTION__);
         }
 
         $this->removeTagEntries($item);
@@ -275,16 +276,18 @@ public function commit()
     protected function validateKey($key)
     {
         if (!is_string($key)) {
-            throw new InvalidArgumentException(sprintf(
+            $e = new InvalidArgumentException(sprintf(
                 'Cache key must be string, "%s" given', gettype($key)
             ));
+            $this->handleException($e, __FUNCTION__);
         }
 
         if (preg_match('|[\{\}\(\)/\\\@\:]|', $key)) {
-            throw new InvalidArgumentException(sprintf(
+            $e = new InvalidArgumentException(sprintf(
                 'Invalid key: "%s". The key contains one or more characters reserved for future extension: {}()/\@:',
                 $key
             ));
+            $this->handleException($e, __FUNCTION__);
         }
     }
 

Changelog.md

@@ -10,14 +10,13 @@ The change log describes what is "Added", "Removed", "Changed" or "Fixed" betwee
 * `AbstractCachePool::invalidateTags` and `AbstractCachePool::invalidateTags` 
 * Added interfaces for our items and pools `PhpCachePool` and `PhpCacheItem`
 * Trait to help adapters to support tags. `TagSupportWithArray`.
-* New interfaces for tags. `TaggableCacheItemInterface` and `TaggableCacheItemPoolInterface`
 
 ### Changed
 
 * First parameter to `AbstractCachePool::storeItemInCache` must be a `PhpCacheItem`. 
 * Return value from `AbstractCachePool::fetchObjectFromCache` must be a an array with 4 values. Added expiration timestamp. 
 * `HasExpirationDateInterface` is replaced by `HasExpirationTimestampInterface`
-* We do not work with `\DateTime` anymore. We work with timestamps. 
+* We do not work with `\DateTime` internally anymore. We work with timestamps. 
 
 ## 0.3.3