cross-solution / YAWIK

module/Core/src/Form/CollectionContainer.php

Doc Comments +1 added lines, -1 removed lines

@@ -179,7 +179,7 @@
     
     /**
      * @throws \RuntimeException
-     * @return Collection
+     * @return \Core\Entity\EntityInterface
      */
     protected function getCollection()
     {

Spacing +2 added lines, -2 removed lines

@@ -80,12 +80,12 @@
             $collection[] = $this->newEntry;
             $form = $this->buildForm($key, $this->newEntry);
             $eventManager = $form->getEventManager();
-            $eventManager->attach(CoreForm::EVENT_IS_VALID, function (Event $event) use ($collection) {
+            $eventManager->attach(CoreForm::EVENT_IS_VALID, function(Event $event) use ($collection) {
                 if (!$event->getParam('isValid')) {
                     $collection->removeElement($this->newEntry);
                 }
             });
-            $eventManager->attach(CoreForm::EVENT_PREPARE, function (Event $event) use ($collection) {
+            $eventManager->attach(CoreForm::EVENT_PREPARE, function(Event $event) use ($collection) {
                 $this->setupForm($event->getTarget(), $collection->indexOf($this->newEntry));
             });
             

module/Core/src/Form/Container.php

Doc Comments +1 added lines, -1 removed lines

@@ -98,7 +98,7 @@
     /**
      * Gets an iterator to iterate over the enabled formulars.
      *
-     * @return \ArrayIterator
+     * @return PriorityList
      * @see IteratorAggregate::getIterator()
      */
     public function getIterator()

Spacing +11 added lines, -11 removed lines

@@ -324,7 +324,7 @@ 
         $formAction = $formInstance->getAttribute('action');
 
         if (empty($formAction)) {
-            $formInstance->setAttribute('action', '?form=' . $formName);
+            $formInstance->setAttribute('action', '?form='.$formName);
         }
 
         // @TODO: [ZF3] which one is correct? $form[options][label] or $form[options]
@@ -396,7 +396,7 @@ 
                 $spec->setParent($this);
             }
 
-            $spec = [ '__instance__' => $spec, 'name' => $key, 'entity' => '*' ];
+            $spec = ['__instance__' => $spec, 'name' => $key, 'entity' => '*'];
         }
 
         if (!is_array($spec)) {
@@ -515,7 +515,7 @@ 
         }
         $this->activeForms = array_filter(
             $this->activeForms,
-            function ($item) use ($key) {
+            function($item) use ($key) {
                 return !in_array($item, $key);
             }
         );
@@ -529,7 +529,7 @@ 
      * @throws \InvalidArgumentException
      * @return Container
      */
-    public function setEntity($entity, $key='*')
+    public function setEntity($entity, $key = '*')
     {
         if (!$entity instanceof EntityInterface) {
             throw new \InvalidArgumentException(sprintf('$entity must be instance of %s', EntityInterface::class));
@@ -551,7 +551,7 @@ 
      *
      * @return \Core\Entity\EntityInterface
      */
-    public function getEntity($key='*')
+    public function getEntity($key = '*')
     {
         return isset($this->entities[$key]) ? $this->entities[$key] : null;
     }
@@ -753,10 +753,10 @@ 
         $actualKey = $this->getActiveFormActual();
         if (isset($actualKey)) {
             $forms = array_keys($this->forms);
-            $formsFlip =  array_flip($forms);
+            $formsFlip = array_flip($forms);
             $index = $formsFlip[$actualKey];
             if (0 < $index) {
-                $key = $forms[$index-1];
+                $key = $forms[$index - 1];
             }
         }
         return $key;
@@ -773,10 +773,10 @@ 
         $actualKey = $this->getActiveFormActual();
         if (isset($actualKey)) {
             $forms = array_keys($this->forms);
-            $formsFlip =  array_flip($forms);
+            $formsFlip = array_flip($forms);
             $index = $formsFlip[$actualKey];
             if ($index < count($forms) - 1) {
-                $key = $forms[$index+1];
+                $key = $forms[$index + 1];
             }
         }
         return $key;
@@ -790,7 +790,7 @@ 
      */
     public function formatAction($name)
     {
-        return sprintf('%s%s', $this->hasParent() ? $this->getName() . '.' : '', $name);
+        return sprintf('%s%s', $this->hasParent() ? $this->getName().'.' : '', $name);
     }
 
     /**
@@ -800,7 +800,7 @@ 
     public function getActionFor($key)
     {
         if (isset($this->forms[$key])) {
-            return '?form=' . $this->formatAction($this->forms[$key]['name']);
+            return '?form='.$this->formatAction($this->forms[$key]['name']);
         }
     }
 }

module/Core/src/Form/Form.php

Doc Comments +2 added lines, -1 removed lines

@@ -77,6 +77,7 @@ 
 
     /**
      * @param string $description
+     * @param double[] $params
      *
      * @return $this
      */
@@ -258,7 +259,7 @@ 
     }
 
     /**
-     * @param $spec
+     * @param string $spec
      *
      * @return $this
      */

Indentation +2 added lines, -2 removed lines

@@ -25,8 +25,8 @@
  */
 class Form extends ZendForm implements
     DescriptionAwareFormInterface,
-                                       DisableElementsCapableInterface,
-                                       FormParentInterface
+                                        DisableElementsCapableInterface,
+                                        FormParentInterface
 {
     use EventManagerAwareTrait, HydratorStrategyAwareTrait;
     

Spacing +2 added lines, -2 removed lines

@@ -368,9 +368,9 @@
                 if (!$inputExists && $required) {
                     $fieldsetName = '';
                     if ($fieldset->hasAttribute('name')) {
-                        $fieldsetName = 'in Fieldset "' . $fieldset->getAttribute('name') . '" ';
+                        $fieldsetName = 'in Fieldset "'.$fieldset->getAttribute('name').'" ';
                     }
-                    throw new \RuntimeException('input for "' . $name . '" ' . $fieldsetName . 'is required but a input-field with this name is not defined');
+                    throw new \RuntimeException('input for "'.$name.'" '.$fieldsetName.'is required but a input-field with this name is not defined');
                 }
             }
         }

module/Core/src/Form/Hydrator/Strategy/TreeSelectStrategy.php

Doc Comments +1 added lines, -1 removed lines

@@ -97,7 +97,7 @@
     /**
      * Set the allow multiple selections flag.
      *
-     * @param Callable|bool $flagOrCallback When a Callable is passed, it must return bool.
+     * @param \Closure $flagOrCallback When a Callable is passed, it must return bool.
      *
      * @return self
      */

Spacing +1 added lines, -1 removed lines

@@ -222,7 +222,7 @@
      */
     private function findLeaf(NodeInterface $leaf, $value)
     {
-        $parts = is_array($value) ? $value : explode($this->shouldUseNames() ? ' | ': '-', $value);
+        $parts = is_array($value) ? $value : explode($this->shouldUseNames() ? ' | ' : '-', $value);
         $value = array_shift($parts);
 
         /* @var NodeInterface $item */

module/Core/src/Form/Hydrator/TreeHydrator.php

Doc Comments +1 added lines, -1 removed lines

@@ -93,7 +93,7 @@
      * @param  array  $data Form values
      * @param  Collection $object
      *
-     * @return object
+     * @return NodeInterface
      */
     public function hydrate(array $data, $object)
     {

Spacing +3 added lines, -3 removed lines

@@ -53,7 +53,7 @@ 
         $data = [];
         $this->flattenTree($object, $data);
 
-        return ['items' => $data ];
+        return ['items' => $data];
     }
 
     /**
@@ -77,7 +77,7 @@ 
 
         if ($tree->hasChildren()) {
             foreach ($tree->getChildren() as $i => $child) {
-                $this->flattenTree($child, $data, $curId . '-' . ($i + 1));
+                $this->flattenTree($child, $data, $curId.'-'.($i + 1));
             }
         }
     }
@@ -114,7 +114,7 @@ 
          * unflatten tree
          */
         $items = $data['items'];
-        $tree = [ '__root__' => array_shift($items) ];
+        $tree = ['__root__' => array_shift($items)];
 
         foreach ($items as $item) {
             $parent = substr($item['current'], 0, strrpos($item['current'], '-'));

module/Core/src/Form/SearchForm.php

Doc Comments +6 added lines

@@ -47,6 +47,9 @@ 
         $this->buttonsIterator->isLIFO(false);
     }
 
+    /**
+     * @param string $name
+     */
     public function setButtonElement($name)
     {
         return $this->setOption('button_element', $name);
@@ -182,6 +185,9 @@ 
         return $this->buttonsIterator;
     }
 
+    /**
+     * @param string $name
+     */
     public function addButton($name, $priority = 0, $type = 'button')
     {
         if (is_array($name)) {

Spacing +1 added lines, -1 removed lines

@@ -167,7 +167,7 @@
                     'label' => $label,
                 ],
                 'attributes' => [
-                    'class' => 'btn btn-' . ('submit' == $type ? 'primary' : 'default'),
+                    'class' => 'btn btn-'.('submit' == $type ? 'primary' : 'default'),
                     'type'  => $type,
                 ],
             ]

module/Core/src/Form/View/Helper/FormContainer.php

Doc Comments +3 added lines

@@ -74,6 +74,9 @@
         return $content;
     }
 
+    /**
+     * @param string $layout
+     */
     public function renderElement($element, $layout, $parameter)
     {
         $parameterPartial = $parameter;

Indentation +2 added lines, -2 removed lines

@@ -84,8 +84,8 @@
         if ($element instanceof ViewPartialProviderInterface) {
             $parameterPartial = array_merge(array('element' => $element, 'layout' => $layout), $parameterPartial);
             $content .= $this->getView()->partial(
-                             $element->getViewPartial(),
-                             $parameterPartial
+                                $element->getViewPartial(),
+                                $parameterPartial
             );
         } elseif ($element instanceof ViewHelperProviderInterface) {
             $helper = $element->getViewHelper();

Spacing +3 added lines, -3 removed lines

@@ -64,7 +64,7 @@ 
         }
 
         if (isset($parameter['render_label']) && $parameter['render_label'] && ($label = $container->getLabel())) {
-            $content .= '<div class="container-headline"><h3>' . $this->getView()->translate($label) . '</h3></div>';
+            $content .= '<div class="container-headline"><h3>'.$this->getView()->translate($label).'</h3></div>';
         }
         foreach ($container as $element) {
             $content .= $this->renderElement($element, $layout, $parameter);
@@ -99,9 +99,9 @@ 
         } elseif ($element instanceof CoreSummaryForm) {
             $content .= $this->getView()->summaryForm($element);
         } elseif ($element instanceof Container) {
-            $content.= $this->render($element, $layout, $parameter);
+            $content .= $this->render($element, $layout, $parameter);
         } else {
-            $content.= $this->getView()->form($element, $layout, $parameter);
+            $content .= $this->getView()->form($element, $layout, $parameter);
         }
 
         return $content;

module/Core/src/Form/View/Helper/FormRowCombined.php

Doc Comments +1 added lines, -1 removed lines

@@ -73,7 +73,7 @@
      *
      * Proxies to {@link render()}.
      *
-     * @param null|ElementInterface $element
+     * @param null|ElementInterface $elements
      * @param null|string           $labelPosition
      * @param bool                  $renderErrors
      * @return string|FormRow

Spacing +1 added lines, -1 removed lines

@@ -62,7 +62,7 @@
         
         
         return sprintf(
-            '<div class="controls controls-row ' . $form_row_class . '">%s%s</div>',
+            '<div class="controls controls-row '.$form_row_class.'">%s%s</div>',
             $labelMarkup,
             $markups
         );

module/Core/src/Form/View/Helper/SearchForm.php

Doc Comments +5 added lines

@@ -88,6 +88,11 @@
         return $content;
     }
 
+    /**
+     * @param FormInterface $form
+     *
+     * @return null|FormInterface
+     */
     public function renderElements($form, $colMap = null, $buttonsSpan = null)
     {
         if ($form instanceof ViewPartialProviderInterface) {

Indentation +11 added lines, -11 removed lines

@@ -67,8 +67,8 @@ 
         $content = $this->renderElements($form, $colMap, $buttonsSpan);
 
         return $this->openTag($form)
-             . '<div class="row" style="padding: 0 15px;">'
-             . $content . '</div>' . $this->closeTag();
+                . '<div class="row" style="padding: 0 15px;">'
+                . $content . '</div>' . $this->closeTag();
     }
 
     public function renderButtons($buttons)
@@ -81,8 +81,8 @@ 
             $attrs = $helper->createAttributesString($button->getAttributes());
 
             $content.= '<button ' . $attrs . '>'
-                       . $this->getTranslator()->translate($button->getLabel(), $this->getTranslatorTextDomain())
-                       . '</button>';
+                        . $this->getTranslator()->translate($button->getLabel(), $this->getTranslatorTextDomain())
+                        . '</button>';
         }
 
         return $content;
@@ -118,15 +118,15 @@ 
 
             if ($element->getName() == $form->getButtonElement()) {
                 $content.='<div class="input-group col-md-' . $cols . '">'
-                              . $formElement($element)
-                              . '<div class="input-group-btn search-form-buttons" style="width: 1px;">'
-                              . $this->renderButtons($form->getButtons()) . '</div>'
-                              . '</div>';
+                                . $formElement($element)
+                                . '<div class="input-group-btn search-form-buttons" style="width: 1px;">'
+                                . $this->renderButtons($form->getButtons()) . '</div>'
+                                . '</div>';
                 $buttonsRendered = true;
             } else {
                 $content .= '<div class="input-group col-md-' . $cols . '">'
-                          . $formElement($element)
-                          . '</div>';
+                            . $formElement($element)
+                            . '</div>';
             }
 
             $i += 1;
@@ -137,7 +137,7 @@ 
                 $buttonsSpan = $form->getOption('buttons_span') ?: 12;
             }
             $content .= '<div class="input-group search-form-buttons col-md-' . $buttonsSpan . ' text-right">'
-                      . '<div class="btn-group">' . $this->renderButtons($form->getButtons()) .'</div></div>';
+                        . '<div class="btn-group">' . $this->renderButtons($form->getButtons()) .'</div></div>';
         }
 
         return $content;

Spacing +12 added lines, -12 removed lines

@@ -35,7 +35,7 @@ 
      * @param array $colMap
      * @return Form|string
      */
-    public function __invoke(FormInterface $form = null, $colMap=null)
+    public function __invoke(FormInterface $form = null, $colMap = null)
     {
         if (!$form) {
             return $this;
@@ -44,7 +44,7 @@ 
         return $this->render($form, $colMap);
     }
 
-    public function render(FormInterface $form, $colMap=null, $buttonsSpan = null)
+    public function render(FormInterface $form, $colMap = null, $buttonsSpan = null)
     {
         $headscript = $this->getView()->plugin('headscript');
         $basepath   = $this->getView()->plugin('basepath');
@@ -61,14 +61,14 @@ 
         $form->prepare();
 
         if ($form instanceof ViewPartialProviderInterface) {
-            return $this->getView()->partial($form->getViewPartial(), [ 'element' => $form, 'colMap' => $colMap, 'buttonsSpan' => $buttonsSpan ]);
+            return $this->getView()->partial($form->getViewPartial(), ['element' => $form, 'colMap' => $colMap, 'buttonsSpan' => $buttonsSpan]);
         }
 
         $content = $this->renderElements($form, $colMap, $buttonsSpan);
 
         return $this->openTag($form)
              . '<div class="row" style="padding: 0 15px;">'
-             . $content . '</div>' . $this->closeTag();
+             . $content.'</div>'.$this->closeTag();
     }
 
     public function renderButtons($buttons)
@@ -80,7 +80,7 @@ 
             $button->removeAttribute('name');
             $attrs = $helper->createAttributesString($button->getAttributes());
 
-            $content.= '<button ' . $attrs . '>'
+            $content .= '<button '.$attrs.'>'
                        . $this->getTranslator()->translate($button->getLabel(), $this->getTranslatorTextDomain())
                        . '</button>';
         }
@@ -91,7 +91,7 @@ 
     public function renderElements($form, $colMap = null, $buttonsSpan = null)
     {
         if ($form instanceof ViewPartialProviderInterface) {
-            return $this->getView()->partial($form->getViewPartial(), [ 'element' => $form, 'colMap' => $colMap, 'buttonsSpan' => $buttonsSpan ]);
+            return $this->getView()->partial($form->getViewPartial(), ['element' => $form, 'colMap' => $colMap, 'buttonsSpan' => $buttonsSpan]);
         }
 
         if (null === $colMap) {
@@ -101,7 +101,7 @@ 
         $formElement = $this->getView()->plugin('formElement');
         $content = '';
         $buttonsRendered = false;
-        $i=0;
+        $i = 0;
         foreach ($form as $element) {
             if ($element instanceof Hidden) {
                 $content .= $formElement($element);
@@ -117,14 +117,14 @@ 
             }
 
             if ($element->getName() == $form->getButtonElement()) {
-                $content.='<div class="input-group col-md-' . $cols . '">'
+                $content .= '<div class="input-group col-md-'.$cols.'">'
                               . $formElement($element)
                               . '<div class="input-group-btn search-form-buttons" style="width: 1px;">'
-                              . $this->renderButtons($form->getButtons()) . '</div>'
+                              . $this->renderButtons($form->getButtons()).'</div>'
                               . '</div>';
                 $buttonsRendered = true;
             } else {
-                $content .= '<div class="input-group col-md-' . $cols . '">'
+                $content .= '<div class="input-group col-md-'.$cols.'">'
                           . $formElement($element)
                           . '</div>';
             }
@@ -136,8 +136,8 @@ 
             if (null === $buttonsSpan) {
                 $buttonsSpan = $form->getOption('buttons_span') ?: 12;
             }
-            $content .= '<div class="input-group search-form-buttons col-md-' . $buttonsSpan . ' text-right">'
-                      . '<div class="btn-group">' . $this->renderButtons($form->getButtons()) .'</div></div>';
+            $content .= '<div class="input-group search-form-buttons col-md-'.$buttonsSpan.' text-right">'
+                      . '<div class="btn-group">'.$this->renderButtons($form->getButtons()).'</div></div>';
         }
 
         return $content;