hfw / asana

src/Api/Laravel/Facade/Asana.php

Spacing +2 added lines, -2 removed lines

@@ -46,14 +46,14 @@
     /**
      * @return Api
      */
-    public static function getApi () {
+    public static function getApi() {
         return static::getFacadeRoot();
     }
 
     /**
      * @return string
      */
-    public static function getFacadeAccessor () {
+    public static function getFacadeAccessor() {
         return AsanaServiceProvider::NAME;
     }
 

src/Api/SimpleCachePool.php

Spacing +9 added lines, -9 removed lines

@@ -33,7 +33,7 @@ 
     /**
      * @param PSR16 $psr
      */
-    public function __construct (PSR16 $psr) {
+    public function __construct(PSR16 $psr) {
         $this->psr = $psr;
     }
 
@@ -41,7 +41,7 @@ 
      * @param AbstractEntity $entity
      * @throws CacheException
      */
-    protected function _add (AbstractEntity $entity): void {
+    protected function _add(AbstractEntity $entity): void {
         $this->psr->set("asana/{$entity->getGid()}", $entity, $this->_getTtl($entity));
         parent::_add($entity);
     }
@@ -51,7 +51,7 @@ 
      * @param string[] $keys
      * @throws CacheException
      */
-    protected function _addKeys (AbstractEntity $entity, ...$keys): void {
+    protected function _addKeys(AbstractEntity $entity, ...$keys): void {
         $gid = $entity->getGid();
         $ttl = $this->_getTtl($entity);
         foreach ($keys as $key) {
@@ -68,7 +68,7 @@ 
      * @return null|AbstractEntity
      * @throws CacheException
      */
-    protected function _get (string $key, $caller) {
+    protected function _get(string $key, $caller) {
         if (!$entity = parent::_get($key, $caller) and $entity = $this->psr->get("asana/{$key}")) {
             if (is_string($entity)) { // gid ref
                 if (!$entity = $this->_get($entity, $caller)) {
@@ -88,7 +88,7 @@ 
      * @param AbstractEntity $entity
      * @return int
      */
-    protected function _getTtl (AbstractEntity $entity): int {
+    protected function _getTtl(AbstractEntity $entity): int {
         if ($entity instanceof ImmutableInterface) {
             return strtotime('tomorrow') - time();
         }
@@ -100,14 +100,14 @@ 
      * @return bool
      * @throws CacheException
      */
-    protected function _has (string $key): bool {
+    protected function _has(string $key): bool {
         return parent::_has($key) or $this->psr->has("asana/{$key}");
     }
 
     /**
      * @return int
      */
-    public function getTtl (): int {
+    public function getTtl(): int {
         return $this->ttl;
     }
 
@@ -115,7 +115,7 @@ 
      * @param string[] $keys
      * @throws CacheException
      */
-    public function remove (array $keys): void {
+    public function remove(array $keys): void {
         parent::remove($keys);
         foreach ($keys as $key) {
             $this->psr->delete("asana/{$key}");
@@ -126,7 +126,7 @@ 
      * @param int $ttl
      * @return $this
      */
-    public function setTtl (int $ttl) {
+    public function setTtl(int $ttl) {
         $this->ttl = $ttl;
         return $this;
     }

src/Portfolio.php

Spacing +11 added lines, -11 removed lines

@@ -63,7 +63,7 @@ 
      *
      * @return string
      */
-    final public function __toString (): string {
+    final public function __toString(): string {
         return "portfolios/{$this->getGid()}";
     }
 
@@ -72,7 +72,7 @@ 
      *
      * @return string
      */
-    final protected function _getDir (): string {
+    final protected function _getDir(): string {
         return 'portfolios';
     }
 
@@ -81,7 +81,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;
     }
@@ -90,7 +90,7 @@ 
      * @param User $user
      * @return $this
      */
-    public function addMember (User $user) {
+    public function addMember(User $user) {
         return $this->addMembers([$user]);
     }
 
@@ -98,7 +98,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);
@@ -108,14 +108,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->api->loadEach($this, Project::class, "{$this}/items");
     }
 
@@ -124,7 +124,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;
     }
@@ -133,7 +133,7 @@ 
      * @param User $user
      * @return $this
      */
-    public function removeMember (User $user) {
+    public function removeMember(User $user) {
         return $this->removeMembers([$user]);
     }
 
@@ -141,7 +141,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);
@@ -151,7 +151,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/Event.php

Spacing +13 added lines, -13 removed lines

@@ -41,11 +41,11 @@ 
  */
 class Event extends Data {
 
-    const ACTION_CHANGED = 'changed';       // no parent
-    const ACTION_ADDED = 'added';           // relational, no change
-    const ACTION_REMOVED = 'removed';       // relational, no change
-    const ACTION_DELETED = 'deleted';       // no parent or change
-    const ACTION_UNDELETED = 'undeleted';   // no parent or change
+    const ACTION_CHANGED = 'changed'; // no parent
+    const ACTION_ADDED = 'added'; // relational, no change
+    const ACTION_REMOVED = 'removed'; // relational, no change
+    const ACTION_DELETED = 'deleted'; // no parent or change
+    const ACTION_UNDELETED = 'undeleted'; // no parent or change
 
     const GRAPH = [
         User::TYPE => User::class,
@@ -63,7 +63,7 @@ 
         'user' => User::class
     ];
 
-    protected function _setData (array $data): void {
+    protected function _setData(array $data): void {
         if (isset($data['parent'])) {
             $type = $data['parent']['resource_type'];
             $data['parent'] = $this->_hydrate(static::GRAPH[$type], $data['parent']);
@@ -80,7 +80,7 @@ 
      *
      * @return null|Project|Section|Task
      */
-    public function getParent () {
+    public function getParent() {
         return $this->data['parent'] ?? null;
     }
 
@@ -89,42 +89,42 @@ 
      *
      * @return User|Project|Section|Task|CustomField|Attachment|Story|Like
      */
-    public function getResource () {
+    public function getResource() {
         return $this->data['resource'];
     }
 
     /**
      * @return bool
      */
-    final public function wasAddition (): bool {
+    final public function wasAddition(): bool {
         return $this->getAction() === self::ACTION_ADDED;
     }
 
     /**
      * @return bool
      */
-    final public function wasChange (): bool {
+    final public function wasChange(): bool {
         return $this->getAction() === self::ACTION_CHANGED;
     }
 
     /**
      * @return bool
      */
-    final public function wasDeletion (): bool {
+    final public function wasDeletion(): bool {
         return $this->getAction() === self::ACTION_DELETED;
     }
 
     /**
      * @return bool
      */
-    final public function wasRemoval (): bool {
+    final public function wasRemoval(): bool {
         return $this->getAction() === self::ACTION_REMOVED;
     }
 
     /**
      * @return bool
      */
-    final public function wasUndeletion (): bool {
+    final public function wasUndeletion(): bool {
         return $this->getAction() === self::ACTION_UNDELETED;
     }
 }
\ No newline at end of file

src/Base/AbstractEntity/DateTrait.php

Spacing +3 added lines, -3 removed lines

@@ -22,7 +22,7 @@ 
      * @param null|string|DateTimeInterface $date
      * @return $this
      */
-    private function _setYmd (string $field, $date) {
+    private function _setYmd(string $field, $date) {
         if ($date instanceof DateTimeInterface) {
             $date = $date->format('Y-m-d');
         }
@@ -33,7 +33,7 @@ 
      * @param null|string|DateTimeInterface $date
      * @return $this
      */
-    public function setDueOn ($date) {
+    public function setDueOn($date) {
         return $this->_setYmd('due_on', $date);
     }
 
@@ -41,7 +41,7 @@ 
      * @param null|string|DateTimeInterface $date
      * @return $this
      */
-    public function setStartOn ($date) {
+    public function setStartOn($date) {
         return $this->_setYmd('start_on', $date);
     }
 

src/Base/AbstractEntity/PostMutatorTrait.php

Spacing +3 added lines, -3 removed lines

@@ -19,7 +19,7 @@ 
      * @param array $diff
      * @return $this
      */
-    private function _addWithPost (string $addPath, array $data, string $field, array $diff) {
+    private function _addWithPost(string $addPath, array $data, string $field, array $diff) {
         if ($this->hasGid()) {
             return $this->_setWithPost($addPath, $data, $field);
         }
@@ -33,7 +33,7 @@ 
      * @param array|Closure $diff An array to diff, or a filter closure.
      * @return $this
      */
-    private function _removeWithPost (string $rmPath, array $data, string $field, $diff) {
+    private function _removeWithPost(string $rmPath, array $data, string $field, $diff) {
         if ($this->hasGid()) {
             return $this->_setWithPost($rmPath, $data, $field);
         }
@@ -53,7 +53,7 @@ 
      * @return $this
      * @internal
      */
-    private function _setWithPost (string $path, array $data, string $field, $value = null) {
+    private function _setWithPost(string $path, array $data, string $field, $value = null) {
         if ($this->hasGid()) {
             /** @var array $remote */
             $remote = $this->api->post($path, $data, ['fields' => static::OPT_FIELDS[$field] ?? $field]);

Braces +1 added lines, -2 removed lines

@@ -36,8 +36,7 @@
     private function _removeWithPost (string $rmPath, array $data, string $field, $diff) {
         if ($this->hasGid()) {
             return $this->_setWithPost($rmPath, $data, $field);
-        }
-        elseif (is_array($diff)) {
+        } elseif (is_array($diff)) {
             return $this->_set($field, array_values(array_diff($this->data[$field] ?? [], $diff)));
         }
         return $this->_set($field, array_values(array_filter($this->data[$field] ?? [], $diff)));

src/Project/Status.php

Spacing +5 added lines, -5 removed lines

@@ -50,7 +50,7 @@ 
      * @param Project $project
      * @param array $data
      */
-    public function __construct (Project $project, array $data = []) {
+    public function __construct(Project $project, array $data = []) {
         $this->project = $project;
         parent::__construct($project, $data);
     }
@@ -60,7 +60,7 @@ 
      *
      * @return string
      */
-    final public function __toString (): string {
+    final public function __toString(): string {
         return "project_statuses/{$this->getGid()}";
     }
 
@@ -69,11 +69,11 @@ 
      *
      * @return string
      */
-    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']);
 
@@ -83,7 +83,7 @@ 
     /**
      * @return Project
      */
-    public function getProject () {
+    public function getProject() {
         return $this->project;
     }
 }
\ No newline at end of file

src/Project/Section.php

Spacing +5 added lines, -5 removed lines

@@ -36,7 +36,7 @@ 
      *
      * @return string
      */
-    final public function __toString (): string {
+    final public function __toString(): string {
         return "sections/{$this->getGid()}";
     }
 
@@ -45,7 +45,7 @@ 
      *
      * @return string
      */
-    final protected function _getDir (): string {
+    final protected function _getDir(): string {
         return "{$this->getProject()}/sections";
     }
 
@@ -53,7 +53,7 @@ 
      * @param array $filter
      * @return Traversable|Task[]
      */
-    public function getIterator (array $filter = Task::GET_INCOMPLETE) {
+    public function getIterator(array $filter = Task::GET_INCOMPLETE) {
         $filter['section'] = $this->getGid();
         return $this->api->loadEach($this, Task::class, 'tasks', $filter);
     }
@@ -62,7 +62,7 @@ 
      * @param array $filter
      * @return Task[]
      */
-    public function getTasks (array $filter = Task::GET_INCOMPLETE) {
+    public function getTasks(array $filter = Task::GET_INCOMPLETE) {
         return iterator_to_array($this->getIterator($filter));
     }
 
@@ -72,7 +72,7 @@ 
      * @depends after-create
      * @return Task
      */
-    public function newTask () {
+    public function newTask() {
         /** @var Task $task */
         $task = $this->api->factory($this, Task::class);
         return $task->addToProject($this);

src/CustomField.php

Spacing +12 added lines, -12 removed lines

@@ -48,7 +48,7 @@ 
      *
      * @return string
      */
-    final public function __toString (): string {
+    final public function __toString(): string {
         return "custom_fields/{$this->getGid()}";
     }
 
@@ -57,11 +57,11 @@ 
      *
      * @return string
      */
-    final protected function _getDir (): string {
+    final protected function _getDir(): string {
         return 'custom_fields';
     }
 
-    protected function _setData (array $data): void {
+    protected function _setData(array $data): void {
         // strip out field entry values if present.
         $data = array_intersect_key($data, array_flip([
             'gid',
@@ -82,7 +82,7 @@ 
      * @param string $name
      * @return EnumOption
      */
-    public function addEnumOption (string $name) {
+    public function addEnumOption(string $name) {
         /** @var EnumOption $option */
         $option = $this->api->factory($this, EnumOption::class);
         $option->setName($name);
@@ -98,35 +98,35 @@ 
     /**
      * @return bool
      */
-    final public function hasNotificationsEnabled (): bool {
+    final 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
      */
-    final public function isGlobalToWorkspace (): bool {
+    final 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;
     }
 
@@ -134,7 +134,7 @@ 
      * @param bool $flag
      * @return $this
      */
-    final public function setGlobalToWorkspace (bool $flag) {
+    final public function setGlobalToWorkspace(bool $flag) {
         return $this->_set('is_global_to_workspace', $flag);
     }
 
@@ -142,7 +142,7 @@ 
      * @param bool $flag
      * @return $this
      */
-    final public function setNotificationsEnabled (bool $flag) {
+    final public function setNotificationsEnabled(bool $flag) {
         return $this->_set('has_notifications_enabled', $flag);
     }
 
@@ -150,7 +150,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);