hfw / asana

src/CustomField/FieldSettingsTrait.php

Spacing +2 added lines, -2 removed lines

@@ -15,7 +15,7 @@ 
     /**
      * @return CustomField[]
      */
-    public function getCustomFields () {
+    public function getCustomFields() {
         return array_map(function(FieldSetting $setting) {
             return $setting->getCustomField();
         }, $this->getCustomFieldSettings());
@@ -25,7 +25,7 @@ 
      * @param callable $filter `fn( CustomField $field ): bool`
      * @return CustomField[]
      */
-    public function selectCustomFields (callable $filter) {
+    public function selectCustomFields(callable $filter) {
         return $this->{'_select'}($this->getCustomFields(), $filter);
     }
 }
\ No newline at end of file

src/Workspace/OrganizationExport.php

Spacing +9 added lines, -9 removed lines

@@ -33,11 +33,11 @@ 
         'organization' => Workspace::class
     ];
 
-    final public function __toString (): string {
+    final public function __toString(): string {
         return "organization_exports/{$this->getGid()}";
     }
 
-    final protected function _getDir (): string {
+    final protected function _getDir(): string {
         return "organization_exports";
     }
 
@@ -45,7 +45,7 @@ 
      * @param Workspace $organization
      * @return $this
      */
-    public function create (Workspace $organization) {
+    public function create(Workspace $organization) {
         $this->_set('organization', $organization);
         return $this->_create();
     }
@@ -55,7 +55,7 @@ 
      *
      * @return bool
      */
-    final public function isActive (): bool {
+    final public function isActive(): bool {
         return $this->getState() === self::STATE_ACTIVE;
     }
 
@@ -64,7 +64,7 @@ 
      *
      * @return bool
      */
-    final public function isDone (): bool {
+    final public function isDone(): bool {
         return $this->isSuccessful() or $this->isFailed();
     }
 
@@ -73,7 +73,7 @@ 
      *
      * @return bool
      */
-    final public function isFailed (): bool {
+    final public function isFailed(): bool {
         return $this->getState() === self::STATE_FAIL;
     }
 
@@ -82,7 +82,7 @@ 
      *
      * @return bool
      */
-    final public function isQueued (): bool {
+    final public function isQueued(): bool {
         return $this->getState() === self::STATE_QUEUED;
     }
 
@@ -91,7 +91,7 @@ 
      *
      * @return bool
      */
-    final public function isSuccessful (): bool {
+    final public function isSuccessful(): bool {
         return $this->getState() === self::STATE_SUCCESS;
     }
 
@@ -103,7 +103,7 @@ 
      * @param callable $spinner `fn( OrganizationExport $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);

src/Job.php

Spacing +9 added lines, -9 removed lines

@@ -33,7 +33,7 @@ 
         'new_task' => Task::class
     ];
 
-    final public function __toString (): string {
+    final public function __toString(): string {
         return "jobs/{$this->getGid()}";
     }
 
@@ -42,7 +42,7 @@ 
      *
      * @return bool
      */
-    public function isActive (): bool {
+    public function isActive(): bool {
         return $this->getStatus() === self::STATUS_ACTIVE;
     }
 
@@ -51,15 +51,15 @@ 
      *
      * @return bool
      */
-    public function isDone (): bool {
+    public function isDone(): bool {
         return $this->isSuccessful() or $this->isFailed();
     }
 
-    final public function isDuplicatingProject (): bool {
+    final public function isDuplicatingProject(): bool {
         return $this->getResourceSubtype() === self::TYPE_DUPLICATE_PROJECT;
     }
 
-    final public function isDuplicatingTask (): bool {
+    final public function isDuplicatingTask(): bool {
         return $this->getResourceSubtype() === self::TYPE_DUPLICATE_TASK;
     }
 
@@ -68,7 +68,7 @@ 
      *
      * @return bool
      */
-    public function isFailed (): bool {
+    public function isFailed(): bool {
         return $this->getStatus() === self::STATUS_FAIL;
     }
 
@@ -77,7 +77,7 @@ 
      *
      * @return bool
      */
-    public function isQueued (): bool {
+    public function isQueued(): bool {
         return $this->getStatus() === self::STATUS_QUEUED;
     }
 
@@ -86,7 +86,7 @@ 
      *
      * @return bool
      */
-    public function isSuccessful (): bool {
+    public function isSuccessful(): bool {
         return $this->getStatus() === self::STATUS_SUCCESS;
     }
 
@@ -98,7 +98,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);

src/Project/Status.php

Spacing +5 added lines, -5 removed lines

@@ -50,20 +50,20 @@ 
      * @param Project $project
      * @param array $data
      */
-    public function __construct (Project $project, array $data = []) {
+    public function __construct(Project $project, array $data = []) {
         parent::__construct($project, $data);
         $this->project = $project;
     }
 
-    final public function __toString (): string {
+    final public function __toString(): string {
         return "project_statuses/{$this->getGid()}";
     }
 
-    final protected function _getDir (): string {
+    final protected function _getDir(): string {
         return "{$this->project}/project_statuses";
     }
 
-    protected function _setData (array $data): void {
+    protected function _setData(array $data): void {
         // redundant, prefer created_by
         unset($data['author']);
         parent::_setData($data);
@@ -72,7 +72,7 @@ 
     /**
      * @return Project
      */
-    public function getProject () {
+    public function getProject() {
         return $this->project;
     }
 }
\ No newline at end of file

src/Tag.php

Spacing +4 added lines, -4 removed lines

@@ -33,25 +33,25 @@
         'workspace' => Workspace::class
     ];
 
-    final public function __toString (): string {
+    final public function __toString(): string {
         return "tags/{$this->getGid()}";
     }
 
-    final protected function _getDir (): string {
+    final protected function _getDir(): string {
         return 'tags';
     }
 
     /**
      * @return Traversable|Task[]
      */
-    public function getIterator () {
+    public function getIterator() {
         return $this->_loadEach(Task::class, "{$this}/tasks");
     }
 
     /**
      * @return Task[]
      */
-    public function getTasks () {
+    public function getTasks() {
         return iterator_to_array($this);
     }
 }
\ No newline at end of file

src/Base/Data.php

Spacing +24 added lines, -24 removed lines

@@ -48,7 +48,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);
     }
@@ -64,9 +64,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|select|set)(.+)$/', $method, $call);
             $call[1] = '_' . $call[1];
             $call[2] = preg_replace_callback('/[A-Z]/', function(array $match) {
@@ -76,7 +76,7 @@ 
         return $this->{$call[1]}($call[2], ...$args);
     }
 
-    public function __debugInfo (): array {
+    public function __debugInfo(): array {
         return $this->data;
     }
 
@@ -86,7 +86,7 @@ 
      * @param $field
      * @return null|Data|mixed
      */
-    final public function __get ($field) {
+    final public function __get($field) {
         return $this->_get($field);
     }
 
@@ -98,7 +98,7 @@ 
      * @param $field
      * @return bool
      */
-    final public function __isset ($field) {
+    final public function __isset($field) {
         return true;
     }
 
@@ -109,7 +109,7 @@ 
      * @param array $data
      * @return mixed|Data|AbstractEntity
      */
-    final protected function _factory (string $class, array $data = []) {
+    final protected function _factory(string $class, array $data = []) {
         return $this->api->factory($class, $this, $data);
     }
 
@@ -121,7 +121,7 @@ 
      * @param string $field
      * @return mixed
      */
-    protected function _get (string $field) {
+    protected function _get(string $field) {
         return $this->data[$field] ?? null;
     }
 
@@ -136,7 +136,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)) {
@@ -159,7 +159,7 @@ 
      * @param string $field
      * @return bool
      */
-    protected function _is (string $field): bool {
+    protected function _is(string $field): bool {
         return (bool)$this->_get($field);
     }
 
@@ -173,7 +173,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);
     }
 
@@ -188,7 +188,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);
     }
 
@@ -203,7 +203,7 @@ 
      * @param int $pages
      * @return Traversable|AbstractEntity[]
      */
-    final protected function _loadEach (string $class, string $path, array $query = [], int $pages = 0) {
+    final protected function _loadEach(string $class, string $path, array $query = [], int $pages = 0) {
         return $this->api->loadEach($class, $this, $path, $query, $pages);
     }
 
@@ -218,7 +218,7 @@ 
      * @param callable $filter `fn( Data $object ): bool`
      * @return array
      */
-    protected function _select ($subject, callable $filter) {
+    protected function _select($subject, callable $filter) {
         if (!is_iterable($subject)) {
             $subject = $this->_get($subject) ?? [];
         }
@@ -240,7 +240,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;
@@ -251,7 +251,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->_setMapped($field, $value);
@@ -264,7 +264,7 @@ 
      * @param string $field
      * @param mixed $value
      */
-    protected function _setMapped (string $field, $value): void {
+    protected function _setMapped(string $field, $value): void {
         unset($this->diff[$field]);
 
         // use value as-is?
@@ -310,7 +310,7 @@ 
      *
      * @return array
      */
-    public function getDiff (): array {
+    public function getDiff(): array {
         $convert = function($each) use (&$convert) {
             // convert existing entities to gids
             if ($each instanceof AbstractEntity and $each->hasGid()) {
@@ -336,7 +336,7 @@ 
      * @param string null $field
      * @return bool
      */
-    final public function isDiff (string $field = null): bool {
+    final public function isDiff(string $field = null): bool {
         if ($field) {
             return isset($this->diff[$field]);
         }
@@ -347,7 +347,7 @@ 
      * @see toArray()
      * @return array
      */
-    public function jsonSerialize (): array {
+    public function jsonSerialize(): array {
         $data = $this->toArray();
         ksort($data);
         return $data;
@@ -358,8 +358,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);
     }
 
     /**
@@ -367,7 +367,7 @@ 
      *
      * @return array
      */
-    public function toArray (): array {
+    public function toArray(): array {
         return array_map($dehydrate = function($each) use (&$dehydrate) {
             if ($each instanceof AbstractEntity and $each->hasGid()) {
                 return $each->getGid();
@@ -387,7 +387,7 @@ 
      *
      * @param $serialized
      */
-    public function unserialize ($serialized): void {
+    public function unserialize($serialized): void {
         $this->api = Api::getDefault();
         $data = json_decode($serialized, true);
         $this->_setData($data);

Braces +3 added lines, -6 removed lines

@@ -315,8 +315,7 @@ 
 
         if ($isList) {
             $this->data[$field] = array_map($hydrate, $value);
-        }
-        else {
+        } else {
             $this->data[$field] = $hydrate($value);
         }
     }
@@ -387,11 +386,9 @@ 
         return array_map($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;

src/Portfolio.php

Spacing +11 added lines, -11 removed lines

@@ -52,11 +52,11 @@ 
         'workspace' => Workspace::class
     ];
 
-    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';
     }
 
@@ -65,7 +65,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;
     }
@@ -74,7 +74,7 @@ 
      * @param User $user
      * @return $this
      */
-    public function addMember (User $user) {
+    public function addMember(User $user) {
         return $this->addMembers([$user]);
     }
 
@@ -82,7 +82,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);
@@ -92,14 +92,14 @@ 
      * @depends after-create
      * @return Project[]
      */
-    public function getItems () {
+    public function getItems() {
         return iterator_to_array($this);
     }
 
     /**
      * @return Traversable|Project[]
      */
-    public function getIterator () {
+    public function getIterator() {
         return $this->_loadEach(Project::class, "{$this}/items");
     }
 
@@ -108,7 +108,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;
     }
@@ -117,7 +117,7 @@ 
      * @param User $user
      * @return $this
      */
-    public function removeMember (User $user) {
+    public function removeMember(User $user) {
         return $this->removeMembers([$user]);
     }
 
@@ -125,7 +125,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);
@@ -135,7 +135,7 @@ 
      * @param callable $filter `fn( Project $project ): bool`
      * @return Project[]
      */
-    public function selectItems (callable $filter) {
+    public function selectItems(callable $filter) {
         return $this->_select($this, $filter);
     }
 }
\ No newline at end of file

src/User.php

Spacing +16 added lines, -16 removed lines

@@ -28,7 +28,7 @@ 
         'workspaces' => [Workspace::class]
     ];
 
-    final public function __toString (): string {
+    final public function __toString(): string {
         return "users/{$this->getGid()}";
     }
 
@@ -36,7 +36,7 @@ 
      * @param Workspace $workspace
      * @return $this
      */
-    public function addToWorkspace (Workspace $workspace) {
+    public function addToWorkspace(Workspace $workspace) {
         return $this->_addWithPost("{$workspace}/addUser", [
             'user' => $this->getGid()
         ], 'workspaces', [$workspace]);
@@ -45,7 +45,7 @@ 
     /**
      * @return string[]
      */
-    public function getCacheKeys () {
+    public function getCacheKeys() {
         $keys = parent::getCacheKeys();
 
         // only include email as a key if it's loaded
@@ -62,7 +62,7 @@ 
      *
      * @return Workspace
      */
-    public function getDefaultWorkspace () {
+    public function getDefaultWorkspace() {
         return $this->getWorkspaces()[0];
     }
 
@@ -70,7 +70,7 @@ 
      * @param null|Workspace $workspace Falls back to the default workspace.
      * @return Portfolio[]
      */
-    public function getFavoritePortfolios (Workspace $workspace = null) {
+    public function getFavoritePortfolios(Workspace $workspace = null) {
         return $this->getFavorites(Portfolio::class, Portfolio::TYPE, $workspace);
     }
 
@@ -78,7 +78,7 @@ 
      * @param null|Workspace $workspace Falls back to the default workspace.
      * @return Project[]
      */
-    public function getFavoriteProjects (Workspace $workspace = null) {
+    public function getFavoriteProjects(Workspace $workspace = null) {
         return $this->getFavorites(Project::class, Project::TYPE, $workspace);
     }
 
@@ -86,7 +86,7 @@ 
      * @param null|Workspace $workspace Falls back to the default workspace.
      * @return Tag[]
      */
-    public function getFavoriteTags (Workspace $workspace = null) {
+    public function getFavoriteTags(Workspace $workspace = null) {
         return $this->getFavorites(Tag::class, Tag::TYPE, $workspace);
     }
 
@@ -94,7 +94,7 @@ 
      * @param null|Workspace $workspace Falls back to the default workspace.
      * @return Team[]
      */
-    public function getFavoriteTeams (Workspace $workspace = null) {
+    public function getFavoriteTeams(Workspace $workspace = null) {
         return $this->getFavorites(Team::class, Team::TYPE, $workspace);
     }
 
@@ -102,7 +102,7 @@ 
      * @param null|Workspace $workspace Falls back to the default workspace.
      * @return User[]
      */
-    public function getFavoriteUsers (Workspace $workspace = null) {
+    public function getFavoriteUsers(Workspace $workspace = null) {
         return $this->getFavorites(self::class, self::TYPE, $workspace);
     }
 
@@ -112,7 +112,7 @@ 
      * @param null|Workspace $workspace Falls back to the default workspace.
      * @return array
      */
-    protected function getFavorites (string $class, string $resourceType, Workspace $workspace = null) {
+    protected function getFavorites(string $class, string $resourceType, Workspace $workspace = null) {
         return $this->_loadAll($class, "{$this}/favorites", [
             'resource_type' => $resourceType,
             'workspace' => ($workspace ?? $this->getDefaultWorkspace())->getGid()
@@ -123,7 +123,7 @@ 
      * @param null|Workspace $workspace
      * @return Portfolio[]
      */
-    public function getPortfolios (Workspace $workspace = null) {
+    public function getPortfolios(Workspace $workspace = null) {
         return $this->_loadAll(Portfolio::class, "portfolios", [
             'workspace' => ($workspace ?? $this->getDefaultWorkspace())->getGid(),
             'owner' => $this->getGid()
@@ -134,7 +134,7 @@ 
      * @param null|Workspace $workspace Falls back to the default workspace.
      * @return TaskList
      */
-    public function getTaskList (Workspace $workspace = null) {
+    public function getTaskList(Workspace $workspace = null) {
         return $this->_load(TaskList::class, "{$this}/user_task_list", [
             'workspace' => ($workspace ?? $this->getDefaultWorkspace())->getGid()
         ]);
@@ -146,7 +146,7 @@ 
      * @param string[] $filter `workspace` falls back to the default.
      * @return Task[]
      */
-    public function getTasks (array $filter = []) {
+    public function getTasks(array $filter = []) {
         $filter['assignee'] = $this->getGid();
         $filter += ['workspace' => $this->getDefaultWorkspace()->getGid()];
         return $this->_loadAll(Task::class, 'tasks', $filter);
@@ -160,13 +160,13 @@ 
      * @param null|Workspace $organization Falls back to the default workspace.
      * @return Team[]
      */
-    public function getTeams (Workspace $organization = null) {
+    public function getTeams(Workspace $organization = null) {
         return $this->_loadAll(Team::class, "{$this}/teams", [
             'organization' => ($organization ?? $this->getDefaultWorkspace())->getGid()
         ]);
     }
 
-    public function getUrl (): string {
+    public function getUrl(): string {
         return "https://app.asana.com/0/{$this->getGid()}/list";
     }
 
@@ -174,7 +174,7 @@ 
      * @param Workspace $workspace
      * @return $this
      */
-    public function removeFromWorkspace (Workspace $workspace) {
+    public function removeFromWorkspace(Workspace $workspace) {
         return $this->_removeWithPost("{$workspace}/removeUser", [
             'user' => $this->getGid()
         ], 'workspaces', [$workspace]);

src/CustomField.php

Spacing +11 added lines, -11 removed lines

@@ -40,11 +40,11 @@ 
         'enum_options' => [EnumOption::class],
     ];
 
-    final public function __toString (): string {
+    final public function __toString(): string {
         return "custom_fields/{$this->getGid()}";
     }
 
-    final protected function _getDir (): string {
+    final protected function _getDir(): string {
         return 'custom_fields';
     }
 
@@ -56,7 +56,7 @@ 
      * @param string $name
      * @return EnumOption
      */
-    public function addEnumOption (string $name) {
+    public function addEnumOption(string $name) {
         /** @var EnumOption $option */
         $option = $this->_factory(EnumOption::class);
         $option->setName($name);
@@ -72,35 +72,35 @@ 
     /**
      * @return bool
      */
-    public function hasNotificationsEnabled (): bool {
+    public function hasNotificationsEnabled(): bool {
         return $this->_is('has_notifications_enabled');
     }
 
     /**
      * @return bool
      */
-    final public function isEnum (): bool {
+    final public function isEnum(): bool {
         return $this->getResourceSubtype() === self::TYPE_ENUM;
     }
 
     /**
      * @return bool
      */
-    public function isGlobalToWorkspace (): bool {
+    public function isGlobalToWorkspace(): bool {
         return $this->_is('is_global_to_workspace');
     }
 
     /**
      * @return bool
      */
-    final public function isNumber (): bool {
+    final public function isNumber(): bool {
         return $this->getResourceSubtype() === self::TYPE_NUMBER;
     }
 
     /**
      * @return bool
      */
-    final public function isText (): bool {
+    final public function isText(): bool {
         return $this->getResourceSubtype() === self::TYPE_TEXT;
     }
 
@@ -108,7 +108,7 @@ 
      * @param bool $flag
      * @return $this
      */
-    public function setGlobalToWorkspace (bool $flag) {
+    public function setGlobalToWorkspace(bool $flag) {
         return $this->_set('is_global_to_workspace', $flag);
     }
 
@@ -116,7 +116,7 @@ 
      * @param bool $flag
      * @return $this
      */
-    public function setNotificationsEnabled (bool $flag) {
+    public function setNotificationsEnabled(bool $flag) {
         return $this->_set('has_notifications_enabled', $flag);
     }
 
@@ -124,7 +124,7 @@ 
      * @param callable $cmp `fn( EnumOption $a, EnumOption $b ): int`
      * @return $this
      */
-    public function sortEnumOptions (callable $cmp) {
+    public function sortEnumOptions(callable $cmp) {
         if ($options = $this->getEnumOptions()) {
             $prev = $options[0]; // first option on remote
             usort($options, $cmp);