hfw / asana

src/Job.php

Spacing +8 added lines, -8 removed lines

@@ -28,11 +28,11 @@ 
     const STATUS_SUCCESS = 'succeeded'; // api docs say "completed" but that's wrong.
     const STATUS_FAIL = 'failed';
 
-    final public function __toString (): string {
+    final public function __toString(): string {
         return "jobs/{$this->getGid()}";
     }
 
-    protected function _getMap (): array {
+    protected function _getMap(): array {
         return [
             'new_project' => Project::class,
             'new_task' => Task::class
@@ -44,7 +44,7 @@ 
      *
      * @return bool
      */
-    public function isActive (): bool {
+    public function isActive(): bool {
         return $this->getStatus() === self::STATUS_ACTIVE;
     }
 
@@ -53,7 +53,7 @@ 
      *
      * @return bool
      */
-    public function isDone (): bool {
+    public function isDone(): bool {
         return $this->isSuccessful() or $this->isFailed();
     }
 
@@ -62,7 +62,7 @@ 
      *
      * @return bool
      */
-    public function isFailed (): bool {
+    public function isFailed(): bool {
         return $this->getStatus() === self::STATUS_FAIL;
     }
 
@@ -71,7 +71,7 @@ 
      *
      * @return bool
      */
-    public function isQueued (): bool {
+    public function isQueued(): bool {
         return $this->getStatus() === self::STATUS_QUEUED;
     }
 
@@ -80,7 +80,7 @@ 
      *
      * @return bool
      */
-    public function isSuccessful (): bool {
+    public function isSuccessful(): bool {
         return $this->getStatus() === self::STATUS_SUCCESS;
     }
 
@@ -89,7 +89,7 @@ 
      *
      * @return $this
      */
-    public function wait () {
+    public function wait() {
         while (!$this->isDone()) {
             sleep(3);
             $this->reload();

src/Base/AbstractEntity/WorkspaceTrait.php

Spacing +1 added lines, -1 removed lines

@@ -17,7 +17,7 @@
      * @param Workspace $workspace
      * @return $this
      */
-    public function setWorkspace (Workspace $workspace) {
+    public function setWorkspace(Workspace $workspace) {
         assert(!$this->hasGid());
         return $this->_set('workspace', $workspace);
     }

src/Base/AbstractEntity.php

Spacing +7 added lines, -7 removed lines

@@ -16,7 +16,7 @@ 
      *
      * @return string
      */
-    abstract public function __toString (): string;
+    abstract public function __toString(): string;
 
     /**
      * Resolves entities to GIDs.
@@ -24,7 +24,7 @@ 
      * @param AbstractEntity[] $entities
      * @return string[]
      */
-    protected static function _getGids (array $entities) {
+    protected static function _getGids(array $entities) {
         return array_map(function(AbstractEntity $entity) {
             return $entity->getGid();
         }, $entities);
@@ -36,7 +36,7 @@ 
      * @param string $key
      * @return mixed
      */
-    protected function _get (string $key) {
+    protected function _get(string $key) {
         if (!array_key_exists($key, $this->data) and isset($this->data['gid'])) { // can't use hasGid(), inf. loop
             $this->reload($key);
         }
@@ -51,7 +51,7 @@ 
      * @param bool $force
      * @return $this
      */
-    protected function _merge (string $key, array $entities, $force = false) {
+    protected function _merge(string $key, array $entities, $force = false) {
         if ($force or isset($this->data[$key])) {
             foreach ($entities as $entity) {
                 $this->data[$key][] = $entity;
@@ -74,7 +74,7 @@ 
      * @param AbstractEntity[] $entities
      * @return $this
      */
-    protected function _remove (string $key, array $entities) {
+    protected function _remove(string $key, array $entities) {
         if (isset($this->data[$key])) {
             $this->data[$key] = array_values(array_diff($this->data[$key], $entities));
             $this->api->getCache()->add($this);
@@ -88,7 +88,7 @@ 
      * @param AbstractEntity $entity
      * @return $this
      */
-    public function merge (AbstractEntity $entity) {
+    public function merge(AbstractEntity $entity) {
         assert($entity->api); // hydrated
         foreach ($entity->data as $key => $value) {
             if (!array_key_exists($key, $this->data) and !isset($entity->diff[$key])) {
@@ -104,7 +104,7 @@ 
      * @param string $key
      * @return $this
      */
-    public function reload (string $key = null) {
+    public function reload(string $key = null) {
         if (isset($key)) {
             $value = $this->api->get($this, [], ['fields' => $key])[$key];
             $map = $this->_getMap();

Braces +2 added lines, -4 removed lines

@@ -59,8 +59,7 @@ 
             $this->data[$key] = array_values(array_unique($this->data[$key]));
             if ($force) {
                 $this->diff[$key] = true;
-            }
-            else {
+            } else {
                 $this->api->getCache()->add($this);
             }
         }
@@ -110,8 +109,7 @@ 
             $map = $this->_getMap();
             $this->data[$key] = isset($map[$key]) ? $this->_getMapped($map[$key], $value) : $value;
             unset($this->diff[$key]);
-        }
-        else {
+        } else {
             $this->_setData($this->api->get($this, [], ['expand' => 'this']));
         }
         $this->api->getCache()->add($this);

src/Base/Data.php

Braces +2 added lines, -4 removed lines

@@ -291,11 +291,9 @@
         $dehydrate = function($each) use (&$dehydrate) {
             if ($each instanceof AbstractEntity and $each->hasGid()) {
                 return $each->getGid();
-            }
-            elseif ($each instanceof self) {
+            } elseif ($each instanceof self) {
                 return $each->toArray();
-            }
-            elseif (is_array($each)) {
+            } elseif (is_array($each)) {
                 return array_map($dehydrate, $each);
             }
             return $each;

Spacing +20 added lines, -20 removed lines

@@ -30,7 +30,7 @@ 
      * @param Api|Data $caller
      * @param array $data
      */
-    public function __construct ($caller, array $data = []) {
+    public function __construct($caller, array $data = []) {
         $this->api = $caller instanceof self ? $caller->api : $caller;
         $this->_setData($data);
     }
@@ -46,9 +46,9 @@ 
      * @param array $args
      * @return mixed
      */
-    public function __call (string $method, array $args) {
+    public function __call(string $method, array $args) {
         static $cache = [];
-        if (!$call =& $cache[$method]) {
+        if (!$call = & $cache[$method]) {
             preg_match('/^(get|has|is|set)(.+)$/', $method, $call);
             $call[1] = '_' . $call[1];
             $call[2] = preg_replace_callback('/[A-Z]/', function(array $match) {
@@ -58,7 +58,7 @@ 
         return $this->{$call[1]}($call[2], ...$args);
     }
 
-    public function __debugInfo (): array {
+    public function __debugInfo(): array {
         return $this->data;
     }
 
@@ -70,7 +70,7 @@ 
      * @param string $key
      * @return mixed
      */
-    protected function _get (string $key) {
+    protected function _get(string $key) {
         return $this->data[$key] ?? null;
     }
 
@@ -82,7 +82,7 @@ 
      *
      * @return array
      */
-    protected function _getMap (): array {
+    protected function _getMap(): array {
         return [];
     }
 
@@ -91,7 +91,7 @@ 
      * @param null|self|array $value
      * @return null|Data|Data[]
      */
-    protected function _getMapped ($class, $value) {
+    protected function _getMapped($class, $value) {
         // use empty|Data|Data[] as-is
         if (!$value or $value instanceof self or current($value) instanceof self) {
             return $value;
@@ -124,7 +124,7 @@ 
      *
      * @return array
      */
-    protected function _getPatch (): array {
+    protected function _getPatch(): array {
         $convert = function($each) use (&$convert) {
             // convert existing entities to gids
             if ($each instanceof AbstractEntity and $each->hasGid()) {
@@ -154,7 +154,7 @@ 
      * @param string $key
      * @return bool
      */
-    protected function _has (string $key): bool {
+    protected function _has(string $key): bool {
         $value = $this->_get($key);
         if (isset($value)) {
             if (is_countable($value)) {
@@ -177,7 +177,7 @@ 
      * @param string $key
      * @return bool
      */
-    protected function _is (string $key): bool {
+    protected function _is(string $key): bool {
         return !empty($this->_get($key));
     }
 
@@ -190,7 +190,7 @@ 
      * @param mixed $value
      * @return $this
      */
-    protected function _set (string $key, $value) {
+    protected function _set(string $key, $value) {
         $this->data[$key] = $value;
         $this->diff[$key] = true;
         return $this;
@@ -202,7 +202,7 @@ 
      * @param array $data
      * @return $this
      */
-    protected function _setData (array $data) {
+    protected function _setData(array $data) {
         $map = $this->_getMap();
         /** @var null|self|array $value */
         foreach (array_intersect_key($data, $map) as $key => $value) {
@@ -220,7 +220,7 @@ 
      * @param array $data
      * @return mixed
      */
-    final protected function factory (string $class, array $data = []) {
+    final protected function factory(string $class, array $data = []) {
         return $this->api->factory($class, $this, $data);
     }
 
@@ -229,7 +229,7 @@ 
      *
      * @return bool
      */
-    final public function isDiff (): bool {
+    final public function isDiff(): bool {
         return (bool)$this->diff;
     }
 
@@ -237,7 +237,7 @@ 
      * @see toArray()
      * @return array
      */
-    public function jsonSerialize (): array {
+    public function jsonSerialize(): array {
         return $this->toArray();
     }
 
@@ -251,7 +251,7 @@ 
      * @param array $query
      * @return null|mixed|AbstractEntity
      */
-    final protected function load (string $class, string $path, array $query = []) {
+    final protected function load(string $class, string $path, array $query = []) {
         return $this->api->load($class, $this, $path, $query);
     }
 
@@ -266,7 +266,7 @@ 
      * @param int $pages
      * @return array|AbstractEntity[]
      */
-    final protected function loadAll (string $class, string $path, array $query = [], int $pages = 0) {
+    final protected function loadAll(string $class, string $path, array $query = [], int $pages = 0) {
         return $this->api->loadAll($class, $this, $path, $query, $pages);
     }
 
@@ -277,7 +277,7 @@ 
      *
      * @return string
      */
-    public function serialize (): string {
+    public function serialize(): string {
         return serialize($this->toArray());
     }
 
@@ -286,7 +286,7 @@ 
      *
      * @return array
      */
-    public function toArray (): array {
+    public function toArray(): array {
         if (!$this->api) {
             return $this->data;
         }
@@ -312,7 +312,7 @@ 
      *
      * @param $serialized
      */
-    public function unserialize ($serialized): void {
+    public function unserialize($serialized): void {
         $this->data = unserialize($serialized);
     }
 }
\ No newline at end of file

src/Portfolio.php

Braces +1 added lines, -2 removed lines

@@ -73,8 +73,7 @@
         if ($this->hasGid()) {
             $this->api->post("{$this}/addMembers", ['members' => static::_getGids($users)]);
             $this->_merge('members', $users);
-        }
-        else {
+        } else {
             $this->_merge('members', $users, true);
         }
         return $this;

Spacing +10 added lines, -10 removed lines

@@ -30,15 +30,15 @@ 
 
     const TYPE = 'portfolio';
 
-    final public function __toString (): string {
+    final public function __toString(): string {
         return "portfolios/{$this->getGid()}";
     }
 
-    final protected function _getDir (): string {
+    final protected function _getDir(): string {
         return 'portfolios';
     }
 
-    protected function _getMap (): array {
+    protected function _getMap(): array {
         return [
             'created_by' => User::class,
             'owner' => User::class,
@@ -52,7 +52,7 @@ 
      * @param Project $item
      * @return $this
      */
-    public function addItem (Project $item) {
+    public function addItem(Project $item) {
         $this->api->post("{$this}/addItem", ['item' => $item->getGid()]);
         return $this;
     }
@@ -61,7 +61,7 @@ 
      * @param User $user
      * @return $this
      */
-    public function addMember (User $user) {
+    public function addMember(User $user) {
         return $this->addMembers([$user]);
     }
 
@@ -69,7 +69,7 @@ 
      * @param User[] $users
      * @return $this
      */
-    public function addMembers (array $users) {
+    public function addMembers(array $users) {
         if ($this->hasGid()) {
             $this->api->post("{$this}/addMembers", ['members' => static::_getGids($users)]);
             $this->_merge('members', $users);
@@ -84,7 +84,7 @@ 
      * @depends after-create
      * @return Project[]
      */
-    public function getItems () {
+    public function getItems() {
         return $this->loadAll(Project::class, "{$this}/items");
     }
 
@@ -93,7 +93,7 @@ 
      * @param Project $item
      * @return $this
      */
-    public function removeItem (Project $item) {
+    public function removeItem(Project $item) {
         $this->api->post("{$this}/removeItem", ['item' => $item->getGid()]);
         return $this;
     }
@@ -102,7 +102,7 @@ 
      * @param User $user
      * @return $this
      */
-    public function removeMember (User $user) {
+    public function removeMember(User $user) {
         return $this->removeMembers([$user]);
     }
 
@@ -110,7 +110,7 @@ 
      * @param User[] $users
      * @return $this
      */
-    public function removeMembers (array $users) {
+    public function removeMembers(array $users) {
         if ($this->hasGid()) {
             $this->api->post("{$this}/removeMembers", ['members' => static::_getGids($users)]);
         }

src/Api/Cache.php

Spacing +4 added lines, -4 removed lines

@@ -32,7 +32,7 @@ 
      *
      * @param AbstractEntity $entity
      */
-    public function add (AbstractEntity $entity): void {
+    public function add(AbstractEntity $entity): void {
         if ($gid = $entity->getGid() and !$entity->isDiff()) {
             $this->entities[$gid] = $entity;
             foreach ($this->getKeys($entity) as $key) {
@@ -52,7 +52,7 @@ 
      * @param Closure $factory `fn($caller): null|AbstractEntity`
      * @return null|mixed|AbstractEntity
      */
-    public function get (string $key, $caller, Closure $factory) {
+    public function get(string $key, $caller, Closure $factory) {
         if ($gid = $this->gids[$key] ?? null) {
             return $this->entities[$gid];
         }
@@ -81,7 +81,7 @@ 
      * @param AbstractEntity $entity
      * @return string[]
      */
-    protected function getKeys (AbstractEntity $entity) {
+    protected function getKeys(AbstractEntity $entity) {
         if (!$gid = $entity->getGid() or $entity->isDiff()) {
             return [];
         }
@@ -97,7 +97,7 @@ 
      *
      * @param AbstractEntity $entity
      */
-    public function remove (AbstractEntity $entity): void {
+    public function remove(AbstractEntity $entity): void {
         unset($this->entities[$entity->getGid()]);
         foreach ($this->getKeys($entity) as $key) {
             unset($this->gids[$key]);

src/Api/LoggerInterface.php

Spacing +1 added lines, -1 removed lines

@@ -12,5 +12,5 @@
      * @param string $path
      * @param null|array $payload
      */
-    public function log (string $info, string $path, ?array $payload): void;
+    public function log(string $info, string $path, ?array $payload): void;
 }
\ No newline at end of file

src/Api/Laravel/AsanaServiceProvider.php

Spacing +2 added lines, -2 removed lines

@@ -11,11 +11,11 @@
 
     const NAME = 'asana';
 
-    public function boot () {
+    public function boot() {
         $this->mergeConfigFrom(App::configPath('asana.php'), self::NAME);
     }
 
-    public function register () {
+    public function register() {
         $this->app->singleton(self::NAME, function($app) {
             $config = $app->config->get(self::NAME);
             return new Api($config['token']);

src/Api/SimpleCache.php

Braces +1 added lines, -2 removed lines

@@ -68,8 +68,7 @@
                     /** @var null|AbstractEntity $entity */
                     if ($entity = $this->get($entity, $caller, $factory)) {
                         $this->cache->set($key, $entity->getGid()); // renew potentially nonstandard key
-                    }
-                    else {
+                    } else {
                         $this->cache->delete($key); // remove invalid key
                     }
                     return $entity; // null or hydrated. parent calls add()

Spacing +5 added lines, -5 removed lines

@@ -36,7 +36,7 @@ 
      * @param Api $api
      * @param PSR16 $cache
      */
-    public function __construct (Api $api, PSR16 $cache) {
+    public function __construct(Api $api, PSR16 $cache) {
         $this->api = $api;
         $this->cache = $cache;
     }
@@ -45,7 +45,7 @@ 
      * @param AbstractEntity $entity
      * @throws InvalidArgumentException
      */
-    public function add (AbstractEntity $entity): void {
+    public function add(AbstractEntity $entity): void {
         if ($gid = $entity->getGid() and !$entity->isDiff()) {
             parent::add($entity);
             foreach ($this->getKeys($entity) as $key) {
@@ -57,7 +57,7 @@ 
         }
     }
 
-    public function get (string $key, $caller, Closure $factory) {
+    public function get(string $key, $caller, Closure $factory) {
         return parent::get($key, $caller, function($caller) use ($key, $factory) {
             /** @var null|string|AbstractEntity $entity */
             if ($entity = $this->cache->get($key)) { // hit?
@@ -101,7 +101,7 @@ 
      * @param Api|Data $caller
      * @return AbstractEntity
      */
-    protected function hydrate (AbstractEntity $proto, $caller) {
+    protected function hydrate(AbstractEntity $proto, $caller) {
         return $this->api->factory(get_class($proto), $caller, $proto->toArray());
     }
 
@@ -109,7 +109,7 @@ 
      * @param AbstractEntity $entity
      * @throws InvalidArgumentException
      */
-    public function remove (AbstractEntity $entity): void {
+    public function remove(AbstractEntity $entity): void {
         parent::remove($entity);
         $this->cache->deleteMultiple($this->getKeys($entity));
     }