Symplify / PHP7_Sculpin

src/Bundle/SculpinBundle/Command/ContainerDebugCommand.php

Doc Comments +3 added lines

@@ -264,6 +264,9 @@
         }
     }
 
+    /**
+     * @param string|null $className
+     */
     protected function buildArgumentsArray($serviceId, $scope, $className, array $tagAttributes = array())
     {
         $arguments = array($serviceId);

Spacing +3 added lines, -3 removed lines

@@ -215,12 +215,12 @@ 
             }
         }
         $format = '%-'.$maxName.'s ';
-        $format .= implode("", array_map(function ($length) { return "%-{$length}s "; }, $maxTags));
+        $format .= implode("", array_map(function($length) { return "%-{$length}s "; }, $maxTags));
         $format .= '%-'.$maxScope.'s %s';
 
         // the title field needs extra space to make up for comment tags
         $format1 = '%-'.($maxName + 19).'s ';
-        $format1 .= implode("", array_map(function ($length) { return '%-'.($length + 19).'s '; }, $maxTags));
+        $format1 .= implode("", array_map(function($length) { return '%-'.($length + 19).'s '; }, $maxTags));
         $format1 .= '%-'.($maxScope + 19).'s %s';
 
         $tags = array();
@@ -296,7 +296,7 @@ 
                 $output->writeln('<comment>Tags</comment>');
                 foreach ($tags as $tagName => $tagData) {
                     foreach ($tagData as $singleTagData) {
-                        $output->writeln(sprintf('    - %-30s (%s)', $tagName, implode(', ', array_map(function ($key, $value) {
+                        $output->writeln(sprintf('    - %-30s (%s)', $tagName, implode(', ', array_map(function($key, $value) {
                             return sprintf('<info>%s</info>: %s', $key, $value);
                         }, array_keys($singleTagData), array_values($singleTagData)))));
                     }

src/Bundle/SculpinBundle/HttpKernel/AbstractKernel.php

Doc Comments +2 added lines

@@ -28,6 +28,8 @@
 
     /**
      * {@inheritdoc}
+     * @param boolean $debug
+     * @param string $projectDir
      */
     public function __construct($environment, $debug, $projectDir = null)
     {

src/Bundle/ThemeBundle/ThemeTwigExtension.php

Doc Comments +3 added lines

@@ -138,6 +138,9 @@
         return array_reverse($paths);
     }
 
+    /**
+     * @param string $resource
+     */
     private function findThemeResource($theme, $resource)
     {
         if (file_exists($theme['path'].'/'.$resource)) {

src/Core/SiteConfiguration/SiteConfigurationFactory.php

Doc Comments +1 added lines, -1 removed lines

@@ -37,7 +37,7 @@
      * Get an instance of the Configuration() class from the given file.
      *
      * @param  string $configFile
-     * @return YamlFileConfigurationBuilder
+     * @return \Dflydev\DotAccessConfiguration\ConfigurationInterface
      */
     private function getConfigFile($configFile)
     {

src/Core/Tests/Permalink/SourcePermalinkFactoryTest.php

Doc Comments +3 added lines

@@ -68,6 +68,9 @@
         );
     }
 
+    /**
+     * @param string $relativePathname
+     */
     private static function makeTestSource($relativePathname, array $configuration = array())
     {
         $configuration = new Configuration($configuration);

src/Core/Tests/Source/CompositeDataSourceTest.php

Doc Comments +5 added lines

@@ -16,6 +16,11 @@
 
 class CompositeDataSourceTest extends \PHPUnit_Framework_TestCase
 {
+    /**
+     * @param string $dataSourceId
+     *
+     * @return \Sculpin\Core\Source\DataSourceInterface
+     */
     public function makeDataSource($dataSourceId)
     {
         $dataSource = $this->getMock('Sculpin\Core\Source\DataSourceInterface');

src/Core/Tests/Source/FileSourceTest.php

Doc Comments +6 added lines

@@ -21,6 +21,9 @@ 
         return $source;
     }
 
+    /**
+     * @return Analyzer
+     */
     public function makeTestAnalyzer()
     {
         $analyzer = $this->getMock('Dflydev\Canal\Analyzer\Analyzer');
@@ -62,6 +65,9 @@ 
         return $factory;
     }
 
+    /**
+     * @return \Sculpin\Core\Source\DataSourceInterface
+     */
     public function makeTestDatasource()
     {
         $datasource = $this->getMock('Sculpin\Core\Source\DataSourceInterface');

Unused Use Statements -1 removed lines

@@ -4,7 +4,6 @@
 use Dflydev\Canal\Analyzer\Analyzer;
 use Sculpin\Core\Source\FileSource;
 use Sculpin\Core\Source\FilesystemDataSource;
-use Sculpin\Core\Source\MemorySource;
 use Symfony\Component\Finder\SplFileInfo;
 
 class FileSourceTest extends \PHPUnit_Framework_TestCase

Spacing +7 added lines, -7 removed lines

@@ -92,12 +92,12 @@ 
     public function provideTestParseYaml()
     {
         return array(
-            array(__DIR__ . '/../Fixtures/valid/no-end-frontmatter.yml', ''),
-            array(__DIR__ . '/../Fixtures/valid/frontmatter-nocontent.yml', ''),
-            array(__DIR__ . '/../Fixtures/valid/frontmatter-content.yml', ''),
-            array(__DIR__ . '/../Fixtures/invalid/one-line-edge-case.yml', 'Yaml could not be parsed, parser detected a string.'),
-            array(__DIR__ . '/../Fixtures/invalid/malformed-yaml.yml', 'Yaml could not be parsed, parser detected a string.'),
-            array(__DIR__ . '/../Fixtures/invalid/malformed-yaml2.yml', 'Unable to parse at line 2 (near "first:fsdqf").'),
+            array(__DIR__.'/../Fixtures/valid/no-end-frontmatter.yml', ''),
+            array(__DIR__.'/../Fixtures/valid/frontmatter-nocontent.yml', ''),
+            array(__DIR__.'/../Fixtures/valid/frontmatter-content.yml', ''),
+            array(__DIR__.'/../Fixtures/invalid/one-line-edge-case.yml', 'Yaml could not be parsed, parser detected a string.'),
+            array(__DIR__.'/../Fixtures/invalid/malformed-yaml.yml', 'Yaml could not be parsed, parser detected a string.'),
+            array(__DIR__.'/../Fixtures/invalid/malformed-yaml2.yml', 'Unable to parse at line 2 (near "first:fsdqf").'),
         );
     }
 
@@ -106,6 +106,6 @@ 
         if ($msg == '') {
             return '';
         }
-        return ' ! FileSource:FilesystemDataSource:test:' . $filename . ' ' . $msg . ' !' . PHP_EOL;
+        return ' ! FileSource:FilesystemDataSource:test:'.$filename.' '.$msg.' !'.PHP_EOL;
     }
 }

src/Core/Tests/Source/Map/CalculatedDateFromFilenameMapTest.php

Doc Comments +6 added lines

@@ -62,6 +62,9 @@ 
         $this->assertEquals(strtotime("2013-12-12"), $source->data()->get('calculated_date'));
     }
 
+    /**
+     * @param integer $timestamp
+     */
     protected function getSourceWithCalculatedDate($timestamp)
     {
         return new MemorySource(
@@ -78,6 +81,9 @@ 
         );
     }
 
+    /**
+     * @param string $path
+     */
     protected function getSourceWithoutCalculatedDateAndPathname($path)
     {
         return new MemorySource(

src/Core/Tests/Source/SourceSetTest.php

Doc Comments +5 added lines

@@ -15,6 +15,11 @@
 
 class SourceSetTest extends \PHPUnit_Framework_TestCase
 {
+    /**
+     * @param string $sourceId
+     *
+     * @return \Sculpin\Core\Source\SourceInterface
+     */
     public function makeTestSource($sourceId, $hasChanged = true)
     {
         $source = $this->getMock('Sculpin\Core\Source\SourceInterface');