Merge pull request #293 from FriendsOfSymfony/styleci

add styleci

Author: dbu

.styleci.yml

@@ -0,0 +1 @@
+preset: symfony

src/CacheInvalidator.php

@@ -76,7 +76,7 @@ class CacheInvalidator
     private $headerLength = 7500;
 
     /**
-     * Constructor
+     * Constructor.
      *
      * @param ProxyClientInterface $cache HTTP cache
      */
@@ -112,12 +112,12 @@ public function supports($operation)
             case self::INVALIDATE:
                 return $this->cache instanceof BanInterface;
             default:
-                throw new InvalidArgumentException('Unknown operation ' . $operation);
+                throw new InvalidArgumentException('Unknown operation '.$operation);
         }
     }
 
     /**
-     * Set event dispatcher
+     * Set event dispatcher.
      *
      * @param EventDispatcherInterface $eventDispatcher
      *
@@ -136,7 +136,7 @@ public function setEventDispatcher(EventDispatcherInterface $eventDispatcher)
     }
 
     /**
-     * Get event dispatcher
+     * Get event dispatcher.
      *
      * @return EventDispatcherInterface
      */
@@ -150,7 +150,7 @@ public function getEventDispatcher()
     }
 
     /**
-     * Add subscriber
+     * Add subscriber.
      *
      * @param EventSubscriberInterface $subscriber
      *
@@ -166,7 +166,7 @@ public function addSubscriber(EventSubscriberInterface $subscriber)
     }
 
     /**
-     * Set the HTTP header name that will hold cache tags
+     * Set the HTTP header name that will hold cache tags.
      *
      * @param string $tagsHeader
      *
@@ -184,7 +184,7 @@ public function setTagsHeader($tagsHeader)
     }
 
     /**
-     * Get the HTTP header name that will hold cache tags
+     * Get the HTTP header name that will hold cache tags.
      *
      * @return string
      *
@@ -196,7 +196,7 @@ public function getTagsHeader()
     }
 
     /**
-     * Invalidate a path or URL
+     * Invalidate a path or URL.
      *
      * @param string $path    Path or URL
      * @param array  $headers HTTP headers (optional)
@@ -217,7 +217,7 @@ public function invalidatePath($path, array $headers = array())
     }
 
     /**
-     * Refresh a path or URL
+     * Refresh a path or URL.
      *
      * @param string $path    Path or URL
      * @param array  $headers HTTP headers (optional)

src/Event.php

@@ -18,7 +18,7 @@ class Event extends BaseEvent
     private $exception;
 
     /**
-     * Set exception
+     * Set exception.
      *
      * @param \Exception $exception
      */
@@ -28,7 +28,7 @@ public function setException(\Exception $exception)
     }
 
     /**
-     * Get exception
+     * Get exception.
      *
      * @return \Exception
      */

src/EventListener/LogSubscriber.php

@@ -36,7 +36,7 @@ public static function getSubscribedEvents()
     {
         return array(
             Events::PROXY_UNREACHABLE_ERROR => 'onProxyUnreachableError',
-            Events::PROXY_RESPONSE_ERROR    => 'onProxyResponseError'
+            Events::PROXY_RESPONSE_ERROR => 'onProxyResponseError',
         );
     }
 
@@ -53,7 +53,7 @@ public function onProxyResponseError(Event $event)
     private function log($level, \Exception $exception)
     {
         $context = array(
-            'exception' => $exception
+            'exception' => $exception,
         );
 
         $this->logger->log($level, $exception->getMessage(), $context);

src/Events.php

@@ -12,10 +12,10 @@
 namespace FOS\HttpCache;
 
 /**
- * Events thrown by the FOSHttpCache library
+ * Events thrown by the FOSHttpCache library.
  */
 final class Events
 {
     const PROXY_UNREACHABLE_ERROR = 'fos_http_cache.error.proxy_unreachable';
-    const PROXY_RESPONSE_ERROR    = 'fos_http_cache.error.response';
+    const PROXY_RESPONSE_ERROR = 'fos_http_cache.error.response';
 }

src/Exception/ExceptionCollection.php

@@ -13,14 +13,14 @@
 
 /**
  * A collection of exceptions that might occur during the flush operation of a
- * ProxyClientInterface implementation
+ * ProxyClientInterface implementation.
  */
-class ExceptionCollection extends \Exception implements \IteratorAggregate, \Countable,  HttpCacheExceptionInterface
+class ExceptionCollection extends \Exception implements \IteratorAggregate, \Countable, HttpCacheExceptionInterface
 {
     private $exceptions = array();
 
     /**
-     * Add an exception to the collection
+     * Add an exception to the collection.
      *
      * @param \Exception $e
      *
@@ -48,11 +48,11 @@ public function getFirst()
             return $this->exceptions[0];
         }
 
-        return null;
+        return;
     }
 
     /**
-     * Get exception iterator
+     * Get exception iterator.
      *
      * @return \ArrayIterator
      */
@@ -62,7 +62,7 @@ public function getIterator()
     }
 
     /**
-     * Get number of exceptions in collection
+     * Get number of exceptions in collection.
      *
      * @return int
      */

src/Exception/HttpCacheExceptionInterface.php

@@ -15,4 +15,5 @@
  * Common interface for all exceptions thrown by this library.
  */
 interface HttpCacheExceptionInterface
-{}
+{
+}

src/Exception/InvalidUrlException.php

@@ -29,7 +29,7 @@ public static function invalidUrl($url, $reason = null)
             $msg .= sprintf('Reason: %s', $reason);
         }
 
-        return new InvalidUrlException($msg);
+        return new self($msg);
     }
 
     /**
@@ -40,7 +40,7 @@ public static function invalidUrl($url, $reason = null)
      */
     public static function invalidUrlParts($server, array $allowed)
     {
-        return new InvalidUrlException(sprintf(
+        return new self(sprintf(
             'Server "%s" is invalid. Only %s URL parts are allowed.',
             $server,
             implode(', ', $allowed)
@@ -56,7 +56,7 @@ public static function invalidUrlParts($server, array $allowed)
      */
     public static function invalidUrlScheme($url, $scheme, array $allowed)
     {
-        return new InvalidUrlException(sprintf(
+        return new self(sprintf(
             'Host "%s" with scheme "%s" is invalid. Only schemes "%s" are supported',
             $url,
             $scheme,

src/Exception/MissingHostException.php

@@ -26,11 +26,11 @@ public static function missingHost($path)
     {
         $msg = sprintf(
             'Path "%s" cannot be invalidated without a host. '
-            . 'Either invalidate full URLs containing hostnames instead of paths '
-            . 'or configure the caching proxy class with a hostname in the base path.',
+            .'Either invalidate full URLs containing hostnames instead of paths '
+            .'or configure the caching proxy class with a hostname in the base path.',
             $path
         );
 
-        return new MissingHostException($msg);
+        return new self($msg);
     }
 }

src/Exception/ProxyResponseException.php

@@ -37,6 +37,6 @@ public static function proxyResponse($host, $statusCode, $statusMessage, $detail
             $message .= ". $details";
         }
 
-        return new ProxyResponseException($message, $statusCode, $previous);
+        return new self($message, $statusCode, $previous);
     }
 }