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/Task/FieldEntry.php

Spacing +13 added lines, -13 removed lines

@@ -31,7 +31,7 @@ 
      * @param Change|FieldEntries $caller
      * @param array $data
      */
-    public function __construct ($caller, array $data = []) {
+    public function __construct($caller, array $data = []) {
         $this->caller = $caller;
         parent::__construct($caller, $data);
     }
@@ -39,7 +39,7 @@ 
     /**
      * @return string
      */
-    public function __toString (): string {
+    public function __toString(): string {
         if ($this->isEnum()) {
             return (string)$this->getCurrentOptionName();
         }
@@ -51,7 +51,7 @@ 
      *
      * @param array $data
      */
-    protected function _setData (array $data): void {
+    protected function _setData(array $data): void {
         if (isset($data['resource_subtype'])) { // sentinel for bloat
             $tiny = array_intersect_key($data, array_flip([
                 'gid',
@@ -77,21 +77,21 @@ 
      * @param null|string $value
      * @return null|string
      */
-    protected function _toEnumOptionGid ($value) {
+    protected function _toEnumOptionGid($value) {
         return $this->getEnumOptionValues()[$value] ?? $value;
     }
 
     /**
      * @return string
      */
-    public function getCurrentOptionName () {
+    public function getCurrentOptionName() {
         return $this->getEnumOptionNames()[$this->getValue()];
     }
 
     /**
      * @return CustomField
      */
-    public function getCustomField () {
+    public function getCustomField() {
         return $this->api->getCustomField($this->getGid());
     }
 
@@ -100,7 +100,7 @@ 
      *
      * @return string[]
      */
-    public function getEnumOptionNames () {
+    public function getEnumOptionNames() {
         static $names = []; // shared
         $gid = $this->data['gid'];
         return $names[$gid] ?? $names[$gid] = array_column($this->data['enum_options'], 'name', 'gid');
@@ -111,7 +111,7 @@ 
      *
      * @return string[]
      */
-    public function getEnumOptionValues () {
+    public function getEnumOptionValues() {
         static $values = []; // shared
         $gid = $this->data['gid'];
         return $values[$gid] ?? $values[$gid] = array_column($this->data['enum_options'], 'gid', 'name');
@@ -120,7 +120,7 @@ 
     /**
      * @return null|number|string
      */
-    public function getValue () {
+    public function getValue() {
         $type = $this->data['type'];
         if ($type === CustomField::TYPE_ENUM) {
             return $this->data['enum_value']['gid'] ?? null;
@@ -131,21 +131,21 @@ 
     /**
      * @return bool
      */
-    final public function isEnum (): bool {
+    final public function isEnum(): bool {
         return $this->getType() === CustomField::TYPE_ENUM;
     }
 
     /**
      * @return bool
      */
-    final public function isNumber (): bool {
+    final public function isNumber(): bool {
         return $this->getType() === CustomField::TYPE_NUMBER;
     }
 
     /**
      * @return bool
      */
-    final public function isText (): bool {
+    final public function isText(): bool {
         return $this->getType() === CustomField::TYPE_TEXT;
     }
 
@@ -153,7 +153,7 @@ 
      * @param null|number|string $value
      * @return $this
      */
-    public function setValue ($value) {
+    public function setValue($value) {
         if ($this->caller instanceof FieldEntries) { // read-only if the entry is in an event change.
             $type = $this->data['type'];
             $this->diff["{$type}_value"] = true;

Braces +1 added lines, -2 removed lines

@@ -160,8 +160,7 @@
             $this->caller->__set($this->data['gid'], true);
             if ($type === CustomField::TYPE_ENUM) {
                 $this->data['enum_value']['gid'] = $this->_toEnumOptionGid($value);
-            }
-            else {
+            } else {
                 $this->data["{$type}_value"] = $value;
             }
         }

src/Task/Like.php

Spacing +1 added lines, -1 removed lines

@@ -18,7 +18,7 @@
         'user' => User::class
     ];
 
-    protected function _setData (array $data): void {
+    protected function _setData(array $data): void {
         // useless. likes aren't entities.
         unset($data['gid'], $data['resource_type']);