@@ -59,7 +59,7 @@ discard block |
||
| 59 | 59 | if (!$this->filterComposite->filter($propertyName)) { |
| 60 | 60 | continue; |
| 61 | 61 | } |
| 62 | - $getter = 'get' . ucfirst($propertyName); |
|
| 62 | + $getter = 'get'.ucfirst($propertyName); |
|
| 63 | 63 | $value = method_exists($object, $getter) |
| 64 | 64 | ? $object->$getter() |
| 65 | 65 | : $property->getValue($object); |
@@ -87,7 +87,7 @@ discard block |
||
| 87 | 87 | foreach ($this->data as $key => $value) { |
| 88 | 88 | if (isset($reflProperties[$key])) { |
| 89 | 89 | $value = $this->hydrateValue($key, $value); |
| 90 | - $setter = 'set' . ucfirst($key); |
|
| 90 | + $setter = 'set'.ucfirst($key); |
|
| 91 | 91 | if (method_exists($object, $setter)) { |
| 92 | 92 | $object->$setter($value); |
| 93 | 93 | } else { |
@@ -310,9 +310,9 @@ |
||
| 310 | 310 | if (!$user) { |
| 311 | 311 | $user = $repository->create(); |
| 312 | 312 | $user->setEmail($email) |
| 313 | - ->setLogin($email) |
|
| 314 | - ->setRole(\Auth\Entity\User::ROLE_RECRUITER) |
|
| 315 | - ->setIsDraft(true); |
|
| 313 | + ->setLogin($email) |
|
| 314 | + ->setRole(\Auth\Entity\User::ROLE_RECRUITER) |
|
| 315 | + ->setIsDraft(true); |
|
| 316 | 316 | $info = $user->getInfo(); |
| 317 | 317 | /* @var $info \Auth\Entity\InfoInterface */ |
| 318 | 318 | $info->setEmail($email); |
@@ -43,7 +43,7 @@ |
||
| 43 | 43 | $setterMethods = $object->getSetters(); |
| 44 | 44 | } |
| 45 | 45 | foreach ($data as $key => $value) { |
| 46 | - $setter = 'set' . ucfirst($key); |
|
| 46 | + $setter = 'set'.ucfirst($key); |
|
| 47 | 47 | if (in_array($setter, $setterMethods)) { |
| 48 | 48 | $object->$setter($value); |
| 49 | 49 | } else { |
@@ -32,9 +32,9 @@ discard block |
||
| 32 | 32 | |
| 33 | 33 | $getters = array_filter( |
| 34 | 34 | get_class_methods($object), |
| 35 | - function ($methodName) use ($object) { |
|
| 35 | + function($methodName) use ($object) { |
|
| 36 | 36 | return "get" === substr($methodName, 0, 3) |
| 37 | - && method_exists($object, 's' . substr($methodName, 1)); |
|
| 37 | + && method_exists($object, 's'.substr($methodName, 1)); |
|
| 38 | 38 | } |
| 39 | 39 | ); |
| 40 | 40 | |
@@ -61,7 +61,7 @@ discard block |
||
| 61 | 61 | } |
| 62 | 62 | $setters = array_filter( |
| 63 | 63 | get_class_methods($object), |
| 64 | - function ($methodName) { |
|
| 64 | + function($methodName) { |
|
| 65 | 65 | return "set" === substr($methodName, 0, 3); |
| 66 | 66 | } |
| 67 | 67 | ); |
@@ -89,7 +89,7 @@ |
||
| 89 | 89 | /** |
| 90 | 90 | * exclude methods from the automatism |
| 91 | 91 | * |
| 92 | - * @param $methods |
|
| 92 | + * @param string[] $methods |
|
| 93 | 93 | */ |
| 94 | 94 | public function setExcludeMethods($methods) |
| 95 | 95 | { |
@@ -149,21 +149,21 @@ discard block |
||
| 149 | 149 | } |
| 150 | 150 | |
| 151 | 151 | if (preg_match('~^is(View|Change|None|All)Granted$~', $method, $match)) { |
| 152 | - $permission = constant('self::PERMISSION_' . strtoupper($match[1])); |
|
| 152 | + $permission = constant('self::PERMISSION_'.strtoupper($match[1])); |
|
| 153 | 153 | return $this->isGranted($params[0], $permission); |
| 154 | 154 | } |
| 155 | 155 | |
| 156 | 156 | if (preg_match('~^grant(View|Change|None|All)$~', $method, $match)) { |
| 157 | - $permission = constant('self::PERMISSION_' . strtoupper($match[1])); |
|
| 157 | + $permission = constant('self::PERMISSION_'.strtoupper($match[1])); |
|
| 158 | 158 | return $this->grant($params[0], $permission); |
| 159 | 159 | } |
| 160 | 160 | |
| 161 | 161 | if (preg_match('~^revoke(View|Change|None|All)$~', $method, $match)) { |
| 162 | - $permission = constant('self::PERMISSION_' . strtoupper($match[1])); |
|
| 162 | + $permission = constant('self::PERMISSION_'.strtoupper($match[1])); |
|
| 163 | 163 | return $this->revoke($params[0], $permission); |
| 164 | 164 | } |
| 165 | 165 | |
| 166 | - throw new \BadMethodCallException('Unknown method "' . $method . '"'); |
|
| 166 | + throw new \BadMethodCallException('Unknown method "'.$method.'"'); |
|
| 167 | 167 | } |
| 168 | 168 | |
| 169 | 169 | /** |
@@ -311,7 +311,7 @@ discard block |
||
| 311 | 311 | if ($build) { |
| 312 | 312 | $this->build(); |
| 313 | 313 | } |
| 314 | - $this->hasChanged= true; |
|
| 314 | + $this->hasChanged = true; |
|
| 315 | 315 | |
| 316 | 316 | // restore orginial permissions type |
| 317 | 317 | $this->type = $oldType; |
@@ -444,10 +444,10 @@ discard block |
||
| 444 | 444 | } |
| 445 | 445 | |
| 446 | 446 | if ($resource instanceof UserInterface) { |
| 447 | - return 'user:' . $resource->getId(); |
|
| 447 | + return 'user:'.$resource->getId(); |
|
| 448 | 448 | } |
| 449 | 449 | |
| 450 | - return 'user:' . $resource; |
|
| 450 | + return 'user:'.$resource; |
|
| 451 | 451 | } |
| 452 | 452 | |
| 453 | 453 | /** |
@@ -467,7 +467,7 @@ discard block |
||
| 467 | 467 | ); |
| 468 | 468 | if (!in_array($permission, $perms)) { |
| 469 | 469 | throw new \InvalidArgumentException( |
| 470 | - 'Invalid permission. Must be one of ' . implode(', ', $perms) |
|
| 470 | + 'Invalid permission. Must be one of '.implode(', ', $perms) |
|
| 471 | 471 | ); |
| 472 | 472 | } |
| 473 | 473 | } |
@@ -354,6 +354,9 @@ |
||
| 354 | 354 | return $this; |
| 355 | 355 | } |
| 356 | 356 | |
| 357 | + /** |
|
| 358 | + * @param string $permission |
|
| 359 | + */ |
|
| 357 | 360 | private function checkIsGranted($userId, $permission) |
| 358 | 361 | { |
| 359 | 362 | if (!$userId) { return false; } |
@@ -49,7 +49,7 @@ discard block |
||
| 49 | 49 | return $this->$method($value); |
| 50 | 50 | } |
| 51 | 51 | |
| 52 | - throw new OutOfBoundsException("'$property' is not a valid property of '" . get_class($this). "'"); |
|
| 52 | + throw new OutOfBoundsException("'$property' is not a valid property of '".get_class($this)."'"); |
|
| 53 | 53 | } |
| 54 | 54 | |
| 55 | 55 | /** |
@@ -76,7 +76,7 @@ discard block |
||
| 76 | 76 | if (method_exists($this, $method)) { |
| 77 | 77 | return $this->$method(); |
| 78 | 78 | } |
| 79 | - throw new OutOfBoundsException("'$property' is not a valid property of '" . get_class($this) . "'"); |
|
| 79 | + throw new OutOfBoundsException("'$property' is not a valid property of '".get_class($this)."'"); |
|
| 80 | 80 | } |
| 81 | 81 | |
| 82 | 82 | /** |
@@ -66,13 +66,13 @@ |
||
| 66 | 66 | } |
| 67 | 67 | |
| 68 | 68 | $output = '<div class="br-widget br-readonly ' |
| 69 | - . ('compact' == $mode ? ' br-compact' : '' ) |
|
| 70 | - . '" title="' . $this->getView()->translate(self::$ratingValueMap[$rating]) . '">'; |
|
| 71 | - for ($i=1; $i<6; $i++) { |
|
| 69 | + . ('compact' == $mode ? ' br-compact' : '') |
|
| 70 | + . '" title="'.$this->getView()->translate(self::$ratingValueMap[$rating]).'">'; |
|
| 71 | + for ($i = 1; $i < 6; $i++) { |
|
| 72 | 72 | $class = $i <= $rating ? 'br-selected' : ''; |
| 73 | 73 | $class .= $i == $rating ? ' br-current' : ''; |
| 74 | 74 | |
| 75 | - $output .= '<a class="' . $class . '"></a>'; |
|
| 75 | + $output .= '<a class="'.$class.'"></a>'; |
|
| 76 | 76 | } |
| 77 | 77 | $output .= '</div>'; |
| 78 | 78 | |
@@ -58,7 +58,7 @@ discard block |
||
| 58 | 58 | : ''; |
| 59 | 59 | |
| 60 | 60 | if (false !== strpos($urlOrEmail, '@')) { |
| 61 | - $urlOrEmail = 'mailto:' . $urlOrEmail; |
|
| 61 | + $urlOrEmail = 'mailto:'.$urlOrEmail; |
|
| 62 | 62 | } |
| 63 | 63 | |
| 64 | 64 | return sprintf('<a %s href="%s">%s</a>', $attributesStr, $urlOrEmail, $label); |
@@ -73,7 +73,7 @@ discard block |
||
| 73 | 73 | $attr = array(); |
| 74 | 74 | |
| 75 | 75 | foreach ($attributes as $name => $value) { |
| 76 | - $attr[] = $escape($name) . '="' . $escapeAttr($value) . '"'; |
|
| 76 | + $attr[] = $escape($name).'="'.$escapeAttr($value).'"'; |
|
| 77 | 77 | } |
| 78 | 78 | |
| 79 | 79 | return implode(' ', $attr); |
@@ -57,7 +57,7 @@ |
||
| 57 | 57 | protected function checkObjectType($object) |
| 58 | 58 | { |
| 59 | 59 | if (!$object instanceof $this->objectType) { |
| 60 | - throw new \InvalidArgumentException('Wrapped entity must be of type ' . $this->objectType); |
|
| 60 | + throw new \InvalidArgumentException('Wrapped entity must be of type '.$this->objectType); |
|
| 61 | 61 | } |
| 62 | 62 | } |
| 63 | 63 | } |