jenschude / HandlebarsBundle

src/HandlebarsEngine.php

Doc Comments -1 removed lines

@@ -26,7 +26,6 @@
      *
      * @param HandlebarsEnvironment       $handlebars   A HandlebarsEnvironment instance
      * @param TemplateNameParserInterface $parser       A TemplateNameParserInterface instance
-     * @param FileLocatorInterface        $locator      A FileLocatorInterface instance
      */
     public function __construct(HandlebarsEnvironment $handlebars, TemplateNameParserInterface $parser)
     {

Unused Use Statements -3 removed lines

@@ -7,13 +7,10 @@
 namespace JaySDe\HandlebarsBundle;
 
 
-use JaySDe\HandlebarsBundle\Cache\Filesystem;
-use LightnCandy\LightnCandy;
 use Symfony\Bundle\FrameworkBundle\Templating\EngineInterface;
 use Symfony\Component\Config\FileLocatorInterface;
 use Symfony\Component\HttpFoundation\Response;
 use Symfony\Component\Templating\TemplateNameParserInterface;
-use Symfony\Component\Templating\TemplateReferenceInterface;
 
 class HandlebarsEngine implements EngineInterface
 {

src/HandlebarsEnvironment.php

Doc Comments +3 added lines

@@ -72,6 +72,9 @@
         $this->profiler = $profiler;
     }
 
+    /**
+     * @param string $name
+     */
     public function compile($name)
     {
         $source = $this->loader->getSource($name);

Spacing +7 added lines, -7 removed lines

@@ -52,18 +52,18 @@ 
             'debug' => true,
             'flags' => $flags,
             'helpers' => $helper->getHelperMethods(),
-            'partialresolver' => function ($cx, $name) use ($loader, &$partials) {
+            'partialresolver' => function($cx, $name) use ($loader, &$partials) {
                 $extension = false;
-                if ($loader->exists($name . '.handlebars')) {
+                if ($loader->exists($name.'.handlebars')) {
                     $extension = '.handlebars';
-                } else if ($loader->exists($name . '.hbs')) {
+                } else if ($loader->exists($name.'.hbs')) {
                     $extension = '.hbs';
                 }
                 if ($extension === false) {
                     return null;
                 }
-                $partials[] = new FileResource($loader->getCacheKey($name . $extension));
-                return $loader->getSource($name . $extension);
+                $partials[] = new FileResource($loader->getCacheKey($name.$extension));
+                return $loader->getSource($name.$extension);
             },
         ], $options);
         $this->debug = (bool) $this->options['debug'];
@@ -84,7 +84,7 @@ 
         } catch (\Exception $e) {
             throw new LoaderException($e->getMessage());
         }
-        $this->cache->write($cacheKey, '<?php // ' . $name . PHP_EOL . $phpStr, $this->partials->getArrayCopy());
+        $this->cache->write($cacheKey, '<?php // '.$name.PHP_EOL.$phpStr, $this->partials->getArrayCopy());
 
         return $phpStr;
     }
@@ -134,7 +134,7 @@ 
      */
     public function loadTemplate($templateName)
     {
-        $name = (string)$templateName;
+        $name = (string) $templateName;
         $cacheKey = $this->getCacheFilename($name);
 
         if (!$this->isAutoReload() && file_exists($cacheKey)) {

src/Loader/FilesystemLoader.php

Doc Comments +8 added lines, -1 removed lines

@@ -26,7 +26,6 @@ 
     /**
      * Constructor.
      *
-     * @param string|array $paths A path or an array of paths where to look for templates
      */
     public function __construct(FileLocatorInterface $locator, TemplateNameParserInterface $parser)
     {
@@ -133,6 +132,7 @@ 
 
     /**
      * {@inheritdoc}
+     * @return string
      */
     public function getCacheKey($name)
     {
@@ -141,6 +141,7 @@ 
 
     /**
      * {@inheritdoc}
+     * @param string $template
      */
     public function exists($template)
     {
@@ -157,6 +158,9 @@ 
         }
     }
 
+    /**
+     * @param string $name
+     */
     protected function parseName($name, $default = self::MAIN_NAMESPACE)
     {
         if (isset($name[0]) && '@' == $name[0]) {
@@ -178,6 +182,9 @@ 
         return preg_replace('#/{2,}#', '/', str_replace('\\', '/', (string) $name));
     }
 
+    /**
+     * @return string
+     */
     protected function findTemplate($template, $throw = true)
     {
         $name = $this->normalizeName($template);

src/HandlebarsHelper.php

Spacing +1 added lines, -1 removed lines

@@ -16,7 +16,7 @@
     {
         if ($helper instanceof HelperInterface) {
             $this->helpers[$id] = [$helper, 'handle'];
-        } elseif(is_callable($helper)) {
+        } elseif (is_callable($helper)) {
             $this->helpers[$id] = $helper;
         }
     }

src/DependencyInjection/Configuration.php

Spacing +1 added lines, -1 removed lines

@@ -34,7 +34,7 @@
                     ->useAttributeAsKey('paths')
                     ->beforeNormalization()
                         ->always()
-                        ->then(function ($paths) {
+                        ->then(function($paths) {
                             $normalized = array();
                             foreach ($paths as $path => $namespace) {
                                 if (is_array($namespace)) {

src/Helper/CmsHelper.php

Spacing +2 added lines, -2 removed lines

@@ -22,11 +22,11 @@
         $options = isset($options['hash']) ? $options['hash'] : [];
         $bundle = '';
         if (isset($options['bundle'])) {
-            $bundle = $options['bundle'] . ':';
+            $bundle = $options['bundle'].':';
             unset($options['bundle']);
         }
 
-        $cmsKey = $bundle . $context;
+        $cmsKey = $bundle.$context;
 
         $result = $this->translationHelper->handle($cmsKey, $options);
 

src/Helper/TranslationHelper.php

Spacing +1 added lines, -1 removed lines

@@ -54,7 +54,7 @@
         $count = isset($options['count']) ? $options['count'] : null;
         $args = [];
         foreach ($options as $key => $value) {
-            $key = $this->interpolationPrefix . $key . $this->interpolationSuffix;
+            $key = $this->interpolationPrefix.$key.$this->interpolationSuffix;
             $args[$key] = $value;
         }
 

src/DependencyInjection/HandlebarsExtension.php

Spacing +3 added lines, -3 removed lines

@@ -29,12 +29,12 @@ 
         $flags = 0;
         if (isset($config['flags'])) {
             foreach ($config['flags'] as $flag) {
-                $flags = $flags | constant('LightnCandy\LightnCandy::' . $flag);
+                $flags = $flags | constant('LightnCandy\LightnCandy::'.$flag);
             }
         }
         if (isset($config['excludeFlags'])) {
             foreach ($config['excludeFlags'] as $flag) {
-                $flags = $flags & ~constant('LightnCandy\LightnCandy::' . $flag);
+                $flags = $flags & ~constant('LightnCandy\LightnCandy::'.$flag);
             }
             unset($config['excludeFlags']);
         }
@@ -59,7 +59,7 @@ 
         $container->setParameter('handlebars.assetic', $config['assetic']);
 
         foreach ($config['translation'] as $key => $value) {
-            $container->setParameter('handlebars.translation.' . $key, $value);
+            $container->setParameter('handlebars.translation.'.$key, $value);
         }
 
         // register user-configured paths