Victoire / victoire

Bundle/FormBundle/Form/Type/LinkType.php

Doc Comments +2 added lines, -2 removed lines

@@ -130,7 +130,7 @@ 
      *
      * @param                                    $linkType
      * @param                                    $locale
-     * @param FormBuilderInterface|FormInterface $form
+     * @param FormInterface $form
      * @param FormBuilderInterface               $builder
      * @param array                              $options
      */
@@ -202,7 +202,7 @@ 
      * Add the types related to the target value.
      *
      * @param string                             $target
-     * @param FormBuilderInterface|FormInterface $form
+     * @param FormInterface $form
      * @param array                              $options
      */
     protected function manageTargetRelatedFields($target, $form, $options)

Spacing +3 added lines, -3 removed lines

@@ -64,7 +64,7 @@ 
                 'choices'           => $options['linkTypeChoices'],
                 'choices_as_values' => true,
             ])
-            ->addEventListener(FormEvents::PRE_SET_DATA, function (FormEvent $event) use ($builder, $options) {
+            ->addEventListener(FormEvents::PRE_SET_DATA, function(FormEvent $event) use ($builder, $options) {
                 /** @var Link $data */
                 $data = $event->getData();
                 $form = $event->getForm();
@@ -74,7 +74,7 @@ 
             });
 
         $builder
-            ->addEventListener(FormEvents::PRE_SUBMIT, function (FormEvent $event) use ($builder, $options) {
+            ->addEventListener(FormEvents::PRE_SUBMIT, function(FormEvent $event) use ($builder, $options) {
                 $form = $event->getForm();
                 $data = $event->getData();
                 $locale = isset($data['locale']) ? $data['locale'] : null;
@@ -212,7 +212,7 @@ 
                 'label'        => 'form.link_type.target.modalLayouts.label',
                 'required'     => true,
                 'choices'      => $this->modalLayouts,
-                'choice_label' => function ($value, $key, $index) {
+                'choice_label' => function($value, $key, $index) {
                     return 'form.link_type.target.modalLayouts.choices.'.$value;
                 },
                 'choices_as_values'              => true,

Bundle/MediaBundle/Helper/RemoteVideo/RemoteVideoHandler.php

Doc Comments +5 added lines, -5 removed lines

@@ -108,11 +108,11 @@ 
     }
 
     /**
-     * @param $link
+     * @param string $link
      *
      * @throws VideoException
      *
-     * @return mixed
+     * @return string|null
      */
     public function isolateVimeoVideoCode($link)
     {
@@ -126,11 +126,11 @@ 
     }
 
     /**
-     * @param $link
+     * @param string $link
      *
      * @throws VideoException
      *
-     * @return mixed
+     * @return string|null
      */
     public function isolateYoutubeVideoCode($link)
     {
@@ -144,7 +144,7 @@ 
     }
 
     /**
-     * @param $link
+     * @param string $link
      *
      * @throws VideoException
      *

Bundle/PageBundle/Entity/Traits/WebViewTrait.php

Doc Comments +3 added lines

@@ -182,6 +182,9 @@
         return PropertyAccess::createPropertyAccessor()->getValue($this->translate($currentLocale, false), 'getUrl');
     }
 
+    /**
+     * @param string $name
+     */
     public function setUrl($name, $locale = null)
     {
         $this->translate($locale)->setUrl($name);

Bundle/WidgetMapBundle/Builder/WidgetMapBuilder.php

Doc Comments +1 added lines, -1 removed lines

@@ -140,7 +140,7 @@
      * Create a $slot array that'll contain, for each slot, a widgetmap id as key and a widgetmap as value
      * Do not keeps widgetMaps that are overwrited.
      *
-     * @param $widgetMaps
+     * @param WidgetMap[] $widgetMaps
      *
      * @return array
      */

Bundle/WidgetMapBundle/Command/WidgetMapOverwriteValidationCommand.php

Doc Comments +1 added lines, -1 removed lines

@@ -30,7 +30,7 @@
      *
      * @throws \Exception
      *
-     * @return bool
+     * @return boolean|null
      */
     protected function execute(InputInterface $input, OutputInterface $output)
     {

Bundle/WidgetMapBundle/Warmer/WidgetDataWarmer.php

Doc Comments +2 added lines, -2 removed lines

@@ -72,7 +72,7 @@ 
      * Inject Widgets in View's builtWidgetMap.
      *
      * @param View $view
-     * @param $viewWidgets
+     * @param Widget[] $viewWidgets
      */
     private function injectWidgets(View $view, $viewWidgets)
     {
@@ -183,7 +183,7 @@ 
      * @throws \Throwable
      * @throws \TypeError
      *
-     * @return array
+     * @return Widget[]
      */
     private function setAssociatedEntities(array $repositories)
     {

Spacing +1 added lines, -1 removed lines

@@ -193,7 +193,7 @@
             foreach ($findMethods as $findMethod => $associatedEntitiesToWarm) {
 
                 //Extract ids to search
-                $idsToSearch = array_map(function ($associatedEntityToWarm) {
+                $idsToSearch = array_map(function($associatedEntityToWarm) {
                     return $associatedEntityToWarm->getEntityId();
                 }, $associatedEntitiesToWarm);
 

Tests/Features/Context/MinkContext.php

Doc Comments +1 added lines, -1 removed lines

@@ -68,7 +68,7 @@
 
     /**
      * @param Element   $page
-     * @param float|int $timeout
+     * @param integer $timeout
      */
     public function assertPageAddress($page, $timeout = 10000)
     {

Bundle/WidgetBundle/Command/CreateWidgetCommand.php

Spacing +6 added lines, -6 removed lines

@@ -238,7 +238,7 @@ 
             ]);
 
             $question = new Question($questionHelper->getQuestion('Widget name', $input->getOption('bundle-name')));
-            $question->setValidator(function ($answer) {
+            $question->setValidator(function($answer) {
                 return self::validateWidgetName($answer, false);
             });
 
@@ -285,7 +285,7 @@ 
             ]);
 
             $question = new Question($questionHelper->getQuestion('Parent widget name', false));
-            $question->setValidator(function ($answer) {
+            $question->setValidator(function($answer) {
                 return self::validateWidgetName($answer, false);
             });
             $parent = $questionHelper->ask($input, $output, $question);
@@ -310,7 +310,7 @@ 
         ]);
 
         $question = new Question($questionHelper->getQuestion('Target directory', $dir), $dir);
-        $question->setValidator(function ($dir) use ($bundle, $namespace) {
+        $question->setValidator(function($dir) use ($bundle, $namespace) {
             return Validators::validateTargetDir($dir, $bundle, $namespace);
         });
         $dir = $questionHelper->ask($input, $output, $question);
@@ -486,7 +486,7 @@ 
         }
         $output->writeln('');
 
-        $fieldValidator = function ($type) use ($types) {
+        $fieldValidator = function($type) use ($types) {
             if (!in_array($type, $types)) {
                 throw new \InvalidArgumentException(sprintf('Invalid type "%s".', $type));
             }
@@ -494,7 +494,7 @@ 
             return $type;
         };
 
-        $lengthValidator = function ($length) {
+        $lengthValidator = function($length) {
             if (!$length) {
                 return $length;
             }
@@ -516,7 +516,7 @@ 
 
             $question = new Question($questionHelper->getQuestion('New field name (press <return> to stop adding fields)', null));
             $question->setValidator(
-                function ($name) use ($fields, $generator) {
+                function($name) use ($fields, $generator) {
                     if (isset($fields[$name]) || 'id' == $name) {
                         throw new \InvalidArgumentException(sprintf('Field "%s" is already defined.', $name));
                     }

Bundle/CoreBundle/Repository/ViewRepository.php

Spacing +1 added lines, -1 removed lines

@@ -89,7 +89,7 @@
             $pageId = $page->getId();
             $viewReference = array_filter(
                 $viewReferences,
-                function ($e) use ($pageId) {
+                function($e) use ($pageId) {
                     return $e->getViewId() == $pageId;
                 });
             if (!empty($viewReference[0])) {