Replace more docblocks by type-hints

Author: nicolas-grekas

Collator/Collator.php

@@ -74,7 +74,7 @@ class Collator
      *
      * @throws MethodArgumentValueNotImplementedException When $locale different than "en" or null is passed
      */
-    public function __construct($locale)
+    public function __construct(?string $locale)
     {
         if ('en' !== $locale && null !== $locale) {
             throw new MethodArgumentValueNotImplementedException(__METHOD__, 'locale', $locale, 'Only the locale "en" is supported');

Data/Bundle/Compiler/GenrbCompiler.php

@@ -32,7 +32,7 @@ class GenrbCompiler implements BundleCompilerInterface
      *
      * @throws RuntimeException if the "genrb" cannot be found
      */
-    public function __construct($genrb = 'genrb', $envVars = '')
+    public function __construct(string $genrb = 'genrb', string $envVars = '')
     {
         exec('which '.$genrb, $output, $status);
 

Data/Bundle/Reader/BufferedBundleReader.php

@@ -29,7 +29,7 @@ class BufferedBundleReader implements BundleReaderInterface
      * @param BundleReaderInterface $reader     The reader to buffer
      * @param int                   $bufferSize The number of entries to store in the buffer
      */
-    public function __construct(BundleReaderInterface $reader, $bufferSize)
+    public function __construct(BundleReaderInterface $reader, int $bufferSize)
     {
         $this->reader = $reader;
         $this->buffer = new RingBuffer($bufferSize);

Data/Generator/AbstractDataGenerator.php

@@ -29,10 +29,10 @@ abstract class AbstractDataGenerator
     private $compiler;
     private $dirName;
 
-    public function __construct(GenrbCompiler $compiler, $dirName)
+    public function __construct(GenrbCompiler $compiler, string $dirName)
     {
         $this->compiler = $compiler;
-        $this->dirName = (string) $dirName;
+        $this->dirName = $dirName;
     }
 
     public function generateData(GeneratorConfig $config)

Data/Generator/GeneratorConfig.php

@@ -30,11 +30,7 @@ class GeneratorConfig
      */
     private $bundleWriters = array();
 
-    /**
-     * @param string $sourceDir
-     * @param string $icuVersion
-     */
-    public function __construct($sourceDir, $icuVersion)
+    public function __construct(string $sourceDir, string $icuVersion)
     {
         $this->sourceDir = $sourceDir;
         $this->icuVersion = $icuVersion;

Data/Generator/LocaleDataGenerator.php

@@ -34,9 +34,9 @@ class LocaleDataGenerator
     private $scriptDataProvider;
     private $regionDataProvider;
 
-    public function __construct($dirName, LanguageDataProvider $languageDataProvider, ScriptDataProvider $scriptDataProvider, RegionDataProvider $regionDataProvider)
+    public function __construct(string $dirName, LanguageDataProvider $languageDataProvider, ScriptDataProvider $scriptDataProvider, RegionDataProvider $regionDataProvider)
     {
-        $this->dirName = (string) $dirName;
+        $this->dirName = $dirName;
         $this->languageDataProvider = $languageDataProvider;
         $this->scriptDataProvider = $scriptDataProvider;
         $this->regionDataProvider = $regionDataProvider;

Data/Provider/CurrencyDataProvider.php

@@ -39,7 +39,7 @@ class CurrencyDataProvider
      * @param string                     $path   The path to the resource bundle
      * @param BundleEntryReaderInterface $reader The reader for reading the resource bundle
      */
-    public function __construct($path, BundleEntryReaderInterface $reader)
+    public function __construct(string $path, BundleEntryReaderInterface $reader)
     {
         $this->path = $path;
         $this->reader = $reader;

Data/Provider/LocaleDataProvider.php

@@ -32,7 +32,7 @@ class LocaleDataProvider
      * @param string                     $path   The path to the directory containing the .res files
      * @param BundleEntryReaderInterface $reader The reader for reading the .res files
      */
-    public function __construct($path, BundleEntryReaderInterface $reader)
+    public function __construct(string $path, BundleEntryReaderInterface $reader)
     {
         $this->path = $path;
         $this->reader = $reader;

Data/Provider/RegionDataProvider.php

@@ -32,7 +32,7 @@ class RegionDataProvider
      * @param string                     $path   The path to the directory containing the .res files
      * @param BundleEntryReaderInterface $reader The reader for reading the .res files
      */
-    public function __construct($path, BundleEntryReaderInterface $reader)
+    public function __construct(string $path, BundleEntryReaderInterface $reader)
     {
         $this->path = $path;
         $this->reader = $reader;

Data/Provider/ScriptDataProvider.php

@@ -32,7 +32,7 @@ class ScriptDataProvider
      * @param string                     $path   The path to the directory containing the .res files
      * @param BundleEntryReaderInterface $reader The reader for reading the .res files
      */
-    public function __construct($path, BundleEntryReaderInterface $reader)
+    public function __construct(string $path, BundleEntryReaderInterface $reader)
     {
         $this->path = $path;
         $this->reader = $reader;