hfw / asana

src/Task/Story.php

Spacing +7 added lines, -7 removed lines

@@ -39,15 +39,15 @@ 
     const TYPE_LIKED = 'liked';
     const TYPE_TAGGED = 'added_to_tag';
 
-    final public function __toString (): string {
+    final public function __toString(): string {
         return "stories/{$this->getGid()}";
     }
 
-    final protected function _getDir (): string {
+    final protected function _getDir(): string {
         return "{$this->getTask()}/stories";
     }
 
-    protected function _getMap (): array {
+    protected function _getMap(): array {
         return [
             'created_by' => User::class,
             'likes' => [User::class],
@@ -55,15 +55,15 @@ 
         ];
     }
 
-    final public function isComment (): bool {
+    final public function isComment(): bool {
         return $this->getResourceSubtype() === self::TYPE_COMMENT_ADDED;
     }
 
-    public function isEdited (): bool {
+    public function isEdited(): bool {
         return $this->_is('is_edited');
     }
 
-    public function isPinned (): bool {
+    public function isPinned(): bool {
         return $this->_is('is_pinned');
     }
 
@@ -71,7 +71,7 @@ 
      * @param bool $pinned
      * @return $this
      */
-    public function setPinned (bool $pinned) {
+    public function setPinned(bool $pinned) {
         return $this->_set('is_pinned', $pinned);
     }
 

src/Tag.php

Spacing +4 added lines, -4 removed lines

@@ -26,15 +26,15 @@ 
 
     const TYPE = 'tag';
 
-    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';
     }
 
-    protected function _getMap (): array {
+    protected function _getMap(): array {
         return [
             'followers' => [User::class],
             'workspace' => Workspace::class
@@ -44,7 +44,7 @@ 
     /**
      * @return Task[]
      */
-    public function getTasks () {
+    public function getTasks() {
         return $this->loadAll(Task::class, "{$this}/tasks");
     }
 }
\ No newline at end of file

src/Project/Section.php

Spacing +6 added lines, -6 removed lines

@@ -23,15 +23,15 @@ 
 
     const TYPE = 'section';
 
-    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";
     }
 
-    protected function _getMap (): array {
+    protected function _getMap(): array {
         return [
             'projects' => [Project::class]
         ];
@@ -40,14 +40,14 @@ 
     /**
      * @return Project
      */
-    public function getProject () {
+    public function getProject() {
         return $this->_get('projects')[0];
     }
 
     /**
      * @return Task[]
      */
-    public function getTasks () {
+    public function getTasks() {
         return $this->loadAll(Task::class, 'tasks', ['section' => $this->getGid()]);
     }
 
@@ -57,7 +57,7 @@ 
      * @after-create
      * @return Task
      */
-    public function newTask () {
+    public function newTask() {
         /** @var Task $task */
         $task = $this->factory(Task::class);
         return $task->addToProject($this);

src/Project/Status.php

Spacing +5 added lines, -5 removed lines

@@ -48,20 +48,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 _getMap (): array {
+    protected function _getMap(): array {
         return [
             'created_by' => User::class
         ];
@@ -70,7 +70,7 @@ 
     /**
      * @return Project
      */
-    public function getProject () {
+    public function getProject() {
         return $this->project;
     }
 }
\ No newline at end of file

src/Job.php

Spacing +8 added lines, -8 removed lines

@@ -28,11 +28,11 @@ 
     const STATUS_SUCCESS = 'succeeded'; // api docs say "completed" but that's wrong.
     const STATUS_FAIL = 'failed';
 
-    final public function __toString (): string {
+    final public function __toString(): string {
         return "jobs/{$this->getGid()}";
     }
 
-    protected function _getMap (): array {
+    protected function _getMap(): array {
         return [
             'new_project' => Project::class,
             'new_task' => Task::class
@@ -44,7 +44,7 @@ 
      *
      * @return bool
      */
-    public function isActive (): bool {
+    public function isActive(): bool {
         return $this->getStatus() === self::STATUS_ACTIVE;
     }
 
@@ -53,7 +53,7 @@ 
      *
      * @return bool
      */
-    public function isDone (): bool {
+    public function isDone(): bool {
         return $this->isSuccessful() or $this->isFailed();
     }
 
@@ -62,7 +62,7 @@ 
      *
      * @return bool
      */
-    public function isFailed (): bool {
+    public function isFailed(): bool {
         return $this->getStatus() === self::STATUS_FAIL;
     }
 
@@ -71,7 +71,7 @@ 
      *
      * @return bool
      */
-    public function isQueued (): bool {
+    public function isQueued(): bool {
         return $this->getStatus() === self::STATUS_QUEUED;
     }
 
@@ -80,7 +80,7 @@ 
      *
      * @return bool
      */
-    public function isSuccessful (): bool {
+    public function isSuccessful(): bool {
         return $this->getStatus() === self::STATUS_SUCCESS;
     }
 
@@ -89,7 +89,7 @@ 
      *
      * @return $this
      */
-    public function wait () {
+    public function wait() {
         while (!$this->isDone()) {
             sleep(3);
             $this->reload();

src/Base/AbstractEntity/WorkspaceTrait.php

Spacing +1 added lines, -1 removed lines

@@ -17,7 +17,7 @@
      * @param Workspace $workspace
      * @return $this
      */
-    public function setWorkspace (Workspace $workspace) {
+    public function setWorkspace(Workspace $workspace) {
         assert(!$this->hasGid());
         return $this->_set('workspace', $workspace);
     }

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;
 
     /**
      * Resolves entities to GIDs.
@@ -24,7 +24,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);
@@ -36,7 +36,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);
         }
@@ -51,7 +51,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;
@@ -74,7 +74,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);
@@ -88,7 +88,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])) {
@@ -104,7 +104,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' => $key])[$key];
             $map = $this->_getMap();

Braces +2 added lines, -4 removed lines

@@ -59,8 +59,7 @@ 
             $this->data[$key] = array_values(array_unique($this->data[$key]));
             if ($force) {
                 $this->diff[$key] = true;
-            }
-            else {
+            } else {
                 $this->api->getCache()->add($this);
             }
         }
@@ -110,8 +109,7 @@ 
             $map = $this->_getMap();
             $this->data[$key] = isset($map[$key]) ? $this->_getMapped($map[$key], $value) : $value;
             unset($this->diff[$key]);
-        }
-        else {
+        } else {
             $this->_setData($this->api->get($this, [], ['expand' => 'this']));
         }
         $this->api->getCache()->add($this);

src/Base/Data.php

Braces +2 added lines, -4 removed lines

@@ -291,11 +291,9 @@
         $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;

Spacing +20 added lines, -20 removed lines

@@ -30,7 +30,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);
     }
@@ -44,9 +44,9 @@ 
      * @uses _is()
      * @uses _set()
      */
-    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) {
@@ -56,7 +56,7 @@ 
         return $this->{$call[1]}($call[2], ...$args);
     }
 
-    public function __debugInfo (): array {
+    public function __debugInfo(): array {
         return $this->data;
     }
 
@@ -68,7 +68,7 @@ 
      * @param string $key
      * @return mixed
      */
-    protected function _get (string $key) {
+    protected function _get(string $key) {
         return $this->data[$key] ?? null;
     }
 
@@ -80,7 +80,7 @@ 
      *
      * @return array
      */
-    protected function _getMap (): array {
+    protected function _getMap(): array {
         return [];
     }
 
@@ -89,7 +89,7 @@ 
      * @param null|self|array $value
      * @return null|Data|Data[]
      */
-    protected function _getMapped ($class, $value) {
+    protected function _getMapped($class, $value) {
         // use empty|Data|Data[] as-is
         if (!$value or $value instanceof self or current($value) instanceof self) {
             return $value;
@@ -122,7 +122,7 @@ 
      *
      * @return array
      */
-    protected function _getPatch (): array {
+    protected function _getPatch(): array {
         $convert = function($each) use (&$convert) {
             // convert existing entities to gids
             if ($each instanceof AbstractEntity and $each->hasGid()) {
@@ -152,7 +152,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)) {
@@ -175,7 +175,7 @@ 
      * @param string $key
      * @return bool
      */
-    protected function _is (string $key): bool {
+    protected function _is(string $key): bool {
         return !empty($this->_get($key));
     }
 
@@ -188,7 +188,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;
@@ -200,7 +200,7 @@ 
      * @param array $data
      * @return $this
      */
-    protected function _setData (array $data) {
+    protected function _setData(array $data) {
         $map = $this->_getMap();
         /** @var null|self|array $value */
         foreach (array_intersect_key($data, $map) as $key => $value) {
@@ -218,7 +218,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);
     }
 
@@ -227,7 +227,7 @@ 
      *
      * @return bool
      */
-    final public function isDiff (): bool {
+    final public function isDiff(): bool {
         return (bool)$this->diff;
     }
 
@@ -235,7 +235,7 @@ 
      * @see toArray()
      * @return array
      */
-    public function jsonSerialize (): array {
+    public function jsonSerialize(): array {
         return $this->toArray();
     }
 
@@ -249,7 +249,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);
     }
 
@@ -264,7 +264,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);
     }
 
@@ -275,7 +275,7 @@ 
      *
      * @return string
      */
-    public function serialize (): string {
+    public function serialize(): string {
         return serialize($this->toArray());
     }
 
@@ -284,7 +284,7 @@ 
      *
      * @return array
      */
-    public function toArray (): array {
+    public function toArray(): array {
         if (!$this->api) {
             return $this->data;
         }
@@ -310,7 +310,7 @@ 
      *
      * @param $serialized
      */
-    public function unserialize ($serialized): void {
+    public function unserialize($serialized): void {
         $this->data = unserialize($serialized);
     }
 }
\ No newline at end of file

src/Portfolio.php

Spacing +10 added lines, -10 removed lines

@@ -30,15 +30,15 @@ 
 
     const TYPE = 'portfolio';
 
-    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';
     }
 
-    protected function _getMap (): array {
+    protected function _getMap(): array {
         return [
             'created_by' => User::class,
             'owner' => User::class,
@@ -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 @@ 
      * @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)]);
         }

Braces +1 added lines, -2 removed lines

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