cross-solution / YAWIK

module/Core/src/Core/Factory/EventManager/EventManagerAbstractFactory.php

Braces +1 added lines, -2 removed lines

@@ -54,8 +54,7 @@
         if (method_exists($events, 'setEventPrototype')) {
             $event = $serviceLocator->has($config['event']) ? $serviceLocator->get($config['event']) : new $config['event']();
             $events->setEventPrototype($event);
-        }
-        else {
+        } else {
             $events->setEventClass($config['event']);
         }
 

Doc Comments +1 added lines, -1 removed lines

@@ -293,7 +293,7 @@
      * @param string|array $options String is either event name or aggregate name (when name is int).
      *                              Array are the options from config. [ [event,..], method, priority, lazy]
      *
-     * @return array
+     * @return \Zend\EventManager\EventManagerInterface
      */
     protected function normalizeListenerOptions($name, $options)
     {

Indentation +1 added lines, -1 removed lines

@@ -278,7 +278,7 @@
             /* @var \Core\Listener\DeferredListenerAggregate $aggregate */
             $aggregate = $services->get('Core/Listener/DeferredListenerAggregate');
             $aggregate->setListeners($lazyListeners)
-                      ->attach($eventManager);
+                        ->attach($eventManager);
         }
     }
 

Spacing +8 added lines, -8 removed lines

@@ -148,7 +148,7 @@ 
         $defaults = [
             'service' => 'EventManager',
             'configure' => true,
-            'identifiers' => [ $name ],
+            'identifiers' => [$name],
             'event' => '\Zend\EventManager\Event',
             'listeners' => [],
         ];
@@ -239,7 +239,7 @@ 
         foreach ($listeners as $name => $options) {
             $options = $this->normalizeListenerOptions($name, $options);
 
-            if ($options['lazy'] && null !== $options['attach'] ) {
+            if ($options['lazy'] && null !== $options['attach']) {
                 foreach ($options['attach'] as $spec) {
                     $lazyListeners[] = [
                         'service' => $options['service'],
@@ -269,7 +269,7 @@ 
             }
 
             foreach ($options['attach'] as $spec) {
-                $callback = $spec['method'] ? [ $listener, $spec['method'] ] : $listener;
+                $callback = $spec['method'] ? [$listener, $spec['method']] : $listener;
                 $eventManager->attach($spec['events'], $callback, $spec['priority']);
             }
         }
@@ -340,7 +340,7 @@ 
 
         if (is_string($options)) {
             /* Only an event name is provided in config */
-            $normalized['attach'] = [ [ 'events' => [ $options ], 'method' => null, 'priority' => 0 ] ];
+            $normalized['attach'] = [['events' => [$options], 'method' => null, 'priority' => 0]];
             return $normalized;
 
         }
@@ -369,7 +369,7 @@ 
             } else if (is_string($opt)) {
                 if (null === $event) {
                     /* first string found is assumed to be the event name */
-                    $event = [ $opt ];
+                    $event = [$opt];
                 } else {
                     /* second string found must be a method name. */
                     $method = $opt;
@@ -385,7 +385,7 @@ 
             }
         }
 
-        $normalized['attach'] = [ [ 'events' => $event, 'method' => $method, 'priority' => $priority ] ];
+        $normalized['attach'] = [['events' => $event, 'method' => $method, 'priority' => $priority]];
         $normalized['lazy']   = $lazy;
 
         return $normalized;
@@ -394,7 +394,7 @@ 
     protected function normalizeEventsSpec($options)
     {
         $listenerPriority = isset($options['priority']) ? $options['priority'] : 0;
-        $listenerMethod   = isset($options['method'])   ? $options['method']   : '__none__';
+        $listenerMethod   = isset($options['method']) ? $options['method'] : '__none__';
         $events = [];
 
         foreach ($options['events'] as $event => $spec) {
@@ -414,7 +414,7 @@ 
             } else if (is_array($spec)) {
                 if (isset($spec['method'])) {
                     if (!is_array($spec['method'])) {
-                        $spec['method'] = [ $spec['method'] ];
+                        $spec['method'] = [$spec['method']];
                     }
 
                     foreach ($spec['method'] as $method => $methodPriority) {

module/Orders/config/router.config.php

Spacing +1 added lines, -1 removed lines

@@ -6,7 +6,7 @@
  * @license MIT
  * @copyright  2013 - 2016 Cross Solution <http://cross-solution.de>
  */
-return [ 'router' => [ 'routes' => [ 'lang' => [ 'child_routes' => [
+return ['router' => ['routes' => ['lang' => ['child_routes' => [
 
     'orders' => [
         'type' => 'Segment',

module/Orders/config/module.config.php

Indentation +1 added lines, -1 removed lines

@@ -60,4 +60,4 @@
         ],
     ],
 ];
-  
\ No newline at end of file
+    
\ No newline at end of file

Spacing +7 added lines, -7 removed lines

@@ -23,7 +23,7 @@ 
                  * for multiple paths.
                  * example https://github.com/doctrine/DoctrineORMModule
                  */
-                'paths' => array( __DIR__ . '/../src/Entity'),
+                'paths' => array(__DIR__.'/../src/Entity'),
             ),
         ],
     ],
@@ -33,7 +33,7 @@ 
         'translation_file_patterns' => array(
             array(
                 'type'     => 'gettext',
-                'base_dir' => __DIR__ . '/../language',
+                'base_dir' => __DIR__.'/../language',
                 'pattern'  => '%s.mo',
             ),
         ),
@@ -42,11 +42,11 @@ 
 
     'event_manager' => [
 
-        'Jobs/Events' => [ 'listeners' => [
-            'Orders/Listener/CreateJobOrder' => [ \Jobs\Listener\Events\JobEvent::EVENT_JOB_CREATED, true ],
+        'Jobs/Events' => ['listeners' => [
+            'Orders/Listener/CreateJobOrder' => [\Jobs\Listener\Events\JobEvent::EVENT_JOB_CREATED, true],
         ]],
 
-        'Jobs/JobContainer/Events' => [ 'listeners' => [
+        'Jobs/JobContainer/Events' => ['listeners' => [
             '\Orders\Form\Listener\DisableJobInvoiceAddress' => 'DisableElements',
             'Orders\Form\Listener\InjectInvoiceAddressInJobContainer' => \Core\Form\Event\FormEvent::EVENT_INIT,
             '\Orders\Form\Listener\ValidateJobInvoiceAddress' => 'ValidateJob',
@@ -59,7 +59,7 @@ 
             ],
         ],
 
-        'Core/AdminController/Events' => [ 'listeners' => [
+        'Core/AdminController/Events' => ['listeners' => [
             'Orders/Listener/AdminWidgetProvider' => \Core\Controller\AdminControllerEvent::EVENT_DASHBOARD,
         ]],
     ],
@@ -82,7 +82,7 @@ 
     'view_helper_config' => [
         'headscript' => [
             'lang/orders' => [
-                [ \Zend\View\Helper\HeadScript::FILE, 'Orders/js/list.index.js', 'PREPEND' ],
+                [\Zend\View\Helper\HeadScript::FILE, 'Orders/js/list.index.js', 'PREPEND'],
                 'js/bootstrap-dialog.min.js'
             ],
         ],

module/Orders/config/acl.config.php

Spacing +1 added lines, -1 removed lines

@@ -7,7 +7,7 @@
  * @copyright  2013 - 2016 Cross Solution <http://cross-solution.de>
  */
 
-return [ 'acl' => [
+return ['acl' => [
 
     'rules' => [
         'admin' => [

module/Orders/src/Entity/Order.php

Spacing +3 added lines, -3 removed lines

@@ -236,7 +236,7 @@ 
      */
     public function setPrice($amount)
     {
-        $tax    = $this->getTaxRate()  / 100;
+        $tax = $this->getTaxRate() / 100;
         $taxAmount = $amount * $tax;
 
         $this->prices = [
@@ -248,7 +248,7 @@ 
         return $this;
     }
 
-    public function getPrice($type="net")
+    public function getPrice($type = "net")
     {
         if (!$this->prices || !array_key_exists($type, $this->prices)) {
             return 0;
@@ -264,7 +264,7 @@ 
      *
      * @return array
      */
-    public function getPrices($calculate=false)
+    public function getPrices($calculate = false)
     {
         /*if (!$this->prices || $calculate) {
             $this->calculatePrices();

module/Orders/src/Repository/Orders.php

Spacing +2 added lines, -2 removed lines

@@ -22,14 +22,14 @@
  */
 class Orders extends AbstractRepository
 {
-    public function createJobOrder(array $data=[])
+    public function createJobOrder(array $data = [])
     {
         $data['type'] = OrderInterface::TYPE_JOB;
 
         return $this->create($data);
     }
 
-    public function create(array $data=[], $counterName = null, $counterFormat = null)
+    public function create(array $data = [], $counterName = null, $counterFormat = null)
     {
         if (isset($data['counter'])) {
             $counterName = $data['counter'];

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

Indentation +16 added lines, -16 removed lines

@@ -139,12 +139,12 @@ 
         $jobEntity = $this->initializeJob()->get($this->params());
 
         $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;
     }
@@ -440,9 +440,9 @@ 
             // we want just some Values to be compared
             $diff = null;
             foreach (array('title', 'organization', 'location',
-                         'templateValues.qualifications', 'templateValues.requirements', 'templateValues.benefits', 'templateValues.title',
-                         'templateValues._freeValues.description',
-                     ) as $prelKey) {
+                            'templateValues.qualifications', 'templateValues.requirements', 'templateValues.benefits', 'templateValues.title',
+                            'templateValues._freeValues.description',
+                        ) as $prelKey) {
                 if (array_key_exists($prelKey, $prelDiff)) {
                     $diff[$prelKey] = $prelDiff[$prelKey];
                 }
@@ -471,28 +471,28 @@ 
             'lang/jobs/view',
             array(),
             array('query' =>
-                      array( 'id' => $jobEntity->getId()))
+                        array( 'id' => $jobEntity->getId()))
         );
 
         $approvalLink = $this->url()->fromRoute(
             'lang/jobs/approval',
             array('state' => 'approved'),
             array('query' =>
-                      array( 'id' => $jobEntity->getId()))
+                        array( 'id' => $jobEntity->getId()))
         );
 
         $declineLink = $this->url()->fromRoute(
             'lang/jobs/approval',
             array('state' => 'declined'),
             array('query' =>
-                      array( 'id' => $jobEntity->getId()))
+                        array( 'id' => $jobEntity->getId()))
         );
 
         return array('job' => $jobEntity,
-                     'diffSnapshot' => $diff,
-                     'viewLink' => $viewLink,
-                     'approvalLink' => $approvalLink,
-                     'declineLink' => $declineLink);
+                        'diffSnapshot' => $diff,
+                        'viewLink' => $viewLink,
+                        'approvalLink' => $approvalLink,
+                        'declineLink' => $declineLink);
     }
 
     /**

Spacing +14 added lines, -14 removed lines

@@ -170,7 +170,7 @@ 
         if (empty($user->info->email)) {
             return $this->getErrorViewModel('no-parent', array('cause' => 'noEmail'));
         }
-        $userOrg            = $user->getOrganization();
+        $userOrg = $user->getOrganization();
         if (!$userOrg->hasAssociation()) {
             return $this->getErrorViewModel('no-parent', array('cause' => 'noCompany'));
         }
@@ -192,11 +192,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();
@@ -250,20 +250,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);
@@ -271,7 +271,7 @@ 
             } else {
                 $viewHelper = 'form';
             }
-            $viewHelperManager  = $serviceLocator->get('ViewHelperManager');
+            $viewHelperManager = $serviceLocator->get('ViewHelperManager');
             $content = $viewHelperManager->get($viewHelper)->__invoke($instanceForm);
             $viewModel = new JsonModel(
                 array(
@@ -294,7 +294,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();
             }
@@ -302,7 +302,7 @@ 
 
             $completionLink = $this->url()->fromRoute(
                 'lang/jobs/completion',
-                [ 'id' => $jobEntity->getId()]
+                ['id' => $jobEntity->getId()]
             );
 
             $viewModel = $this->getViewModel($form);
@@ -437,7 +437,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;
         // preliminary difference, contain all differences
         $prelDiff = $this->entitySnapshot()->diff($jobEntity);
         if (isset($prelDiff)) {
@@ -475,21 +475,21 @@ 
             'lang/jobs/view',
             array(),
             array('query' =>
-                      array( 'id' => $jobEntity->getId()))
+                      array('id' => $jobEntity->getId()))
         );
 
         $approvalLink = $this->url()->fromRoute(
             'lang/jobs/approval',
             array('state' => 'approved'),
             array('query' =>
-                      array( 'id' => $jobEntity->getId()))
+                      array('id' => $jobEntity->getId()))
         );
 
         $declineLink = $this->url()->fromRoute(
             'lang/jobs/approval',
             array('state' => 'declined'),
             array('query' =>
-                      array( 'id' => $jobEntity->getId()))
+                      array('id' => $jobEntity->getId()))
         );
 
         return array('job' => $jobEntity,
@@ -506,7 +506,7 @@ 
      */
     public function deactivateAction()
     {
-        $user           = $this->auth->getUser();
+        $user = $this->auth->getUser();
 
         $jobEntity = $this->initializeJob()->get($this->params());
 

Doc Comments +3 added lines, -4 removed lines

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

module/Jobs/src/Jobs/Factory/Form/Hydrator/OrganizationNameHydratorFactory.php

Indentation +1 added lines, -1 removed lines

@@ -15,7 +15,7 @@
 /**
  *
  * @ODM\MappedSuperclass
-  */
+ */
 abstract class AbstractIdentifiableEntity extends AbstractEntity implements IdentifiableEntityInterface
 {
        

module/Core/src/Core/Controller/Plugin/ContentCollector.php

Indentation +4 added lines, -4 removed lines

@@ -42,7 +42,7 @@ 
     public function trigger($event, $target = null)
     {
         if (empty($this->_template) || !is_string($this->_template)) {
-              throw new \InvalidArgumentException('ContentCollector must have a template-name');
+                throw new \InvalidArgumentException('ContentCollector must have a template-name');
         }
           
         $responseCollection = $this->getController()->getEventManager()->trigger($event, $target);
@@ -50,9 +50,9 @@ 
         $viewModel->setTemplate($this->_template);
         foreach ($responseCollection as $i => $response) {
             if (is_string($response)) {
-                      $template = $response;
-                      $response = new ViewModel(array('target' => $target));
-                      $response->setTemplate($template);
+                        $template = $response;
+                        $response = new ViewModel(array('target' => $target));
+                        $response->setTemplate($template);
             }
                     $viewModel->addChild($response, $this->_captureTo . $i);
         }

Spacing +1 added lines, -1 removed lines

@@ -54,7 +54,7 @@
                       $response = new ViewModel(array('target' => $target));
                       $response->setTemplate($template);
             }
-                    $viewModel->addChild($response, $this->_captureTo . $i);
+                    $viewModel->addChild($response, $this->_captureTo.$i);
         }
                 
         return $viewModel;