cross-solution / YAWIK

module/Core/src/Core/Entity/Hydrator/EntityHydrator.php

Spacing +3 added lines, -3 removed lines

@@ -32,9 +32,9 @@ 
         
         $getters = array_filter(
             get_class_methods($object),
-            function ($methodName) use ($object) {
+            function($methodName) use ($object) {
                 return "get" === substr($methodName, 0, 3)
-                && method_exists($object, 's' . substr($methodName, 1));
+                && method_exists($object, 's'.substr($methodName, 1));
             }
         );
 
@@ -61,7 +61,7 @@ 
         }
         $setters = array_filter(
             get_class_methods($object),
-            function ($methodName) {
+            function($methodName) {
                 return "set" === substr($methodName, 0, 3);
             }
         );

Doc Comments +1 added lines, -1 removed lines

@@ -89,7 +89,7 @@
     /**
      * exclude methods from the automatism
      *
-     * @param $methods
+     * @param string[] $methods
      */
     public function setExcludeMethods($methods)
     {

module/Core/src/Core/Entity/Permissions.php

Spacing +8 added lines, -8 removed lines

@@ -149,21 +149,21 @@ 
         }
 
         if (preg_match('~^is(View|Change|None|All)Granted$~', $method, $match)) {
-            $permission = constant('self::PERMISSION_' . strtoupper($match[1]));
+            $permission = constant('self::PERMISSION_'.strtoupper($match[1]));
             return $this->isGranted($params[0], $permission);
         }
         
         if (preg_match('~^grant(View|Change|None|All)$~', $method, $match)) {
-            $permission = constant('self::PERMISSION_' . strtoupper($match[1]));
+            $permission = constant('self::PERMISSION_'.strtoupper($match[1]));
             return $this->grant($params[0], $permission);
         }
         
         if (preg_match('~^revoke(View|Change|None|All)$~', $method, $match)) {
-            $permission = constant('self::PERMISSION_' . strtoupper($match[1]));
+            $permission = constant('self::PERMISSION_'.strtoupper($match[1]));
             return $this->revoke($params[0], $permission);
         }
         
-        throw new \BadMethodCallException('Unknown method "' . $method . '"');
+        throw new \BadMethodCallException('Unknown method "'.$method.'"');
     }
 
     /**
@@ -311,7 +311,7 @@ 
         if ($build) {
             $this->build();
         }
-        $this->hasChanged= true;
+        $this->hasChanged = true;
 
         // restore orginial permissions type
         $this->type = $oldType;
@@ -444,10 +444,10 @@ 
         }
         
         if ($resource instanceof UserInterface) {
-            return 'user:' . $resource->getId();
+            return 'user:'.$resource->getId();
         }
         
-        return 'user:' . $resource;
+        return 'user:'.$resource;
     }
 
     /**
@@ -467,7 +467,7 @@ 
         );
         if (!in_array($permission, $perms)) {
             throw new \InvalidArgumentException(
-                'Invalid permission. Must be one of ' . implode(', ', $perms)
+                'Invalid permission. Must be one of '.implode(', ', $perms)
             );
         }
     }

Doc Comments +3 added lines

@@ -354,6 +354,9 @@
         return $this;
     }
     
+    /**
+     * @param string $permission
+     */
     private function checkIsGranted($userId, $permission)
     {
         if (!$userId) { return false; }

module/Core/src/Core/Entity/AbstractEntity.php

Spacing +2 added lines, -2 removed lines

@@ -49,7 +49,7 @@ 
             return $this->$method($value);
         }
         
-        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)."'");
     }
     
     /**
@@ -76,7 +76,7 @@ 
         if (method_exists($this, $method)) {
             return $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)."'");
     }
     
     /**

module/Core/src/Core/View/Helper/Rating.php

Spacing +4 added lines, -4 removed lines

@@ -66,13 +66,13 @@
         }
         
         $output = '<div class="br-widget br-readonly '
-                . ('compact' == $mode ? ' br-compact' : '' )
-                . '" title="' . $this->getView()->translate(self::$ratingValueMap[$rating]) . '">';
-        for ($i=1; $i<6; $i++) {
+                . ('compact' == $mode ? ' br-compact' : '')
+                . '" title="'.$this->getView()->translate(self::$ratingValueMap[$rating]).'">';
+        for ($i = 1; $i < 6; $i++) {
             $class = $i <= $rating ? 'br-selected' : '';
             $class .= $i == $rating ? ' br-current' : '';
             
-            $output .= '<a class="' . $class . '"></a>';
+            $output .= '<a class="'.$class.'"></a>';
         }
         $output .= '</div>';
         

module/Core/src/Core/View/Helper/Link.php

Spacing +2 added lines, -2 removed lines

@@ -58,7 +58,7 @@ 
                        : '';
         
         if (false !== strpos($urlOrEmail, '@')) {
-            $urlOrEmail = 'mailto:' . $urlOrEmail;
+            $urlOrEmail = 'mailto:'.$urlOrEmail;
         }
         
         return sprintf('<a %s href="%s">%s</a>', $attributesStr, $urlOrEmail, $label);
@@ -73,7 +73,7 @@ 
         $attr       = array();
         
         foreach ($attributes as $name => $value) {
-            $attr[] = $escape($name) . '="' . $escapeAttr($value) . '"';
+            $attr[] = $escape($name).'="'.$escapeAttr($value).'"';
         }
         
         return implode(' ', $attr);

module/Core/src/Core/Decorator/Decorator.php

Spacing +1 added lines, -1 removed lines

@@ -57,7 +57,7 @@
     protected function checkObjectType($object)
     {
         if (!$object instanceof $this->objectType) {
-            throw new \InvalidArgumentException('Wrapped entity must be of type ' . $this->objectType);
+            throw new \InvalidArgumentException('Wrapped entity must be of type '.$this->objectType);
         }
     }
 }

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

Spacing +2 added lines, -2 removed lines

@@ -39,11 +39,11 @@
                     $element->setLabel('');
                     $element->setAttribute('placeholder', $label);
                 }
-                $formContent .= $this->getView()->formRow($element, null, null, Form::LAYOUT_BARE) . ' ';
+                $formContent .= $this->getView()->formRow($element, null, null, Form::LAYOUT_BARE).' ';
             }
         }
         
         
-        return $this->openTag($form) . $formContent . $this->closeTag();
+        return $this->openTag($form).$formContent.$this->closeTag();
     }
 }

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

Indentation +8 added lines, -8 removed lines

@@ -52,10 +52,10 @@ 
 
         if ($element instanceof ViewPartialProviderInterface && !$ignoreViewPartial) {
             return $this->getView()->partial($element->getViewPartial(),
-                                             array(
-                                                 'element' => $element,
-                                                 'labelWitdh' => $labelWidth,
-                                                 'label_attributes' => $labelAttributes
+                                                array(
+                                                    'element' => $element,
+                                                    'labelWitdh' => $labelWidth,
+                                                    'label_attributes' => $labelAttributes
                                                             )
             );
         }
@@ -107,10 +107,10 @@ 
         $desc = $element->getOption('description', false);
         if ($desc && $this->layout != Form::LAYOUT_BARE) {
             if (null !== ($translator = $this->getTranslator())) {
-                             $desc = $translator->translate(
-                                 $desc,
-                                 $this->getTranslatorTextDomain()
-                             );
+                                $desc = $translator->translate(
+                                    $desc,
+                                    $this->getTranslatorTextDomain()
+                                );
             }
             $elementString .= sprintf(
                 '<div id="%s-desc" class="cam-description alert alert-info">%s</div>',

Spacing +8 added lines, -8 removed lines

@@ -76,8 +76,8 @@ 
         
         // Does this element have errors ?
         if (!empty($elementErrors) && !empty($inputErrorClass) && $this->layout != Form::LAYOUT_BARE) {
-            $classAttributes = ($element->hasAttribute('class') ? $element->getAttribute('class') . ' ' : '');
-            $classAttributes = $classAttributes . $inputErrorClass;
+            $classAttributes = ($element->hasAttribute('class') ? $element->getAttribute('class').' ' : '');
+            $classAttributes = $classAttributes.$inputErrorClass;
     
             $element->setAttribute('class', $classAttributes);
         }
@@ -85,7 +85,7 @@ 
         if (!$element->hasAttribute('id')) {
             $elementId = preg_replace(
                 array('~[^A-Za-z0-9_-]~', '~--+~', '~^-|-$~'),
-                array('-'             , '-',     ''),
+                array('-', '-', ''),
                 $element->getName()
             );
             $element->setAttribute('id', $elementId);
@@ -131,7 +131,7 @@ 
         }
         
         // moved label here so we can change it in the ElementViewHelper
-        $label           = $element->getLabel();
+        $label = $element->getLabel();
         if (isset($label) && '' !== $label && !$element instanceof \Zend\Form\Element\Button) {
             // Translate the label
             if (null !== ($translator = $this->getTranslator())) {
@@ -173,11 +173,11 @@ 
                     }
                     $element->setLabelAttributes($labelAttributes);
 
-                    $label = $labelHelper($element,$label);
+                    $label = $labelHelper($element, $label);
                     if ($this->shouldWrap) {
                         $spanWidth = 12 - $labelWidth;
                         $elementString = sprintf(
-                            '<div class="col-md-%d%s" id="' . $elementId . '-span">%s</div>',
+                            '<div class="col-md-%d%s" id="'.$elementId.'-span">%s</div>',
                             $spanWidth,
                             $elementErrors ? " $inputErrorClass" : '',
                             $elementString
@@ -189,7 +189,7 @@ 
                         );
 
                     }
-                        $markup = $label . $elementString;
+                        $markup = $label.$elementString;
                 }
             }
         } else {
@@ -210,7 +210,7 @@ 
             && !$element instanceof \Zend\Form\Element\Button
             && $this->layout != Form::LAYOUT_BARE
             ) {
-            $markup = sprintf('<div class="controls controls-row ' . $form_row_class . '">%s</div>', $markup);
+            $markup = sprintf('<div class="controls controls-row '.$form_row_class.'">%s</div>', $markup);
         }
     
         return $markup;

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

Spacing +2 added lines, -2 removed lines

@@ -26,7 +26,7 @@ 
      *
      * @var string
      */
-    protected $language="de";
+    protected $language = "de";
 
     public function __invoke(ElementInterface $element = null)
     {
@@ -60,7 +60,7 @@ 
         /*
          *
          */
-        $markup = '<div class="input-group date">%s<div data-toggle="description" data-target="%s" class="input-group-addon" onClick="$(this).parent().find(\':input\').datepicker(\'show\')">' .
+        $markup = '<div class="input-group date">%s<div data-toggle="description" data-target="%s" class="input-group-addon" onClick="$(this).parent().find(\':input\').datepicker(\'show\')">'.
             '<i class="fa fa-calendar"></i></div></div><div class="checkbox"></div>';
         
         $markup = sprintf(