hfw / asana

src/Base/AbstractEntity/UpdateTrait.php

Spacing +1 added lines, -1 removed lines

@@ -12,7 +12,7 @@
      *
      * @return $this
      */
-    public function update () {
+    public function update() {
         return $this->{'_save'}();
     }
 }
\ No newline at end of file

src/Base/Data.php

Spacing +21 added lines, -21 removed lines

@@ -47,7 +47,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);
     }
@@ -63,9 +63,9 @@ 
      * @param array $args
      * @return mixed
      */
-    public function __call (string $method, array $args) {
+    public function __call(string $method, array $args) {
         static $cache = [];
-        if (!$call =& $cache[$method]) {
+        if (!$call = & $cache[$method]) {
             preg_match('/^(get|has|is|set)(.+)$/', $method, $call);
             $call[1] = '_' . $call[1];
             $call[2] = preg_replace_callback('/[A-Z]/', function(array $match) {
@@ -75,7 +75,7 @@ 
         return $this->{$call[1]}($call[2], ...$args);
     }
 
-    public function __debugInfo (): array {
+    public function __debugInfo(): array {
         return $this->data;
     }
 
@@ -85,7 +85,7 @@ 
      * @param $field
      * @return null|Data|mixed
      */
-    final public function __get ($field) {
+    final public function __get($field) {
         return $this->_get($field);
     }
 
@@ -97,7 +97,7 @@ 
      * @param $field
      * @return bool
      */
-    final public function __isset ($field) {
+    final public function __isset($field) {
         return true;
     }
 
@@ -109,7 +109,7 @@ 
      * @param string $field
      * @return mixed
      */
-    protected function _get (string $field) {
+    protected function _get(string $field) {
         return $this->data[$field] ?? null;
     }
 
@@ -123,7 +123,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)) {
@@ -146,7 +146,7 @@ 
      * @param string $field
      * @return bool
      */
-    protected function _is (string $field): bool {
+    protected function _is(string $field): bool {
         return !empty($this->_get($field));
     }
 
@@ -159,7 +159,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;
@@ -170,7 +170,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);
@@ -183,7 +183,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?
@@ -231,7 +231,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);
     }
 
@@ -240,7 +240,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()) {
@@ -265,7 +265,7 @@ 
      *
      * @return bool
      */
-    final public function isDiff (): bool {
+    final public function isDiff(): bool {
         return (bool)$this->diff;
     }
 
@@ -273,7 +273,7 @@ 
      * @see toArray()
      * @return array
      */
-    public function jsonSerialize (): array {
+    public function jsonSerialize(): array {
         return $this->toArray();
     }
 
@@ -287,7 +287,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);
     }
 
@@ -302,7 +302,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);
     }
 
@@ -311,7 +311,7 @@ 
      *
      * @return string
      */
-    public function serialize (): string {
+    public function serialize(): string {
         return json_encode($this);
     }
 
@@ -320,7 +320,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();
@@ -340,7 +340,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

@@ -218,8 +218,7 @@ 
         if (is_array($class)) {
             $class = $class[0];
             $this->data[$field] = array_map($hydrate, $value);
-        }
-        else {
+        } else {
             $this->data[$field] = $hydrate($value);
         }
     }
@@ -324,11 +323,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/CustomField/EnumOption.php

Spacing +8 added lines, -8 removed lines

@@ -33,23 +33,23 @@ 
      */
     protected $customField;
 
-    public function __construct (CustomField $customField, array $data = []) {
+    public function __construct(CustomField $customField, array $data = []) {
         parent::__construct($customField, $data);
         $this->customField = $customField;
     }
 
-    final public function __toString (): string {
+    final public function __toString(): string {
         return "enum_options/{$this->getGid()}";
     }
 
-    final protected function _getDir (): string {
+    final protected function _getDir(): string {
         return "{$this->customField}/enum_options";
     }
 
     /**
      * @return CustomField
      */
-    public function getCustomField () {
+    public function getCustomField() {
         return $this->customField;
     }
 
@@ -58,7 +58,7 @@ 
      * @param EnumOption $option
      * @return $this
      */
-    public function moveAfter (EnumOption $option) {
+    public function moveAfter(EnumOption $option) {
         $this->api->post("{$this->_getDir()}/insert", [
             'after_enum_option' => $option->getGid(),
             'enum_option' => $this->getGid()
@@ -72,7 +72,7 @@ 
      * @param EnumOption $option
      * @return $this
      */
-    public function moveBefore (EnumOption $option) {
+    public function moveBefore(EnumOption $option) {
         $this->api->post("{$this->_getDir()}/insert", [
             'before_enum_option' => $option->getGid(),
             'enum_option' => $this->getGid()
@@ -84,7 +84,7 @@ 
     /**
      * @return $this
      */
-    public function moveFirst () {
+    public function moveFirst() {
         $first = $this->customField->getEnumOptions()[0];
         if ($first !== $this) {
             $this->moveBefore($first);
@@ -95,7 +95,7 @@ 
     /**
      * @return $this
      */
-    public function moveLast () {
+    public function moveLast() {
         $options = $this->customField->getEnumOptions();
         $last = $options[count($options) - 1];
         if ($last !== $this) {

src/Task.php

Spacing +40 added lines, -40 removed lines

@@ -91,15 +91,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'])) {
@@ -111,7 +111,7 @@ 
         return $this;
     }
 
-    protected function _setData (array $data): void {
+    protected function _setData(array $data): void {
         unset($data['hearted']); // deprecated for likes
         unset($data['hearts']); // deprecated for likes
         unset($data['projects']); // redundant, memberships are used instead
@@ -125,7 +125,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);
@@ -137,7 +137,7 @@ 
      * @param string $text
      * @return Story
      */
-    public function addComment (string $text) {
+    public function addComment(string $text) {
         return $this->newComment()->setText($text)->create();
     }
 
@@ -148,7 +148,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;
     }
@@ -160,7 +160,7 @@ 
      * @param Task $task
      * @return $this
      */
-    public function addDependency (Task $task) {
+    public function addDependency(Task $task) {
         return $this->addDependencies([$task]);
     }
 
@@ -171,7 +171,7 @@ 
      * @param Task $task
      * @return $this
      */
-    public function addDependent (Task $task) {
+    public function addDependent(Task $task) {
         return $this->addDependents([$task]);
     }
 
@@ -182,7 +182,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;
     }
@@ -193,7 +193,7 @@ 
      * @param User $user
      * @return $this
      */
-    public function addFollower (User $user) {
+    public function addFollower(User $user) {
         return $this->addFollowers([$user]);
     }
 
@@ -205,7 +205,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);
@@ -219,7 +219,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]);
@@ -234,7 +234,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->factory(Membership::class)
             ->_set('project', $project)
@@ -249,7 +249,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);
@@ -265,7 +265,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,
@@ -280,7 +280,7 @@ 
      * @depends after-create
      * @return Attachment[]
      */
-    public function getAttachments () {
+    public function getAttachments() {
         return $this->loadAll(Attachment::class, "{$this}/attachments");
     }
 
@@ -290,7 +290,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();
         }));
@@ -302,7 +302,7 @@ 
      * @depends after-create
      * @return Task[]
      */
-    public function getDependencies () {
+    public function getDependencies() {
         return $this->loadAll(self::class, "{$this}/dependencies");
     }
 
@@ -312,7 +312,7 @@ 
      * @depends after-create
      * @return Task[]
      */
-    public function getDependents () {
+    public function getDependents() {
         return $this->loadAll(self::class, "{$this}/dependents");
     }
 
@@ -323,7 +323,7 @@ 
      * @param null|string $token
      * @return TaskEvent[]|StoryEvent[]
      */
-    public function getEvents (&$token) {
+    public function getEvents(&$token) {
         return $this->api->sync($this, $token);
     }
 
@@ -335,14 +335,14 @@ 
      *
      * @return External
      */
-    public function getExternal () {
+    public function getExternal() {
         return $this->_get('external') ?? $this->data['external'] = $this->factory(External::class);
     }
 
     /**
      * @return Project[]
      */
-    public function getProjects () {
+    public function getProjects() {
         return array_map(function(Membership $membership) {
             return $membership->getProject();
         }, $this->getMemberships());
@@ -354,7 +354,7 @@ 
      * @depends after-create
      * @return Story[]
      */
-    public function getStories () {
+    public function getStories() {
         return $this->loadAll(Story::class, "{$this}/stories");
     }
 
@@ -364,7 +364,7 @@ 
      * @depends after-create
      * @return Task[]
      */
-    public function getSubTasks () {
+    public function getSubTasks() {
         return $this->loadAll(self::class, "{$this}/subtasks");
     }
 
@@ -374,7 +374,7 @@ 
      * @depends after-create
      * @return string
      */
-    public function getUrl (): string {
+    public function getUrl(): string {
         return "https://app.asana.com/0/0/{$this->getGid()}";
     }
 
@@ -384,7 +384,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()
@@ -394,7 +394,7 @@ 
     /**
      * @return bool
      */
-    public function isRenderedAsSeparator (): bool {
+    public function isRenderedAsSeparator(): bool {
         return $this->_is('is_rendered_as_separator');
     }
 
@@ -404,7 +404,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
@@ -418,7 +418,7 @@ 
      * @depends after-create
      * @return Task
      */
-    public function newSubTask () {
+    public function newSubTask() {
         /** @var Task $sub */
         $sub = $this->factory(self::class);
         return $sub->setParent($this);
@@ -431,7 +431,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;
     }
@@ -443,7 +443,7 @@ 
      * @param Task $task
      * @return $this
      */
-    public function removeDependency (Task $task) {
+    public function removeDependency(Task $task) {
         return $this->removeDependencies([$task]);
     }
 
@@ -454,7 +454,7 @@ 
      * @param Task $task
      * @return $this
      */
-    public function removeDependent (Task $task) {
+    public function removeDependent(Task $task) {
         return $this->removeDependents([$task]);
     }
 
@@ -465,7 +465,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;
     }
@@ -476,7 +476,7 @@ 
      * @param User $user
      * @return $this
      */
-    public function removeFollower (User $user) {
+    public function removeFollower(User $user) {
         return $this->removeFollowers([$user]);
     }
 
@@ -488,7 +488,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);
@@ -502,7 +502,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) {
@@ -518,7 +518,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]);
@@ -531,7 +531,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);
@@ -541,7 +541,7 @@ 
      * @param bool $flag
      * @return $this
      */
-    public function setRenderedAsSeparator (bool $flag) {
+    public function setRenderedAsSeparator(bool $flag) {
         return $this->_set('is_rendered_as_separator', $flag);
     }