zicht / framework-extra-bundle

src/Zicht/Bundle/FrameworkExtraBundle/Command/AbstractCronCommand.php

Unused Use Statements -2 removed lines

@@ -6,12 +6,10 @@
 namespace Zicht\Bundle\FrameworkExtraBundle\Command;
 
 use Exception;
-
 use Monolog\Logger;
 use Monolog\Handler\StreamHandler;
 use Monolog\Processor\MemoryUsageProcessor;
 use Monolog\Processor\MemoryPeakUsageProcessor;
-
 use Symfony\Bundle\FrameworkBundle\Command\ContainerAwareCommand;
 use Symfony\Component\Console\Output\OutputInterface;
 use Symfony\Component\Console\Input\InputInterface;

Spacing +2 added lines, -2 removed lines

@@ -163,13 +163,13 @@
             $mutex = new Mutex($mutexFile, false);
             $isLockAcquired = false;
             $mutex->run(
-                function () use ($self, $input, $output, &$result) {
+                function() use ($self, $input, $output, &$result) {
                     return $self->doParentRun($input, $output);
                 },
                 $isLockAcquired
             );
             if (!$isLockAcquired && $this->logger) {
-                $this->logger->addWarning("Mutex failed in " . get_class($this) . ", job was not run");
+                $this->logger->addWarning("Mutex failed in ".get_class($this).", job was not run");
             }
         } else {
             $result = $this->doParentRun($input, $output);

src/Zicht/Bundle/FrameworkExtraBundle/Helper/AnnotationRegistry.php

Doc Comments +1 added lines, -1 removed lines

@@ -54,7 +54,7 @@
      * Get annotation
      *
      * @param string $name
-     * @return array
+     * @return string
      */
     public function getAnnotation($name)
     {

Spacing +1 added lines, -1 removed lines

@@ -46,7 +46,7 @@
                 $this->setAnnotation($annotation['key'], $new_annotation);
             }
         } else {
-            $this->annotations[]= $new_annotation;
+            $this->annotations[] = $new_annotation;
         }
     }
 

src/Zicht/Bundle/FrameworkExtraBundle/Helper/EmbedHelper.php

Doc Comments +2 added lines, -2 removed lines

@@ -53,7 +53,7 @@ 
      * Get the top most (root) element of the form view
      *
      * @param FormView $formView
-     * @return mixed
+     * @return FormView
      */
     public static function getFormRoot($formView)
     {
@@ -296,7 +296,7 @@ 
      * Returns the ID the form's state is stored by in the session
      *
      * @param \Symfony\Component\Form\FormInterface $form
-     * @return mixed
+     * @return string
      */
     public function getFormId(FormInterface $form)
     {

Spacing +1 added lines, -1 removed lines

@@ -280,7 +280,7 @@
             $viewVars['messages'] = [];
             if ($request->hasPreviousSession() && ($messages = $this->container->get('session')->getFlashBag()->get($formId))) {
                 foreach ($messages as $value) {
-                    $viewVars['messages'][] = $prefix . $value;
+                    $viewVars['messages'][] = $prefix.$value;
                 }
             }
 

src/Zicht/Bundle/FrameworkExtraBundle/Pager/Pager.php

Doc Comments +3 added lines, -3 removed lines

@@ -123,7 +123,7 @@ 
     /**
      * Returns the 1-indexed start of the displayed range, used for displaying in templates
      *
-     * @return int
+     * @return double
      */
     public function getRangeStart()
     {
@@ -134,7 +134,7 @@ 
     /**
      * Returns the 1-indexed end of the displayed range, used for displaying in templates
      *
-     * @return int
+     * @return double
      */
     public function getRangeEnd()
     {
@@ -145,7 +145,7 @@ 
     /**
      * Combines the getRangeStart() and getRangeEnd() in one array
      *
-     * @return array
+     * @return double[]
      */
     public function getRange()
     {

Spacing +4 added lines, -4 removed lines

@@ -82,7 +82,7 @@ 
         }
         if ((int)$page != $page) {
             throw new \InvalidArgumentException(
-                "Invalid argument \$page, expected integer number, got " . gettype($page)
+                "Invalid argument \$page, expected integer number, got ".gettype($page)
             );
         }
         $this->numPages = (int)ceil($this->total / $this->itemsPerPage);
@@ -343,7 +343,7 @@ 
      */
     public function offsetSet($offset, $value)
     {
-        throw new \BadMethodCallException(__CLASS__ . ' is read only');
+        throw new \BadMethodCallException(__CLASS__.' is read only');
     }
 
 
@@ -357,7 +357,7 @@ 
      */
     public function offsetUnset($offset)
     {
-        throw new \BadMethodCallException(__CLASS__ . ' is read only');
+        throw new \BadMethodCallException(__CLASS__.' is read only');
     }
 
 
@@ -398,7 +398,7 @@ 
                 $isPreviousGap = false;
             } elseif (!$isPreviousGap) {
                 $isPreviousGap = true;
-                $ret[$i]= null;
+                $ret[$i] = null;
             }
         }
 

src/Zicht/Bundle/FrameworkExtraBundle/Twig/ControlStructures/SwitchTokenParser.php

Doc Comments +1 added lines, -1 removed lines

@@ -95,7 +95,7 @@
      * Checks if the token is part of the current control structure.
      *
      * @param Twig_Token $token
-     * @return mixed
+     * @return boolean
      */
     public function decideSwitchFork($token)
     {

src/Zicht/Bundle/FrameworkExtraBundle/Twig/Extension.php

Doc Comments +4 added lines, -4 removed lines

@@ -104,7 +104,7 @@ 
     }
 
     /**
-     * @return array
+     * @return Twig_SimpleFilter[]
      */
     public function getFilters()
     {
@@ -303,7 +303,7 @@ 
      *
      * @param array|Collection $items
      * @param array $keyValuePairs
-     * @return array
+     * @return Collection[]
      */
     public function whereSplit($items, $keyValuePairs)
     {
@@ -325,7 +325,7 @@ 
      * </form>
      *
      * @param string $url
-     * @return array
+     * @return string
      */
     public function urlStripQuery($url)
     {
@@ -772,7 +772,7 @@ 
      *
      * @param mixed $var
      * @param string $mode
-     * @return mixed
+     * @return string|null
      */
     public function dump($var, $mode = null)
     {

Spacing +9 added lines, -9 removed lines

@@ -158,7 +158,7 @@ 
     public function prefixMultiple($values, $prefix)
     {
         return iter\map(
-            function ($value) use ($prefix) {
+            function($value) use ($prefix) {
                 return sprintf('%s%s', $prefix, $value);
             },
             $values
@@ -178,7 +178,7 @@ 
     {
         $translator = $this->translator;
         return iter\map(
-            function ($message) use ($translator, $parameters, $domain, $locale) {
+            function($message) use ($translator, $parameters, $domain, $locale) {
                 return $this->translator->trans($message, $parameters, $domain, $locale);
             },
             $messages
@@ -266,7 +266,7 @@ 
             $items->initialize();
         }
 
-        $whereMethod = $booleanOperator . 'Where';
+        $whereMethod = $booleanOperator.'Where';
 
         $eb = new ExpressionBuilder();
         $criteria = new Criteria();
@@ -330,7 +330,7 @@ 
     public function urlStripQuery($url)
     {
         $query = parse_url($url, PHP_URL_QUERY);
-        return str_replace('?' . $query, '', $url);
+        return str_replace('?'.$query, '', $url);
     }
 
     /**
@@ -483,7 +483,7 @@ 
 
         usort(
             $collection,
-            function ($left, $right) use ($types, $idToIndexMap, $numTypes) {
+            function($left, $right) use ($types, $idToIndexMap, $numTypes) {
                 $localClassNameLeft = Str::classname(get_class($left));
                 $localClassNameRight = Str::classname(get_class($right));
 
@@ -617,7 +617,7 @@ 
 
                 if (null !== $this->translator) {
                     $message = $this->translator->trans(
-                        '%count% ' . $denominator . ' ago',
+                        '%count% '.$denominator.' ago',
                         array('%count%' => $diff->$part)
                     );
                 } else {
@@ -655,8 +655,8 @@ 
     {
         $result = '';
         foreach (preg_split('/\b/U', $str) as $part) {
-            if (strlen($result . $part) > $length) {
-                $result = rtrim($result) . $ellipsis;
+            if (strlen($result.$part) > $length) {
+                $result = rtrim($result).$ellipsis;
                 break;
             } else {
                 $result .= $part;
@@ -731,7 +731,7 @@ 
             parse_str($query, $currentParams);
             $currentParams += $embedParams;
 
-            return preg_replace('/\?$/', '', $urlOrArray) . '?' . http_build_query($currentParams);
+            return preg_replace('/\?$/', '', $urlOrArray).'?'.http_build_query($currentParams);
         } else {
             throw new \InvalidArgumentException("Only supports arrays or strings");
         }

src/Zicht/Bundle/FrameworkExtraBundle/Util/SortedList.php

Spacing +3 added lines, -3 removed lines

@@ -24,7 +24,7 @@ 
      */
     public function insert($item, $priority)
     {
-        $this->items[]= array($priority, $item);
+        $this->items[] = array($priority, $item);
 
         $this->sort();
     }
@@ -39,7 +39,7 @@ 
     {
         usort(
             $this->items,
-            function ($a, $b) {
+            function($a, $b) {
                 return $a[0] > $b[0] ? 1 : $a[0] === $b[0] ? 0 : -1;
             }
         );
@@ -52,7 +52,7 @@ 
     {
         return new \ArrayIterator(
             array_map(
-                function ($item) {
+                function($item) {
                     return $item[1];
                 },
                 $this->items

src/Zicht/Bundle/FrameworkExtraBundle/Controller/TranslationController.php

Spacing +1 added lines, -1 removed lines

@@ -96,7 +96,7 @@
         $allMessages = $this->getTranslationsForLocale($locale);
 
         if (!array_key_exists($domain, $allMessages)) {
-            throw new \Exception('Domain ' . $domain . ' not found in the translations for locale ' . $locale);
+            throw new \Exception('Domain '.$domain.' not found in the translations for locale '.$locale);
         }
 
         return $allMessages[$domain];

src/Zicht/Bundle/FrameworkExtraBundle/Command/ValidateEntityCommand.php

Spacing +3 added lines, -3 removed lines

@@ -36,7 +36,7 @@ 
             ->addOption(
                 'group',
                 'g',
-                InputOption::VALUE_REQUIRED | InputOption::VALUE_IS_ARRAY,
+                InputOption::VALUE_REQUIRED|InputOption::VALUE_IS_ARRAY,
                 "Optional validation group(s)"
             );
     }
@@ -56,7 +56,7 @@ 
                 $violations = $this->getContainer()->get('validator')->validate($entity, $groups ? $groups : null);
 
                 if (count($violations)) {
-                    $output->writeln(get_class($entity) . "::" . $entity->getId());
+                    $output->writeln(get_class($entity)."::".$entity->getId());
                     foreach ($violations as $error) {
                         $output->writeln(" -> {$error}");
                     }
@@ -79,7 +79,7 @@ 
             ->getMetadataFactory()
             ->getAllMetadata();
 
-        $isAcceptable = function ($meta) {
+        $isAcceptable = function($meta) {
             return !$meta->isMappedSuperclass && empty($meta->subClasses);
         };