cross-solution / YAWIK

module/Core/src/Factory/OptionsAbstractFactory.php

Doc Comments +1 added lines, -1 removed lines

@@ -176,7 +176,7 @@
      *
      * @param string $fullName
      *
-     * @return array|bool
+     * @return string
      */
     protected function getOptionsConfig($fullName)
     {

module/Core/src/Factory/Service/RestClientFactory.php

Doc Comments +1 added lines, -1 removed lines

@@ -58,7 +58,7 @@
     }
 
     /**
-     * @return mixed
+     * @return null|string
      */
     abstract protected function getUri();
 

module/Core/src/Filter/File/Entity.php

Doc Comments +2 added lines, -2 removed lines

@@ -38,7 +38,7 @@ 
     }
 
     /**
-     * @return mixed
+     * @return \Auth\Entity\UserInterface
      */
     public function getUser()
     {
@@ -68,7 +68,7 @@ 
     }
 
     /**
-     * @param mixed $repositories
+     * @param mixed $repository
      *
      * @return self
      */

Spacing +2 added lines, -2 removed lines

@@ -92,7 +92,7 @@ 
 
     public function getFileEntity()
     {
-        if (! $this->fileEntity instanceof FileInterface) {
+        if (!$this->fileEntity instanceof FileInterface) {
             throw new \RuntimeException('No file entity set or it does not implement \Core\Entity\FileInterface.');
         }
 
@@ -101,7 +101,7 @@ 
 
     public function filter($value)
     {
-        if (! is_array($value) || ! isset($value['tmp_name']) || ((isset($value['error']) && UPLOAD_ERR_NO_FILE == $value['error']))) {
+        if (!is_array($value) || !isset($value['tmp_name']) || ((isset($value['error']) && UPLOAD_ERR_NO_FILE == $value['error']))) {
             return null;
         }
 

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,
                 ],
             ]