@@ -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 | } |
@@ -39,11 +39,11 @@ |
||
39 | 39 | $element->setLabel(''); |
40 | 40 | $element->setAttribute('placeholder', $label); |
41 | 41 | } |
42 | - $formContent .= $this->getView()->formRow($element, null, null, Form::LAYOUT_BARE) . ' '; |
|
42 | + $formContent .= $this->getView()->formRow($element, null, null, Form::LAYOUT_BARE).' '; |
|
43 | 43 | } |
44 | 44 | } |
45 | 45 | |
46 | 46 | |
47 | - return $this->openTag($form) . $formContent . $this->closeTag(); |
|
47 | + return $this->openTag($form).$formContent.$this->closeTag(); |
|
48 | 48 | } |
49 | 49 | } |