cross-solution / YAWIK

module/Core/src/Core/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 +2 added lines, -2 removed lines

@@ -123,7 +123,7 @@ 
     public function extract($value)
     {
         if (!$value instanceOf AbstractLeafs) {
-            throw new \InvalidArgumentException('$value must be an instance of ' . AbstractLeafs::class);
+            throw new \InvalidArgumentException('$value must be an instance of '.AbstractLeafs::class);
         }
         /* @var AbstractLeafs $value
          * @var NodeInterface $item */
@@ -175,7 +175,7 @@ 
      */
     private function getItemValue(NodeInterface $item)
     {
-        $parts = [ $item->getValue() ];
+        $parts = [$item->getValue()];
 
         while ($item = $item->getParent()) {
             $parts[] = $item->getValue();

module/Core/src/Core/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];
 
     }
 
@@ -79,7 +79,7 @@ 
 
         if ($tree->hasChildren()) {
             foreach ($tree->getChildren() as $i => $child) {
-                $this->flattenTree($child, $data, $curId . '-' . ($i + 1) );
+                $this->flattenTree($child, $data, $curId.'-'.($i + 1));
             }
         }
     }
@@ -116,7 +116,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/config/module.config.php

Spacing +30 added lines, -30 removed lines

@@ -12,7 +12,7 @@ 
  * @license   MIT
  */
 
-$doctrineConfig = include __DIR__ . '/doctrine.config.php';
+$doctrineConfig = include __DIR__.'/doctrine.config.php';
 
 
 return array(
@@ -20,7 +20,7 @@ 
     'doctrine' => $doctrineConfig,
 
     'options' => [
-        'Core/MailServiceOptions' => [ 'class' => '\Core\Options\MailServiceOptions' ],
+        'Core/MailServiceOptions' => ['class' => '\Core\Options\MailServiceOptions'],
         ],
     
     'Core' => array(
@@ -40,7 +40,7 @@ 
                      'name' => 'stream',
                     'priority' => 1000,
                     'options' => array(
-                         'stream' => __DIR__ .'/../../../log/yawik.log',
+                         'stream' => __DIR__.'/../../../log/yawik.log',
                     ),
                  ),
             ),
@@ -51,7 +51,7 @@ 
                      'name' => 'stream',
                     'priority' => 1000,
                     'options' => array(
-                         'stream' => __DIR__ .'/../../../log/mails.log',
+                         'stream' => __DIR__.'/../../../log/mails.log',
                     ),
                  ),
             ),
@@ -59,7 +59,7 @@ 
         'ErrorLogger' => array(
             'service' => 'Core/ErrorLogger',
             'config'  => array(
-                'stream' => __DIR__ . '/../../../log/error.log',
+                'stream' => __DIR__.'/../../../log/error.log',
                 'log_errors' => true,
                 'log_exceptions' => true,
             ),
@@ -221,17 +221,17 @@ 
         'translation_file_patterns' => array(
             [
                 'type' => 'gettext',
-                'base_dir' => __DIR__ . '/../language',
+                'base_dir' => __DIR__.'/../language',
                 'pattern' => '%s.mo',
             ],
             [
                 'type'     => 'phparray',
-                'base_dir' => __DIR__ . '/../language',
+                'base_dir' => __DIR__.'/../language',
                 'pattern' => 'Zend_Validate.%s.php',
             ],
             [
                 'type'     => 'phparray',
-                'base_dir' => __DIR__ . '/../language',
+                'base_dir' => __DIR__.'/../language',
                 'pattern' => 'Zend_Captcha.%s.php',
             ]
         ),
@@ -300,32 +300,32 @@ 
         'exception_template' => 'error/index',
         // Map template to files. Speeds up the lookup through the template stack.
         'template_map' => array(
-            'noscript-notice' => __DIR__ . '/../view/layout/_noscript-notice.phtml',
-            'layout/layout' => __DIR__ . '/../view/layout/layout.phtml',
-            'error/404' => __DIR__ . '/../view/error/404.phtml',
-            'error/403' => __DIR__ . '/../view/error/403.phtml',
-            'error/index' => __DIR__ . '/../view/error/index.phtml',
-            'main-navigation' => __DIR__ . '/../view/partial/main-navigation.phtml',
-            'pagination-control' => __DIR__ . '/../view/partial/pagination-control.phtml',
-            'core/loading-popup' => __DIR__ . '/../view/partial/loading-popup.phtml',
-            'core/notifications' => __DIR__ . '/../view/partial/notifications.phtml',
-            'form/core/buttons' => __DIR__ . '/../view/form/buttons.phtml',
-            'core/social-buttons' => __DIR__ . '/../view/partial/social-buttons.phtml',
-            'form/core/privacy' => __DIR__ . '/../view/form/privacy.phtml',
-            'core/form/permissions-fieldset' => __DIR__ . '/../view/form/permissions-fieldset.phtml',
-            'core/form/permissions-collection' => __DIR__ . '/../view/form/permissions-collection.phtml',
-            'core/form/container-view' => __DIR__ . '/../view/form/container.view.phtml',
-            'core/form/tree-manage.view' => __DIR__ . '/../view/form/tree-manage.view.phtml',
-            'core/form/tree-manage.form' => __DIR__ . '/../view/form/tree-manage.form.phtml',
-            'core/form/tree-add-item' => __DIR__ . '/../view/form/tree-add-item.phtml',
-            'mail/header' =>  __DIR__ . '/../view/mail/header.phtml',
-            'mail/footer' =>  __DIR__ . '/../view/mail/footer.phtml',
-            'mail/footer.en' =>  __DIR__ . '/../view/mail/footer.en.phtml',
+            'noscript-notice' => __DIR__.'/../view/layout/_noscript-notice.phtml',
+            'layout/layout' => __DIR__.'/../view/layout/layout.phtml',
+            'error/404' => __DIR__.'/../view/error/404.phtml',
+            'error/403' => __DIR__.'/../view/error/403.phtml',
+            'error/index' => __DIR__.'/../view/error/index.phtml',
+            'main-navigation' => __DIR__.'/../view/partial/main-navigation.phtml',
+            'pagination-control' => __DIR__.'/../view/partial/pagination-control.phtml',
+            'core/loading-popup' => __DIR__.'/../view/partial/loading-popup.phtml',
+            'core/notifications' => __DIR__.'/../view/partial/notifications.phtml',
+            'form/core/buttons' => __DIR__.'/../view/form/buttons.phtml',
+            'core/social-buttons' => __DIR__.'/../view/partial/social-buttons.phtml',
+            'form/core/privacy' => __DIR__.'/../view/form/privacy.phtml',
+            'core/form/permissions-fieldset' => __DIR__.'/../view/form/permissions-fieldset.phtml',
+            'core/form/permissions-collection' => __DIR__.'/../view/form/permissions-collection.phtml',
+            'core/form/container-view' => __DIR__.'/../view/form/container.view.phtml',
+            'core/form/tree-manage.view' => __DIR__.'/../view/form/tree-manage.view.phtml',
+            'core/form/tree-manage.form' => __DIR__.'/../view/form/tree-manage.form.phtml',
+            'core/form/tree-add-item' => __DIR__.'/../view/form/tree-add-item.phtml',
+            'mail/header' =>  __DIR__.'/../view/mail/header.phtml',
+            'mail/footer' =>  __DIR__.'/../view/mail/footer.phtml',
+            'mail/footer.en' =>  __DIR__.'/../view/mail/footer.en.phtml',
             //'startpage' => __DIR__ . '/../view/layout/startpage.phtml',
         ),
         // Where to look for view templates not mapped above
         'template_path_stack' => array(
-            __DIR__ . '/../view',
+            __DIR__.'/../view',
         ),
     ),
     'view_helpers' => array(

module/Core/src/Core/Factory/Form/Tree/SelectFactory.php

Spacing +2 added lines, -2 removed lines

@@ -142,7 +142,7 @@ 
      *
      * @return array
      */
-    protected function createValueOptions(NodeInterface $node, $allowSelectNodes = false, $isRoot=true)
+    protected function createValueOptions(NodeInterface $node, $allowSelectNodes = false, $isRoot = true)
     {
         $key    = $isRoot ? $node->getValue() : $this->getItemValue($node);
         $name   = $node->getName();
@@ -181,7 +181,7 @@ 
      */
     private function getItemValue(NodeInterface $item)
     {
-        $parts = [ $item->getValue() ];
+        $parts = [$item->getValue()];
 
         while ($item = $item->getParent()) {
             $parts[] = $item->getValue();

module/Core/src/Core/Entity/Tree/AbstractLeafs.php

Spacing +1 added lines, -1 removed lines

@@ -71,7 +71,7 @@
                 $parent = $nextParent;
             }
             $nameParts[] = $item->getName();
-            $items[]     = join(' / ' , $nameParts);
+            $items[]     = join(' / ', $nameParts);
 
         }
 

module/Core/src/Core/Repository/RepositoryService.php

Spacing +1 added lines, -1 removed lines

@@ -38,7 +38,7 @@
             $name = "\\$namespace\\Entity\\$entityName";
         }
         
-        $repository  = $this->dm->getRepository($name);
+        $repository = $this->dm->getRepository($name);
         return $repository;
     }
     

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

Spacing +15 added lines, -15 removed lines

@@ -67,7 +67,7 @@ 
         $renderer->headscript()->appendFile($renderer->basePath('Core/js/jquery.summary-form.js'));
         
         $label = $form->getLabel();
-        $labelContent = $label ? '<div class="sf-headline"><h3>' . $this->getView()->translate($label) . '</h3></div>' : '';
+        $labelContent = $label ? '<div class="sf-headline"><h3>'.$this->getView()->translate($label).'</h3></div>' : '';
         $formContent  = $this->renderForm($form, $layout, $parameter);
         $summaryContent = $this->renderSummary($form);
         
@@ -131,8 +131,8 @@ 
     public function renderForm(SummaryFormInterface $form, $layout = Form::LAYOUT_HORIZONTAL, $parameter = array())
     {
                                                     /* @var $form SummaryFormInterface|\Core\Form\SummaryForm */
-        $renderer     = $this->getView();           /* @var $renderer \Zend\View\Renderer\PhpRenderer */
-        $formHelper   = $renderer->plugin('form');  /* @var $formHelper \Core\Form\View\Helper\Form */
+        $renderer     = $this->getView(); /* @var $renderer \Zend\View\Renderer\PhpRenderer */
+        $formHelper   = $renderer->plugin('form'); /* @var $formHelper \Core\Form\View\Helper\Form */
         $fieldset     = $form->getBaseFieldset();
         $resetPartial = false;
 
@@ -166,7 +166,7 @@ 
         $form->prepare();
         $baseFieldset = $form->getBaseFieldset();
         if (!isset($baseFieldset)) {
-            throw new \InvalidArgumentException('For the Form ' . get_class($form) . ' there is no Basefieldset');
+            throw new \InvalidArgumentException('For the Form '.get_class($form).' there is no Basefieldset');
         }
 
         $dataAttributesMarkup = '';
@@ -179,7 +179,7 @@ 
             }
         }
         
-        $markup = '<div class="panel panel-default" style="min-height: 100px;"' . $dataAttributesMarkup . '>
+        $markup = '<div class="panel panel-default" style="min-height: 100px;"'.$dataAttributesMarkup.'>
                     <div class="panel-body"><div class="sf-controls">%s</div>%s</div></div>';
 
         $view = $this->getView();
@@ -192,9 +192,9 @@ 
         
         if (($controlButtons = $form->getOption('control_buttons')) !== null)
         {
-            $buttonMarkup .= PHP_EOL . implode(PHP_EOL, array_map(function (array $buttonSpec) use ($view) {
-                return '<button type="button" class="btn btn-default btn-xs' . (isset($buttonSpec['class']) ? ' ' . $buttonSpec['class'] : '') . '">'
-                    . (isset($buttonSpec['icon']) ? '<span class="yk-icon yk-icon-' . $buttonSpec['icon'] . '"></span> ' : '')
+            $buttonMarkup .= PHP_EOL.implode(PHP_EOL, array_map(function(array $buttonSpec) use ($view) {
+                return '<button type="button" class="btn btn-default btn-xs'.(isset($buttonSpec['class']) ? ' '.$buttonSpec['class'] : '').'">'
+                    . (isset($buttonSpec['icon']) ? '<span class="yk-icon yk-icon-'.$buttonSpec['icon'].'"></span> ' : '')
                     . $view->translate($buttonSpec['label'])
                     . '</button>';
             }, $controlButtons));
@@ -234,10 +234,10 @@ 
         }
 
         if ($element instanceof ViewPartialProviderInterface) {
-            $renderer    = $this->getView();                 /* @var $renderer \Zend\View\Renderer\PhpRenderer */
+            $renderer    = $this->getView(); /* @var $renderer \Zend\View\Renderer\PhpRenderer */
             $origPartial = $element->getViewPartial();
             $partial     = "$origPartial.view";
-            $partialParams  = array(
+            $partialParams = array(
                 'element' => $element
             );
             if (!$renderer->resolver($partial)) {
@@ -253,7 +253,7 @@ 
         
         if ($element instanceof FieldsetInterface) {
             if (!$element instanceof FormInterface && $label) {
-                $markup .= '<h4>' . $label . '</h4>';
+                $markup .= '<h4>'.$label.'</h4>';
             }
             foreach ($element as $el) {
                 $markup .= $this->renderSummaryElement($el);
@@ -282,11 +282,11 @@ 
                         if (!is_array($optVal) || !array_key_exists($optionKey, $optVal['options'])) { continue; }
 
                         $optGroupLabel = isset($optVal['label']) ? $translator->translate($optVal['label']) : $optKey;
-                        $multiOptions[] = $optGroupLabel . ' / ' . $translator->translate($optVal['options'][$optionKey]);
+                        $multiOptions[] = $optGroupLabel.' / '.$translator->translate($optVal['options'][$optionKey]);
                     }
                 }
 
-                $elementValue = join(', ' , $multiOptions);
+                $elementValue = join(', ', $multiOptions);
 //                $numberOfmultiOptions = count($multiOptions);
 //                foreach ($multiOptions as $optGroupLabel => $vals) {
 //                    $elementValue[] = ($numberOfmultiOptions > 1 || $optGroupLabel !== $generalOptGroupName ? "<b>$optGroupLabel</b><br>" : '') . join(', ', $vals);
@@ -305,10 +305,10 @@ 
         $markup .= '<div class="row">';
         $col = 12;
         if ($label) {
-            $markup .= '<div class="col-md-3 yk-label"><label>' . $label . '</label></div>';
+            $markup .= '<div class="col-md-3 yk-label"><label>'.$label.'</label></div>';
             $col = 9;
         }
-        $markup .= '<div class="col-md-' . $col . '">' . $elementValue . '</div>'
+        $markup .= '<div class="col-md-'.$col.'">'.$elementValue.'</div>'
             . '</div>';
         return $markup;
     }

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

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/Core/Form/View/Helper/FormWizardContainer.php

Indentation +10 added lines, -10 removed lines

@@ -90,24 +90,24 @@
             $tabId = $containerId . '-' . strtolower($tabElement->getName());
             $tabsNav .= '<li><a data-toggle="tab" href="#' . $tabId . '">' . $translate($tabElement->getLabel()) . '</a></li>';
             $tabsContent .= '<div class="tab-pane" id="' . $tabId . '">'
-                          . $formContainer($tabElement, $layout, $parameter)
-                          . '</div>';
+                            . $formContainer($tabElement, $layout, $parameter)
+                            . '</div>';
         }
 
         $content .= '<style type="text/css">.tab-content > div > div:first-child { margin-top: 10px; }</style><div class="wizard-container" id="' . $containerId . '">'
-                  . '<ul>' . $tabsNav . '</ul>'
-                  . '<div class="tab-content">' . $tabsContent . '</div>';
+                    . '<ul>' . $tabsNav . '</ul>'
+                    . '<div class="tab-content">' . $tabsContent . '</div>';
         if ($containerParams['pager']) {
             $content .='<ul class="pager wizard">'
-                  . '<li class="previous"><a href="javascript:;">&larr; ' . $translate('previous') . '</a></li>'
-                  . '<li class="next"><a href="javascript:;">' . $translate('Next') . ' &rarr;</a></li>'
-                  . '<li class="finish' . ($containerParams['finish_enabled'] ? '' : ' disabled') . '">'
-                  . (false !== $containerParams['finish_label']
+                    . '<li class="previous"><a href="javascript:;">&larr; ' . $translate('previous') . '</a></li>'
+                    . '<li class="next"><a href="javascript:;">' . $translate('Next') . ' &rarr;</a></li>'
+                    . '<li class="finish' . ($containerParams['finish_enabled'] ? '' : ' disabled') . '">'
+                    . (false !== $containerParams['finish_label']
                      ? '<a class="pull-right" href="' . $containerParams['finish_href'] . '">'
-                       . $translate($containerParams['finish_label']) . ' &bull;</a>'
+                        . $translate($containerParams['finish_label']) . ' &bull;</a>'
                      : ''
                     )
-                  . '</li></ul>';
+                    . '</li></ul>';
         }
         $content .= '</div>';
 

Spacing +13 added lines, -13 removed lines

@@ -83,28 +83,28 @@
 
         $containerId = $container->getAttribute('id');
         if (!$containerId) {
-            $containerId = 'wizardcontainer-' . strtolower(str_replace('\\', '-', get_class($container)));
+            $containerId = 'wizardcontainer-'.strtolower(str_replace('\\', '-', get_class($container)));
         }
 
         foreach ($container as $tabElement) {
-            $tabId = $containerId . '-' . strtolower($tabElement->getName());
-            $tabsNav .= '<li><a data-toggle="tab" href="#' . $tabId . '">' . $translate($tabElement->getLabel()) . '</a></li>';
-            $tabsContent .= '<div class="tab-pane" id="' . $tabId . '">'
+            $tabId = $containerId.'-'.strtolower($tabElement->getName());
+            $tabsNav .= '<li><a data-toggle="tab" href="#'.$tabId.'">'.$translate($tabElement->getLabel()).'</a></li>';
+            $tabsContent .= '<div class="tab-pane" id="'.$tabId.'">'
                           . $formContainer($tabElement, $layout, $parameter)
                           . '</div>';
         }
 
-        $content .= '<style type="text/css">.tab-content > div > div:first-child { margin-top: 10px; }</style><div class="wizard-container" id="' . $containerId . '">'
-                  . '<ul>' . $tabsNav . '</ul>'
-                  . '<div class="tab-content">' . $tabsContent . '</div>';
+        $content .= '<style type="text/css">.tab-content > div > div:first-child { margin-top: 10px; }</style><div class="wizard-container" id="'.$containerId.'">'
+                  . '<ul>'.$tabsNav.'</ul>'
+                  . '<div class="tab-content">'.$tabsContent.'</div>';
         if ($containerParams['pager']) {
-            $content .='<ul class="pager wizard">'
-                  . '<li class="previous"><a href="javascript:;">&larr; ' . $translate('previous') . '</a></li>'
-                  . '<li class="next"><a href="javascript:;">' . $translate('Next') . ' &rarr;</a></li>'
-                  . '<li class="finish' . ($containerParams['finish_enabled'] ? '' : ' disabled') . '">'
+            $content .= '<ul class="pager wizard">'
+                  . '<li class="previous"><a href="javascript:;">&larr; '.$translate('previous').'</a></li>'
+                  . '<li class="next"><a href="javascript:;">'.$translate('Next').' &rarr;</a></li>'
+                  . '<li class="finish'.($containerParams['finish_enabled'] ? '' : ' disabled').'">'
                   . (false !== $containerParams['finish_label']
-                     ? '<a class="pull-right" href="' . $containerParams['finish_href'] . '">'
-                       . $translate($containerParams['finish_label']) . ' &bull;</a>'
+                     ? '<a class="pull-right" href="'.$containerParams['finish_href'].'">'
+                       . $translate($containerParams['finish_label']).' &bull;</a>'
                      : ''
                     )
                   . '</li></ul>';