@@ -35,7 +35,7 @@ |
||
35 | 35 | /** |
36 | 36 | * @return string |
37 | 37 | */ |
38 | - public static function random () { |
|
38 | + public static function random() { |
|
39 | 39 | try { |
40 | 40 | $colors = (new ReflectionClass(self::class))->getConstants(); |
41 | 41 | return $colors[array_rand($colors)]; |
@@ -39,8 +39,7 @@ |
||
39 | 39 | try { |
40 | 40 | $colors = (new ReflectionClass(self::class))->getConstants(); |
41 | 41 | return $colors[array_rand($colors)]; |
42 | - } |
|
43 | - catch (ReflectionException $exception) { |
|
42 | + } catch (ReflectionException $exception) { |
|
44 | 43 | return 'none'; // unreachable |
45 | 44 | } |
46 | 45 | } |
@@ -37,23 +37,23 @@ discard block |
||
37 | 37 | * @param CustomField $customField |
38 | 38 | * @param array $data |
39 | 39 | */ |
40 | - public function __construct (CustomField $customField, array $data = []) { |
|
40 | + public function __construct(CustomField $customField, array $data = []) { |
|
41 | 41 | parent::__construct($customField, $data); |
42 | 42 | $this->customField = $customField; |
43 | 43 | } |
44 | 44 | |
45 | - final public function __toString (): string { |
|
45 | + final public function __toString(): string { |
|
46 | 46 | return "enum_options/{$this->getGid()}"; |
47 | 47 | } |
48 | 48 | |
49 | - final protected function _getDir (): string { |
|
49 | + final protected function _getDir(): string { |
|
50 | 50 | return "{$this->customField}/enum_options"; |
51 | 51 | } |
52 | 52 | |
53 | 53 | /** |
54 | 54 | * @return CustomField |
55 | 55 | */ |
56 | - public function getCustomField () { |
|
56 | + public function getCustomField() { |
|
57 | 57 | return $this->customField; |
58 | 58 | } |
59 | 59 | |
@@ -61,7 +61,7 @@ discard block |
||
61 | 61 | * @param EnumOption $option |
62 | 62 | * @return $this |
63 | 63 | */ |
64 | - public function insertAfter (EnumOption $option) { |
|
64 | + public function insertAfter(EnumOption $option) { |
|
65 | 65 | assert($option->hasGid()); |
66 | 66 | if ($this->hasGid()) { |
67 | 67 | $this->api->post("{$this->_getDir()}/insert", [ |
@@ -80,7 +80,7 @@ discard block |
||
80 | 80 | * @param EnumOption $option |
81 | 81 | * @return $this |
82 | 82 | */ |
83 | - public function insertBefore (EnumOption $option) { |
|
83 | + public function insertBefore(EnumOption $option) { |
|
84 | 84 | assert($option->hasGid()); |
85 | 85 | if ($this->hasGid()) { |
86 | 86 | $this->api->post("{$this->_getDir()}/insert", [ |
@@ -68,8 +68,7 @@ discard block |
||
68 | 68 | 'enum_option' => $this->getGid(), |
69 | 69 | 'after_enum_option' => $option->getGid() |
70 | 70 | ]); |
71 | - } |
|
72 | - else { |
|
71 | + } else { |
|
73 | 72 | unset($this->data['insert_before'], $this->diff['insert_before']); |
74 | 73 | $this->_set('insert_after', $option->getGid()); |
75 | 74 | } |
@@ -87,8 +86,7 @@ discard block |
||
87 | 86 | 'enum_option' => $this->getGid(), |
88 | 87 | 'before_enum_option' => $option->getGid() |
89 | 88 | ]); |
90 | - } |
|
91 | - else { |
|
89 | + } else { |
|
92 | 90 | unset($this->data['insert_after'], $this->diff['insert_after']); |
93 | 91 | $this->_set('insert_before', $option->getGid()); |
94 | 92 | } |
@@ -30,11 +30,11 @@ discard block |
||
30 | 30 | } |
31 | 31 | use DeleteTrait; |
32 | 32 | |
33 | - final public function __toString (): string { |
|
33 | + final public function __toString(): string { |
|
34 | 34 | return "webhooks/{$this->getGid()}"; |
35 | 35 | } |
36 | 36 | |
37 | - final protected function _getDir (): string { |
|
37 | + final protected function _getDir(): string { |
|
38 | 38 | return 'webhooks'; |
39 | 39 | } |
40 | 40 | |
@@ -43,7 +43,7 @@ discard block |
||
43 | 43 | * @param string $target |
44 | 44 | * @return $this |
45 | 45 | */ |
46 | - public function create ($resource, string $target) { |
|
46 | + public function create($resource, string $target) { |
|
47 | 47 | $this->_set('resource', $resource); |
48 | 48 | $this->_set('target', $target); |
49 | 49 | return $this->_create(); |
@@ -62,8 +62,7 @@ |
||
62 | 62 | $option->setName($name); |
63 | 63 | if ($this->hasGid()) { |
64 | 64 | $option->create(); |
65 | - } |
|
66 | - else { |
|
65 | + } else { |
|
67 | 66 | $this->diff['enum_options'] = true; |
68 | 67 | } |
69 | 68 | $this->data['enum_options'][] = $option; |
@@ -40,11 +40,11 @@ discard block |
||
40 | 40 | 'enum_options' => [EnumOption::class], |
41 | 41 | ]; |
42 | 42 | |
43 | - final public function __toString (): string { |
|
43 | + final public function __toString(): string { |
|
44 | 44 | return "custom_fields/{$this->getGid()}"; |
45 | 45 | } |
46 | 46 | |
47 | - final protected function _getDir (): string { |
|
47 | + final protected function _getDir(): string { |
|
48 | 48 | return 'custom_fields'; |
49 | 49 | } |
50 | 50 | |
@@ -52,7 +52,7 @@ discard block |
||
52 | 52 | * @param string $name |
53 | 53 | * @return EnumOption |
54 | 54 | */ |
55 | - public function addEnumOption (string $name) { |
|
55 | + public function addEnumOption(string $name) { |
|
56 | 56 | /** @var EnumOption $option */ |
57 | 57 | $option = $this->factory(EnumOption::class); |
58 | 58 | $option->setName($name); |
@@ -69,28 +69,28 @@ discard block |
||
69 | 69 | /** |
70 | 70 | * @return bool |
71 | 71 | */ |
72 | - public function hasNotificationsEnabled (): bool { |
|
72 | + public function hasNotificationsEnabled(): bool { |
|
73 | 73 | return $this->_is('has_notifications_enabled'); |
74 | 74 | } |
75 | 75 | |
76 | 76 | /** |
77 | 77 | * @return bool |
78 | 78 | */ |
79 | - public function isEnum (): bool { |
|
79 | + public function isEnum(): bool { |
|
80 | 80 | return $this->getResourceSubtype() === self::TYPE_ENUM; |
81 | 81 | } |
82 | 82 | |
83 | 83 | /** |
84 | 84 | * @return bool |
85 | 85 | */ |
86 | - public function isGlobalToWorkspace (): bool { |
|
86 | + public function isGlobalToWorkspace(): bool { |
|
87 | 87 | return $this->_is('is_global_to_workspace'); |
88 | 88 | } |
89 | 89 | |
90 | 90 | /** |
91 | 91 | * @return bool |
92 | 92 | */ |
93 | - final public function isText (): bool { |
|
93 | + final public function isText(): bool { |
|
94 | 94 | return $this->getResourceSubtype() === self::TYPE_TEXT; |
95 | 95 | } |
96 | 96 | |
@@ -98,7 +98,7 @@ discard block |
||
98 | 98 | * @param bool $flag |
99 | 99 | * @return $this |
100 | 100 | */ |
101 | - public function setGlobalToWorkspace (bool $flag) { |
|
101 | + public function setGlobalToWorkspace(bool $flag) { |
|
102 | 102 | return $this->_set('is_global_to_workspace', $flag); |
103 | 103 | } |
104 | 104 | |
@@ -106,7 +106,7 @@ discard block |
||
106 | 106 | * @param bool $flag |
107 | 107 | * @return $this |
108 | 108 | */ |
109 | - public function setNotificationsEnabled (bool $flag) { |
|
109 | + public function setNotificationsEnabled(bool $flag) { |
|
110 | 110 | return $this->_set('has_notifications_enabled', $flag); |
111 | 111 | } |
112 | 112 |
@@ -460,8 +460,7 @@ |
||
460 | 460 | 'sync' => $token, |
461 | 461 | 'opt_expand' => 'this' |
462 | 462 | ])); |
463 | - } |
|
464 | - catch (Error $error) { |
|
463 | + } catch (Error $error) { |
|
465 | 464 | if ($error->getCode() !== 412) { |
466 | 465 | throw $error; |
467 | 466 | } |
@@ -52,24 +52,24 @@ discard block |
||
52 | 52 | /** |
53 | 53 | * @return Cache |
54 | 54 | */ |
55 | - public static function getCache () { |
|
55 | + public static function getCache() { |
|
56 | 56 | return static::$cache ?? static::$cache = new Cache(); |
57 | 57 | } |
58 | 58 | |
59 | 59 | /** |
60 | 60 | * @return Api |
61 | 61 | */ |
62 | - public static function getDefault () { |
|
62 | + public static function getDefault() { |
|
63 | 63 | return self::$default; |
64 | 64 | } |
65 | 65 | |
66 | 66 | /** |
67 | 67 | * @return LoggerInterface |
68 | 68 | */ |
69 | - public static function getLogger () { |
|
69 | + public static function getLogger() { |
|
70 | 70 | return static::$logger ?? static::$logger = new class implements LoggerInterface { |
71 | 71 | |
72 | - public function log (string $info, string $path, ?array $payload): void { |
|
72 | + public function log(string $info, string $path, ?array $payload): void { |
|
73 | 73 | // stub |
74 | 74 | } |
75 | 75 | |
@@ -79,28 +79,28 @@ discard block |
||
79 | 79 | /** |
80 | 80 | * @param Cache $cache |
81 | 81 | */ |
82 | - public static function setCache (Cache $cache) { |
|
82 | + public static function setCache(Cache $cache) { |
|
83 | 83 | static::$cache = $cache; |
84 | 84 | } |
85 | 85 | |
86 | 86 | /** |
87 | 87 | * @param Api $default |
88 | 88 | */ |
89 | - public static function setDefault (Api $default) { |
|
89 | + public static function setDefault(Api $default) { |
|
90 | 90 | self::$default = $default; |
91 | 91 | } |
92 | 92 | |
93 | 93 | /** |
94 | 94 | * @param LoggerInterface $logger |
95 | 95 | */ |
96 | - public static function setLogger (LoggerInterface $logger) { |
|
96 | + public static function setLogger(LoggerInterface $logger) { |
|
97 | 97 | static::$logger = $logger; |
98 | 98 | } |
99 | 99 | |
100 | 100 | /** |
101 | 101 | * @param string $token |
102 | 102 | */ |
103 | - public function __construct (string $token) { |
|
103 | + public function __construct(string $token) { |
|
104 | 104 | $this->token = $token; |
105 | 105 | if (!static::$default) { |
106 | 106 | static::$default = $this; |
@@ -112,7 +112,7 @@ discard block |
||
112 | 112 | * |
113 | 113 | * @param string $path |
114 | 114 | */ |
115 | - public function delete (string $path): void { |
|
115 | + public function delete(string $path): void { |
|
116 | 116 | $this->exec('DELETE', $path); |
117 | 117 | } |
118 | 118 | |
@@ -123,7 +123,7 @@ discard block |
||
123 | 123 | * @return null|array |
124 | 124 | * @internal |
125 | 125 | */ |
126 | - protected function exec (string $method, string $path, array $opts = null) { |
|
126 | + protected function exec(string $method, string $path, array $opts = null) { |
|
127 | 127 | $ch = curl_init(); |
128 | 128 | curl_setopt_array($ch, [ |
129 | 129 | CURLOPT_CUSTOMREQUEST => $method, |
@@ -174,7 +174,7 @@ discard block |
||
174 | 174 | * @param array $data |
175 | 175 | * @return mixed |
176 | 176 | */ |
177 | - public function factory (string $class, $caller, array $data = []) { |
|
177 | + public function factory(string $class, $caller, array $data = []) { |
|
178 | 178 | return new $class($caller, $data); |
179 | 179 | } |
180 | 180 | |
@@ -186,7 +186,7 @@ discard block |
||
186 | 186 | * @param array $options |
187 | 187 | * @return null|array |
188 | 188 | */ |
189 | - public function get (string $path, array $query = [], array $options = []) { |
|
189 | + public function get(string $path, array $query = [], array $options = []) { |
|
190 | 190 | foreach ($options as $name => $value) { |
191 | 191 | $query["opt_{$name}"] = $value; |
192 | 192 | } |
@@ -200,7 +200,7 @@ discard block |
||
200 | 200 | * @param string $gid |
201 | 201 | * @return null|Attachment |
202 | 202 | */ |
203 | - public function getAttachment (string $gid) { |
|
203 | + public function getAttachment(string $gid) { |
|
204 | 204 | return $this->load(Attachment::class, $this, "attachments/{$gid}"); |
205 | 205 | } |
206 | 206 | |
@@ -210,7 +210,7 @@ discard block |
||
210 | 210 | * @param string $gid |
211 | 211 | * @return null|CustomField |
212 | 212 | */ |
213 | - public function getCustomField (string $gid) { |
|
213 | + public function getCustomField(string $gid) { |
|
214 | 214 | return $this->load(CustomField::class, $this, "custom_fields/{$gid}"); |
215 | 215 | } |
216 | 216 | |
@@ -219,7 +219,7 @@ discard block |
||
219 | 219 | * |
220 | 220 | * @return User |
221 | 221 | */ |
222 | - public function getMe () { |
|
222 | + public function getMe() { |
|
223 | 223 | return $this->getUser('me'); |
224 | 224 | } |
225 | 225 | |
@@ -230,7 +230,7 @@ discard block |
||
230 | 230 | * @param array $query |
231 | 231 | * @return string |
232 | 232 | */ |
233 | - protected function getPath (string $path, array $query): string { |
|
233 | + protected function getPath(string $path, array $query): string { |
|
234 | 234 | return $query ? $path . '?' . http_build_query($query) : $path; |
235 | 235 | } |
236 | 236 | |
@@ -240,7 +240,7 @@ discard block |
||
240 | 240 | * @param string $gid |
241 | 241 | * @return null|Portfolio |
242 | 242 | */ |
243 | - public function getPortfolio (string $gid) { |
|
243 | + public function getPortfolio(string $gid) { |
|
244 | 244 | return $this->load(Portfolio::class, $this, "portfolios/{$gid}"); |
245 | 245 | } |
246 | 246 | |
@@ -250,7 +250,7 @@ discard block |
||
250 | 250 | * @param string $gid |
251 | 251 | * @return null|Project |
252 | 252 | */ |
253 | - public function getProject (string $gid) { |
|
253 | + public function getProject(string $gid) { |
|
254 | 254 | return $this->load(Project::class, $this, "projects/{$gid}"); |
255 | 255 | } |
256 | 256 | |
@@ -260,7 +260,7 @@ discard block |
||
260 | 260 | * @param string $gid |
261 | 261 | * @return null|Section |
262 | 262 | */ |
263 | - public function getSection (string $gid) { |
|
263 | + public function getSection(string $gid) { |
|
264 | 264 | return $this->load(Section::class, $this, "sections/{$gid}"); |
265 | 265 | } |
266 | 266 | |
@@ -270,7 +270,7 @@ discard block |
||
270 | 270 | * @param string $gid |
271 | 271 | * @return null|Story |
272 | 272 | */ |
273 | - public function getStory (string $gid) { |
|
273 | + public function getStory(string $gid) { |
|
274 | 274 | return $this->load(Story::class, $this, "stories/{$gid}"); |
275 | 275 | } |
276 | 276 | |
@@ -280,7 +280,7 @@ discard block |
||
280 | 280 | * @param string $gid |
281 | 281 | * @return null|Tag |
282 | 282 | */ |
283 | - public function getTag (string $gid) { |
|
283 | + public function getTag(string $gid) { |
|
284 | 284 | return $this->load(Tag::class, $this, "tags/{$gid}"); |
285 | 285 | } |
286 | 286 | |
@@ -290,7 +290,7 @@ discard block |
||
290 | 290 | * @param string $gid |
291 | 291 | * @return null|Task |
292 | 292 | */ |
293 | - public function getTask (string $gid) { |
|
293 | + public function getTask(string $gid) { |
|
294 | 294 | return $this->load(Task::class, $this, "tasks/{$gid}"); |
295 | 295 | } |
296 | 296 | |
@@ -300,7 +300,7 @@ discard block |
||
300 | 300 | * @param string $gid |
301 | 301 | * @return null|Team |
302 | 302 | */ |
303 | - public function getTeam (string $gid) { |
|
303 | + public function getTeam(string $gid) { |
|
304 | 304 | return $this->load(Team::class, $this, "teams/{$gid}"); |
305 | 305 | } |
306 | 306 | |
@@ -310,7 +310,7 @@ discard block |
||
310 | 310 | * @param string $gid |
311 | 311 | * @return null|User |
312 | 312 | */ |
313 | - public function getUser (string $gid) { |
|
313 | + public function getUser(string $gid) { |
|
314 | 314 | return $this->load(User::class, $this, "users/{$gid}"); |
315 | 315 | } |
316 | 316 | |
@@ -322,7 +322,7 @@ discard block |
||
322 | 322 | * @param array $data |
323 | 323 | * @return ProjectEvent|TaskEvent|StoryEvent |
324 | 324 | */ |
325 | - public function getWebhookEvent (array $data) { |
|
325 | + public function getWebhookEvent(array $data) { |
|
326 | 326 | static $classes = [ |
327 | 327 | Project::TYPE => ProjectEvent::class, |
328 | 328 | Task::TYPE => TaskEvent::class, |
@@ -335,7 +335,7 @@ discard block |
||
335 | 335 | * @param string $gid |
336 | 336 | * @return null|Workspace |
337 | 337 | */ |
338 | - public function getWorkspace (string $gid) { |
|
338 | + public function getWorkspace(string $gid) { |
|
339 | 339 | return $this->load(Workspace::class, $this, "workspaces/{$gid}"); |
340 | 340 | } |
341 | 341 | |
@@ -343,7 +343,7 @@ discard block |
||
343 | 343 | * @param string $name |
344 | 344 | * @return null|Workspace |
345 | 345 | */ |
346 | - public function getWorkspaceByName (string $name) { |
|
346 | + public function getWorkspaceByName(string $name) { |
|
347 | 347 | foreach ($this->getMe()->getWorkspaces() as $workspace) { |
348 | 348 | if ($workspace->getName() === $name) { |
349 | 349 | return $workspace; |
@@ -356,16 +356,16 @@ discard block |
||
356 | 356 | * @param string $json |
357 | 357 | * @return null|array |
358 | 358 | */ |
359 | - protected function jsonDecode (string $json) { |
|
360 | - return json_decode($json, true, JSON_BIGINT_AS_STRING | JSON_THROW_ON_ERROR); |
|
359 | + protected function jsonDecode(string $json) { |
|
360 | + return json_decode($json, true, JSON_BIGINT_AS_STRING|JSON_THROW_ON_ERROR); |
|
361 | 361 | } |
362 | 362 | |
363 | 363 | /** |
364 | 364 | * @param array $data |
365 | 365 | * @return string |
366 | 366 | */ |
367 | - protected function jsonEncode (array $data): string { |
|
368 | - return json_encode($data, JSON_PRETTY_PRINT | JSON_THROW_ON_ERROR); |
|
367 | + protected function jsonEncode(array $data): string { |
|
368 | + return json_encode($data, JSON_PRETTY_PRINT|JSON_THROW_ON_ERROR); |
|
369 | 369 | } |
370 | 370 | |
371 | 371 | /** |
@@ -379,7 +379,7 @@ discard block |
||
379 | 379 | * @param array $query |
380 | 380 | * @return null|mixed|AbstractEntity |
381 | 381 | */ |
382 | - public function load (string $class, $caller, string $path, array $query = []) { |
|
382 | + public function load(string $class, $caller, string $path, array $query = []) { |
|
383 | 383 | $key = $this->getPath($path, $query); |
384 | 384 | return $this->getCache()->get($key, $caller, function($caller) use ($class, $path, $query) { |
385 | 385 | $data = $this->get($path, $query, ['expand' => 'this']); |
@@ -399,7 +399,7 @@ discard block |
||
399 | 399 | * @param int $pages If positive, stops after this many pages have been fetched. |
400 | 400 | * @return array|AbstractEntity[] |
401 | 401 | */ |
402 | - public function loadAll (string $class, $caller, string $path, array $query = [], int $pages = 0) { |
|
402 | + public function loadAll(string $class, $caller, string $path, array $query = [], int $pages = 0) { |
|
403 | 403 | $query['opt_expand'] = 'this'; |
404 | 404 | $query += ['limit' => 100]; |
405 | 405 | $path = $this->getPath($path, $query); |
@@ -425,7 +425,7 @@ discard block |
||
425 | 425 | * @param array $options |
426 | 426 | * @return null|array |
427 | 427 | */ |
428 | - public function post (string $path, array $data = [], array $options = []) { |
|
428 | + public function post(string $path, array $data = [], array $options = []) { |
|
429 | 429 | $response = $this->exec('POST', $path, [ |
430 | 430 | CURLOPT_HTTPHEADER => ['Content-Type: application/json'], |
431 | 431 | CURLOPT_POSTFIELDS => $this->jsonEncode(['options' => $options, 'data' => $data]) |
@@ -441,7 +441,7 @@ discard block |
||
441 | 441 | * @param array $options |
442 | 442 | * @return null|array |
443 | 443 | */ |
444 | - public function put (string $path, array $data = [], array $options = []) { |
|
444 | + public function put(string $path, array $data = [], array $options = []) { |
|
445 | 445 | $response = $this->exec('PUT', $path, [ |
446 | 446 | CURLOPT_HTTPHEADER => ['Content-Type: application/json'], |
447 | 447 | CURLOPT_POSTFIELDS => $this->jsonEncode(['options' => $options, 'data' => $data]) |
@@ -460,7 +460,7 @@ discard block |
||
460 | 460 | * @param null|string $token |
461 | 461 | * @return ProjectEvent[]|TaskEvent[]|StoryEvent[] |
462 | 462 | */ |
463 | - public function sync ($entity, ?string &$token) { |
|
463 | + public function sync($entity, ?string &$token) { |
|
464 | 464 | try { |
465 | 465 | $response = $this->exec('GET', $this->getPath('events', [ |
466 | 466 | 'resource' => $entity->getGid(), |
@@ -495,7 +495,7 @@ discard block |
||
495 | 495 | * @param string $file |
496 | 496 | * @return null|array |
497 | 497 | */ |
498 | - public function upload (string $path, string $file) { |
|
498 | + public function upload(string $path, string $file) { |
|
499 | 499 | $response = $this->exec('POST', $path, [ |
500 | 500 | CURLOPT_POSTFIELDS => ['file' => new CURLFile(realpath($file))] // multipart/form-data |
501 | 501 | ]); |
@@ -179,8 +179,7 @@ discard block |
||
179 | 179 | if ($this->hasGid()) { |
180 | 180 | $this->api->post("{$this}/addFollowers", ['followers' => static::_getGids($users)]); |
181 | 181 | $this->_merge('followers', $users); |
182 | - } |
|
183 | - else { |
|
182 | + } else { |
|
184 | 183 | $this->_merge('followers', $users, true); |
185 | 184 | } |
186 | 185 | return $this; |
@@ -196,8 +195,7 @@ discard block |
||
196 | 195 | if ($this->hasGid()) { |
197 | 196 | $this->api->post("{$this}/addTag", ['tag' => $tag->getGid()]); |
198 | 197 | $this->_merge('tags', [$tag]); |
199 | - } |
|
200 | - else { |
|
198 | + } else { |
|
201 | 199 | $this->_merge('tags', [$tag], true); |
202 | 200 | } |
203 | 201 | return $this; |
@@ -222,8 +220,7 @@ discard block |
||
222 | 220 | 'section' => $section |
223 | 221 | ]); |
224 | 222 | } |
225 | - } |
|
226 | - else { |
|
223 | + } else { |
|
227 | 224 | if (!$this->hasWorkspace()) { |
228 | 225 | $this->setWorkspace($project->getWorkspace()); |
229 | 226 | } |
@@ -509,8 +506,7 @@ discard block |
||
509 | 506 | if ($this->hasGid()) { |
510 | 507 | $this->api->post("{$this}/setParent", ['parent' => $parent ? $parent->getGid() : null]); |
511 | 508 | $this->data['parent'] = $parent; |
512 | - } |
|
513 | - else { |
|
509 | + } else { |
|
514 | 510 | $this->_set('parent', $parent); |
515 | 511 | } |
516 | 512 | return $this; |
@@ -95,14 +95,14 @@ discard block |
||
95 | 95 | 'memberships' => 'memberships.(project|section)' |
96 | 96 | ]; |
97 | 97 | |
98 | - final public function __toString (): string { |
|
98 | + final public function __toString(): string { |
|
99 | 99 | return "tasks/{$this->getGid()}"; |
100 | 100 | } |
101 | 101 | |
102 | 102 | /** |
103 | 103 | * Clears the diffs for `custom_fields` and `external` sub-objects after saving. |
104 | 104 | */ |
105 | - protected function _clearSubDiffs (): void { |
|
105 | + protected function _clearSubDiffs(): void { |
|
106 | 106 | // use isset() to avoid has() fetch. |
107 | 107 | if (isset($this->data['custom_fields'])) { |
108 | 108 | $this->getCustomFields()->diff = []; |
@@ -112,7 +112,7 @@ discard block |
||
112 | 112 | } |
113 | 113 | } |
114 | 114 | |
115 | - final protected function _getDir (): string { |
|
115 | + final protected function _getDir(): string { |
|
116 | 116 | return 'tasks'; |
117 | 117 | } |
118 | 118 | |
@@ -123,7 +123,7 @@ discard block |
||
123 | 123 | * @param string $file |
124 | 124 | * @return Attachment |
125 | 125 | */ |
126 | - public function addAttachment (string $file) { |
|
126 | + public function addAttachment(string $file) { |
|
127 | 127 | /** @var Attachment $attachment */ |
128 | 128 | $attachment = $this->factory(Attachment::class, ['parent' => $this]); |
129 | 129 | return $attachment->upload($file); |
@@ -135,7 +135,7 @@ discard block |
||
135 | 135 | * @param string $text |
136 | 136 | * @return Story |
137 | 137 | */ |
138 | - public function addComment (string $text) { |
|
138 | + public function addComment(string $text) { |
|
139 | 139 | return $this->newComment()->setText($text)->create(); |
140 | 140 | } |
141 | 141 | |
@@ -146,7 +146,7 @@ discard block |
||
146 | 146 | * @param Task[] $tasks |
147 | 147 | * @return $this |
148 | 148 | */ |
149 | - public function addDependencies (array $tasks) { |
|
149 | + public function addDependencies(array $tasks) { |
|
150 | 150 | $this->api->post("{$this}/addDependencies", ['dependents' => static::_getGids($tasks)]); |
151 | 151 | return $this; |
152 | 152 | } |
@@ -158,7 +158,7 @@ discard block |
||
158 | 158 | * @param Task $task |
159 | 159 | * @return $this |
160 | 160 | */ |
161 | - public function addDependency (Task $task) { |
|
161 | + public function addDependency(Task $task) { |
|
162 | 162 | return $this->addDependencies([$task]); |
163 | 163 | } |
164 | 164 | |
@@ -169,7 +169,7 @@ discard block |
||
169 | 169 | * @param Task $task |
170 | 170 | * @return $this |
171 | 171 | */ |
172 | - public function addDependent (Task $task) { |
|
172 | + public function addDependent(Task $task) { |
|
173 | 173 | return $this->addDependents([$task]); |
174 | 174 | } |
175 | 175 | |
@@ -180,7 +180,7 @@ discard block |
||
180 | 180 | * @param Task[] $tasks |
181 | 181 | * @return $this |
182 | 182 | */ |
183 | - public function addDependents (array $tasks) { |
|
183 | + public function addDependents(array $tasks) { |
|
184 | 184 | $this->api->post("{$this}/addDependents", ['dependents' => static::_getGids($tasks)]); |
185 | 185 | return $this; |
186 | 186 | } |
@@ -191,7 +191,7 @@ discard block |
||
191 | 191 | * @param User $user |
192 | 192 | * @return $this |
193 | 193 | */ |
194 | - public function addFollower (User $user) { |
|
194 | + public function addFollower(User $user) { |
|
195 | 195 | return $this->addFollowers([$user]); |
196 | 196 | } |
197 | 197 | |
@@ -201,7 +201,7 @@ discard block |
||
201 | 201 | * @param User[] $users |
202 | 202 | * @return $this |
203 | 203 | */ |
204 | - public function addFollowers (array $users) { |
|
204 | + public function addFollowers(array $users) { |
|
205 | 205 | if ($this->hasGid()) { |
206 | 206 | $this->api->post("{$this}/addFollowers", ['followers' => static::_getGids($users)]); |
207 | 207 | $this->_merge('followers', $users); |
@@ -218,7 +218,7 @@ discard block |
||
218 | 218 | * @param Tag $tag |
219 | 219 | * @return $this |
220 | 220 | */ |
221 | - public function addTag (Tag $tag) { |
|
221 | + public function addTag(Tag $tag) { |
|
222 | 222 | if ($this->hasGid()) { |
223 | 223 | $this->api->post("{$this}/addTag", ['tag' => $tag->getGid()]); |
224 | 224 | $this->_merge('tags', [$tag]); |
@@ -235,7 +235,7 @@ discard block |
||
235 | 235 | * @param Section $section |
236 | 236 | * @return $this |
237 | 237 | */ |
238 | - public function addToProject (Section $section) { |
|
238 | + public function addToProject(Section $section) { |
|
239 | 239 | $project = $section->getProject(); |
240 | 240 | if ($this->hasGid()) { |
241 | 241 | $this->api->post("{$this}/addProject", [ |
@@ -269,7 +269,7 @@ discard block |
||
269 | 269 | * @param string $target |
270 | 270 | * @return TaskWebhook |
271 | 271 | */ |
272 | - public function addWebhook (string $target) { |
|
272 | + public function addWebhook(string $target) { |
|
273 | 273 | /** @var TaskWebhook $webhook */ |
274 | 274 | $webhook = $this->factory(TaskWebhook::class); |
275 | 275 | return $webhook->create($this, $target); |
@@ -278,7 +278,7 @@ discard block |
||
278 | 278 | /** |
279 | 279 | * @return $this |
280 | 280 | */ |
281 | - public function create () { |
|
281 | + public function create() { |
|
282 | 282 | $this->_create(); |
283 | 283 | $this->_clearSubDiffs(); |
284 | 284 | return $this; |
@@ -294,7 +294,7 @@ discard block |
||
294 | 294 | * @param string[] $include |
295 | 295 | * @return Job |
296 | 296 | */ |
297 | - public function duplicate (string $name, array $include) { |
|
297 | + public function duplicate(string $name, array $include) { |
|
298 | 298 | $remote = $this->api->post("{$this}/duplicate", [ |
299 | 299 | 'name' => $name, |
300 | 300 | 'include' => array_values($include) |
@@ -308,7 +308,7 @@ discard block |
||
308 | 308 | * @depends after-create |
309 | 309 | * @return Attachment[] |
310 | 310 | */ |
311 | - public function getAttachments () { |
|
311 | + public function getAttachments() { |
|
312 | 312 | return $this->loadAll(Attachment::class, "{$this}/attachments"); |
313 | 313 | } |
314 | 314 | |
@@ -318,7 +318,7 @@ discard block |
||
318 | 318 | * @depends after-create |
319 | 319 | * @return Story[] |
320 | 320 | */ |
321 | - public function getComments () { |
|
321 | + public function getComments() { |
|
322 | 322 | return array_values(array_filter($this->getStories(), function(Story $story) { |
323 | 323 | return $story->isComment(); |
324 | 324 | })); |
@@ -330,7 +330,7 @@ discard block |
||
330 | 330 | * @depends after-create |
331 | 331 | * @return Task[] |
332 | 332 | */ |
333 | - public function getDependencies () { |
|
333 | + public function getDependencies() { |
|
334 | 334 | return $this->loadAll(self::class, "{$this}/dependencies"); |
335 | 335 | } |
336 | 336 | |
@@ -340,7 +340,7 @@ discard block |
||
340 | 340 | * @depends after-create |
341 | 341 | * @return Task[] |
342 | 342 | */ |
343 | - public function getDependents () { |
|
343 | + public function getDependents() { |
|
344 | 344 | return $this->loadAll(self::class, "{$this}/dependents"); |
345 | 345 | } |
346 | 346 | |
@@ -351,21 +351,21 @@ discard block |
||
351 | 351 | * @param null|string $token |
352 | 352 | * @return TaskEvent[]|StoryEvent[] |
353 | 353 | */ |
354 | - public function getEvents (&$token) { |
|
354 | + public function getEvents(&$token) { |
|
355 | 355 | return $this->api->sync($this, $token); |
356 | 356 | } |
357 | 357 | |
358 | 358 | /** |
359 | 359 | * @return External |
360 | 360 | */ |
361 | - public function getExternal () { |
|
361 | + public function getExternal() { |
|
362 | 362 | return $this->_get('external') ?? $this->data['external'] = $this->factory(static::$map['external']); |
363 | 363 | } |
364 | 364 | |
365 | 365 | /** |
366 | 366 | * @return Project[] |
367 | 367 | */ |
368 | - public function getProjects () { |
|
368 | + public function getProjects() { |
|
369 | 369 | return array_map(function(Membership $membership) { |
370 | 370 | return $membership->getProject(); |
371 | 371 | }, $this->getMemberships()); |
@@ -377,7 +377,7 @@ discard block |
||
377 | 377 | * @depends after-create |
378 | 378 | * @return Story[] |
379 | 379 | */ |
380 | - public function getStories () { |
|
380 | + public function getStories() { |
|
381 | 381 | return $this->loadAll(Story::class, "{$this}/stories"); |
382 | 382 | } |
383 | 383 | |
@@ -387,7 +387,7 @@ discard block |
||
387 | 387 | * @depends after-create |
388 | 388 | * @return Task[] |
389 | 389 | */ |
390 | - public function getSubTasks () { |
|
390 | + public function getSubTasks() { |
|
391 | 391 | return $this->loadAll(self::class, "{$this}/subtasks"); |
392 | 392 | } |
393 | 393 | |
@@ -397,7 +397,7 @@ discard block |
||
397 | 397 | * @depends after-create |
398 | 398 | * @return string |
399 | 399 | */ |
400 | - public function getUrl (): string { |
|
400 | + public function getUrl(): string { |
|
401 | 401 | return "https://app.asana.com/0/0/{$this->getGid()}"; |
402 | 402 | } |
403 | 403 | |
@@ -407,7 +407,7 @@ discard block |
||
407 | 407 | * @depends after-create |
408 | 408 | * @return TaskWebhook[] |
409 | 409 | */ |
410 | - public function getWebhooks () { |
|
410 | + public function getWebhooks() { |
|
411 | 411 | return $this->loadAll(TaskWebhook::class, 'webhooks', [ |
412 | 412 | 'workspace' => $this->getWorkspace()->getGid(), |
413 | 413 | 'resource' => $this->getGid() |
@@ -417,7 +417,7 @@ discard block |
||
417 | 417 | /** |
418 | 418 | * @return bool |
419 | 419 | */ |
420 | - public function isRenderedAsSeparator (): bool { |
|
420 | + public function isRenderedAsSeparator(): bool { |
|
421 | 421 | return $this->_is('is_rendered_as_separator'); |
422 | 422 | } |
423 | 423 | |
@@ -427,7 +427,7 @@ discard block |
||
427 | 427 | * @depends after-create |
428 | 428 | * @return Story |
429 | 429 | */ |
430 | - public function newComment () { |
|
430 | + public function newComment() { |
|
431 | 431 | /** @var Story $comment */ |
432 | 432 | $comment = $this->factory(Story::class, [ |
433 | 433 | 'resource_subtype' => Story::TYPE_COMMENT_ADDED |
@@ -441,7 +441,7 @@ discard block |
||
441 | 441 | * @depends after-create |
442 | 442 | * @return Task |
443 | 443 | */ |
444 | - public function newSubTask () { |
|
444 | + public function newSubTask() { |
|
445 | 445 | /** @var Task $sub */ |
446 | 446 | $sub = $this->factory(self::class); |
447 | 447 | return $sub->setParent($this); |
@@ -454,7 +454,7 @@ discard block |
||
454 | 454 | * @param Task[] $tasks |
455 | 455 | * @return $this |
456 | 456 | */ |
457 | - public function removeDependencies (array $tasks) { |
|
457 | + public function removeDependencies(array $tasks) { |
|
458 | 458 | $this->api->post("{$this}/removeDependencies", ['dependencies' => static::_getGids($tasks)]); |
459 | 459 | return $this; |
460 | 460 | } |
@@ -466,7 +466,7 @@ discard block |
||
466 | 466 | * @param Task $task |
467 | 467 | * @return $this |
468 | 468 | */ |
469 | - public function removeDependency (Task $task) { |
|
469 | + public function removeDependency(Task $task) { |
|
470 | 470 | return $this->removeDependencies([$task]); |
471 | 471 | } |
472 | 472 | |
@@ -477,7 +477,7 @@ discard block |
||
477 | 477 | * @param Task $task |
478 | 478 | * @return $this |
479 | 479 | */ |
480 | - public function removeDependent (Task $task) { |
|
480 | + public function removeDependent(Task $task) { |
|
481 | 481 | return $this->removeDependents([$task]); |
482 | 482 | } |
483 | 483 | |
@@ -488,7 +488,7 @@ discard block |
||
488 | 488 | * @param Task[] $tasks |
489 | 489 | * @return $this |
490 | 490 | */ |
491 | - public function removeDependents (array $tasks) { |
|
491 | + public function removeDependents(array $tasks) { |
|
492 | 492 | $this->api->post("{$this}/removeDependents", ['dependents' => static::_getGids($tasks)]); |
493 | 493 | return $this; |
494 | 494 | } |
@@ -499,7 +499,7 @@ discard block |
||
499 | 499 | * @param User $user |
500 | 500 | * @return $this |
501 | 501 | */ |
502 | - public function removeFollower (User $user) { |
|
502 | + public function removeFollower(User $user) { |
|
503 | 503 | return $this->removeFollowers([$user]); |
504 | 504 | } |
505 | 505 | |
@@ -509,7 +509,7 @@ discard block |
||
509 | 509 | * @param User[] $users |
510 | 510 | * @return $this |
511 | 511 | */ |
512 | - public function removeFollowers (array $users) { |
|
512 | + public function removeFollowers(array $users) { |
|
513 | 513 | if ($this->hasGid()) { |
514 | 514 | $this->api->post("{$this}/removeFollowers", ['followers' => static::_getGids($users)]); |
515 | 515 | } |
@@ -523,7 +523,7 @@ discard block |
||
523 | 523 | * @param Project $project |
524 | 524 | * @return $this |
525 | 525 | */ |
526 | - public function removeFromProject (Project $project) { |
|
526 | + public function removeFromProject(Project $project) { |
|
527 | 527 | $gid = $project->getGid(); |
528 | 528 | if ($this->hasGid()) { |
529 | 529 | $this->api->post("{$this}/removeProject", ['project' => $gid]); |
@@ -542,7 +542,7 @@ discard block |
||
542 | 542 | * @param Tag $tag |
543 | 543 | * @return $this |
544 | 544 | */ |
545 | - public function removeTag (Tag $tag) { |
|
545 | + public function removeTag(Tag $tag) { |
|
546 | 546 | if ($this->hasGid()) { |
547 | 547 | $this->api->post("{$this}/removeTag", ['tag' => $tag->getGid()]); |
548 | 548 | } |
@@ -556,7 +556,7 @@ discard block |
||
556 | 556 | * @param null|Task $parent |
557 | 557 | * @return $this |
558 | 558 | */ |
559 | - public function setParent (?self $parent) { |
|
559 | + public function setParent(?self $parent) { |
|
560 | 560 | if ($this->hasGid()) { |
561 | 561 | $this->api->post("{$this}/setParent", ['parent' => $parent ? $parent->getGid() : null]); |
562 | 562 | $this->data['parent'] = $parent; |
@@ -571,14 +571,14 @@ discard block |
||
571 | 571 | * @param bool $flag |
572 | 572 | * @return $this |
573 | 573 | */ |
574 | - public function setRenderedAsSeparator (bool $flag) { |
|
574 | + public function setRenderedAsSeparator(bool $flag) { |
|
575 | 575 | return $this->_set('is_rendered_as_separator', $flag); |
576 | 576 | } |
577 | 577 | |
578 | 578 | /** |
579 | 579 | * @return $this |
580 | 580 | */ |
581 | - public function update () { |
|
581 | + public function update() { |
|
582 | 582 | $this->_update(); |
583 | 583 | $this->_clearSubDiffs(); |
584 | 584 | return $this; |
@@ -17,7 +17,7 @@ |
||
17 | 17 | * @param Workspace $workspace |
18 | 18 | * @return $this |
19 | 19 | */ |
20 | - public function setWorkspace (Workspace $workspace) { |
|
20 | + public function setWorkspace(Workspace $workspace) { |
|
21 | 21 | assert(!$this->hasGid()); |
22 | 22 | return $this->_set('workspace', $workspace); |
23 | 23 | } |
@@ -291,11 +291,9 @@ |
||
291 | 291 | $dehydrate = function($each) use (&$dehydrate) { |
292 | 292 | if ($each instanceof AbstractEntity and $each->hasGid()) { |
293 | 293 | return $each->getGid(); |
294 | - } |
|
295 | - elseif ($each instanceof self) { |
|
294 | + } elseif ($each instanceof self) { |
|
296 | 295 | return $each->toArray(); |
297 | - } |
|
298 | - elseif (is_array($each)) { |
|
296 | + } elseif (is_array($each)) { |
|
299 | 297 | return array_map($dehydrate, $each); |
300 | 298 | } |
301 | 299 | return $each; |
@@ -40,7 +40,7 @@ discard block |
||
40 | 40 | * @param Api|Data $caller |
41 | 41 | * @param array $data |
42 | 42 | */ |
43 | - public function __construct ($caller, array $data = []) { |
|
43 | + public function __construct($caller, array $data = []) { |
|
44 | 44 | $this->api = $caller instanceof self ? $caller->api : $caller; |
45 | 45 | $this->_setData($data); |
46 | 46 | } |
@@ -56,9 +56,9 @@ discard block |
||
56 | 56 | * @param array $args |
57 | 57 | * @return mixed |
58 | 58 | */ |
59 | - public function __call (string $method, array $args) { |
|
59 | + public function __call(string $method, array $args) { |
|
60 | 60 | static $cache = []; |
61 | - if (!$call =& $cache[$method]) { |
|
61 | + if (!$call = & $cache[$method]) { |
|
62 | 62 | preg_match('/^(get|has|is|set)(.+)$/', $method, $call); |
63 | 63 | $call[1] = '_' . $call[1]; |
64 | 64 | $call[2] = preg_replace_callback('/[A-Z]/', function(array $match) { |
@@ -68,7 +68,7 @@ discard block |
||
68 | 68 | return $this->{$call[1]}($call[2], ...$args); |
69 | 69 | } |
70 | 70 | |
71 | - public function __debugInfo (): array { |
|
71 | + public function __debugInfo(): array { |
|
72 | 72 | return $this->data; |
73 | 73 | } |
74 | 74 | |
@@ -78,7 +78,7 @@ discard block |
||
78 | 78 | * @param $key |
79 | 79 | * @return null|Data|mixed |
80 | 80 | */ |
81 | - final public function __get ($key) { |
|
81 | + final public function __get($key) { |
|
82 | 82 | return $this->_get($key); |
83 | 83 | } |
84 | 84 | |
@@ -90,7 +90,7 @@ discard block |
||
90 | 90 | * @param $key |
91 | 91 | * @return bool |
92 | 92 | */ |
93 | - final public function __isset ($key) { |
|
93 | + final public function __isset($key) { |
|
94 | 94 | return true; |
95 | 95 | } |
96 | 96 | |
@@ -102,7 +102,7 @@ discard block |
||
102 | 102 | * @param string $key |
103 | 103 | * @return mixed |
104 | 104 | */ |
105 | - protected function _get (string $key) { |
|
105 | + protected function _get(string $key) { |
|
106 | 106 | return $this->data[$key] ?? null; |
107 | 107 | } |
108 | 108 | |
@@ -116,7 +116,7 @@ discard block |
||
116 | 116 | * @param string $key |
117 | 117 | * @return bool |
118 | 118 | */ |
119 | - protected function _has (string $key): bool { |
|
119 | + protected function _has(string $key): bool { |
|
120 | 120 | $value = $this->_get($key); |
121 | 121 | if (isset($value)) { |
122 | 122 | if (is_countable($value)) { |
@@ -139,7 +139,7 @@ discard block |
||
139 | 139 | * @param string $key |
140 | 140 | * @return bool |
141 | 141 | */ |
142 | - protected function _is (string $key): bool { |
|
142 | + protected function _is(string $key): bool { |
|
143 | 143 | return !empty($this->_get($key)); |
144 | 144 | } |
145 | 145 | |
@@ -152,7 +152,7 @@ discard block |
||
152 | 152 | * @param mixed $value |
153 | 153 | * @return $this |
154 | 154 | */ |
155 | - protected function _set (string $key, $value) { |
|
155 | + protected function _set(string $key, $value) { |
|
156 | 156 | $this->data[$key] = $value; |
157 | 157 | $this->diff[$key] = true; |
158 | 158 | return $this; |
@@ -163,7 +163,7 @@ discard block |
||
163 | 163 | * |
164 | 164 | * @param array $data |
165 | 165 | */ |
166 | - protected function _setData (array $data): void { |
|
166 | + protected function _setData(array $data): void { |
|
167 | 167 | $this->data = $this->diff = []; |
168 | 168 | foreach ($data as $key => $value) { |
169 | 169 | $this->_setMapped($key, $value); |
@@ -176,7 +176,7 @@ discard block |
||
176 | 176 | * @param string $key |
177 | 177 | * @param null|Data|array $value |
178 | 178 | */ |
179 | - protected function _setMapped (string $key, $value): void { |
|
179 | + protected function _setMapped(string $key, $value): void { |
|
180 | 180 | unset($this->diff[$key]); |
181 | 181 | |
182 | 182 | // use unmapped values, null, [], Data, and Data[] as-is |
@@ -216,7 +216,7 @@ discard block |
||
216 | 216 | * @param array $data |
217 | 217 | * @return mixed |
218 | 218 | */ |
219 | - final protected function factory (string $class, array $data = []) { |
|
219 | + final protected function factory(string $class, array $data = []) { |
|
220 | 220 | return $this->api->factory($class, $this, $data); |
221 | 221 | } |
222 | 222 | |
@@ -225,7 +225,7 @@ discard block |
||
225 | 225 | * |
226 | 226 | * @return array |
227 | 227 | */ |
228 | - public function getDiff (): array { |
|
228 | + public function getDiff(): array { |
|
229 | 229 | $convert = function($each) use (&$convert) { |
230 | 230 | // convert existing entities to gids |
231 | 231 | if ($each instanceof AbstractEntity and $each->hasGid()) { |
@@ -250,7 +250,7 @@ discard block |
||
250 | 250 | * |
251 | 251 | * @return bool |
252 | 252 | */ |
253 | - final public function isDiff (): bool { |
|
253 | + final public function isDiff(): bool { |
|
254 | 254 | return (bool)$this->diff; |
255 | 255 | } |
256 | 256 | |
@@ -258,7 +258,7 @@ discard block |
||
258 | 258 | * @see toArray() |
259 | 259 | * @return array |
260 | 260 | */ |
261 | - public function jsonSerialize (): array { |
|
261 | + public function jsonSerialize(): array { |
|
262 | 262 | return $this->toArray(); |
263 | 263 | } |
264 | 264 | |
@@ -272,7 +272,7 @@ discard block |
||
272 | 272 | * @param array $query |
273 | 273 | * @return null|mixed|AbstractEntity |
274 | 274 | */ |
275 | - final protected function load (string $class, string $path, array $query = []) { |
|
275 | + final protected function load(string $class, string $path, array $query = []) { |
|
276 | 276 | return $this->api->load($class, $this, $path, $query); |
277 | 277 | } |
278 | 278 | |
@@ -287,7 +287,7 @@ discard block |
||
287 | 287 | * @param int $pages |
288 | 288 | * @return array|AbstractEntity[] |
289 | 289 | */ |
290 | - final protected function loadAll (string $class, string $path, array $query = [], int $pages = 0) { |
|
290 | + final protected function loadAll(string $class, string $path, array $query = [], int $pages = 0) { |
|
291 | 291 | return $this->api->loadAll($class, $this, $path, $query, $pages); |
292 | 292 | } |
293 | 293 | |
@@ -296,7 +296,7 @@ discard block |
||
296 | 296 | * |
297 | 297 | * @return string |
298 | 298 | */ |
299 | - public function serialize (): string { |
|
299 | + public function serialize(): string { |
|
300 | 300 | return serialize($this->toArray()); |
301 | 301 | } |
302 | 302 | |
@@ -305,7 +305,7 @@ discard block |
||
305 | 305 | * |
306 | 306 | * @return array |
307 | 307 | */ |
308 | - public function toArray (): array { |
|
308 | + public function toArray(): array { |
|
309 | 309 | if (!$this->api) { |
310 | 310 | return $this->data; |
311 | 311 | } |
@@ -329,7 +329,7 @@ discard block |
||
329 | 329 | * |
330 | 330 | * @param $serialized |
331 | 331 | */ |
332 | - public function unserialize ($serialized): void { |
|
332 | + public function unserialize($serialized): void { |
|
333 | 333 | $this->api = Api::getDefault(); |
334 | 334 | $this->data = unserialize($serialized); |
335 | 335 | } |
@@ -73,8 +73,7 @@ |
||
73 | 73 | if ($this->hasGid()) { |
74 | 74 | $this->api->post("{$this}/addMembers", ['members' => static::_getGids($users)]); |
75 | 75 | $this->_merge('members', $users); |
76 | - } |
|
77 | - else { |
|
76 | + } else { |
|
78 | 77 | $this->_merge('members', $users, true); |
79 | 78 | } |
80 | 79 | return $this; |
@@ -39,11 +39,11 @@ discard block |
||
39 | 39 | 'workspace' => Workspace::class |
40 | 40 | ]; |
41 | 41 | |
42 | - final public function __toString (): string { |
|
42 | + final public function __toString(): string { |
|
43 | 43 | return "portfolios/{$this->getGid()}"; |
44 | 44 | } |
45 | 45 | |
46 | - final protected function _getDir (): string { |
|
46 | + final protected function _getDir(): string { |
|
47 | 47 | return 'portfolios'; |
48 | 48 | } |
49 | 49 | |
@@ -52,7 +52,7 @@ discard block |
||
52 | 52 | * @param Project $item |
53 | 53 | * @return $this |
54 | 54 | */ |
55 | - public function addItem (Project $item) { |
|
55 | + public function addItem(Project $item) { |
|
56 | 56 | $this->api->post("{$this}/addItem", ['item' => $item->getGid()]); |
57 | 57 | return $this; |
58 | 58 | } |
@@ -61,7 +61,7 @@ discard block |
||
61 | 61 | * @param User $user |
62 | 62 | * @return $this |
63 | 63 | */ |
64 | - public function addMember (User $user) { |
|
64 | + public function addMember(User $user) { |
|
65 | 65 | return $this->addMembers([$user]); |
66 | 66 | } |
67 | 67 | |
@@ -69,7 +69,7 @@ discard block |
||
69 | 69 | * @param User[] $users |
70 | 70 | * @return $this |
71 | 71 | */ |
72 | - public function addMembers (array $users) { |
|
72 | + public function addMembers(array $users) { |
|
73 | 73 | if ($this->hasGid()) { |
74 | 74 | $this->api->post("{$this}/addMembers", ['members' => static::_getGids($users)]); |
75 | 75 | $this->_merge('members', $users); |
@@ -84,7 +84,7 @@ discard block |
||
84 | 84 | * @depends after-create |
85 | 85 | * @return Project[] |
86 | 86 | */ |
87 | - public function getItems () { |
|
87 | + public function getItems() { |
|
88 | 88 | return $this->loadAll(Project::class, "{$this}/items"); |
89 | 89 | } |
90 | 90 | |
@@ -93,7 +93,7 @@ discard block |
||
93 | 93 | * @param Project $item |
94 | 94 | * @return $this |
95 | 95 | */ |
96 | - public function removeItem (Project $item) { |
|
96 | + public function removeItem(Project $item) { |
|
97 | 97 | $this->api->post("{$this}/removeItem", ['item' => $item->getGid()]); |
98 | 98 | return $this; |
99 | 99 | } |
@@ -102,7 +102,7 @@ discard block |
||
102 | 102 | * @param User $user |
103 | 103 | * @return $this |
104 | 104 | */ |
105 | - public function removeMember (User $user) { |
|
105 | + public function removeMember(User $user) { |
|
106 | 106 | return $this->removeMembers([$user]); |
107 | 107 | } |
108 | 108 | |
@@ -110,7 +110,7 @@ discard block |
||
110 | 110 | * @param User[] $users |
111 | 111 | * @return $this |
112 | 112 | */ |
113 | - public function removeMembers (array $users) { |
|
113 | + public function removeMembers(array $users) { |
|
114 | 114 | if ($this->hasGid()) { |
115 | 115 | $this->api->post("{$this}/removeMembers", ['members' => static::_getGids($users)]); |
116 | 116 | } |