hfw / asana

src/Event.php

Spacing +10 added lines, -10 removed lines

@@ -42,11 +42,11 @@ 
  */
 class Event extends Data {
 
-    const ACTION_CHANGED = 'changed';       // no parent
-    const ACTION_ADDED = 'added';           // relational, no change
-    const ACTION_REMOVED = 'removed';       // relational, no change
-    const ACTION_DELETED = 'deleted';       // no parent or change
-    const ACTION_UNDELETED = 'undeleted';   // no parent or change
+    const ACTION_CHANGED = 'changed'; // no parent
+    const ACTION_ADDED = 'added'; // relational, no change
+    const ACTION_REMOVED = 'removed'; // relational, no change
+    const ACTION_DELETED = 'deleted'; // no parent or change
+    const ACTION_UNDELETED = 'undeleted'; // no parent or change
 
     const GRAPH = [
         User::TYPE => User::class,
@@ -71,35 +71,35 @@ 
     /**
      * @return bool
      */
-    final public function wasAddition (): bool {
+    final public function wasAddition(): bool {
         return $this->getAction() === self::ACTION_ADDED;
     }
 
     /**
      * @return bool
      */
-    final public function wasChange (): bool {
+    final public function wasChange(): bool {
         return $this->getAction() === self::ACTION_CHANGED;
     }
 
     /**
      * @return bool
      */
-    final public function wasDeletion (): bool {
+    final public function wasDeletion(): bool {
         return $this->getAction() === self::ACTION_DELETED;
     }
 
     /**
      * @return bool
      */
-    final public function wasRemoval (): bool {
+    final public function wasRemoval(): bool {
         return $this->getAction() === self::ACTION_REMOVED;
     }
 
     /**
      * @return bool
      */
-    final public function wasUndeletion (): bool {
+    final public function wasUndeletion(): bool {
         return $this->getAction() === self::ACTION_UNDELETED;
     }
 

src/Project.php

Spacing +26 added lines, -26 removed lines

@@ -77,15 +77,15 @@ 
      */
     private $defaultSection;
 
-    final public function __toString (): string {
+    final public function __toString(): string {
         return "projects/{$this->getGid()}";
     }
 
-    final protected function _getDir (): string {
+    final protected function _getDir(): string {
         return 'projects';
     }
 
-    protected function _setData (array $data): void {
+    protected function _setData(array $data): void {
         // this is always empty. fields are in the settings, values are in tasks.
         unset($data['custom_fields']);
 
@@ -100,7 +100,7 @@ 
      * @param User $user
      * @return $this
      */
-    public function addMember (User $user) {
+    public function addMember(User $user) {
         return $this->addMembers([$user]);
     }
 
@@ -109,7 +109,7 @@ 
      * @param User[] $users
      * @return $this
      */
-    public function addMembers (array $users) {
+    public function addMembers(array $users) {
         return $this->_addWithPost("{$this}/addMembers", [
             'members' => array_column($users, 'gid')
         ], 'members', $users);
@@ -120,7 +120,7 @@ 
      * @param string $target
      * @return ProjectWebhook
      */
-    public function addWebhook (string $target) {
+    public function addWebhook(string $target) {
         /** @var ProjectWebhook $webhook */
         $webhook = $this->api->factory($this, ProjectWebhook::class);
         return $webhook->create($this, $target);
@@ -145,7 +145,7 @@ 
      * @param array $schedule
      * @return Job
      */
-    public function duplicate (string $name, array $include, Team $team = null, array $schedule = []) {
+    public function duplicate(string $name, array $include, Team $team = null, array $schedule = []) {
         $data = ['name' => $name];
         if ($team) {
             $data['team'] = $team->getGid();
@@ -164,7 +164,7 @@ 
      * @depends after-create
      * @return Section
      */
-    public function getDefaultSection () {
+    public function getDefaultSection() {
         return $this->defaultSection ??
             $this->defaultSection = $this->api->loadAll($this, Section::class, "{$this}/sections", ['limit' => 1])[0];
     }
@@ -178,14 +178,14 @@ 
      * @param null|string $token
      * @return Event[]
      */
-    public function getEvents (&$token) {
+    public function getEvents(&$token) {
         return $this->api->sync($this->getGid(), $token);
     }
 
     /**
      * @return Traversable|Section[]
      */
-    public function getIterator () {
+    public function getIterator() {
         return $this->api->loadEach($this, Section::class, "{$this}/sections");
     }
 
@@ -193,7 +193,7 @@ 
      * @depends after-create
      * @return Section[]
      */
-    public function getSections () {
+    public function getSections() {
         return iterator_to_array($this);
     }
 
@@ -201,7 +201,7 @@ 
      * @depends after-create
      * @return Status[]
      */
-    public function getStatuses () {
+    public function getStatuses() {
         return $this->api->loadAll($this, Status::class, "{$this}/project_statuses");
     }
 
@@ -212,7 +212,7 @@ 
      * @param array $filter
      * @return Task[]
      */
-    public function getTasks (array $filter = []) {
+    public function getTasks(array $filter = []) {
         $filter['project'] = $this->getGid();
         return $this->api->loadAll($this, Task::class, "tasks", $filter);
     }
@@ -221,25 +221,25 @@ 
      * @depends after-create
      * @return string
      */
-    public function getUrl (): string {
+    public function getUrl(): string {
         return "https://app.asana.com/0/{$this->getGid()}";
     }
 
     /**
      * @return ProjectWebhook[]
      */
-    public function getWebhooks () {
+    public function getWebhooks() {
         return $this->api->loadAll($this, ProjectWebhook::class, 'webhooks', [
             'workspace' => $this->getWorkspace()->getGid(),
             'resource' => $this->getGid()
         ]);
     }
 
-    public function isBoard (): bool {
+    public function isBoard(): bool {
         return $this->getLayout() === self::LAYOUT_BOARD;
     }
 
-    public function isList (): bool {
+    public function isList(): bool {
         return $this->getLayout() === self::LAYOUT_LIST;
     }
 
@@ -247,7 +247,7 @@ 
      * @depends after-create
      * @return Section
      */
-    public function newSection () {
+    public function newSection() {
         return $this->api->factory($this, Section::class, ['project' => $this]);
     }
 
@@ -255,7 +255,7 @@ 
      * @depends after-create
      * @return Status
      */
-    public function newStatus () {
+    public function newStatus() {
         return $this->api->factory($this, Status::class);
     }
 
@@ -265,7 +265,7 @@ 
      * @depends after-create
      * @return Task
      */
-    public function newTask () {
+    public function newTask() {
         return $this->getDefaultSection()->newTask();
     }
 
@@ -274,7 +274,7 @@ 
      * @param User $user
      * @return $this
      */
-    public function removeMember (User $user) {
+    public function removeMember(User $user) {
         return $this->removeMembers([$user]);
     }
 
@@ -283,7 +283,7 @@ 
      * @param User[] $users
      * @return $this
      */
-    public function removeMembers (array $users) {
+    public function removeMembers(array $users) {
         return $this->_removeWithPost("{$this}/removeMembers", [
             'members' => array_column($users, 'gid')
         ], 'members', $users);
@@ -293,7 +293,7 @@ 
      * @param callable $filter `fn( Section $section ): bool`
      * @return Section[]
      */
-    public function selectSections (callable $filter) {
+    public function selectSections(callable $filter) {
         return $this->_select($this->getSections(), $filter);
     }
 
@@ -301,7 +301,7 @@ 
      * @param callable $filter `fn( Status $status ): bool`
      * @return Status[]
      */
-    public function selectStatuses (callable $filter) {
+    public function selectStatuses(callable $filter) {
         return $this->_select($this->getStatuses(), $filter);
     }
 
@@ -310,7 +310,7 @@ 
      * @param array $apiFilter Pre-filter given to the API to reduce network load.
      * @return Task[]
      */
-    public function selectTasks (callable $filter, array $apiFilter = []) {
+    public function selectTasks(callable $filter, array $apiFilter = []) {
         return $this->_select($this->getTasks($apiFilter), $filter);
     }
 
@@ -319,7 +319,7 @@ 
      * @param null|Team $team
      * @return $this
      */
-    public function setTeam (?Team $team) {
+    public function setTeam(?Team $team) {
         if ($team and !$this->hasWorkspace()) {
             $this->setWorkspace($team->getOrganization());
         }

src/Api/Cache.php

Spacing +4 added lines, -4 removed lines

@@ -34,7 +34,7 @@ 
      * @param AbstractEntity $entity
      * @return bool Success?
      */
-    public function add (AbstractEntity $entity): bool {
+    public function add(AbstractEntity $entity): bool {
         if (!$entity->isDiff() and $gid = $entity->getGid()) {
             $this->entities[$gid] = $entity;
             $this->addKeys($gid, $entity->getCacheKeys());
@@ -49,7 +49,7 @@ 
      * @param string $gid
      * @param string[] $keys
      */
-    protected function addKeys (string $gid, array $keys): void {
+    protected function addKeys(string $gid, array $keys): void {
         $this->gids += array_fill_keys($keys, $gid);
     }
 
@@ -64,7 +64,7 @@ 
      * @param Closure $factory `fn( Api|Data $caller ): null|AbstractEntity`
      * @return null|mixed|AbstractEntity
      */
-    public function get (string $key, $caller, Closure $factory) {
+    public function get(string $key, $caller, Closure $factory) {
         // POOL HIT
         if ($gid = $this->gids[$key] ?? null) {
             return $this->entities[$gid];
@@ -93,7 +93,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/Api/FileCache.php

Spacing +14 added lines, -14 removed lines

@@ -36,26 +36,26 @@ 
      * @param string $dir
      * @param bool $gz
      */
-    public function __construct (string $dir, $gz = false) {
+    public function __construct(string $dir, $gz = false) {
         $this->dir = $dir;
         $this->gz = $gz;
     }
 
-    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);
@@ -68,11 +68,11 @@ 
         }
     }
 
-    public function deleteMultiple ($keys) {
+    public function deleteMultiple($keys) {
         // unused
     }
 
-    public function get ($key, $default = null) {
+    public function get($key, $default = null) {
         $path = $this->_path($key);
         if (is_file($path)) {
             if (filemtime($path) > time()) {
@@ -95,19 +95,19 @@ 
     /**
      * @return LoggerInterface
      */
-    public function getLogger () {
+    public function getLogger() {
         return $this->logger;
     }
 
-    public function getMultiple ($keys, $default = null) {
+    public function getMultiple($keys, $default = null) {
         // unused
     }
 
-    public function has ($key) {
+    public function has($key) {
         // unused
     }
 
-    private function log (string $msg, string $key): void {
+    private function log(string $msg, string $key): void {
         if ($this->logger) {
             $this->logger->log($msg, $key);
         }
@@ -119,7 +119,7 @@ 
      * @param DateInterval $ttl
      * @return void
      */
-    public function set ($key, $value, $ttl = null): void {
+    public function set($key, $value, $ttl = null): void {
         assert($ttl instanceof DateInterval);
         $path = $this->_path($key);
         $data = serialize($value);
@@ -146,12 +146,12 @@ 
      * @param null|LoggerInterface $logger
      * @return $this
      */
-    public function setLogger (?LoggerInterface $logger) {
+    public function setLogger(?LoggerInterface $logger) {
         $this->logger = $logger;
         return $this;
     }
 
-    public function setMultiple ($values, $ttl = null) {
+    public function setMultiple($values, $ttl = null) {
         // unused
     }
 }
\ No newline at end of file

Braces +3 added lines, -6 removed lines

@@ -61,8 +61,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);
         }
@@ -85,8 +84,7 @@ 
             }
             $this->log('CACHE EXPIRE', $key);
             unlink($path);
-        }
-        else {
+        } else {
             $this->log('CACHE MISS', $key);
         }
         return $default;
@@ -132,8 +130,7 @@ 
         if (is_object($value)) {
             $this->log('CACHE SET', $key);
             file_put_contents($path, $data);
-        }
-        elseif (!file_exists($path)) {
+        } elseif (!file_exists($path)) {
             $this->log('CACHE LINK', "{$key} => asana/{$value}");
             file_put_contents($path, $data);
             symlink($this->_path("asana/{$value}"), $this->_ref($key));

src/Api/SimpleCache.php

Spacing +10 added lines, -10 removed lines

@@ -34,7 +34,7 @@ 
     /**
      * @param PSR16 $psr
      */
-    public function __construct (PSR16 $psr) {
+    public function __construct(PSR16 $psr) {
         $this->psr = $psr;
         $this->ttl = new DateInterval('PT1H');
     }
@@ -43,7 +43,7 @@ 
      * @param string $key
      * @throws CacheException
      */
-    protected function _delete (string $key): void {
+    protected function _delete(string $key): void {
         $this->psr->delete('asana/' . $key);
     }
 
@@ -52,7 +52,7 @@ 
      * @return null|string|AbstractEntity
      * @throws CacheException
      */
-    protected function _get (string $key) {
+    protected function _get(string $key) {
         return $this->psr->get('asana/' . $key);
     }
 
@@ -61,7 +61,7 @@ 
      * @param mixed $value
      * @throws CacheException
      */
-    protected function _set (string $key, $value): void {
+    protected function _set(string $key, $value): void {
         $this->psr->set('asana/' . $key, $value, $this->ttl);
     }
 
@@ -70,7 +70,7 @@ 
      * @return bool
      * @throws CacheException
      */
-    public function add (AbstractEntity $entity): bool {
+    public function add(AbstractEntity $entity): bool {
         if (!$entity->isDiff() and $gid = $entity->getGid()) {
             $this->_set($gid, $entity);
             return parent::add($entity);
@@ -83,7 +83,7 @@ 
      * @param string[] $keys
      * @throws CacheException
      */
-    protected function addKeys (string $gid, array $keys): void {
+    protected function addKeys(string $gid, array $keys): void {
         parent::addKeys($gid, $keys);
         // stash gid refs
         foreach ($keys as $key) {
@@ -100,7 +100,7 @@ 
      * @return null|AbstractEntity
      * @throws CacheException
      */
-    public function get (string $key, $caller, Closure $factory) {
+    public function get(string $key, $caller, Closure $factory) {
         // POOL MISS && CACHE HIT
         if (!isset($this->gids[$key]) and $entity = $this->_get($key)) {
             if ($entity instanceof AbstractEntity) {
@@ -125,7 +125,7 @@ 
     /**
      * @return DateInterval
      */
-    public function getTtl () {
+    public function getTtl() {
         return $this->ttl;
     }
 
@@ -133,7 +133,7 @@ 
      * @param array $keys
      * @throws CacheException
      */
-    public function remove (array $keys): void {
+    public function remove(array $keys): void {
         parent::remove($keys);
         foreach ($keys as $key) {
             $this->_delete($key);
@@ -144,7 +144,7 @@ 
      * @param DateInterval $ttl
      * @return $this
      */
-    public function setTtl (DateInterval $ttl) {
+    public function setTtl(DateInterval $ttl) {
         $this->ttl = $ttl;
         return $this;
     }

src/Base/AbstractEntity.php

Spacing +15 added lines, -15 removed lines

@@ -21,7 +21,7 @@ 
      *
      * @return string
      */
-    abstract public function __toString (): string;
+    abstract public function __toString(): string;
 
     /**
      * Maps lazy-loaded / reloadable mapped {@see Data} to their proper expanded field expression.
@@ -42,7 +42,7 @@ 
      * @return $this
      * @internal
      */
-    public function __set ($unused, self $entity) {
+    public function __set($unused, self $entity) {
         if ($entity !== $this) {
             $this->data = array_merge($this->data, array_diff_key($entity->data, $this->diff));
         }
@@ -57,7 +57,7 @@ 
      * @return $this
      * @internal
      */
-    protected function _addWithPost (string $addPath, array $data, string $field, array $diff) {
+    protected function _addWithPost(string $addPath, array $data, string $field, array $diff) {
         if ($this->hasGid()) {
             return $this->_setWithPost($addPath, $data, $field);
         }
@@ -69,7 +69,7 @@ 
      *
      * @internal
      */
-    protected function _cache (): void {
+    protected function _cache(): void {
         $this->api->getCache()->add($this);
     }
 
@@ -78,7 +78,7 @@ 
      *
      * @internal
      */
-    protected function _delete (): void {
+    protected function _delete(): void {
         $this->api->delete($this);
         $this->api->getCache()->remove($this->getCacheKeys());
     }
@@ -89,7 +89,7 @@ 
      * @param string $field
      * @return mixed
      */
-    protected function _get (string $field) {
+    protected function _get(string $field) {
         if (!array_key_exists($field, $this->data) and $this->hasGid()) {
             $this->reload($field);
         }
@@ -104,7 +104,7 @@ 
      * @return $this
      * @internal
      */
-    protected function _removeWithPost (string $rmPath, array $data, string $field, $diff) {
+    protected function _removeWithPost(string $rmPath, array $data, string $field, $diff) {
         if ($this->hasGid()) {
             return $this->_setWithPost($rmPath, $data, $field);
         }
@@ -121,7 +121,7 @@ 
      * @return $this
      * @internal
      */
-    protected function _save (string $dir = null) {
+    protected function _save(string $dir = null) {
         if (isset($dir)) {
             $remote = $this->api->post($dir, $this->getDiff(), ['expand' => 'this']);
         }
@@ -137,7 +137,7 @@ 
         return $this;
     }
 
-    protected function _setData (array $data): void {
+    protected function _setData(array $data): void {
         // make sure gid is consistently null|string across all entities.
         $data['gid'] = empty($data['gid']) ? null : (string)$data['gid'];
 
@@ -157,7 +157,7 @@ 
      * @return $this
      * @internal
      */
-    protected function _setWithPost (string $path, array $data, string $field, $value = null) {
+    protected function _setWithPost(string $path, array $data, string $field, $value = null) {
         if ($this->hasGid()) {
             /** @var array $remote */
             $remote = $this->api->post($path, $data, ['fields' => static::$optFields[$field] ?? $field]);
@@ -175,14 +175,14 @@ 
      *
      * @return string[]
      */
-    public function getCacheKeys () {
+    public function getCacheKeys() {
         return [$this->getGid(), (string)$this];
     }
 
     /**
      * @return null|string
      */
-    final public function getGid (): ?string {
+    final public function getGid(): ?string {
         return $this->data['gid'] ?? null;
     }
 
@@ -191,14 +191,14 @@ 
      *
      * @return string
      */
-    final public function getResourceType (): string {
+    final public function getResourceType(): string {
         return $this::TYPE;
     }
 
     /**
      * @return bool
      */
-    final public function hasGid (): bool {
+    final public function hasGid(): bool {
         return isset($this->data['gid']);
     }
 
@@ -209,7 +209,7 @@ 
      * @param string $field
      * @return $this
      */
-    public function reload (string $field = null) {
+    public function reload(string $field = null) {
         if (!$this->hasGid()) {
             throw new LogicException(static::class . " has no GID, it can't be reloaded.");
         }

Braces +5 added lines, -10 removed lines

@@ -107,8 +107,7 @@ 
     protected function _removeWithPost (string $rmPath, array $data, string $field, $diff) {
         if ($this->hasGid()) {
             return $this->_setWithPost($rmPath, $data, $field);
-        }
-        elseif (is_array($diff)) {
+        } elseif (is_array($diff)) {
             return $this->_set($field, array_values(array_diff($this->data[$field] ?? [], $diff)));
         }
         return $this->_set($field, array_values(array_filter($this->data[$field] ?? [], $diff)));
@@ -124,11 +123,9 @@ 
     protected function _save (string $dir = null) {
         if (isset($dir)) {
             $remote = $this->api->post($dir, $this->getDiff(), ['expand' => 'this']);
-        }
-        elseif ($this->isDiff()) {
+        } elseif ($this->isDiff()) {
             $remote = $this->api->put($this, $this->getDiff(), ['expand' => 'this']);
-        }
-        else {
+        } else {
             return $this;
         }
         /** @var array $remote */
@@ -217,11 +214,9 @@ 
             $optField = static::$optFields[$field] ?? $field;
             $value = $this->api->get($this, [], ['fields' => $optField])[$field] ?? null;
             $this->_setMapped($field, $value);
-        }
-        elseif ($remote = $this->api->get($this, [], ['expand' => 'this'])) {
+        } elseif ($remote = $this->api->get($this, [], ['expand' => 'this'])) {
             $this->_setData($remote);
-        }
-        else { // deleted upstream.
+        } else { // deleted upstream.
             $this->api->getCache()->remove($this->getCacheKeys());
             throw new RuntimeException("{$this} was deleted upstream.");
         }

src/Api.php

Spacing +42 added lines, -42 removed lines

@@ -53,25 +53,25 @@ 
     /**
      * @return Cache
      */
-    public static function getCache () {
+    public static function getCache() {
         return static::$cache ?? static::$cache = new Cache();
     }
 
     /**
      * @return Api
      */
-    public static function getDefault () {
+    public static function getDefault() {
         return self::$default;
     }
 
     /**
      * @return LoggerInterface
      */
-    public static function getLogger () {
+    public static function getLogger() {
         return static::$logger
             ?? static::$logger = new class implements LoggerInterface {
 
-                public function log (string $message, string $path, array $data = null): void {
+                public function log(string $message, string $path, array $data = null): void {
                     // stub
                 }
 
@@ -81,28 +81,28 @@ 
     /**
      * @param Cache $cache
      */
-    public static function setCache (Cache $cache) {
+    public static function setCache(Cache $cache) {
         static::$cache = $cache;
     }
 
     /**
      * @param Api $default
      */
-    public static function setDefault (Api $default) {
+    public static function setDefault(Api $default) {
         self::$default = $default;
     }
 
     /**
      * @param LoggerInterface $logger
      */
-    public static function setLogger (LoggerInterface $logger) {
+    public static function setLogger(LoggerInterface $logger) {
         static::$logger = $logger;
     }
 
     /**
      * @param string $token
      */
-    public function __construct (string $token) {
+    public function __construct(string $token) {
         $this->token = $token;
         if (!static::$default) {
             static::$default = $this;
@@ -118,7 +118,7 @@ 
      * @return null|array
      * @internal
      */
-    protected function _exec (string $method, string $path, array $opts = []) {
+    protected function _exec(string $method, string $path, array $opts = []) {
         static::getLogger()->log($method, $path, $opts);
         /** @var resource $ch */
         $ch = curl_init();
@@ -167,7 +167,7 @@ 
      * @return string
      * @internal
      */
-    protected function _getPath (string $path, array $query): string {
+    protected function _getPath(string $path, array $query): string {
         return $query ? $path . '?' . http_build_query($query) : $path;
     }
 
@@ -176,8 +176,8 @@ 
      * @return null|array
      * @internal
      */
-    protected function _jsonDecode (string $json) {
-        return json_decode($json, true, JSON_BIGINT_AS_STRING | JSON_THROW_ON_ERROR);
+    protected function _jsonDecode(string $json) {
+        return json_decode($json, true, JSON_BIGINT_AS_STRING|JSON_THROW_ON_ERROR);
     }
 
     /**
@@ -185,8 +185,8 @@ 
      * @return string
      * @internal
      */
-    protected function _jsonEncode (array $data): string {
-        return json_encode($data, JSON_PRETTY_PRINT | JSON_THROW_ON_ERROR);
+    protected function _jsonEncode(array $data): string {
+        return json_encode($data, JSON_PRETTY_PRINT|JSON_THROW_ON_ERROR);
     }
 
     /**
@@ -194,7 +194,7 @@ 
      *
      * @param string $path
      */
-    public function delete (string $path): void {
+    public function delete(string $path): void {
         $this->_exec('DELETE', $path);
     }
 
@@ -208,7 +208,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);
     }
 
@@ -220,7 +220,7 @@ 
      * @param array $options
      * @return null|array
      */
-    public function get (string $path, array $query = [], array $options = []) {
+    public function get(string $path, array $query = [], array $options = []) {
         foreach ($options as $name => $value) {
             $query["opt_{$name}"] = $value;
         }
@@ -234,7 +234,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}");
     }
 
@@ -244,7 +244,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}");
     }
 
@@ -254,7 +254,7 @@ 
      *
      * @return Workspace
      */
-    public function getDefaultWorkspace () {
+    public function getDefaultWorkspace() {
         return $this->getMe()->getDefaultWorkspace();
     }
 
@@ -263,7 +263,7 @@ 
      *
      * @return User
      */
-    public function getMe () {
+    public function getMe() {
         return $this->getUser('me');
     }
 
@@ -273,7 +273,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}");
     }
 
@@ -283,7 +283,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}");
     }
 
@@ -293,7 +293,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}");
     }
 
@@ -303,7 +303,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}");
     }
 
@@ -313,7 +313,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}");
     }
 
@@ -323,7 +323,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}");
     }
 
@@ -333,7 +333,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}");
     }
 
@@ -343,7 +343,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}");
     }
 
@@ -353,7 +353,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}");
     }
 
@@ -361,7 +361,7 @@ 
      * @param string $gid
      * @return ProjectWebhook|TaskWebhook
      */
-    public function getWebhook (string $gid) {
+    public function getWebhook(string $gid) {
         return $this->getCache()->get($gid, $this, function() use ($gid) {
             static $classes = [
                 Project::TYPE => ProjectWebhook::class,
@@ -382,7 +382,7 @@ 
      * @param array $data
      * @return Event
      */
-    public function getWebhookEvent (array $data) {
+    public function getWebhookEvent(array $data) {
         return $this->factory($this, Event::class, $data);
     }
 
@@ -392,7 +392,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}");
     }
 
@@ -402,7 +402,7 @@ 
      * @param string $name
      * @return null|Workspace
      */
-    public function getWorkspaceByName (string $name) {
+    public function getWorkspaceByName(string $name) {
         foreach ($this->getWorkspaces() as $workspace) {
             if ($workspace->getName() === $name) {
                 return $workspace;
@@ -416,7 +416,7 @@ 
      *
      * @return Workspace[]
      */
-    public function getWorkspaces () {
+    public function getWorkspaces() {
         return $this->getMe()->getWorkspaces();
     }
 
@@ -431,7 +431,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 = $this->_getPath($path, $query);
         return $this->getCache()->get($key, $caller, function($caller) use ($class, $path, $query) {
             $data = $this->get($path, $query, ['expand' => 'this']);
@@ -449,7 +449,7 @@ 
      * @param int $pages
      * @return array|AbstractEntity[]
      */
-    public function loadAll ($caller, string $class, string $path, array $query = [], int $pages = 0) {
+    public function loadAll($caller, string $class, string $path, array $query = [], int $pages = 0) {
         return iterator_to_array($this->loadEach(...func_get_args()));
     }
 
@@ -465,7 +465,7 @@ 
      * @param int $pages If positive, stops after this many pages have been fetched.
      * @return Generator|AbstractEntity[]
      */
-    public function loadEach ($caller, string $class, string $path, array $query = [], int $pages = 0) {
+    public function loadEach($caller, string $class, string $path, array $query = [], int $pages = 0) {
         $hardLimit = isset($query['limit']);
         $query['opt_expand'] = 'this';
         $query += ['limit' => 100];
@@ -490,7 +490,7 @@ 
      * @param array $options
      * @return null|array
      */
-    public function post (string $path, array $data = [], array $options = []) {
+    public function post(string $path, array $data = [], array $options = []) {
         $response = $this->_exec('POST', $path, [
             CURLOPT_HTTPHEADER => ['Content-Type: application/json'],
             CURLOPT_POSTFIELDS => $this->_jsonEncode(['options' => $options, 'data' => $data])
@@ -506,7 +506,7 @@ 
      * @param array $options
      * @return null|array
      */
-    public function put (string $path, array $data = [], array $options = []) {
+    public function put(string $path, array $data = [], array $options = []) {
         $response = $this->_exec('PUT', $path, [
             CURLOPT_HTTPHEADER => ['Content-Type: application/json'],
             CURLOPT_POSTFIELDS => $this->_jsonEncode(['options' => $options, 'data' => $data])
@@ -525,7 +525,7 @@ 
      * @param null|string $token
      * @return Event[]
      */
-    public function sync (string $gid, ?string &$token) {
+    public function sync(string $gid, ?string &$token) {
         try {
             $response = $this->_exec('GET', $this->_getPath('events', [
                 'resource' => $gid,
@@ -559,7 +559,7 @@ 
      * @param string $to
      * @return null|array
      */
-    public function upload (string $file, string $to) {
+    public function upload(string $file, string $to) {
         $response = $this->_exec('POST', $to, [
             CURLOPT_POSTFIELDS => ['file' => new CURLFile(realpath($file))] // multipart/form-data
         ]);

src/Event/Change.php

Spacing +2 added lines, -2 removed lines

@@ -30,7 +30,7 @@ 
         'new_value*' => Event::GRAPH
     ];
 
-    protected function _setMapped (string $field, $value): void {
+    protected function _setMapped(string $field, $value): void {
         if (!is_array($value)) {
             $this->data[$field] = $value;
         }
@@ -42,7 +42,7 @@ 
     /**
      * @return null|number|string|User|Project|Section|Task|FieldEntry|Attachment|Story|Like
      */
-    public function getValue () {
+    public function getValue() {
         $key = [
             'added' => 'added_value',
             'removed' => 'removed_value',

Braces +1 added lines, -2 removed lines

@@ -33,8 +33,7 @@
     protected function _setMapped (string $field, $value): void {
         if (!is_array($value)) {
             $this->data[$field] = $value;
-        }
-        else {
+        } else {
             parent::_setMapped($field, $value);
         }
     }

src/Task.php

Spacing +47 added lines, -47 removed lines

@@ -98,15 +98,15 @@ 
         'memberships' => 'memberships.(project|section)'
     ];
 
-    final public function __toString (): string {
+    final public function __toString(): string {
         return "tasks/{$this->getGid()}";
     }
 
-    final protected function _getDir (): string {
+    final protected function _getDir(): string {
         return 'tasks';
     }
 
-    protected function _save (string $dir = null) {
+    protected function _save(string $dir = null) {
         parent::_save($dir);
         // use isset() to avoid has() fetch.
         if (isset($this->data['custom_fields'])) {
@@ -118,7 +118,7 @@ 
         return $this;
     }
 
-    protected function _setData (array $data): void {
+    protected function _setData(array $data): void {
         // hearts were deprecated for likes
         unset($data['hearted'], $data['hearts'], $data['num_hearts']);
 
@@ -138,7 +138,7 @@ 
      * @param string $file
      * @return Attachment
      */
-    public function addAttachment (string $file) {
+    public function addAttachment(string $file) {
         /** @var Attachment $attachment */
         $attachment = $this->api->factory($this, Attachment::class, ['parent' => $this]);
         return $attachment->upload($file);
@@ -150,7 +150,7 @@ 
      * @param string $text
      * @return Story
      */
-    public function addComment (string $text) {
+    public function addComment(string $text) {
         return $this->newComment()->setText($text)->create();
     }
 
@@ -161,7 +161,7 @@ 
      * @param Task[] $tasks
      * @return $this
      */
-    public function addDependencies (array $tasks) {
+    public function addDependencies(array $tasks) {
         $this->api->post("{$this}/addDependencies", ['dependents' => array_column($tasks, 'gid')]);
         return $this;
     }
@@ -173,7 +173,7 @@ 
      * @param Task $task
      * @return $this
      */
-    public function addDependency (Task $task) {
+    public function addDependency(Task $task) {
         return $this->addDependencies([$task]);
     }
 
@@ -184,7 +184,7 @@ 
      * @param Task $task
      * @return $this
      */
-    public function addDependent (Task $task) {
+    public function addDependent(Task $task) {
         return $this->addDependents([$task]);
     }
 
@@ -195,7 +195,7 @@ 
      * @param Task[] $tasks
      * @return $this
      */
-    public function addDependents (array $tasks) {
+    public function addDependents(array $tasks) {
         $this->api->post("{$this}/addDependents", ['dependents' => array_column($tasks, 'gid')]);
         return $this;
     }
@@ -206,7 +206,7 @@ 
      * @param User $user
      * @return $this
      */
-    public function addFollower (User $user) {
+    public function addFollower(User $user) {
         return $this->addFollowers([$user]);
     }
 
@@ -218,7 +218,7 @@ 
      * @param User[] $users
      * @return $this
      */
-    public function addFollowers (array $users) {
+    public function addFollowers(array $users) {
         return $this->_addWithPost("{$this}/addFollowers", [
             'followers' => array_column($users, 'gid')
         ], 'followers', $users);
@@ -232,7 +232,7 @@ 
      * @param Tag $tag
      * @return $this
      */
-    public function addTag (Tag $tag) {
+    public function addTag(Tag $tag) {
         return $this->_addWithPost("{$this}/addTag", [
             'tag' => $tag->getGid()
         ], 'tags', [$tag]);
@@ -247,7 +247,7 @@ 
      * @param Section|null $section
      * @return $this
      */
-    public function addToProject (Project $project, Section $section = null) {
+    public function addToProject(Project $project, Section $section = null) {
         /** @var Membership $membership */
         $membership = $this->api->factory($this, Membership::class)
             ->_set('project', $project)
@@ -262,7 +262,7 @@ 
      * @param string $target
      * @return TaskWebhook
      */
-    public function addWebhook (string $target) {
+    public function addWebhook(string $target) {
         /** @var TaskWebhook $webhook */
         $webhook = $this->api->factory($this, TaskWebhook::class);
         return $webhook->create($this, $target);
@@ -278,7 +278,7 @@ 
      * @param string[] $include
      * @return Job
      */
-    public function duplicate (string $name, array $include) {
+    public function duplicate(string $name, array $include) {
         /** @var array $remote */
         $remote = $this->api->post("{$this}/duplicate", [
             'name' => $name,
@@ -293,7 +293,7 @@ 
      * @depends after-create
      * @return Attachment[]
      */
-    public function getAttachments () {
+    public function getAttachments() {
         return $this->api->loadAll($this, Attachment::class, "{$this}/attachments");
     }
 
@@ -303,7 +303,7 @@ 
      * @depends after-create
      * @return Story[]
      */
-    public function getComments () {
+    public function getComments() {
         return $this->selectStories(function(Story $story) {
             return $story->isComment();
         });
@@ -315,7 +315,7 @@ 
      * @depends after-create
      * @return Task[]
      */
-    public function getDependencies () {
+    public function getDependencies() {
         return $this->api->loadAll($this, self::class, "{$this}/dependencies");
     }
 
@@ -325,7 +325,7 @@ 
      * @depends after-create
      * @return Task[]
      */
-    public function getDependents () {
+    public function getDependents() {
         return $this->api->loadAll($this, self::class, "{$this}/dependents");
     }
 
@@ -336,7 +336,7 @@ 
      * @param null|string $token
      * @return Event[]
      */
-    public function getEvents (&$token) {
+    public function getEvents(&$token) {
         return $this->api->sync($this->getGid(), $token);
     }
 
@@ -348,14 +348,14 @@ 
      *
      * @return External
      */
-    public function getExternal () {
+    public function getExternal() {
         return $this->_get('external') ?? $this->data['external'] = $this->api->factory($this, External::class);
     }
 
     /**
      * @return Project[]
      */
-    public function getProjects () {
+    public function getProjects() {
         return array_map(function(Membership $membership) {
             return $membership->getProject();
         }, $this->getMemberships());
@@ -367,7 +367,7 @@ 
      * @depends after-create
      * @return Story[]
      */
-    public function getStories () {
+    public function getStories() {
         return $this->api->loadAll($this, Story::class, "{$this}/stories");
     }
 
@@ -377,7 +377,7 @@ 
      * @depends after-create
      * @return Task[]
      */
-    public function getSubTasks () {
+    public function getSubTasks() {
         return $this->api->loadAll($this, self::class, "{$this}/subtasks");
     }
 
@@ -387,7 +387,7 @@ 
      * @depends after-create
      * @return string
      */
-    public function getUrl (): string {
+    public function getUrl(): string {
         return "https://app.asana.com/0/0/{$this->getGid()}";
     }
 
@@ -397,7 +397,7 @@ 
      * @depends after-create
      * @return TaskWebhook[]
      */
-    public function getWebhooks () {
+    public function getWebhooks() {
         return $this->api->loadAll($this, TaskWebhook::class, 'webhooks', [
             'workspace' => $this->getWorkspace()->getGid(),
             'resource' => $this->getGid()
@@ -407,7 +407,7 @@ 
     /**
      * @return bool
      */
-    public function isRenderedAsSeparator (): bool {
+    public function isRenderedAsSeparator(): bool {
         return $this->_is('is_rendered_as_separator');
     }
 
@@ -417,7 +417,7 @@ 
      * @depends after-create
      * @return Story
      */
-    public function newComment () {
+    public function newComment() {
         return $this->api->factory($this, Story::class, [
             'resource_subtype' => Story::TYPE_COMMENT_ADDED,
             'target' => $this
@@ -430,7 +430,7 @@ 
      * @depends after-create
      * @return Task
      */
-    public function newSubTask () {
+    public function newSubTask() {
         /** @var Task $sub */
         $sub = $this->api->factory($this, self::class);
         return $sub->setParent($this);
@@ -443,7 +443,7 @@ 
      * @param Task[] $tasks
      * @return $this
      */
-    public function removeDependencies (array $tasks) {
+    public function removeDependencies(array $tasks) {
         $this->api->post("{$this}/removeDependencies", ['dependencies' => array_column($tasks, 'gid')]);
         return $this;
     }
@@ -455,7 +455,7 @@ 
      * @param Task $task
      * @return $this
      */
-    public function removeDependency (Task $task) {
+    public function removeDependency(Task $task) {
         return $this->removeDependencies([$task]);
     }
 
@@ -466,7 +466,7 @@ 
      * @param Task $task
      * @return $this
      */
-    public function removeDependent (Task $task) {
+    public function removeDependent(Task $task) {
         return $this->removeDependents([$task]);
     }
 
@@ -477,7 +477,7 @@ 
      * @param Task[] $tasks
      * @return $this
      */
-    public function removeDependents (array $tasks) {
+    public function removeDependents(array $tasks) {
         $this->api->post("{$this}/removeDependents", ['dependents' => array_column($tasks, 'gid')]);
         return $this;
     }
@@ -488,7 +488,7 @@ 
      * @param User $user
      * @return $this
      */
-    public function removeFollower (User $user) {
+    public function removeFollower(User $user) {
         return $this->removeFollowers([$user]);
     }
 
@@ -500,7 +500,7 @@ 
      * @param User[] $users
      * @return $this
      */
-    public function removeFollowers (array $users) {
+    public function removeFollowers(array $users) {
         return $this->_removeWithPost("{$this}/removeFollowers", [
             'followers' => array_column($users, 'gid')
         ], 'followers', $users);
@@ -514,7 +514,7 @@ 
      * @param Project $project
      * @return $this
      */
-    public function removeFromProject (Project $project) {
+    public function removeFromProject(Project $project) {
         return $this->_removeWithPost("{$this}/removeProject", [
             'project' => $project->getGid()
         ], 'memberships', function(Membership $membership) use ($project) {
@@ -530,7 +530,7 @@ 
      * @param Tag $tag
      * @return $this
      */
-    public function removeTag (Tag $tag) {
+    public function removeTag(Tag $tag) {
         return $this->_removeWithPost("{$this}/removeTag", [
             'tag' => $tag->getGid()
         ], 'tags', [$tag]);
@@ -540,7 +540,7 @@ 
      * @param callable $filter `fn( Attachment $attachment): bool`
      * @return Attachment[]
      */
-    public function selectAttachments (callable $filter) {
+    public function selectAttachments(callable $filter) {
         return $this->_select($this->getAttachments(), $filter);
     }
 
@@ -548,7 +548,7 @@ 
      * @param callable $filter `fn( Story $comment ): bool`
      * @return Story[]
      */
-    public function selectComments (callable $filter) {
+    public function selectComments(callable $filter) {
         return $this->_select($this->getComments(), $filter);
     }
 
@@ -556,7 +556,7 @@ 
      * @param callable $filter `fn( Task $dependency ): bool`
      * @return Task[]
      */
-    public function selectDependencies (callable $filter) {
+    public function selectDependencies(callable $filter) {
         return $this->_select($this->getDependencies(), $filter);
     }
 
@@ -564,7 +564,7 @@ 
      * @param callable $filter `fn( Task $dependent ): bool`
      * @return Task[]
      */
-    public function selectDependents (callable $filter) {
+    public function selectDependents(callable $filter) {
         return $this->_select($this->getDependents(), $filter);
     }
 
@@ -572,7 +572,7 @@ 
      * @param callable $filter `fn( Project $project ): bool`
      * @return Project[]
      */
-    public function selectProjects (callable $filter) {
+    public function selectProjects(callable $filter) {
         return $this->_select($this->getProjects(), $filter);
     }
 
@@ -580,7 +580,7 @@ 
      * @param callable $filter `fn( Story $story ): bool`
      * @return Story[]
      */
-    public function selectStories (callable $filter) {
+    public function selectStories(callable $filter) {
         return $this->_select($this->getStories(), $filter);
     }
 
@@ -588,7 +588,7 @@ 
      * @param callable $filter `fn( Task $subtask ): bool`
      * @return Task[]
      */
-    public function selectSubTasks (callable $filter) {
+    public function selectSubTasks(callable $filter) {
         return $this->_select($this->getSubTasks(), $filter);
     }
 
@@ -599,7 +599,7 @@ 
      * @param null|Task $parent
      * @return $this
      */
-    public function setParent (?self $parent) {
+    public function setParent(?self $parent) {
         return $this->_setWithPost("{$this}/setParent", [
             'parent' => $parent
         ], 'parent', $parent);
@@ -609,7 +609,7 @@ 
      * @param bool $flag
      * @return $this
      */
-    public function setRenderedAsSeparator (bool $flag) {
+    public function setRenderedAsSeparator(bool $flag) {
         return $this->_set('is_rendered_as_separator', $flag);
     }