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/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/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']);

src/Base/AbstractEntity/CreateTrait.php

Spacing +2 added lines, -2 removed lines

@@ -16,14 +16,14 @@
      *
      * @return string
      */
-    abstract protected function _getDir (): string;
+    abstract protected function _getDir(): string;
 
     /**
      * `POST` the new entity to Asana.
      *
      * @return $this
      */
-    public function create () {
+    public function create() {
         /** @var array $remote */
         $remote = $this->api->post($this->_getDir(), $this->toArray(true), ['expand' => 'this']);
         $this->_setData($remote);

src/Job.php

Spacing +9 added lines, -9 removed lines

@@ -39,7 +39,7 @@ 
      *
      * @return string
      */
-    final public function __toString (): string {
+    final public function __toString(): string {
         return "jobs/{$this->getGid()}";
     }
 
@@ -48,7 +48,7 @@ 
      *
      * @return bool
      */
-    final public function isActive (): bool {
+    final public function isActive(): bool {
         return $this->getStatus() === self::STATUS_ACTIVE;
     }
 
@@ -57,21 +57,21 @@ 
      *
      * @return bool
      */
-    final public function isDone (): bool {
+    final public function isDone(): bool {
         return $this->isSuccessful() or $this->isFailed();
     }
 
     /**
      * @return bool
      */
-    final public function isDuplicatingProject (): bool {
+    final public function isDuplicatingProject(): bool {
         return $this->getResourceSubtype() === self::TYPE_DUPLICATE_PROJECT;
     }
 
     /**
      * @return bool
      */
-    final public function isDuplicatingTask (): bool {
+    final public function isDuplicatingTask(): bool {
         return $this->getResourceSubtype() === self::TYPE_DUPLICATE_TASK;
     }
 
@@ -80,7 +80,7 @@ 
      *
      * @return bool
      */
-    final public function isFailed (): bool {
+    final public function isFailed(): bool {
         return $this->getStatus() === self::STATUS_FAIL;
     }
 
@@ -89,7 +89,7 @@ 
      *
      * @return bool
      */
-    final public function isQueued (): bool {
+    final public function isQueued(): bool {
         return $this->getStatus() === self::STATUS_QUEUED;
     }
 
@@ -98,7 +98,7 @@ 
      *
      * @return bool
      */
-    final public function isSuccessful (): bool {
+    final public function isSuccessful(): bool {
         return $this->getStatus() === self::STATUS_SUCCESS;
     }
 
@@ -110,7 +110,7 @@ 
      * @param callable $spinner `fn( Job $this ): void`
      * @return $this
      */
-    public function wait (callable $spinner = null) {
+    public function wait(callable $spinner = null) {
         while (!$this->isDone()) {
             if ($spinner) {
                 call_user_func($spinner, $this);