cross-solution / YAWIK

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

Doc Comments +1 added lines, -2 removed lines

@@ -46,7 +46,7 @@ 
      * @param array $defaultParams
      * @param bool  $usePostParams
      *
-     * @return mixed
+     * @return ZendPaginator
      */
     public function __invoke($paginatorName, $defaultParams = array(), $usePostParams = false)
     {
@@ -106,7 +106,6 @@ 
     }
     
     /**
-     * @param ControllerManager $controllerManager
      * @return CreatePaginatorService
      */
     public static function factory(ContainerInterface $container)

Indentation +1 added lines, -1 removed lines

@@ -81,7 +81,7 @@
 
         $adapter->setParams($params);
         $paginator->setCurrentPageNumber($params->get('page', 1))
-                  ->setItemCountPerPage($params->get('count', 10));
+                    ->setItemCountPerPage($params->get('count', 10));
 
         return $paginator;
     }

Spacing +2 added lines, -2 removed lines

@@ -61,11 +61,11 @@
         $paginatorManager = $this->serviceManager->get('Core/PaginatorService');
         $paginator = $paginatorManager->get($paginatorName);
         if (!isset($paginator) || !$paginator instanceof ZendPaginator) {
-            throw new \RuntimeException('Could not create paginator ' . $paginatorName);
+            throw new \RuntimeException('Could not create paginator '.$paginatorName);
         }
         $adapter = $paginator->getAdapter();
         if (!isset($adapter) || !$adapter instanceof AdapterInterface) {
-            throw new \RuntimeException('Paginator ' . $paginatorName . ' has no Adapter');
+            throw new \RuntimeException('Paginator '.$paginatorName.' has no Adapter');
         }
 
         $params     = $usePostParams

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

Doc Comments +2 added lines, -2 removed lines

@@ -50,7 +50,7 @@ 
     protected $generator;
 
     /**
-     * @param $serviceLocator
+     * @param \Interop\Container\ContainerInterface $serviceLocator
      * @return $this
      */
     public function setServiceLocator($serviceLocator)
@@ -78,7 +78,7 @@ 
     }
 
     /**
-     * @return mixed
+     * @return RepositoryInterface
      */
     public function getRepositories()
     {

Spacing +6 added lines, -6 removed lines

@@ -149,7 +149,7 @@ 
                 $target->__invoke($data);
                 $className = get_class($this->entity);
                 // @TODO, have to be abstract
-                $snapShotMetaClassName = '\\' . $className . 'SnapshotMeta';
+                $snapShotMetaClassName = '\\'.$className.'SnapshotMeta';
                 $meta = new $snapShotMetaClassName;
                 $meta->setEntity($target);
                 $meta->setSourceId($this->entity->id);
@@ -177,7 +177,7 @@ 
             if (empty($dataHead) || empty($targetClass)) {
                 return null;
             }
-            $repositorySnapshotMeta = $this->getRepositories()->getRepository($targetClass . "Meta");
+            $repositorySnapshotMeta = $this->getRepositories()->getRepository($targetClass."Meta");
             $snapshot = $repositorySnapshotMeta->findSnapshot($this->entity);
             // an snapshot has to be so simple that there is no need for a special hydrator
             $hydrator = new EntityHydrator();
@@ -247,8 +247,8 @@ 
             // the snapshotgenerator is a service defined by the name of the entity
             // this is the highest means, all subsequent means just add what is not set
             $className = get_class($this->entity);
-            if ($serviceLocator->has('snapshotgenerator' . $className)) {
-                $generator = $this->serviceLocator->get('snapshotgenerator' . $className);
+            if ($serviceLocator->has('snapshotgenerator'.$className)) {
+                $generator = $this->serviceLocator->get('snapshotgenerator'.$className);
                 if (is_array($generator)) {
                     $this->options = ArrayUtils::merge($generator, $this->options);
                     $generator = null;
@@ -348,13 +348,13 @@ 
                         }
                     } else {
                         if ($array1[$key] != $array2[$key]) {
-                            $result[$key] = array( $array1[$key], $array2[$key]);
+                            $result[$key] = array($array1[$key], $array2[$key]);
                         }
                     }
                     if (!empty($subResult)) {
                         foreach ($subResult as $subKey => $subValue) {
                             if (!empty($subKey) && is_string($subKey)) {
-                                $result[$key . '.' . $subKey] = $subValue;
+                                $result[$key.'.'.$subKey] = $subValue;
                             }
                         }
                     }

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

Doc Comments +6 added lines

@@ -72,6 +72,9 @@ 
         return $this;
     }
     
+    /**
+     * @param AddressList $header
+     */
     protected function stringFromMailHeader($header)
     {
         $erg = '';
@@ -160,6 +163,9 @@ 
         }
     }
     
+    /**
+     * @return string
+     */
     protected function getTemplate()
     {
         $template = null;

Spacing +18 added lines, -18 removed lines

@@ -153,18 +153,18 @@ 
         $replyTo = $this->stringFromMailHeader($this->getReplyTo());
         
         return str_pad($template, 30)
-                . 'to: ' . str_pad($to, 50)
-                . 'cc: ' . str_pad($cc, 50)
-                . 'bcc: ' . str_pad($bcc, 50)
-                . 'from: ' . str_pad($from, 50)
-                . 'replyTo: ' . str_pad($replyTo, 50)
+                . 'to: '.str_pad($to, 50)
+                . 'cc: '.str_pad($cc, 50)
+                . 'bcc: '.str_pad($bcc, 50)
+                . 'from: '.str_pad($from, 50)
+                . 'replyTo: '.str_pad($replyTo, 50)
                 //. str_pad(implode(',', ArrayUtils::iteratorToArray($this->getSender())),50)
-                . 'subject: ' . str_pad($this->getSubject(), 50);
+                . 'subject: '.str_pad($this->getSubject(), 50);
     }
     
     public function template($template)
     {
-        $controller =  is_object($this->controller) ? get_class($this->controller):'null';
+        $controller = is_object($this->controller) ? get_class($this->controller) : 'null';
         
         $event = new Event();
         $eventManager = $this->eventManager;
@@ -181,11 +181,11 @@ 
         $controllerIdentifier = strtolower(substr($controller, 0, strpos($controller, '\\')));
         $viewResolver = $this->viewResolver;
                 
-        $templateHalf = 'mail/' . $template;
+        $templateHalf = 'mail/'.$template;
         $resource = $viewResolver->resolve($templateHalf);
         
         if (empty($resource)) {
-            $templateFull = $controllerIdentifier . '/mail/' . $template;
+            $templateFull = $controllerIdentifier.'/mail/'.$template;
             $resource = $viewResolver->resolve($templateFull);
         }
         
@@ -212,7 +212,7 @@ 
                     unset($__vars[$key]);
                 }
             }
-            unset($__vars['content'],$__vars['controllerIdentifier'],$__vars['controller'],$__vars['resource'],$__vars['template'],$__vars['viewResolver']);
+            unset($__vars['content'], $__vars['controllerIdentifier'], $__vars['controller'], $__vars['resource'], $__vars['template'], $__vars['viewResolver']);
             $this->config = $__vars;
         }
     }
@@ -237,20 +237,20 @@ 
         if (isset($this->config['from'])) {
             $from = $this->config['from'];
         } else {
-            $log->err('A from email address must be provided (Variable $from) in Template: ' . $template);
-            throw new \InvalidArgumentException('A from email address must be provided (Variable $from) in Template: ' . $template);
+            $log->err('A from email address must be provided (Variable $from) in Template: '.$template);
+            throw new \InvalidArgumentException('A from email address must be provided (Variable $from) in Template: '.$template);
         }
         if (isset($this->config['fromName'])) {
             $fromName = $this->config['fromName'];
         } else {
-            $log->err('A from name must be provided (Variable $fromName) in Template: ' . $template);
-            throw new \InvalidArgumentException('A from name must be provided (Variable $fromName) in Template: ' . $template);
+            $log->err('A from name must be provided (Variable $fromName) in Template: '.$template);
+            throw new \InvalidArgumentException('A from name must be provided (Variable $fromName) in Template: '.$template);
         }
         if (isset($this->config['subject'])) {
             $subject = $this->config['subject'];
         } else {
-            $log->err('A subject must be provided (Variable $subject) in Template: ' . $template);
-            throw new \InvalidArgumentException('A subject must be provided (Variable $subject) in Template: ' . $template);
+            $log->err('A subject must be provided (Variable $subject) in Template: '.$template);
+            throw new \InvalidArgumentException('A subject must be provided (Variable $subject) in Template: '.$template);
         }
         $this->setFrom($from, $fromName);
         $this->setSubject($subject);
@@ -271,7 +271,7 @@ 
             $erg = true;
             $log->info($this);
         } catch (\Exception $e) {
-            $log->err('Mail failure ' . $e->getMessage());
+            $log->err('Mail failure '.$e->getMessage());
         }
         return $erg;
     }
@@ -288,6 +288,6 @@ 
         $viewResolver   = $container->get('ViewResolver');
         $eventManager   = $container->get('EventManager');
         $moduleManager  = $container->get('ModuleManager');
-        return new static($mailLog,$viewResolver,$eventManager,$moduleManager);
+        return new static($mailLog, $viewResolver, $eventManager, $moduleManager);
     }
 }

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

Doc Comments -1 removed lines

@@ -110,7 +110,6 @@
      * @see \Core\Controller\Plugin\SearchForm::get()
      *
      * @param        $elementsFieldset
-     * @param null   $buttonsFieldset
      * @param string $as The name of the key in the result array.
      *
      * @return self

Spacing +3 added lines, -3 removed lines

@@ -78,7 +78,7 @@ 
         }
 
         if (!is_array($stack)) {
-            throw new \InvalidArgumentException('Expected argument to be of type array, but received ' . gettype($stack));
+            throw new \InvalidArgumentException('Expected argument to be of type array, but received '.gettype($stack));
         }
 
         $stack = array_intersect_key($stack, ['params' => true, 'form' => true, 'paginator' => true]);
@@ -147,7 +147,7 @@ 
      *
      * @return self
      */
-    public function params($namespace, $defaults = [ 'page' => 1 ])
+    public function params($namespace, $defaults = ['page' => 1])
     {
         $this->stack['params'] = [$namespace, $defaults];
         return $this;
@@ -242,7 +242,7 @@ 
                         $queryArray[$match['name']][$v] = 1;
                     }
                 } else {
-                    $queryArray[ $match[ 'name' ] ] = $value;
+                    $queryArray[$match['name']] = $value;
                 }
                 unset($queryArray[$key]);
             }

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

Doc Comments +1 added lines, -3 removed lines

@@ -46,10 +46,8 @@
      *
      * Proxies to {@link get()}
      *
-     * @param string|array     $elementsFieldset
-     * @param null|string $buttonsFieldset
      *
-     * @return \Core\Form\TextSearchForm
+     * @return \Core\Form\SearchForm
      */
     public function __invoke($form, $options = null, $params = null)
     {

Spacing +2 added lines, -2 removed lines

@@ -72,11 +72,11 @@
     public function get($form, $options = null, $params = null)
     {
         if (!is_object($form)) {
-            $form             = $this->formElementManager->get($form, $options);
+            $form = $this->formElementManager->get($form, $options);
         }
 
         /** @noinspection PhpUndefinedMethodInspection */
-        $params           = $params ?: clone $this->getController()->getRequest()->getQuery();
+        $params = $params ?: clone $this->getController()->getRequest()->getQuery();
 
         /* I tried using form methods (bind, isValid)...
          * but because the search form could be in an invalidated state

module/Core/src/Entity/AddressInterface.php

Doc Comments -1 removed lines

@@ -55,7 +55,6 @@
      * Two-letter codes from the ISO 3166 standard as implemented by The Internet Assigned
      * Numbers Authority.
      *
-     * @param string $postalCode {'AC'|'AD'|'AE'|'AF'|'AG'|'AI'|'AL'|'AM'|'AN'|'AO'|'AQ'|'AR'|'AS'|'AT'|'AU'|'AW'|'AX'|'AZ'|'BA'|'BB'|'BD'|'BE'|'BF'|'BG'|'BH'|'BI'|'BJ'|'BL'|'BM'|'BN'|'BO'|'BR'|'BS'|'BT'|'BV'|'BW'|'BY'|'BZ'|'CA'|'CC'|'CD'|'CF'|'CG'|'CH'|'CI'|'CK'|'CL'|'CM'|'CN'|'CO'|'CR'|'CU'|'CV'|'CX'|'CY'|'CZ'|'DE'|'DJ'|'DK'|'DM'|'DO'|'DZ'|'EC'|'EE'|'EG'|'EH'|'ER'|'ES'|'ET'|'EU'|'FI'|'FJ'|'FK'|'FM'|'FO'|'FR'|'GA'|'GB'|'GD'|'GE'|'GF'|'GG'|'GH'|'GI'|'GL'|'GM'|'GN'|'GP'|'GQ'|'GR'|'GS'|'GT'|'GU'|'GW'|'GY'|'HK'|'HM'|'HN'|'HR'|'HT'|'HU'|'ID'|'IE'|'IL'|'IM'|'IN'|'IO'|'IQ'|'IR'|'IS'|'IT'|'JE'|'JM'|'JO'|'JP'|'KE'|'KG'|'KH'|'KI'|'KM'|'KN'|'KP'|'KR'|'KW'|'KY'|'KZ'|'LA'|'LB'|'LC'|'LI'|'LK'|'LR'|'LS'|'LT'|'LU'|'LV'|'LY'|'MA'|'MC'|'MD'|'ME'|'MF'|'MG'|'MH'|'MK'|'ML'|'MM'|'MN'|'MO'|'MP'|'MQ'|'MR'|'MS'|'MT'|'MU'|'MV'|'MW'|'MX'|'MY'|'MZ'|'NA'|'NC'|'NE'|'NF'|'NG'|'NI'|'NL'|'NO'|'NP'|'NR'|'NU'|'NZ'|'OM'|'PA'|'PE'|'PF'|'PG'|'PH'|'PK'|'PL'|'PM'|'PN'|'PR'|'PS'|'PT'|'PW'|'PY'|'QA'|'RE'|'RO'|'RS'|'RU'|'RW'|'SA'|'SB'|'SC'|'SD'|'SE'|'SG'|'SH'|'SI'|'SJ'|'SK'|'SL'|'SM'|'SN'|'SO'|'SR'|'ST'|'SU'|'SV'|'SY'|'SZ'|'TC'|'TD'|'TF'|'TG'|'TH'|'TJ'|'TK'|'TL'|'TM'|'TN'|'TO'|'TP'|'TR'|'TT'|'TV'|'TW'|'TZ'|'UA'|'UG'|'UK'|'UM'|'US'|'UY'|'UZ'|'VA'|'VC'|'VE'|'VG'|'VI'|'VN'|'VU'|'WF'|'WS'|'YE'|'YT'|'YU'|'ZA'|'ZM'|'ZW'}
      * @return AddressInterface
      */
     public function setCountryCode($countryCode);

Indentation +6 added lines, -6 removed lines

@@ -14,12 +14,12 @@
 interface AddressInterface extends EntityInterface
 {
        
-   /**
-    * Postal Code of the Address.
-    *
-    * @param string $postalCode
-    * @return AddressInterface
-    */
+    /**
+     * Postal Code of the Address.
+     *
+     * @param string $postalCode
+     * @return AddressInterface
+     */
     public function setPostalCode($postalCode);
     public function getPostalCode();
    

module/Core/src/Entity/AttachableEntityInterface.php

Doc Comments +1 added lines

@@ -22,6 +22,7 @@
     /**
      * @param AttachableEntityManager $attachableEntityManager
      * @throws \LogicException If attachable entity manager is already set
+     * @return AttachableEntityTrait
      */
     public function setAttachableEntityManager(AttachableEntityManager $attachableEntityManager);
 

module/Core/src/Entity/EntityTrait.php

Doc Comments +3 added lines

@@ -22,6 +22,9 @@
  */
 trait EntityTrait
 {
+    /**
+     * @param string $property
+     */
     public function notEmpty($property, array $args=[])
     {
         $method = "get$property";

Spacing +3 added lines, -3 removed lines

@@ -20,16 +20,16 @@
  */
 trait EntityTrait
 {
-    public function notEmpty($property, array $args=[])
+    public function notEmpty($property, array $args = [])
     {
         $method = "get$property";
 
         if (!method_exists($this, $method)) {
-            throw new OutOfBoundsException("'$property' is not a valid property of '" . get_class($this) . "'");
+            throw new OutOfBoundsException("'$property' is not a valid property of '".get_class($this)."'");
         }
 
         $value = count($args)
-            ? call_user_func_array([ $this, $method ], $args)
+            ? call_user_func_array([$this, $method], $args)
             : $this->$method();
 
         if (null === $value) { // is_scalar does not consider 'null' to be scalar value.

module/Core/src/Entity/FileEntity.php

Doc Comments +3 added lines, -3 removed lines

@@ -129,7 +129,7 @@ 
     /**
      * Gets the name of a file
      *
-     * @return mixed
+     * @return string
      */
     public function getName()
     {
@@ -197,7 +197,7 @@ 
     /**
      * Gets the uploaded date of a file
      *
-     * @return mixed
+     * @return null|\DateTime
      */
     public function getDateUploaded()
     {
@@ -233,7 +233,7 @@ 
     /**
      * Gets the length of the file
      *
-     * @return mixed
+     * @return string|null
      */
     public function getLength()
     {

Spacing +4 added lines, -4 removed lines

@@ -146,18 +146,18 @@
         $size = $this->getLength();
         
         if ($size >= 1073741824) {
-            return round($size / 1073741824, 2) . ' GB';
+            return round($size / 1073741824, 2).' GB';
         }
         
         if ($size >= 1048576) {
-            return round($size / 1048576, 2) . ' MB';
+            return round($size / 1048576, 2).' MB';
         }
         
         if ($size >= 1024) {
-            return round($size / 1024, 2) . ' kB';
+            return round($size / 1024, 2).' kB';
         }
         
-        return (string)$size;
+        return (string) $size;
     }
 
     /**