hfw / asana

src/Job.php

Spacing +7 added lines, -7 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,7 +51,7 @@ 
      *
      * @return bool
      */
-    public function isDone (): bool {
+    public function isDone(): bool {
         return $this->isSuccessful() or $this->isFailed();
     }
 
@@ -60,7 +60,7 @@ 
      *
      * @return bool
      */
-    public function isFailed (): bool {
+    public function isFailed(): bool {
         return $this->getStatus() === self::STATUS_FAIL;
     }
 
@@ -69,7 +69,7 @@ 
      *
      * @return bool
      */
-    public function isQueued (): bool {
+    public function isQueued(): bool {
         return $this->getStatus() === self::STATUS_QUEUED;
     }
 
@@ -78,7 +78,7 @@ 
      *
      * @return bool
      */
-    public function isSuccessful (): bool {
+    public function isSuccessful(): bool {
         return $this->getStatus() === self::STATUS_SUCCESS;
     }
 
@@ -87,7 +87,7 @@ 
      *
      * @return $this
      */
-    public function wait () {
+    public function wait() {
         while (!$this->isDone()) {
             sleep(3);
             $this->reload();

src/User/TaskList.php

Spacing +4 added lines, -4 removed lines

@@ -30,14 +30,14 @@ 
         'workspace' => Workspace::class
     ];
 
-    final public function __toString (): string {
+    final public function __toString(): string {
         return "user_task_lists/{$this->getGid()}";
     }
 
     /**
      * @return Task[]
      */
-    public function getIncompleteTasks () {
+    public function getIncompleteTasks() {
         return $this->loadAll(Task::class, "{$this}/tasks", ['completed_since' => 'now']);
     }
 
@@ -46,14 +46,14 @@ 
      *
      * @return ArrayIterator|Task[]
      */
-    public function getIterator () {
+    public function getIterator() {
         return new ArrayIterator($this->getTasks());
     }
 
     /**
      * @return Task[]
      */
-    public function getTasks () {
+    public function getTasks() {
         return $this->loadAll(Task::class, "{$this}/tasks");
     }
 }
\ No newline at end of file

src/Tag.php

Spacing +3 added lines, -3 removed lines

@@ -31,18 +31,18 @@
         '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 Task[]
      */
-    public function getTasks () {
+    public function getTasks() {
         return $this->loadAll(Task::class, "{$this}/tasks");
     }
 }
\ No newline at end of file

src/CustomField.php

Spacing +9 added lines, -9 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';
     }
 
@@ -52,7 +52,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);
@@ -69,28 +69,28 @@ 
     /**
      * @return bool
      */
-    public function hasNotificationsEnabled (): bool {
+    public function hasNotificationsEnabled(): bool {
         return $this->_is('has_notifications_enabled');
     }
 
     /**
      * @return bool
      */
-    public function isEnum (): bool {
+    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 isText (): bool {
+    final public function isText(): bool {
         return $this->getResourceSubtype() === self::TYPE_TEXT;
     }
 
@@ -98,7 +98,7 @@ 
      * @param bool $flag
      * @return $this
      */
-    public function setGlobalToWorkspace (bool $flag) {
+    public function setGlobalToWorkspace(bool $flag) {
         return $this->_set('is_global_to_workspace', $flag);
     }
 
@@ -106,7 +106,7 @@ 
      * @param bool $flag
      * @return $this
      */
-    public function setNotificationsEnabled (bool $flag) {
+    public function setNotificationsEnabled(bool $flag) {
         return $this->_set('has_notifications_enabled', $flag);
     }
 

src/Project/Status.php

Spacing +4 added lines, -4 removed lines

@@ -52,23 +52,23 @@
      * @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";
     }
 
     /**
      * @return Project
      */
-    public function getProject () {
+    public function getProject() {
         return $this->project;
     }
 }
\ No newline at end of file

src/Project/Section.php

Spacing +4 added lines, -4 removed lines

@@ -28,18 +28,18 @@ 
         'project' => Project::class
     ];
 
-    final public function __toString (): string {
+    final public function __toString(): string {
         return "sections/{$this->getGid()}";
     }
 
-    final protected function _getDir (): string {
+    final protected function _getDir(): string {
         return "{$this->getProject()}/sections";
     }
 
     /**
      * @return Task[]
      */
-    public function getTasks () {
+    public function getTasks() {
         return $this->loadAll(Task::class, 'tasks', ['section' => $this->getGid()]);
     }
 
@@ -49,7 +49,7 @@ 
      * @depends after-create
      * @return Task
      */
-    public function newTask () {
+    public function newTask() {
         /** @var Task $task */
         $task = $this->factory(Task::class);
         return $task->addToProject($this);

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' => static::_getGids($users)]);
             $this->_merge('members', $users);
@@ -84,7 +84,7 @@ 
      * @depends after-create
      * @return Project[]
      */
-    public function getItems () {
+    public function getItems() {
         return $this->loadAll(Project::class, "{$this}/items");
     }
 
@@ -93,7 +93,7 @@ 
      * @param Project $item
      * @return $this
      */
-    public function removeItem (Project $item) {
+    public function removeItem(Project $item) {
         $this->api->post("{$this}/removeItem", ['item' => $item->getGid()]);
         return $this;
     }
@@ -102,7 +102,7 @@ 
      * @param User $user
      * @return $this
      */
-    public function removeMember (User $user) {
+    public function removeMember(User $user) {
         return $this->removeMembers([$user]);
     }
 
@@ -110,7 +110,7 @@ 
      * @param User[] $users
      * @return $this
      */
-    public function removeMembers (array $users) {
+    public function removeMembers(array $users) {
         if ($this->hasGid()) {
             $this->api->post("{$this}/removeMembers", ['members' => static::_getGids($users)]);
         }

src/Base/Data.php

Spacing +21 added lines, -21 removed lines

@@ -40,7 +40,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);
     }
@@ -56,9 +56,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) {
@@ -68,7 +68,7 @@ 
         return $this->{$call[1]}($call[2], ...$args);
     }
 
-    public function __debugInfo (): array {
+    public function __debugInfo(): array {
         return $this->data;
     }
 
@@ -78,7 +78,7 @@ 
      * @param $key
      * @return null|Data|mixed
      */
-    final public function __get ($key) {
+    final public function __get($key) {
         return $this->_get($key);
     }
 
@@ -90,7 +90,7 @@ 
      * @param $key
      * @return bool
      */
-    final public function __isset ($key) {
+    final public function __isset($key) {
         return true;
     }
 
@@ -102,7 +102,7 @@ 
      * @param string $key
      * @return mixed
      */
-    protected function _get (string $key) {
+    protected function _get(string $key) {
         return $this->data[$key] ?? null;
     }
 
@@ -116,7 +116,7 @@ 
      * @param string $key
      * @return bool
      */
-    protected function _has (string $key): bool {
+    protected function _has(string $key): bool {
         $value = $this->_get($key);
         if (isset($value)) {
             if (is_countable($value)) {
@@ -139,7 +139,7 @@ 
      * @param string $key
      * @return bool
      */
-    protected function _is (string $key): bool {
+    protected function _is(string $key): bool {
         return !empty($this->_get($key));
     }
 
@@ -152,7 +152,7 @@ 
      * @param mixed $value
      * @return $this
      */
-    protected function _set (string $key, $value) {
+    protected function _set(string $key, $value) {
         $this->data[$key] = $value;
         $this->diff[$key] = true;
         return $this;
@@ -163,7 +163,7 @@ 
      *
      * @param array $data
      */
-    protected function _setData (array $data): void {
+    protected function _setData(array $data): void {
         $this->data = $this->diff = [];
         foreach ($data as $key => $value) {
             $this->_setMapped($key, $value);
@@ -176,7 +176,7 @@ 
      * @param string $key
      * @param null|Data|array $value
      */
-    protected function _setMapped (string $key, $value): void {
+    protected function _setMapped(string $key, $value): void {
         unset($this->diff[$key]);
 
         // use unmapped values, null, [], Data, and Data[] as-is
@@ -216,7 +216,7 @@ 
      * @param array $data
      * @return mixed
      */
-    final protected function factory (string $class, array $data = []) {
+    final protected function factory(string $class, array $data = []) {
         return $this->api->factory($class, $this, $data);
     }
 
@@ -225,7 +225,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()) {
@@ -250,7 +250,7 @@ 
      *
      * @return bool
      */
-    final public function isDiff (): bool {
+    final public function isDiff(): bool {
         return (bool)$this->diff;
     }
 
@@ -258,7 +258,7 @@ 
      * @see toArray()
      * @return array
      */
-    public function jsonSerialize (): array {
+    public function jsonSerialize(): array {
         return $this->toArray();
     }
 
@@ -272,7 +272,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);
     }
 
@@ -287,7 +287,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);
     }
 
@@ -296,7 +296,7 @@ 
      *
      * @return string
      */
-    public function serialize (): string {
+    public function serialize(): string {
         return serialize($this->toArray());
     }
 
@@ -305,7 +305,7 @@ 
      *
      * @return array
      */
-    public function toArray (): array {
+    public function toArray(): array {
         if (!$this->api) {
             return $this->data;
         }
@@ -329,7 +329,7 @@ 
      *
      * @param $serialized
      */
-    public function unserialize ($serialized): void {
+    public function unserialize($serialized): void {
         $this->api = Api::getDefault();
         $this->data = unserialize($serialized);
     }

src/Base/AbstractEntity.php

Spacing +7 added lines, -7 removed lines

@@ -16,7 +16,7 @@ 
      *
      * @return string
      */
-    abstract public function __toString (): string;
+    abstract public function __toString(): string;
 
     /**
      * Maps lazy-loaded / reloadable fields to their proper expanded field expression.
@@ -33,7 +33,7 @@ 
      * @param AbstractEntity[] $entities
      * @return string[]
      */
-    protected static function _getGids (array $entities) {
+    protected static function _getGids(array $entities) {
         return array_map(function(AbstractEntity $entity) {
             return $entity->getGid();
         }, $entities);
@@ -45,7 +45,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);
         }
@@ -60,7 +60,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;
@@ -83,7 +83,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);
@@ -97,7 +97,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])) {
@@ -113,7 +113,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);

Braces +2 added lines, -4 removed lines

@@ -68,8 +68,7 @@ 
             $this->data[$key] = array_values(array_unique($this->data[$key]));
             if ($force) {
                 $this->diff[$key] = true;
-            }
-            else {
+            } else {
                 $this->api->getCache()->add($this);
             }
         }
@@ -117,8 +116,7 @@ 
         if (isset($key)) {
             $value = $this->api->get($this, [], ['fields' => static::$optFields[$key] ?? $key])[$key] ?? null;
             $this->_setMapped($key, $value);
-        }
-        else {
+        } else {
             $this->_setData($this->api->get($this, [], ['expand' => 'this']));
         }
         $this->api->getCache()->add($this);