cross-solution / YAWIK

module/Jobs/src/Jobs/Form/BaseFieldset.php

Spacing +3 added lines, -3 removed lines

@@ -54,13 +54,13 @@ 
 
             $locationsStrategy = new \Zend\Hydrator\Strategy\ClosureStrategy(
                 /* extract */
-                function ($value) use ($geoLocationStrategy)
+                function($value) use ($geoLocationStrategy)
                 {
                     return $geoLocationStrategy->extract($value->first());
                 },
 
                 /* hydrate */
-                function ($value) use ($geoLocationStrategy)
+                function($value) use ($geoLocationStrategy)
                 {
                     return new ArrayCollection([$geoLocationStrategy->hydrate($value)]);
                 }
@@ -111,7 +111,7 @@ 
                             $loc = $this->object->getLocations()->first();
                             if (!$loc) { return ''; }
 
-                            $value = $loc->getPostalCode() . ' ' . $loc->getCity() . ', ' . $loc->getRegion();
+                            $value = $loc->getPostalCode().' '.$loc->getCity().', '.$loc->getRegion();
                             $value = trim($value, ' ,');
 
                             return $value;

module/Core/src/Core/Entity/AbstractLocation.php

Indentation +1 added lines, -1 removed lines

@@ -69,7 +69,7 @@
                     ? [
                         'type' => $coords->getType(),
                         'coordinates' => $coords->getCoordinates(),
-                      ]
+                        ]
                     : null
 
         ];

Spacing +1 added lines, -1 removed lines

@@ -85,7 +85,7 @@
             if (!$value) { continue; }
 
             if ('coordinates' == $key) {
-                $class = '\\Geo\\Entity\\Geometry\\' . $value['type'];
+                $class = '\\Geo\\Entity\\Geometry\\'.$value['type'];
                 $value = new $class($value['coordinates']);
             }
 

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);
@@ -284,11 +284,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 = '<ul><li>' . join('</li><li>' , $multiOptions) . '</li></ul>';
+                    $elementValue = '<ul><li>'.join('</li><li>', $multiOptions).'</li></ul>';
 
                 } else {
                     $elementValue = $translator->translate($options[$elementValue]);
@@ -303,10 +303,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/Geo/src/Geo/Service/Geo.php

Spacing +1 added lines, -1 removed lines

@@ -40,7 +40,7 @@
     {
         $result = json_decode($result, JSON_OBJECT_AS_ARRAY);
 
-        $r=[];
+        $r = [];
         foreach ($result["result"] as $val) {
             $coords = $this->queryCoords($val);
             if (false !== strpos($val, ',')) {

module/Geo/src/Geo/Service/Photon.php

Indentation +9 added lines, -9 removed lines

@@ -63,16 +63,16 @@
 
         foreach ($result as $key => $val) {
             $row=[
-                  'postalCode' => (property_exists($val->properties, 'postcode') ? $val->properties->postcode:''),
-                  'city' =>(property_exists($val->properties, 'city') ? $val->properties->city:''),
-                  'street' => (property_exists($val->properties, 'street') ? $val->properties->street : ''),
-                  'region' => (property_exists($val->properties, 'state') ? $val->properties->state : ''),
-                  'country' => (property_exists($val->properties, 'country') ? $val->properties->country : ''),
+                    'postalCode' => (property_exists($val->properties, 'postcode') ? $val->properties->postcode:''),
+                    'city' =>(property_exists($val->properties, 'city') ? $val->properties->city:''),
+                    'street' => (property_exists($val->properties, 'street') ? $val->properties->street : ''),
+                    'region' => (property_exists($val->properties, 'state') ? $val->properties->state : ''),
+                    'country' => (property_exists($val->properties, 'country') ? $val->properties->country : ''),
 
-                  'osm_key' => (property_exists($val->properties, 'osm_key') ? $val->properties->osm_key : ''),
-                  'osm_value' => (property_exists($val->properties, 'osm_value') ? $val->properties->osm_value : ''),
-                  //'id' => (property_exists($val->properties, 'osm_id') ? $val->properties->osm_id : uniqid()),
-                  //'data' => json_encode($val),
+                    'osm_key' => (property_exists($val->properties, 'osm_key') ? $val->properties->osm_key : ''),
+                    'osm_value' => (property_exists($val->properties, 'osm_value') ? $val->properties->osm_value : ''),
+                    //'id' => (property_exists($val->properties, 'osm_id') ? $val->properties->osm_id : uniqid()),
+                    //'data' => json_encode($val),
             ];
             if ($val->geometry) {
                 $row['coordinates'] = [

Spacing +7 added lines, -7 removed lines

@@ -28,11 +28,11 @@ 
         $client->setMethod('GET');
 
         $osmTags = [
-            'tourism','aeroway','railway', 'amenity', 'historic', 'tunnel', 'mountain_pass',
+            'tourism', 'aeroway', 'railway', 'amenity', 'historic', 'tunnel', 'mountain_pass',
             'leisure', 'natural', 'bridge', 'waterway'
         ];
 
-        $osmTags = array_map(function($i) { return urlencode('!' . $i); }, $osmTags);
+        $osmTags = array_map(function($i) { return urlencode('!'.$i); }, $osmTags);
 
         $uri = sprintf(
             '%s?osm_tag=%s',
@@ -59,12 +59,12 @@ 
     {
         $result = json_decode($result);
         $result = $result->features;
-        $r=[];
+        $r = [];
 
         foreach ($result as $key => $val) {
-            $row=[
-                  'postalCode' => (property_exists($val->properties, 'postcode') ? $val->properties->postcode:''),
-                  'city' =>(property_exists($val->properties, 'city') ? $val->properties->city:''),
+            $row = [
+                  'postalCode' => (property_exists($val->properties, 'postcode') ? $val->properties->postcode : ''),
+                  'city' =>(property_exists($val->properties, 'city') ? $val->properties->city : ''),
                   'street' => (property_exists($val->properties, 'street') ? $val->properties->street : ''),
                   'region' => (property_exists($val->properties, 'state') ? $val->properties->state : ''),
                   'country' => (property_exists($val->properties, 'country') ? $val->properties->country : ''),
@@ -81,7 +81,7 @@ 
                 ];
             }
             //$row['id'] = json_encode($row);
-            $r[]=$row;
+            $r[] = $row;
         }
         return $r;
     }

module/Geo/src/Geo/Form/GeoSelectHydratorStrategy.php

Spacing +2 added lines, -2 removed lines

@@ -72,10 +72,10 @@
         if (0 === strpos($value, '{')) {
             return $value;
         }
-        if ($value){
+        if ($value) {
             $data = $this->geoClient->queryOne($value);
             return $data['id'];
-        }else{
+        } else {
             return;
         }
     }

Braces +1 added lines, -1 removed lines

@@ -75,7 +75,7 @@
         if ($value){
             $data = $this->geoClient->queryOne($value);
             return $data['id'];
-        }else{
+        } else{
             return;
         }
     }

module/Jobs/src/Jobs/Controller/ManageController.php

Indentation +11 added lines, -11 removed lines

@@ -148,12 +148,12 @@ 
         $jobEntity = $this->initializeJob()->get($this->params(), true);
 
         $model = new ViewModel([
-                                   'portals' => $jobEntity->getPortals(),
-                                   'channels' => $channels,
-                                   'defaultCurrencyCode' => $options->defaultCurrencyCode,
-                                   'defaultTaxRate' =>  $options->defaultTaxRate,
-                                   'jobId' => $jobEntity->getId()
-                               ]);
+                                    'portals' => $jobEntity->getPortals(),
+                                    'channels' => $channels,
+                                    'defaultCurrencyCode' => $options->defaultCurrencyCode,
+                                    'defaultTaxRate' =>  $options->defaultTaxRate,
+                                    'jobId' => $jobEntity->getId()
+                                ]);
         $model->setTemplate('jobs/partials/channel-list')->setTerminal(true);
         return $model;
     }
@@ -513,7 +513,7 @@ 
             'lang/jobs/view',
             array(),
             array('query' => $query
-                      )
+                        )
         );
 
         $approvalLink = $this->url()->fromRoute(
@@ -529,10 +529,10 @@ 
         );
 
         return array('job' => $jobEntity,
-                     'diffSnapshot' => $diff,
-                     'viewLink' => $viewLink,
-                     'approvalLink' => $approvalLink,
-                     'declineLink' => $declineLink);
+                        'diffSnapshot' => $diff,
+                        'viewLink' => $viewLink,
+                        'approvalLink' => $approvalLink,
+                        'declineLink' => $declineLink);
     }
 
     /**

Doc Comments +3 added lines, -4 removed lines

@@ -165,7 +165,6 @@ 
      *
      * parameter are arbitrary elements for defaults or programming flow
      *
-     * @param array $parameter
      * @return null|ViewModel
      * @throws \RuntimeException
      */
@@ -388,7 +387,7 @@ 
 
     /**
      * @param  $job \Jobs\Entity\Job
-     * @return mixed
+     * @return \Jobs\Form\Job
      */
     protected function getFormular($job)
     {
@@ -411,7 +410,7 @@ 
     }
 
     /**
-     * @param $form
+     * @param \Jobs\Form\Job $form
      * @param array $params
      * @return ViewModel
      */
@@ -604,7 +603,7 @@ 
     }
 
     /**
-     * @param $script
+     * @param string $script
      * @param array $parameter
      * @return ViewModel
      */

Spacing +12 added lines, -12 removed lines

@@ -178,7 +178,7 @@ 
         if (empty($user->getInfo()->getEmail())) {
             return $this->getErrorViewModel('no-parent', array('cause' => 'noEmail'));
         }
-        $userOrg            = $user->getOrganization();
+        $userOrg = $user->getOrganization();
         if (!$userOrg->hasAssociation() || $userOrg->getOrganization()->isDraft()) {
             return $this->getErrorViewModel('no-parent', array('cause' => 'noCompany'));
         }
@@ -208,7 +208,7 @@ 
         }
 
 
-        $viewModel          = null;
+        $viewModel = null;
         $this->acl($jobEntity, 'edit');
         if ($status = $params->fromQuery('status')) {
             $this->changeStatus($jobEntity, $status);
@@ -220,11 +220,11 @@ 
         $instanceForm       = null;
         $formErrorMessages = array();
 
-        if (isset($formIdentifier) &&  $request->isPost()) {
+        if (isset($formIdentifier) && $request->isPost()) {
             // at this point the form get instantiated and immediately accumulated
             $instanceForm = $form->getForm($formIdentifier);
             if (!isset($instanceForm)) {
-                throw new \RuntimeException('No form found for "' . $formIdentifier . '"');
+                throw new \RuntimeException('No form found for "'.$formIdentifier.'"');
             }
             // the id may be part of the postData, but it never should be altered
             $postData = $request->getPost();
@@ -278,20 +278,20 @@ 
             $jobValid = false;
             $errorMessage[] = $this->translator->translate('Accept the Terms');
         }
-        $result = $formEvents->trigger('ValidateJob', $this, [ 'form' => $form ]);
+        $result = $formEvents->trigger('ValidateJob', $this, ['form' => $form]);
         foreach ($result as $messages) {
             if (!$messages) {
                 continue;
             }
             if (!is_array($messages)) {
-                $messages = [ $messages ];
+                $messages = [$messages];
             }
 
             $errorMessage = array_merge($errorMessage, $messages);
             $jobValid = false;
         }
 
-        $errorMessage = '<br />' . implode('<br />', $errorMessage);
+        $errorMessage = '<br />'.implode('<br />', $errorMessage);
         if ($isAjax) {
             if ($instanceForm instanceof SummaryForm) {
                 $instanceForm->setRenderMode(SummaryForm::RENDER_SUMMARY);
@@ -299,7 +299,7 @@ 
             } else {
                 $viewHelper = 'form';
             }
-            $viewHelperManager  = $serviceLocator->get('ViewHelperManager');
+            $viewHelperManager = $serviceLocator->get('ViewHelperManager');
             $content = $viewHelperManager->get($viewHelper)->__invoke($instanceForm);
             $viewModel = new JsonModel(
                 array(
@@ -322,7 +322,7 @@ 
                     }
                 }
             } else {
-                $formEvents->trigger('DisableElements', $this, [ 'form' => $form, 'job'=>$jobEntity ]);
+                $formEvents->trigger('DisableElements', $this, ['form' => $form, 'job'=>$jobEntity]);
                 // Job is deployed, some changes are now disabled
                 $form->enableAll();
             }
@@ -330,7 +330,7 @@ 
 
             $completionLink = $this->url()->fromRoute(
                 'lang/jobs/completion',
-                [ 'id' => $jobEntity->getId()]
+                ['id' => $jobEntity->getId()]
             );
 
             $viewModel = $this->getViewModel($form);
@@ -501,7 +501,7 @@ 
         // array with differences between the last snapshot and the actual entity
         // is remains Null if there is no snapshot
         // it will be an empty array if the snapshot and the actual entity do not differ
-        $diff           = null;
+        $diff = null;
 
 
         if ($params == 'declined') {
@@ -560,7 +560,7 @@ 
      */
     public function deactivateAction()
     {
-        $user           = $this->auth->getUser();
+        $user = $this->auth->getUser();
 
         $jobEntity = $this->initializeJob()->get($this->params());
 

module/Jobs/src/Jobs/Controller/Plugin/InitializeJob.php

Spacing +5 added lines, -5 removed lines

@@ -43,9 +43,9 @@ 
 
     public function __construct(RepositoryService $repositoryService, AuthenticationService $auth, Acl $acl)
     {
-        $this->repositoryService=$repositoryService;
-        $this->auth=$auth;
-        $this->acl=$acl;
+        $this->repositoryService = $repositoryService;
+        $this->auth = $auth;
+        $this->acl = $acl;
     }
 
     public function __invoke()
@@ -64,12 +64,12 @@ 
     public function get(Params $params, $allowDraft = false, $getSnapshot = false)
     {
         /* @var \Jobs\Repository\Job $jobRepository */
-        $jobRepository  = $this->repositoryService->get('Jobs/Job');
+        $jobRepository = $this->repositoryService->get('Jobs/Job');
         $idFromRoute   = $params('id', 0);
         $idFromQuery   = $params->fromQuery('id', 0);
         $idFromSubForm = $params->fromPost('job', 0);
 
-        $id = empty($idFromRoute)? (empty($idFromQuery)?$idFromSubForm:$idFromQuery) : $idFromRoute;
+        $id = empty($idFromRoute) ? (empty($idFromQuery) ? $idFromSubForm : $idFromQuery) : $idFromRoute;
         $snapshotId = $params->fromPost('snapshot') ?: ($params->fromQuery('snapshot') ?: null);
 
         if (empty($id) && empty($snapshotId) && $allowDraft) {

module/Auth/src/Auth/Controller/Plugin/UserSwitcher.php

Doc Comments +2 added lines, -2 removed lines

@@ -93,7 +93,7 @@ 
     /**
      * Switch to another user.
      *
-     * @param string|UserInterface $id user id of the user to switch to.
+     * @param string $id user id of the user to switch to.
      * @param array $params Additional parameters to store in the session container.
      *
      * @return bool
@@ -167,7 +167,7 @@ 
      * @param string $key
      * @param mixed $default Value to return if param $key is not set.
      *
-     * @return null
+     * @return string
      */
     public function getSessionParam($key, $default = null)
     {