hfw / asana

src/Color.php

Braces +1 added lines, -2 removed lines

@@ -39,8 +39,7 @@
         try {
             $colors = (new ReflectionClass(self::class))->getConstants();
             return $colors[array_rand($colors)];
-        }
-        catch (Exception $exception) {
+        } catch (Exception $exception) {
             return 'none'; // unreachable
         }
     }

Spacing +1 added lines, -1 removed lines

@@ -35,7 +35,7 @@
     /**
      * @return string
      */
-    public static function random (): string {
+    public static function random(): string {
         try {
             $colors = (new ReflectionClass(self::class))->getConstants();
             return $colors[array_rand($colors)];

src/Api/Laravel/AsanaServiceProvider.php

Spacing +3 added lines, -3 removed lines

@@ -17,7 +17,7 @@ 
 
     const NAME = 'asana';
 
-    public function boot () {
+    public function boot() {
         $this->publishes([
             __DIR__ . '/config/asana.php' => $this->app->configPath('asana.php')
         ]);
@@ -30,11 +30,11 @@ 
         }
     }
 
-    public function provides () {
+    public function provides() {
         return [self::NAME];
     }
 
-    public function register () {
+    public function register() {
         $this->app->singleton(self::NAME, function(Application $app) {
             $config = $app['config'][self::NAME];
             $pool = null;

src/Api/AsanaError.php

Spacing +2 added lines, -2 removed lines

@@ -20,7 +20,7 @@ 
      */
     protected $curlInfo;
 
-    public function __construct (int $code, string $message, array $curlInfo) {
+    public function __construct(int $code, string $message, array $curlInfo) {
         parent::__construct($message, $code);
         $this->curlInfo = $curlInfo;
     }
@@ -28,7 +28,7 @@ 
     /**
      * @return array
      */
-    public function getCurlInfo (): array {
+    public function getCurlInfo(): array {
         return $this->curlInfo;
     }
 }
\ No newline at end of file

src/Api/FileCache.php

Braces +2 added lines, -4 removed lines

@@ -58,8 +58,7 @@ 
             $this->_log("CACHE DELINK {$key}");
             unlink($ref);
             unlink($path);
-        }
-        elseif (is_file($path)) {
+        } elseif (is_file($path)) {
             $this->_log("CACHE DELETE {$key}");
             unlink($path);
         }
@@ -117,8 +116,7 @@ 
                 ["CACHE SET {$key}", "CACHE BURN {$key}"][$value instanceof ImmutableInterface],
                 "CACHE UPDATE {$key}"
             ][is_file($path)]);
-        }
-        else {
+        } else {
             $this->_log([
                 "CACHE LINK {$key} => {$value}",
                 "CACHE RENEW LINK {$key} => {$value}"

Spacing +13 added lines, -13 removed lines

@@ -29,30 +29,30 @@ 
     /**
      * @param string $dir
      */
-    public function __construct (string $dir) {
+    public function __construct(string $dir) {
         $this->dir = $dir;
         $this->log = new NullLogger();
     }
 
-    private function _log (string $msg): void {
+    private function _log(string $msg): void {
         $this->log->log(LOG_DEBUG, $msg);
     }
 
-    private function _path ($key): string {
+    private function _path($key): string {
         $path = "{$this->dir}/{$key}~";
         clearstatcache(true, $path);
         return $path;
     }
 
-    private function _ref ($key): string {
+    private function _ref($key): string {
         return "{$this->dir}/{$key}.ref";
     }
 
-    public function clear () {
+    public function clear() {
         // unused. just delete the dir.
     }
 
-    public function delete ($key) {
+    public function delete($key) {
         $path = $this->_path($key);
         if (is_link($ref = $this->_ref($key))) {
             $this->_log("CACHE DELINK {$key}");
@@ -65,7 +65,7 @@ 
         }
     }
 
-    public function deleteMultiple ($keys) {
+    public function deleteMultiple($keys) {
         // unused
     }
 
@@ -74,7 +74,7 @@ 
      * @param mixed $default Unused.
      * @return null|string|object
      */
-    public function get ($key, $default = null) {
+    public function get($key, $default = null) {
         $path = $this->_path($key);
         if (!is_file($path)) {
             $this->_log("CACHE MISS {$key}");
@@ -93,11 +93,11 @@ 
         return $data;
     }
 
-    public function getMultiple ($keys, $default = null) {
+    public function getMultiple($keys, $default = null) {
         // unused
     }
 
-    public function has ($key): bool {
+    public function has($key): bool {
         return is_file($this->_path($key));
     }
 
@@ -107,7 +107,7 @@ 
      * @param int $ttl
      * @return void
      */
-    public function set ($key, $value, $ttl = null): void {
+    public function set($key, $value, $ttl = null): void {
         $path = $this->_path($key);
         if (!is_dir(dirname($path))) {
             mkdir(dirname($path), 0770, true);
@@ -136,12 +136,12 @@ 
      * @param LoggerInterface $log
      * @return $this
      */
-    public function setLog (LoggerInterface $log) {
+    public function setLog(LoggerInterface $log) {
         $this->log = $log;
         return $this;
     }
 
-    public function setMultiple ($values, $ttl = null) {
+    public function setMultiple($values, $ttl = null) {
         // unused
     }
 }
\ No newline at end of file

src/Api/Pool.php

Spacing +7 added lines, -7 removed lines

@@ -29,7 +29,7 @@ 
     /**
      * @param AbstractEntity $entity
      */
-    protected function _add (AbstractEntity $entity): void {
+    protected function _add(AbstractEntity $entity): void {
         $gid = $entity->getGid();
         $this->entities[$gid] = $entity;
         $this->gids[$gid] = $gid;
@@ -39,7 +39,7 @@ 
      * @param AbstractEntity $entity
      * @param string[] $keys
      */
-    protected function _addKeys (AbstractEntity $entity, ...$keys): void {
+    protected function _addKeys(AbstractEntity $entity, ...$keys): void {
         $this->gids += array_fill_keys($keys, $entity->getGid());
     }
 
@@ -48,7 +48,7 @@ 
      * @param Api|Data $caller For hydration if needed.
      * @return null|AbstractEntity
      */
-    protected function _get (string $key, $caller) {
+    protected function _get(string $key, $caller) {
         if (isset($this->gids[$key])) {
             return $this->entities[$this->gids[$key]];
         }
@@ -62,14 +62,14 @@ 
      * @param string $key
      * @return bool
      */
-    protected function _has (string $key): bool {
+    protected function _has(string $key): bool {
         return isset($this->gids[$key]);
     }
 
     /**
      * @param AbstractEntity $entity
      */
-    final public function add (AbstractEntity $entity): void {
+    final public function add(AbstractEntity $entity): void {
         if (!$entity->isDiff()) {
             $this->_add($entity);
             $this->_addKeys($entity, ...$entity->getPoolKeys());
@@ -82,7 +82,7 @@ 
      * @param Closure $factory `fn( Api|Data $caller ): null|AbstractEntity`
      * @return null|mixed|AbstractEntity
      */
-    final public function get (string $key, $caller, Closure $factory) {
+    final public function get(string $key, $caller, Closure $factory) {
         /** @var AbstractEntity $entity */
         if (!$entity = $this->_get($key, $caller) and $entity = $factory($caller)) {
             $gid = $entity->getGid();
@@ -103,7 +103,7 @@ 
     /**
      * @param string[] $keys
      */
-    public function remove (array $keys): void {
+    public function remove(array $keys): void {
         foreach ($keys as $key) {
             unset($this->entities[$key]);
             unset($this->gids[$key]);

src/Base/AbstractEntity/DeleteTrait.php

Spacing +1 added lines, -1 removed lines

@@ -14,7 +14,7 @@
     /**
      * `DELETE`
      */
-    public function delete (): void {
+    public function delete(): void {
         $this->api->delete($this);
         $this->api->getPool()->remove($this->getPoolKeys());
     }

src/Base/Data.php

Braces +2 added lines, -4 removed lines

@@ -49,8 +49,7 @@ 
     public function __construct ($caller, array $data = []) {
         if ($caller instanceof self) {
             $this->api = $caller->api;
-        }
-        else {
+        } else {
             $this->api = $caller;
         }
         $this->_setData($data);
@@ -242,8 +241,7 @@ 
                 $value = array_map(function($each) use ($class) {
                     return $this->_hydrate($class[0], $each);
                 }, $value);
-            }
-            elseif (isset($value)) {
+            } elseif (isset($value)) {
                 $value = $this->_hydrate($class, $value);
             }
         }

Spacing +20 added lines, -20 removed lines

@@ -46,7 +46,7 @@ 
      * @param Api|Data $caller
      * @param array $data
      */
-    public function __construct ($caller, array $data = []) {
+    public function __construct($caller, array $data = []) {
         if ($caller instanceof self) {
             $this->api = $caller->api;
         }
@@ -69,9 +69,9 @@ 
      * @param array $args
      * @return mixed
      */
-    public function __call (string $method, array $args) {
+    public function __call(string $method, array $args) {
         static $magic = [];
-        if (!$call =& $magic[$method]) {
+        if (!$call = & $magic[$method]) {
             preg_match('/^(get|has|is|select|set)(.+)$/', $method, $call);
             if ('_select' !== $call[1] = '_' . $call[1]) { // _select() calls getters
                 $call[2] = preg_replace_callback('/[A-Z]/', function(array $match) {
@@ -86,7 +86,7 @@ 
      * @return array
      * @internal pool, `var_export()`
      */
-    final public function __debugInfo (): array {
+    final public function __debugInfo(): array {
         return $this->data;
     }
 
@@ -95,7 +95,7 @@ 
      * @return null|Data|mixed
      * @internal for `array_column()`
      */
-    final public function __get ($field) {
+    final public function __get($field) {
         return $this->_get($field);
     }
 
@@ -104,7 +104,7 @@ 
      * @return bool
      * @internal for `array_column()`
      */
-    final public function __isset ($field) {
+    final public function __isset($field) {
         return true; // fields may be lazy-loaded or coalesced to null.
     }
 
@@ -116,7 +116,7 @@ 
      * @param string $field
      * @return mixed
      */
-    protected function _get (string $field) {
+    protected function _get(string $field) {
         return $this->data[$field] ?? null;
     }
 
@@ -131,7 +131,7 @@ 
      * @param string $field
      * @return bool
      */
-    protected function _has (string $field): bool {
+    protected function _has(string $field): bool {
         $value = $this->_get($field);
         if (isset($value)) {
             if (is_countable($value)) {
@@ -149,7 +149,7 @@ 
      * @param mixed $item
      * @return mixed
      */
-    protected function _hydrate (string $class, $item) {
+    protected function _hydrate(string $class, $item) {
         if (!isset($item) or $item instanceof self) {
             return $item;
         }
@@ -178,7 +178,7 @@ 
      * @param string $field
      * @return bool
      */
-    protected function _is (string $field): bool {
+    protected function _is(string $field): bool {
         return (bool)$this->_get($field);
     }
 
@@ -196,7 +196,7 @@ 
      * @param array $args
      * @return array
      */
-    protected function _select ($subject, callable $filter, ...$args) {
+    protected function _select($subject, callable $filter, ...$args) {
         if (is_string($subject)) {
             $subject = $this->{'get' . $subject}(...$args) ?? [];
         }
@@ -218,7 +218,7 @@ 
      * @param mixed $value
      * @return $this
      */
-    protected function _set (string $field, $value) {
+    protected function _set(string $field, $value) {
         $this->data[$field] = $value;
         $this->diff[$field] = true;
         return $this;
@@ -229,7 +229,7 @@ 
      *
      * @param array $data
      */
-    protected function _setData (array $data): void {
+    protected function _setData(array $data): void {
         $this->data = $this->diff = [];
         foreach ($data as $field => $value) {
             $this->_setField($field, $value);
@@ -242,7 +242,7 @@ 
      * @param string $field
      * @param mixed $value
      */
-    protected function _setField (string $field, $value): void {
+    protected function _setField(string $field, $value): void {
         if (isset(static::MAP[$field])) {
             $class = static::MAP[$field];
             if (is_array($class)) {
@@ -263,14 +263,14 @@ 
      *
      * @return bool
      */
-    final public function isDiff (): bool {
+    final public function isDiff(): bool {
         return (bool)$this->diff;
     }
 
     /**
      * @return array
      */
-    public function jsonSerialize (): array {
+    public function jsonSerialize(): array {
         $data = $this->toArray();
         ksort($data);
         return $data;
@@ -281,8 +281,8 @@ 
      *
      * @return string
      */
-    public function serialize (): string {
-        return json_encode($this, JSON_PRETTY_PRINT | JSON_UNESCAPED_SLASHES | JSON_UNESCAPED_UNICODE);
+    public function serialize(): string {
+        return json_encode($this, JSON_PRETTY_PRINT|JSON_UNESCAPED_SLASHES|JSON_UNESCAPED_UNICODE);
     }
 
     /**
@@ -291,7 +291,7 @@ 
      * @param bool $diff
      * @return array
      */
-    public function toArray (bool $diff = false): array {
+    public function toArray(bool $diff = false): array {
         $dehydrate = function($each) use (&$dehydrate, $diff) {
             // convert entities to gids
             if ($each instanceof AbstractEntity and $each->hasGid()) {
@@ -317,7 +317,7 @@ 
     /**
      * @param $serialized
      */
-    public function unserialize ($serialized): void {
+    public function unserialize($serialized): void {
         $this->data = json_decode($serialized, true);
     }
 }
\ No newline at end of file

src/Api.php

Braces +1 added lines, -2 removed lines

@@ -484,8 +484,7 @@
                     'sync' => $token,
                     'opt_expand' => 'this'
                 ]));
-        }
-        catch (AsanaError $error) {
+        } catch (AsanaError $error) {
             if ($error->getCode() === 412) {
                 $remote = json_decode($error->getMessage(), true, JSON_BIGINT_AS_STRING | JSON_THROW_ON_ERROR);
                 if (!isset($token)) {

Spacing +35 added lines, -35 removed lines

@@ -43,7 +43,7 @@ 
      * @param string $token
      * @param null|Pool $pool
      */
-    public function __construct (string $token, Pool $pool = null) {
+    public function __construct(string $token, Pool $pool = null) {
         $this->token = $token;
         $this->pool = $pool ?? new Pool();
     }
@@ -57,7 +57,7 @@ 
      * @return null|array
      * @internal
      */
-    protected function _exec (string $method, string $path, array $curlOpts = []) {
+    protected function _exec(string $method, string $path, array $curlOpts = []) {
         $log = $this->getLog();
         $log->log(LOG_DEBUG, "{$method} {$path}", $curlOpts);
         /** @var resource $ch */
@@ -88,7 +88,7 @@ 
         switch ($curlInfo['http_code']) {
             case 200:
             case 201:
-                return json_decode($body, true, JSON_BIGINT_AS_STRING | JSON_THROW_ON_ERROR);
+                return json_decode($body, true, JSON_BIGINT_AS_STRING|JSON_THROW_ON_ERROR);
             case 404:
                 return null;
             case 429:
@@ -107,7 +107,7 @@ 
      *
      * @param string $path
      */
-    public function delete (string $path): void {
+    public function delete(string $path): void {
         $this->_exec('DELETE', $path);
     }
 
@@ -121,7 +121,7 @@ 
      * @param array $data
      * @return mixed|Data|AbstractEntity
      */
-    public function factory ($caller, string $class, array $data = []) {
+    public function factory($caller, string $class, array $data = []) {
         return new $class($caller, $data);
     }
 
@@ -133,7 +133,7 @@ 
      * @param array $opt
      * @return null|array
      */
-    public function get (string $path, array $query = [], array $opt = []) {
+    public function get(string $path, array $query = [], array $opt = []) {
         foreach ($opt as $name => $value) {
             $query["opt_{$name}"] = $value;
         }
@@ -147,7 +147,7 @@ 
      * @param string $gid
      * @return null|Attachment
      */
-    public function getAttachment (string $gid) {
+    public function getAttachment(string $gid) {
         return $this->load($this, Attachment::class, "attachments/{$gid}");
     }
 
@@ -157,7 +157,7 @@ 
      * @param string $gid
      * @return null|CustomField
      */
-    public function getCustomField (string $gid) {
+    public function getCustomField(string $gid) {
         return $this->load($this, CustomField::class, "custom_fields/{$gid}");
     }
 
@@ -168,28 +168,28 @@ 
      *
      * @return Workspace
      */
-    public function getDefaultWorkspace () {
+    public function getDefaultWorkspace() {
         return $this->getMe()->getDefaultWorkspace();
     }
 
     /**
      * @return LoggerInterface
      */
-    public function getLog () {
+    public function getLog() {
         return $this->log ?? $this->log = new NullLogger();
     }
 
     /**
      * @return User
      */
-    public function getMe () {
+    public function getMe() {
         return $this->getUser('me');
     }
 
     /**
      * @return Pool
      */
-    public function getPool () {
+    public function getPool() {
         return $this->pool;
     }
 
@@ -199,7 +199,7 @@ 
      * @param string $gid
      * @return null|Portfolio
      */
-    public function getPortfolio (string $gid) {
+    public function getPortfolio(string $gid) {
         return $this->load($this, Portfolio::class, "portfolios/{$gid}");
     }
 
@@ -209,7 +209,7 @@ 
      * @param string $gid
      * @return null|Project
      */
-    public function getProject (string $gid) {
+    public function getProject(string $gid) {
         return $this->load($this, Project::class, "projects/{$gid}");
     }
 
@@ -219,7 +219,7 @@ 
      * @param string $gid
      * @return null|Section
      */
-    public function getSection (string $gid) {
+    public function getSection(string $gid) {
         return $this->load($this, Section::class, "sections/{$gid}");
     }
 
@@ -229,7 +229,7 @@ 
      * @param string $gid
      * @return null|Story
      */
-    public function getStory (string $gid) {
+    public function getStory(string $gid) {
         return $this->load($this, Story::class, "stories/{$gid}");
     }
 
@@ -239,7 +239,7 @@ 
      * @param string $gid
      * @return null|Tag
      */
-    public function getTag (string $gid) {
+    public function getTag(string $gid) {
         return $this->load($this, Tag::class, "tags/{$gid}");
     }
 
@@ -249,7 +249,7 @@ 
      * @param string $gid
      * @return null|Task
      */
-    public function getTask (string $gid) {
+    public function getTask(string $gid) {
         return $this->load($this, Task::class, "tasks/{$gid}");
     }
 
@@ -259,7 +259,7 @@ 
      * @param string $gid
      * @return null|TaskList
      */
-    public function getTaskList (string $gid) {
+    public function getTaskList(string $gid) {
         return $this->load($this, TaskList::class, "user_task_lists/{$gid}");
     }
 
@@ -269,7 +269,7 @@ 
      * @param string $gid
      * @return null|Team
      */
-    public function getTeam (string $gid) {
+    public function getTeam(string $gid) {
         return $this->load($this, Team::class, "teams/{$gid}");
     }
 
@@ -279,7 +279,7 @@ 
      * @param string $gid
      * @return null|User
      */
-    public function getUser (string $gid) {
+    public function getUser(string $gid) {
         return $this->load($this, User::class, "users/{$gid}");
     }
 
@@ -287,7 +287,7 @@ 
      * @param string $gid
      * @return ProjectWebhook|TaskWebhook
      */
-    public function getWebhook (string $gid) {
+    public function getWebhook(string $gid) {
         return $this->pool->get($gid, $this, function() use ($gid) {
             static $classes = [
                 Project::TYPE => ProjectWebhook::class,
@@ -308,7 +308,7 @@ 
      * @param array $data
      * @return Event
      */
-    public function getWebhookEvent (array $data) {
+    public function getWebhookEvent(array $data) {
         return $this->factory($this, Event::class, $data);
     }
 
@@ -318,7 +318,7 @@ 
      * @param string $gid
      * @return null|Workspace
      */
-    public function getWorkspace (string $gid) {
+    public function getWorkspace(string $gid) {
         return $this->load($this, Workspace::class, "workspaces/{$gid}");
     }
 
@@ -331,7 +331,7 @@ 
      * @param array $query
      * @return null|mixed|AbstractEntity
      */
-    public function load ($caller, string $class, string $path, array $query = []) {
+    public function load($caller, string $class, string $path, array $query = []) {
         $key = rtrim($path . '?' . http_build_query($query), '?');
         return $this->pool->get($key, $caller, function($caller) use ($class, $path, $query) {
             if ($data = $this->get($path, $query, ['expand' => 'this'])) {
@@ -350,7 +350,7 @@ 
      * @param array $query
      * @return array|AbstractEntity[]
      */
-    public function loadAll ($caller, string $class, string $path, array $query = []) {
+    public function loadAll($caller, string $class, string $path, array $query = []) {
         return iterator_to_array($this->loadEach(...func_get_args()));
     }
 
@@ -365,7 +365,7 @@ 
      * @param array $query `limit` can exceed `100` here.
      * @return Generator|AbstractEntity[]
      */
-    public function loadEach ($caller, string $class, string $path, array $query = []) {
+    public function loadEach($caller, string $class, string $path, array $query = []) {
         $query['opt_expand'] = 'this';
         $remain = $query['limit'] ?? PHP_INT_MAX;
         do {
@@ -389,13 +389,13 @@ 
      * @param array $opt
      * @return null|array
      */
-    public function post (string $path, array $data = [], array $opt = []) {
+    public function post(string $path, array $data = [], array $opt = []) {
         $response = $this->_exec('POST', $path, [
             CURLOPT_HTTPHEADER => ['Content-Type: application/json'],
             CURLOPT_POSTFIELDS => json_encode([
                 'options' => $opt,
                 'data' => $data
-            ], JSON_PRETTY_PRINT | JSON_THROW_ON_ERROR)
+            ], JSON_PRETTY_PRINT|JSON_THROW_ON_ERROR)
         ]);
         return $response['data'] ?? null;
     }
@@ -408,13 +408,13 @@ 
      * @param array $opt
      * @return null|array
      */
-    public function put (string $path, array $data = [], array $opt = []) {
+    public function put(string $path, array $data = [], array $opt = []) {
         $response = $this->_exec('PUT', $path, [
             CURLOPT_HTTPHEADER => ['Content-Type: application/json'],
             CURLOPT_POSTFIELDS => json_encode([
                 'options' => $opt,
                 'data' => $data
-            ], JSON_PRETTY_PRINT | JSON_THROW_ON_ERROR)
+            ], JSON_PRETTY_PRINT|JSON_THROW_ON_ERROR)
         ]);
         return $response['data'] ?? null;
     }
@@ -422,7 +422,7 @@ 
     /**
      * @param LoggerInterface $log
      */
-    public function setLog (LoggerInterface $log) {
+    public function setLog(LoggerInterface $log) {
         $this->log = $log;
     }
 
@@ -438,7 +438,7 @@ 
      * @param null|string $token Updated to the new token.
      * @return Event[]
      */
-    public function sync (string $gid, ?string &$token) {
+    public function sync(string $gid, ?string &$token) {
         try {
             /** @var array $remote Asana throws 400 for missing entities. */
             $remote = $this->_exec('GET', 'events?' . http_build_query([
@@ -449,7 +449,7 @@ 
         }
         catch (AsanaError $error) {
             if ($error->getCode() === 412) {
-                $remote = json_decode($error->getMessage(), true, JSON_BIGINT_AS_STRING | JSON_THROW_ON_ERROR);
+                $remote = json_decode($error->getMessage(), true, JSON_BIGINT_AS_STRING|JSON_THROW_ON_ERROR);
                 if (!isset($token)) {
                     // API docs say: "The response will be the same as for an expired sync token."
                     // The caller knowingly gave a null token, so we don't need to rethrow.
@@ -478,7 +478,7 @@ 
      * @param string $to
      * @return array
      */
-    public function upload (string $file, string $to) {
+    public function upload(string $file, string $to) {
         return $this->_exec('POST', $to, [
             CURLOPT_POSTFIELDS => ['file' => new CURLFile(realpath($file))] // multipart/form-data
         ])['data'];

src/Base/AbstractEntity/UpdateTrait.php

Spacing +1 added lines, -1 removed lines

@@ -16,7 +16,7 @@
      *
      * @return $this
      */
-    public function update () {
+    public function update() {
         if ($this->isDiff()) {
             /** @var array $remote */
             $remote = $this->api->put($this, $this->toArray(true), ['expand' => 'this']);