hfw / asana

src/Portfolio.php

Spacing +9 added lines, -9 removed lines

@@ -39,11 +39,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';
     }
 
@@ -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' => array_column($users, 'gid')]);
             $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' => array_column($users, 'gid')]);
         }

Braces +1 added lines, -2 removed lines

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

src/Base/AbstractEntity.php

Spacing +6 added lines, -6 removed lines

@@ -16,7 +16,7 @@ 
      *
      * @return string
      */
-    abstract public function __toString (): string;
+    abstract public function __toString(): string;
 
     /**
      * Maps lazy-loaded / reloadable fields to their proper expanded field expression.
@@ -33,7 +33,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);
         }
@@ -48,7 +48,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;
@@ -71,7 +71,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);
@@ -85,7 +85,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])) {
@@ -101,7 +101,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' => static::$optFields[$key] ?? $key])[$key] ?? null;
             $this->_setMapped($key, $value);

src/Task.php

Spacing +41 added lines, -41 removed lines

@@ -95,14 +95,14 @@ 
         'memberships' => 'memberships.(project|section)'
     ];
 
-    final public function __toString (): string {
+    final public function __toString(): string {
         return "tasks/{$this->getGid()}";
     }
 
     /**
      * Clears the diffs for `custom_fields` and `external` sub-objects after saving.
      */
-    protected function _clearSubDiffs (): void {
+    protected function _clearSubDiffs(): void {
         // use isset() to avoid has() fetch.
         if (isset($this->data['custom_fields'])) {
             $this->getCustomFields()->diff = [];
@@ -112,7 +112,7 @@ 
         }
     }
 
-    final protected function _getDir (): string {
+    final protected function _getDir(): string {
         return 'tasks';
     }
 
@@ -123,7 +123,7 @@ 
      * @param string $file
      * @return Attachment
      */
-    public function addAttachment (string $file) {
+    public function addAttachment(string $file) {
         /** @var Attachment $attachment */
         $attachment = $this->factory(Attachment::class, ['parent' => $this]);
         return $attachment->upload($file);
@@ -135,7 +135,7 @@ 
      * @param string $text
      * @return Story
      */
-    public function addComment (string $text) {
+    public function addComment(string $text) {
         return $this->newComment()->setText($text)->create();
     }
 
@@ -146,7 +146,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;
     }
@@ -158,7 +158,7 @@ 
      * @param Task $task
      * @return $this
      */
-    public function addDependency (Task $task) {
+    public function addDependency(Task $task) {
         return $this->addDependencies([$task]);
     }
 
@@ -169,7 +169,7 @@ 
      * @param Task $task
      * @return $this
      */
-    public function addDependent (Task $task) {
+    public function addDependent(Task $task) {
         return $this->addDependents([$task]);
     }
 
@@ -180,7 +180,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;
     }
@@ -191,7 +191,7 @@ 
      * @param User $user
      * @return $this
      */
-    public function addFollower (User $user) {
+    public function addFollower(User $user) {
         return $this->addFollowers([$user]);
     }
 
@@ -201,7 +201,7 @@ 
      * @param User[] $users
      * @return $this
      */
-    public function addFollowers (array $users) {
+    public function addFollowers(array $users) {
         if ($this->hasGid()) {
             $this->api->post("{$this}/addFollowers", ['followers' => array_column($users, 'gid')]);
             $this->_merge('followers', $users);
@@ -218,7 +218,7 @@ 
      * @param Tag $tag
      * @return $this
      */
-    public function addTag (Tag $tag) {
+    public function addTag(Tag $tag) {
         if ($this->hasGid()) {
             $this->api->post("{$this}/addTag", ['tag' => $tag->getGid()]);
             $this->_merge('tags', [$tag]);
@@ -235,7 +235,7 @@ 
      * @param Section $section
      * @return $this
      */
-    public function addToProject (Section $section) {
+    public function addToProject(Section $section) {
         $project = $section->getProject();
         if ($this->hasGid()) {
             $this->api->post("{$this}/addProject", [
@@ -269,7 +269,7 @@ 
      * @param string $target
      * @return TaskWebhook
      */
-    public function addWebhook (string $target) {
+    public function addWebhook(string $target) {
         /** @var TaskWebhook $webhook */
         $webhook = $this->factory(TaskWebhook::class);
         return $webhook->create($this, $target);
@@ -278,7 +278,7 @@ 
     /**
      * @return $this
      */
-    public function create () {
+    public function create() {
         $this->_create();
         $this->_clearSubDiffs();
         return $this;
@@ -294,7 +294,7 @@ 
      * @param string[] $include
      * @return Job
      */
-    public function duplicate (string $name, array $include) {
+    public function duplicate(string $name, array $include) {
         $remote = $this->api->post("{$this}/duplicate", [
             'name' => $name,
             'include' => array_values($include)
@@ -308,7 +308,7 @@ 
      * @depends after-create
      * @return Attachment[]
      */
-    public function getAttachments () {
+    public function getAttachments() {
         return $this->loadAll(Attachment::class, "{$this}/attachments");
     }
 
@@ -318,7 +318,7 @@ 
      * @depends after-create
      * @return Story[]
      */
-    public function getComments () {
+    public function getComments() {
         return array_values(array_filter($this->getStories(), function(Story $story) {
             return $story->isComment();
         }));
@@ -330,7 +330,7 @@ 
      * @depends after-create
      * @return Task[]
      */
-    public function getDependencies () {
+    public function getDependencies() {
         return $this->loadAll(self::class, "{$this}/dependencies");
     }
 
@@ -340,7 +340,7 @@ 
      * @depends after-create
      * @return Task[]
      */
-    public function getDependents () {
+    public function getDependents() {
         return $this->loadAll(self::class, "{$this}/dependents");
     }
 
@@ -351,21 +351,21 @@ 
      * @param null|string $token
      * @return TaskEvent[]|StoryEvent[]
      */
-    public function getEvents (&$token) {
+    public function getEvents(&$token) {
         return $this->api->sync($this, $token);
     }
 
     /**
      * @return External
      */
-    public function getExternal () {
+    public function getExternal() {
         return $this->_get('external') ?? $this->data['external'] = $this->factory(static::$map['external']);
     }
 
     /**
      * @return Project[]
      */
-    public function getProjects () {
+    public function getProjects() {
         return array_map(function(Membership $membership) {
             return $membership->getProject();
         }, $this->getMemberships());
@@ -377,7 +377,7 @@ 
      * @depends after-create
      * @return Story[]
      */
-    public function getStories () {
+    public function getStories() {
         return $this->loadAll(Story::class, "{$this}/stories");
     }
 
@@ -387,7 +387,7 @@ 
      * @depends after-create
      * @return Task[]
      */
-    public function getSubTasks () {
+    public function getSubTasks() {
         return $this->loadAll(self::class, "{$this}/subtasks");
     }
 
@@ -397,7 +397,7 @@ 
      * @depends after-create
      * @return string
      */
-    public function getUrl (): string {
+    public function getUrl(): string {
         return "https://app.asana.com/0/0/{$this->getGid()}";
     }
 
@@ -407,7 +407,7 @@ 
      * @depends after-create
      * @return TaskWebhook[]
      */
-    public function getWebhooks () {
+    public function getWebhooks() {
         return $this->loadAll(TaskWebhook::class, 'webhooks', [
             'workspace' => $this->getWorkspace()->getGid(),
             'resource' => $this->getGid()
@@ -417,7 +417,7 @@ 
     /**
      * @return bool
      */
-    public function isRenderedAsSeparator (): bool {
+    public function isRenderedAsSeparator(): bool {
         return $this->_is('is_rendered_as_separator');
     }
 
@@ -427,7 +427,7 @@ 
      * @depends after-create
      * @return Story
      */
-    public function newComment () {
+    public function newComment() {
         /** @var Story $comment */
         $comment = $this->factory(Story::class, [
             'resource_subtype' => Story::TYPE_COMMENT_ADDED
@@ -441,7 +441,7 @@ 
      * @depends after-create
      * @return Task
      */
-    public function newSubTask () {
+    public function newSubTask() {
         /** @var Task $sub */
         $sub = $this->factory(self::class);
         return $sub->setParent($this);
@@ -454,7 +454,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;
     }
@@ -466,7 +466,7 @@ 
      * @param Task $task
      * @return $this
      */
-    public function removeDependency (Task $task) {
+    public function removeDependency(Task $task) {
         return $this->removeDependencies([$task]);
     }
 
@@ -477,7 +477,7 @@ 
      * @param Task $task
      * @return $this
      */
-    public function removeDependent (Task $task) {
+    public function removeDependent(Task $task) {
         return $this->removeDependents([$task]);
     }
 
@@ -488,7 +488,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;
     }
@@ -499,7 +499,7 @@ 
      * @param User $user
      * @return $this
      */
-    public function removeFollower (User $user) {
+    public function removeFollower(User $user) {
         return $this->removeFollowers([$user]);
     }
 
@@ -509,7 +509,7 @@ 
      * @param User[] $users
      * @return $this
      */
-    public function removeFollowers (array $users) {
+    public function removeFollowers(array $users) {
         if ($this->hasGid()) {
             $this->api->post("{$this}/removeFollowers", ['followers' => array_column($users, 'gid')]);
         }
@@ -523,7 +523,7 @@ 
      * @param Project $project
      * @return $this
      */
-    public function removeFromProject (Project $project) {
+    public function removeFromProject(Project $project) {
         $gid = $project->getGid();
         if ($this->hasGid()) {
             $this->api->post("{$this}/removeProject", ['project' => $gid]);
@@ -542,7 +542,7 @@ 
      * @param Tag $tag
      * @return $this
      */
-    public function removeTag (Tag $tag) {
+    public function removeTag(Tag $tag) {
         if ($this->hasGid()) {
             $this->api->post("{$this}/removeTag", ['tag' => $tag->getGid()]);
         }
@@ -556,7 +556,7 @@ 
      * @param null|Task $parent
      * @return $this
      */
-    public function setParent (?self $parent) {
+    public function setParent(?self $parent) {
         if ($this->hasGid()) {
             $this->api->post("{$this}/setParent", ['parent' => $parent ? $parent->getGid() : null]);
             $this->data['parent'] = $parent;
@@ -571,14 +571,14 @@ 
      * @param bool $flag
      * @return $this
      */
-    public function setRenderedAsSeparator (bool $flag) {
+    public function setRenderedAsSeparator(bool $flag) {
         return $this->_set('is_rendered_as_separator', $flag);
     }
 
     /**
      * @return $this
      */
-    public function update () {
+    public function update() {
         $this->_update();
         $this->_clearSubDiffs();
         return $this;

Braces +4 added lines, -8 removed lines

@@ -205,8 +205,7 @@ 
         if ($this->hasGid()) {
             $this->api->post("{$this}/addFollowers", ['followers' => array_column($users, 'gid')]);
             $this->_merge('followers', $users);
-        }
-        else {
+        } else {
             $this->_merge('followers', $users, true);
         }
         return $this;
@@ -222,8 +221,7 @@ 
         if ($this->hasGid()) {
             $this->api->post("{$this}/addTag", ['tag' => $tag->getGid()]);
             $this->_merge('tags', [$tag]);
-        }
-        else {
+        } else {
             $this->_merge('tags', [$tag], true);
         }
         return $this;
@@ -248,8 +246,7 @@ 
                     'section' => $section
                 ]);
             }
-        }
-        else {
+        } else {
             if (!$this->hasWorkspace()) {
                 $this->setWorkspace($project->getWorkspace());
             }
@@ -560,8 +557,7 @@ 
         if ($this->hasGid()) {
             $this->api->post("{$this}/setParent", ['parent' => $parent ? $parent->getGid() : null]);
             $this->data['parent'] = $parent;
-        }
-        else {
+        } else {
             $this->_set('parent', $parent);
         }
         return $this;

src/Project.php

Spacing +20 added lines, -20 removed lines

@@ -67,11 +67,11 @@ 
         'workspace' => Workspace::class
     ];
 
-    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';
     }
 
@@ -80,7 +80,7 @@ 
      * @param User $user
      * @return $this
      */
-    public function addMember (User $user) {
+    public function addMember(User $user) {
         return $this->addMembers([$user]);
     }
 
@@ -89,7 +89,7 @@ 
      * @param User[] $users
      * @return $this
      */
-    public function addMembers (array $users) {
+    public function addMembers(array $users) {
         $this->api->post("{$this}/addMembers", ['members' => array_column($users, 'gid')]);
         $this->_merge('members', $users);
         return $this;
@@ -100,7 +100,7 @@ 
      * @param string $target
      * @return ProjectWebhook
      */
-    public function addWebhook (string $target) {
+    public function addWebhook(string $target) {
         /** @var ProjectWebhook $webhook */
         $webhook = $this->factory(ProjectWebhook::class);
         return $webhook->create($this, $target);
@@ -125,7 +125,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();
@@ -147,7 +147,7 @@ 
      * @param null|string $token
      * @return ProjectEvent[]|TaskEvent[]|StoryEvent[]
      */
-    public function getEvents (&$token) {
+    public function getEvents(&$token) {
         return $this->api->sync($this, $token);
     }
 
@@ -155,7 +155,7 @@ 
      * @depends after-create
      * @return Section[]
      */
-    public function getSections () {
+    public function getSections() {
         return $this->loadAll(Section::class, "{$this}/sections");
     }
 
@@ -163,7 +163,7 @@ 
      * @depends after-create
      * @return Status[]
      */
-    public function getStatuses () {
+    public function getStatuses() {
         return $this->loadAll(Status::class, "{$this}/project_statuses");
     }
 
@@ -172,7 +172,7 @@ 
      * @param array $query
      * @return Task[]
      */
-    public function getTasks (array $query = []) {
+    public function getTasks(array $query = []) {
         $query['project'] = $this->getGid();
         return $this->loadAll(Task::class, "tasks", $query);
     }
@@ -181,25 +181,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->loadAll(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;
     }
 
@@ -207,7 +207,7 @@ 
      * @depends after-create
      * @return Section
      */
-    public function newSection () {
+    public function newSection() {
         return $this->factory(Section::class, ['projects' => [$this]]);
     }
 
@@ -215,7 +215,7 @@ 
      * @depends after-create
      * @return Status
      */
-    public function newStatus () {
+    public function newStatus() {
         return $this->factory(Status::class);
     }
 
@@ -225,7 +225,7 @@ 
      * @depends after-create
      * @return Task
      */
-    public function newTask () {
+    public function newTask() {
         return $this->getSections()[0]->newTask();
     }
 
@@ -234,7 +234,7 @@ 
      * @param User $user
      * @return $this
      */
-    public function removeMember (User $user) {
+    public function removeMember(User $user) {
         return $this->removeMembers([$user]);
     }
 
@@ -243,7 +243,7 @@ 
      * @param User[] $users
      * @return $this
      */
-    public function removeMembers (array $users) {
+    public function removeMembers(array $users) {
         $this->api->post("{$this}/removeMembers", ['members' => array_column($users, 'gid')]);
         $this->_remove('members', $users);
         return $this;
@@ -254,7 +254,7 @@ 
      * @param null|Team $team
      * @return $this
      */
-    public function setTeam (?Team $team) {
+    public function setTeam(?Team $team) {
         assert(!$this->hasGid());
         if ($team and !$this->hasWorkspace()) {
             $this->setWorkspace($team->getOrganization());