Victoire / victoire

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])) {

Bundle/CoreBundle/Form/ViewType.php

Spacing +5 added lines, -5 removed lines

@@ -54,18 +54,18 @@ 
      */
     public function buildForm(FormBuilderInterface $builder, array $options)
     {
-        $builder->addEventListener(FormEvents::PRE_SET_DATA, function (FormEvent $event) {
+        $builder->addEventListener(FormEvents::PRE_SET_DATA, function(FormEvent $event) {
             $view = $event->getData();
             $form = $event->getForm();
 
             $this->isNew = !$view || null === $view->getId();
 
             if ($this->isNew) {
-                $getAllTemplateWithoutMe = function (EntityRepository $tr) {
+                $getAllTemplateWithoutMe = function(EntityRepository $tr) {
                     return $tr->getAll()->getInstance();
                 };
             } else {
-                $getAllTemplateWithoutMe = function (EntityRepository $tr) use ($view) {
+                $getAllTemplateWithoutMe = function(EntityRepository $tr) use ($view) {
                     return $tr->getAll()
                         ->getInstance()
                         ->andWhere('template.id != :templateId')
@@ -84,11 +84,11 @@ 
             //If view is an Article BEP, we do not allow to choose parent because it will be set automatically
             if (!$view instanceof ArticleTemplate && ClassUtils::getClass($view) != 'Victoire\Bundle\TemplateBundle\Entity\Template') {
                 if (!$view || null === $view->getId()) {
-                    $getAllPageWithoutMe = function (EntityRepository $repo) {
+                    $getAllPageWithoutMe = function(EntityRepository $repo) {
                         return $repo->getAll()->getInstance();
                     };
                 } else {
-                    $getAllPageWithoutMe = function (EntityRepository $repo) use ($view) {
+                    $getAllPageWithoutMe = function(EntityRepository $repo) use ($view) {
                         return $repo->getAll()
                             ->getInstance()
                             ->andWhere('view.id != :pageId')

Bundle/BlogBundle/Form/ArticleType.php

Spacing +7 added lines, -7 removed lines

@@ -52,7 +52,7 @@ 
             ->add('author', null, [
                 'label'         => 'form.article.type.author.label',
                 'required'      => true,
-                'query_builder' => function (EntityRepository $er) {
+                'query_builder' => function(EntityRepository $er) {
                     return $er->createQueryBuilder('user')
                         ->where('user.roles LIKE :roles')
                         ->setParameter('roles', '%ROLE_VICTOIRE%');
@@ -60,7 +60,7 @@ 
             ])
             ->remove('visibleOnFront');
 
-        $builder->get('blog')->addEventListener(FormEvents::PRE_SET_DATA, function (FormEvent $event) {
+        $builder->get('blog')->addEventListener(FormEvents::PRE_SET_DATA, function(FormEvent $event) {
             $data = $event->getData();
             $parent = $event->getForm()->getParent();
             $this->manageCategories($data, $parent);
@@ -68,19 +68,19 @@ 
             $this->manageLocales($data, $parent);
         });
 
-        $builder->get('blog')->addEventListener(FormEvents::PRE_SUBMIT, function (FormEvent $event) {
+        $builder->get('blog')->addEventListener(FormEvents::PRE_SUBMIT, function(FormEvent $event) {
             $data = $event->getData();
             $parent = $event->getForm()->getParent();
             $this->manageCategories($data, $parent);
             $this->manageLocales($data, $parent);
         });
 
-        $builder->addEventListener(FormEvents::PRE_SET_DATA, function (FormEvent $event) {
+        $builder->addEventListener(FormEvents::PRE_SET_DATA, function(FormEvent $event) {
             $data = $event->getData();
             $form = $event->getForm();
             $this->manageTags($data, $form);
         });
-        $builder->addEventListener(FormEvents::PRE_SUBMIT, function (FormEvent $event) {
+        $builder->addEventListener(FormEvents::PRE_SUBMIT, function(FormEvent $event) {
             $form = $event->getForm();
             $data = $form->getData();
             $this->manageTags($data, $form);
@@ -95,7 +95,7 @@ 
         $form->add('tags', TagsType::class, [
             'required'      => false,
             'multiple'      => true,
-            'query_builder' => function (TagRepository $er) use ($data) {
+            'query_builder' => function(TagRepository $er) use ($data) {
                 $qb = $er->filterByBlog($data->getBlog())->getInstance();
                 $er->clearInstance();
 
@@ -178,7 +178,7 @@ 
 
         if (!$form->getData()->getTemplate()) {
             if ($articleTemplateRepo->filterByBlog($blog_id)->getCount('parent')->run('getSingleScalarResult') > 1) {
-                $articleTemplates = function (ArticleTemplateRepository $repo) use ($blog_id) {
+                $articleTemplates = function(ArticleTemplateRepository $repo) use ($blog_id) {
                     return $repo->filterByBlog($blog_id)->getInstance();
                 };
                 $form->add('template', null, [