cross-solution / YAWIK

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

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) {

Braces +1 added lines, -2 removed lines

@@ -209,8 +209,7 @@
             /* @var \Zend\EventManager\EventInterface $event */
             $event = $services->has($config['event']) ? $services->get($config['event']) : new $config['event']();
             $events->setEventPrototype($event);
-        }
-        else {
+        } else {
             $events->setEventClass($config['event']);
         }
 

module/Core/Module.php

Spacing +10 added lines, -10 removed lines

@@ -36,7 +36,7 @@ 
     public function getConsoleBanner(Console $console)
     {
         $version = `git describe 2>/dev/null`;
-        $name = 'YAWIK ' . trim($version);
+        $name = 'YAWIK '.trim($version);
         $width = $console->getWidth();
         return sprintf(
             "==%1\$s==\n%2\$s%3\$s\n**%1\$s**\n",
@@ -72,12 +72,12 @@ 
         $sharedManager       = $eventManager->getSharedManager();
         
         if (!\Zend\Console\Console::isConsole()) {
-            $redirectCallback = function () use ($e) {
+            $redirectCallback = function() use ($e) {
                 $routeMatch = $e->getRouteMatch();
                 $lang = $routeMatch ? $routeMatch->getParam('lang', 'en') : 'en';
-                $uri    = $e->getRouter()->getBaseUrl() . '/' . $lang . '/error';
+                $uri = $e->getRouter()->getBaseUrl().'/'.$lang.'/error';
                 
-                header('Location: ' . $uri);
+                header('Location: '.$uri);
             };
             
             $errorHandlerListener = new ErrorHandlerListener($sm->get('ErrorLogger'), $redirectCallback);
@@ -109,7 +109,7 @@ 
         
         $eventManager->attach(
             MvcEvent::EVENT_DISPATCH_ERROR,
-            function ($event) {
+            function($event) {
                 $application = $event->getApplication();
                 if ($application::ERROR_EXCEPTION == $event->getError()) {
                     $ex = $event->getParam('exception');
@@ -123,7 +123,7 @@ 
         );
         $eventManager->attach(
             MvcEvent::EVENT_DISPATCH,
-            function ($event) use ($eventManager) {
+            function($event) use ($eventManager) {
                 $eventManager->trigger('postDispatch', $event);
             },
             -150
@@ -138,7 +138,7 @@ 
      */
     public function getConfig()
     {
-        $config = include __DIR__ . '/config/module.config.php';
+        $config = include __DIR__.'/config/module.config.php';
         return $config;
         if (\Zend\Console\Console::isConsole()) {
             $config['doctrine']['configuration']['odm_default']['generate_proxies'] = false;
@@ -158,9 +158,9 @@ 
         return array(
             'Zend\Loader\StandardAutoloader' => array(
                 'namespaces' => array(
-                    __NAMESPACE__ => __DIR__ . '/src/' . __NAMESPACE__,
-                    'CoreTest' => __DIR__ . '/test/' . 'CoreTest',
-                    'CoreTestUtils' => __DIR__ . '/test/CoreTestUtils',
+                    __NAMESPACE__ => __DIR__.'/src/'.__NAMESPACE__,
+                    'CoreTest' => __DIR__.'/test/'.'CoreTest',
+                    'CoreTestUtils' => __DIR__.'/test/CoreTestUtils',
                 ),
             ),
         );

module/Applications/src/Applications/Factory/Listener/StatusChangeFactory.php

Doc Comments +1 added lines, -1 removed lines

@@ -27,7 +27,7 @@
      *
      * @param ServiceLocatorInterface $serviceLocator
      *
-     * @return mixed
+     * @return StatusChange
      */
     public function createService(ServiceLocatorInterface $serviceLocator)
     {

module/Applications/src/Applications/Listener/StatusChange.php

Doc Comments +1 added lines, -1 removed lines

@@ -157,7 +157,7 @@
 
     /**
      * @param Application $application
-     * @param             $status
+     * @param             string $status
      *
      * @return AddressList
      */

Spacing +8 added lines, -8 removed lines

@@ -68,7 +68,7 @@ 
      *
      * @param ApplicationEvent $e
      */
-    public function prepareFormData(ApplicationEvent $e){
+    public function prepareFormData(ApplicationEvent $e) {
         if ($e->isPostRequest()) {
             return;
         }
@@ -95,13 +95,13 @@ 
                 $key = 'mailRejectionText';
                 break;
             default:
-                throw new \InvalidArgumentException('Unknown status value: ' .$status);
+                throw new \InvalidArgumentException('Unknown status value: '.$status);
         }
-        $mailText      = $settings->$key ? $settings->$key : '';
+        $mailText = $settings->$key ? $settings->$key : '';
         $mail->setBody($mailText);
         $mail->setApplication($this->application);
         $mailText = $mail->getBodyText();
-        $mailSubject   = sprintf(
+        $mailSubject = sprintf(
             $this->translator->translate('Your application dated %s'),
             strftime('%x', $this->application->getDateCreated()->getTimestamp())
         );
@@ -121,7 +121,7 @@ 
      *
      * @param ApplicationEvent $event
      */
-    public function  sendMail(ApplicationEvent $event){
+    public function  sendMail(ApplicationEvent $event) {
         if (!$event->isPostRequest()) {
             return;
         }
@@ -167,7 +167,7 @@ 
         $this->mailService->send($mail);
 
 
-        $historyText = sprintf($this->translator->translate('Mail was sent to %s'), key($recipient) ?: $recipient[0] );
+        $historyText = sprintf($this->translator->translate('Mail was sent to %s'), key($recipient) ?: $recipient[0]);
         $this->application->changeStatus($status, $historyText);
         $event->setNotification($historyText);
     }
@@ -178,7 +178,7 @@ 
      *
      * @return AddressList
      */
-    protected function getRecipient( Application $application, $status) {
+    protected function getRecipient(Application $application, $status) {
 
         $recipient = Status::ACCEPTED == $status
             ? $application->getJob()->getUser()->getInfo()
@@ -187,7 +187,7 @@ 
         $email = $recipient->getEmail();
         $name  = $recipient->getDisplayName(false);
 
-        return $name ? [ $email => $name ] : [ $email ];
+        return $name ? [$email => $name] : [$email];
     }
 
 }
\ No newline at end of file

module/Organizations/src/Organizations/Entity/WorkflowSettingsInterface.php

Spacing +2 added lines, -2 removed lines

@@ -17,7 +17,7 @@ 
      * @param bool $acceptApplicationByDepartmentManager
      * @return WorkflowSettings
      */
-    public function setAcceptApplicationByDepartmentManager( $acceptApplicationByDepartmentManager );
+    public function setAcceptApplicationByDepartmentManager($acceptApplicationByDepartmentManager);
 
     /**
      * Gets AcceptApplicationByDepartmentManager
@@ -32,7 +32,7 @@ 
      * @param bool $assignDepartmentManagersToJobs
      * @return WorkflowSettings
      */
-    public function setAssignDepartmentManagersToJobs( $assignDepartmentManagersToJobs );
+    public function setAssignDepartmentManagersToJobs($assignDepartmentManagersToJobs);
 
     /**
      * Gets AssignDepartmentManagersToJobs

module/Organizations/src/Organizations/Entity/WorkflowSettings.php

Spacing +3 added lines, -3 removed lines

@@ -40,9 +40,9 @@ 
      * @param bool $acceptApplicationByDepartmentManager
      * @return WorkflowSettings
      */
-    public function setAcceptApplicationByDepartmentManager( $acceptApplicationByDepartmentManager )
+    public function setAcceptApplicationByDepartmentManager($acceptApplicationByDepartmentManager)
     {
-        $this->acceptApplicationByDepartmentManager= $acceptApplicationByDepartmentManager;
+        $this->acceptApplicationByDepartmentManager = $acceptApplicationByDepartmentManager;
         return $this;
     }
     
@@ -62,7 +62,7 @@ 
      * @param bool $assignDepartmentManagersToJobs
      * @return WorkflowSettings
      */
-    public function setAssignDepartmentManagersToJobs( $assignDepartmentManagersToJobs )
+    public function setAssignDepartmentManagersToJobs($assignDepartmentManagersToJobs)
     {
         $this->assignDepartmentManagersToJobs = $assignDepartmentManagersToJobs;
         return $this;

module/Applications/src/Applications/Entity/Settings.php

Spacing +1 added lines, -1 removed lines

@@ -142,7 +142,7 @@
     /**
      * @return bool
      */
-    public function getAutoConfirmMail(){
+    public function getAutoConfirmMail() {
         return $this->autoConfirmMail;
     }
 }

module/Applications/src/Applications/Listener/Events/ApplicationEvent.php

Spacing +5 added lines, -5 removed lines

@@ -30,17 +30,17 @@ 
     /**
      * Event is fired when a new application is saved.
      */
-    const EVENT_APPLICATION_POST_CREATE   = 'application.post.create';
+    const EVENT_APPLICATION_POST_CREATE = 'application.post.create';
 
     /**
      * Event is fired when a users deleted application
      */
-    const EVENT_APPLICATION_PRE_DELETE   = 'application.pre.delete';
+    const EVENT_APPLICATION_PRE_DELETE = 'application.pre.delete';
 
     /**
      * Event is fired when the status of an application is changed
      */
-    const EVENT_APPLICATION_STATUS_CHANGE   = 'application.status.change';
+    const EVENT_APPLICATION_STATUS_CHANGE = 'application.status.change';
 
     /**
      * @var Application $application
@@ -233,12 +233,12 @@ 
             $this->setApplicationEntity($params->application);
         }
 
-        if(is_array($params) && isset($params['user'])) {
+        if (is_array($params) && isset($params['user'])) {
             $this->setUser($params['user']);
             unset($params['user']);
         }
 
-        if(is_array($params) && isset($params['status'])) {
+        if (is_array($params) && isset($params['status'])) {
             $this->setStatus($params['status']);
             unset($params['status']);
         }

Doc Comments +3 added lines, -3 removed lines

@@ -138,7 +138,7 @@ 
     }
 
     /**
-     * @param $notification
+     * @param string $notification
      *
      * @return $this
      */
@@ -168,7 +168,7 @@ 
     }
 
     /**
-     * @return mixed
+     * @return string
      */
     public function getStatus()
     {
@@ -176,7 +176,7 @@ 
     }
 
     /**
-     * @return mixed
+     * @return boolean
      */
     public function isPostRequest()
     {

module/Applications/src/Applications/Controller/IndexController.php

Indentation +1 added lines, -1 removed lines

@@ -40,7 +40,7 @@
             $params->set('by', 'me');
         }
 
-         //default sorting
+            //default sorting
         if (!isset($params['sort'])) {
             $params['sort']="-date";
         }

Spacing +2 added lines, -2 removed lines

@@ -42,10 +42,10 @@
 
          //default sorting
         if (!isset($params['sort'])) {
-            $params['sort']="-date";
+            $params['sort'] = "-date";
         }
         $params->count = 5;
-        $params->pageRange=5;
+        $params->pageRange = 5;
 
         $this->paginationParams()->setParams('Applications\Index', $params);