From 6e62f035c01cc7eaf17f69cea41e4c97e4c33f58 Mon Sep 17 00:00:00 2001
From: Alex Masterov <alex.masterow@gmail.com>
Date: Sat, 7 Jan 2017 11:38:47 +0300
Subject: [PATCH] Add strict types declaration

---
 src/Configuration/TwigConfiguration.php | 25 ++++---------------------
 src/Configuration/TwigExtensionSet.php  | 10 +---------
 src/Exception/ExtensionException.php    |  4 +---
 src/Exception/LoaderException.php       |  9 ++-------
 src/Loader/FilesystemLoader.php         | 18 ++++++++----------
 src/TwigFormatter.php                   |  3 ---
 6 files changed, 16 insertions(+), 53 deletions(-)

diff --git a/src/Configuration/TwigConfiguration.php b/src/Configuration/TwigConfiguration.php
index a3c55a8..8cef7b0 100644
--- a/src/Configuration/TwigConfiguration.php
+++ b/src/Configuration/TwigConfiguration.php
@@ -1,4 +1,5 @@
 <?php
+declare(strict_types = 1);
 
 namespace AlexMasterov\EquipTwig\Configuration;
 
@@ -16,9 +17,6 @@ final class TwigConfiguration implements ConfigurationInterface
 
     const PREFIX = 'TWIG_';
 
-    /**
-     * @param Injector $injector
-     */
     public function apply(Injector $injector)
     {
         $env = $this->env;
@@ -36,24 +34,14 @@ public function apply(Injector $injector)
         ]);
     }
 
-    /**
-     * @param Env $env
-     *
-     * @return array
-     */
-    private function fileExtensions(Env $env)
+    private function fileExtensions(Env $env): array
     {
         $fileExtensions = $env->getValue('TWIG_FILE_EXTENSIONS', 'html.twig,twig');
 
         return explode(',', $fileExtensions);
     }
 
-    /**
-     * @param Env $env
-     *
-     * @return array
-     */
-    private function options(Env $env)
+    private function options(Env $env): array
     {
        static $options = [
             'debug'               => false,
@@ -76,12 +64,7 @@ private function options(Env $env)
         return $options;
     }
 
-    /**
-     * @param Env $env
-     *
-     * @return array
-     */
-    private function envTwig(Env $env)
+    private function envTwig(Env $env): array
     {
         $twigFilter = static function ($value) {
             return stristr($value, self::PREFIX);
diff --git a/src/Configuration/TwigExtensionSet.php b/src/Configuration/TwigExtensionSet.php
index cd1e5dd..3a5c3de 100644
--- a/src/Configuration/TwigExtensionSet.php
+++ b/src/Configuration/TwigExtensionSet.php
@@ -1,4 +1,5 @@
 <?php
+declare(strict_types = 1);
 
 namespace AlexMasterov\EquipTwig\Configuration;
 
@@ -12,20 +13,11 @@
 
 class TwigExtensionSet extends Set implements ConfigurationInterface
 {
-    /**
-     * @param Injector $injector
-     */
     public function apply(Injector $injector)
     {
         $injector->prepare(Twig_Environment::class, [$this, 'prepareExtension']);
     }
 
-    /**
-     * @param Twig_Environment  $environment
-     * @param Injector          $injector
-     *
-     * @return void
-     */
     public function prepareExtension(
         Twig_Environment $environment,
         Injector $injector
diff --git a/src/Exception/ExtensionException.php b/src/Exception/ExtensionException.php
index 45a28b9..071f030 100644
--- a/src/Exception/ExtensionException.php
+++ b/src/Exception/ExtensionException.php
@@ -10,10 +10,8 @@ class ExtensionException extends InvalidArgumentException implements ExceptionIn
 {
     /**
      * @param string|object $spec
-     *
-     * @return static
      */
-    public static function invalidClass($spec)
+    public static function invalidClass($spec): self
     {
         if (is_object($spec)) {
             $spec = get_class($spec);
diff --git a/src/Exception/LoaderException.php b/src/Exception/LoaderException.php
index 3e40c13..e2a4a11 100644
--- a/src/Exception/LoaderException.php
+++ b/src/Exception/LoaderException.php
@@ -1,4 +1,5 @@
 <?php
+declare(strict_types = 1);
 
 namespace AlexMasterov\EquipTwig\Exception;
 
@@ -7,13 +8,7 @@
 
 class LoaderException extends Twig_Error_Loader implements ExceptionInterface
 {
-    /**
-     * @param string $name  The template name
-     * @param string $where The template directory path
-     *
-     * @return static
-     */
-    public static function notFound($name, $where)
+    public static function notFound(string $name, string $where): self
     {
         return new static(sprintf(
             'Unable to find template `%s` (looked into: %s).',
diff --git a/src/Loader/FilesystemLoader.php b/src/Loader/FilesystemLoader.php
index 6b91413..d641a8c 100644
--- a/src/Loader/FilesystemLoader.php
+++ b/src/Loader/FilesystemLoader.php
@@ -1,4 +1,5 @@
 <?php
+declare(strict_types = 1);
 
 namespace AlexMasterov\EquipTwig\Loader;
 
@@ -16,23 +17,20 @@ final class FilesystemLoader implements Twig_LoaderInterface
     /**
      * @var array
      */
-    private $fileExtensions = [];
+    private $fileExtensions = ['html.twig', 'twig'];
 
     /**
      * @var array
      */
     private $cache = [];
 
-    /**
-     * @param string $path           The template directory path
-     * @param array  $fileExtensions The template file extensions
-     */
-    public function __construct(
-        $path,
-        array $fileExtensions = ['html.twig', 'twig']
-    ) {
+    public function __construct(string $path, array $fileExtensions = null)
+    {
         $this->path = $path;
-        $this->fileExtensions = $fileExtensions;
+
+        if ($fileExtensions) {
+            $this->fileExtensions = $fileExtensions;
+        }
     }
 
     /**
diff --git a/src/TwigFormatter.php b/src/TwigFormatter.php
index 5883af6..4643fc5 100644
--- a/src/TwigFormatter.php
+++ b/src/TwigFormatter.php
@@ -17,9 +17,6 @@ final class TwigFormatter extends HtmlFormatter
      */
     private $template;
 
-    /**
-     * @param Twig_Environment $environment
-     */
     public function __construct(Twig_Environment $environment)
     {
         $this->environment = $environment;