hfw / asana

src/Job.php

Spacing +9 added lines, -9 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,15 +51,15 @@ 
      *
      * @return bool
      */
-    public function isDone (): bool {
+    public function isDone(): bool {
         return $this->isSuccessful() or $this->isFailed();
     }
 
-    final public function isDuplicatingProject (): bool {
+    final public function isDuplicatingProject(): bool {
         return $this->getResourceSubtype() === self::TYPE_DUPLICATE_PROJECT;
     }
 
-    final public function isDuplicatingTask (): bool {
+    final public function isDuplicatingTask(): bool {
         return $this->getResourceSubtype() === self::TYPE_DUPLICATE_TASK;
     }
 
@@ -68,7 +68,7 @@ 
      *
      * @return bool
      */
-    public function isFailed (): bool {
+    public function isFailed(): bool {
         return $this->getStatus() === self::STATUS_FAIL;
     }
 
@@ -77,7 +77,7 @@ 
      *
      * @return bool
      */
-    public function isQueued (): bool {
+    public function isQueued(): bool {
         return $this->getStatus() === self::STATUS_QUEUED;
     }
 
@@ -86,7 +86,7 @@ 
      *
      * @return bool
      */
-    public function isSuccessful (): bool {
+    public function isSuccessful(): bool {
         return $this->getStatus() === self::STATUS_SUCCESS;
     }
 
@@ -98,7 +98,7 @@ 
      * @param callable $spinner `fn( Job $this ): void`
      * @return $this
      */
-    public function wait (callable $spinner = null) {
+    public function wait(callable $spinner = null) {
         while (!$this->isDone()) {
             if ($spinner) {
                 call_user_func($spinner, $this);

src/Project/Status.php

Spacing +5 added lines, -5 removed lines

@@ -50,20 +50,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 _setData (array $data): void {
+    protected function _setData(array $data): void {
         // redundant, prefer created_by
         unset($data['author']);
         parent::_setData($data);
@@ -72,7 +72,7 @@ 
     /**
      * @return Project
      */
-    public function getProject () {
+    public function getProject() {
         return $this->project;
     }
 }
\ No newline at end of file

src/Api/Laravel/AsanaServiceProvider.php

Spacing +3 added lines, -3 removed lines

@@ -15,15 +15,15 @@
 
     const NAME = 'asana';
 
-    public function boot () {
+    public function boot() {
         $this->publishes([__DIR__ . '/config/asana.php' => App::configPath('asana.php')]);
     }
 
-    public function provides () {
+    public function provides() {
         return [self::NAME];
     }
 
-    public function register () {
+    public function register() {
         $this->app->singleton(self::NAME, function(Application $app) {
             $config = $app['config'][self::NAME];
 

src/Base/Data.php

Braces +3 added lines, -6 removed lines

@@ -315,8 +315,7 @@ 
 
         if ($isList) {
             $this->data[$field] = array_map($hydrate, $value);
-        }
-        else {
+        } else {
             $this->data[$field] = $hydrate($value);
         }
     }
@@ -387,11 +386,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;

Spacing +20 added lines, -20 removed lines

@@ -58,7 +58,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);
     }
@@ -74,9 +74,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|select|set)(.+)$/', $method, $call);
             $call[1] = '_' . $call[1];
             $call[2] = preg_replace_callback('/[A-Z]/', function(array $match) {
@@ -86,7 +86,7 @@ 
         return $this->{$call[1]}($call[2], ...$args);
     }
 
-    public function __debugInfo (): array {
+    public function __debugInfo(): array {
         return $this->data;
     }
 
@@ -96,7 +96,7 @@ 
      * @param $field
      * @return null|Data|mixed
      */
-    final public function __get ($field) {
+    final public function __get($field) {
         return $this->_get($field);
     }
 
@@ -108,7 +108,7 @@ 
      * @param $field
      * @return bool
      */
-    final public function __isset ($field) {
+    final public function __isset($field) {
         return true;
     }
 
@@ -120,7 +120,7 @@ 
      * @param string $field
      * @return mixed
      */
-    protected function _get (string $field) {
+    protected function _get(string $field) {
         return $this->data[$field] ?? null;
     }
 
@@ -135,7 +135,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)) {
@@ -158,7 +158,7 @@ 
      * @param string $field
      * @return bool
      */
-    protected function _is (string $field): bool {
+    protected function _is(string $field): bool {
         return (bool)$this->_get($field);
     }
 
@@ -173,7 +173,7 @@ 
      * @param callable $filter `fn( Data $object ): bool`
      * @return array
      */
-    protected function _select ($subject, callable $filter) {
+    protected function _select($subject, callable $filter) {
         if (!is_iterable($subject)) {
             $subject = $this->_get($subject) ?? [];
         }
@@ -195,7 +195,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;
@@ -206,7 +206,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);
@@ -219,7 +219,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?
@@ -271,7 +271,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()) {
@@ -297,7 +297,7 @@ 
      * @param string null $field
      * @return bool
      */
-    final public function isDiff (string $field = null): bool {
+    final public function isDiff(string $field = null): bool {
         if ($field) {
             return isset($this->diff[$field]);
         }
@@ -308,7 +308,7 @@ 
      * @see toArray()
      * @return array
      */
-    public function jsonSerialize (): array {
+    public function jsonSerialize(): array {
         $data = $this->toArray();
         ksort($data);
         return $data;
@@ -319,8 +319,8 @@ 
      *
      * @return string
      */
-    public function serialize (): string {
-        return json_encode($this, JSON_PRETTY_PRINT | JSON_UNESCAPED_SLASHES | JSON_UNESCAPED_UNICODE);
+    public function serialize(): string {
+        return json_encode($this, JSON_PRETTY_PRINT|JSON_UNESCAPED_SLASHES|JSON_UNESCAPED_UNICODE);
     }
 
     /**
@@ -328,7 +328,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();
@@ -348,7 +348,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);

src/Task/FieldEntry.php

Spacing +9 added lines, -9 removed lines

@@ -23,7 +23,7 @@ 
     /**
      * @return string
      */
-    public function getCurrentOptionName () {
+    public function getCurrentOptionName() {
         return $this->getEnumOptionNames()[$this->getValue()];
     }
 
@@ -32,7 +32,7 @@ 
      *
      * @return string[]
      */
-    public function getEnumOptionNames () {
+    public function getEnumOptionNames() {
         static $names = []; // shared
         $gid = $this->data['gid'];
         return $names[$gid] ?? $names[$gid] = array_column($this->data['enum_options'], 'name', 'gid');
@@ -43,7 +43,7 @@ 
      *
      * @return string[]
      */
-    public function getEnumOptionValues () {
+    public function getEnumOptionValues() {
         static $values = []; // shared
         $gid = $this->data['gid'];
         return $values[$gid] ?? $values[$gid] = array_column($this->data['enum_options'], 'gid', 'name');
@@ -52,7 +52,7 @@ 
     /**
      * @return null|number|string
      */
-    public function getValue () {
+    public function getValue() {
         $type = $this->data['type'];
         if ($type === CustomField::TYPE_ENUM) {
             return $this->data['enum_value']['gid'] ?? null;
@@ -60,15 +60,15 @@ 
         return $this->data["{$type}_value"];
     }
 
-    final public function isEnum (): bool {
+    final public function isEnum(): bool {
         return $this->getType() === CustomField::TYPE_ENUM;
     }
 
-    final public function isNumber (): bool {
+    final public function isNumber(): bool {
         return $this->getType() === CustomField::TYPE_NUMBER;
     }
 
-    final public function isText (): bool {
+    final public function isText(): bool {
         return $this->getType() === CustomField::TYPE_TEXT;
     }
 
@@ -76,7 +76,7 @@ 
      * @param null|number|string $value
      * @return $this
      */
-    public function setValue ($value) {
+    public function setValue($value) {
         $type = $this->data['type'];
         $this->diff["{$type}_value"] = true;
         if ($type === CustomField::TYPE_ENUM) {
@@ -92,7 +92,7 @@ 
      *
      * @return array
      */
-    public function toArray (): array {
+    public function toArray(): array {
         // only strip if needed.
         if (!isset($this->data['resource_subtype'])) {
             return $this->data;

src/Task/FieldEntries.php

Spacing +10 added lines, -10 removed lines

@@ -39,7 +39,7 @@ 
      */
     protected $types = [];
 
-    public function __construct (Task $task, array $data = []) {
+    public function __construct(Task $task, array $data = []) {
         $this->task = $task;
         parent::__construct($task, $data);
     }
@@ -47,7 +47,7 @@ 
     /**
      * @param array[] $values
      */
-    protected function _setData (array $values): void {
+    protected function _setData(array $values): void {
         // ensure data is keyed by field gid
         $values = array_combine(array_column($values, 'gid'), $values);
         parent::_setData($values);
@@ -56,11 +56,11 @@ 
     /**
      * @return int
      */
-    public function count () {
+    public function count() {
         return count($this->data);
     }
 
-    public function getDiff (): array {
+    public function getDiff(): array {
         return array_map(function(FieldEntry $value) {
             return $value->getValue();
         }, array_intersect_key($this->data, $this->diff));
@@ -70,14 +70,14 @@ 
      * @param $fieldGid
      * @return FieldEntry
      */
-    public function getField ($fieldGid) {
+    public function getField($fieldGid) {
         return $this->data[$fieldGid];
     }
 
     /**
      * @return Generator
      */
-    public function getIterator () {
+    public function getIterator() {
         foreach ($this->data as $gid => $field) {
             yield $gid => $field->getValue();
         }
@@ -87,7 +87,7 @@ 
      * @param string $fieldGid
      * @return bool
      */
-    public function offsetExists ($fieldGid) {
+    public function offsetExists($fieldGid) {
         return array_key_exists($fieldGid, $this->data);
     }
 
@@ -95,7 +95,7 @@ 
      * @param string $fieldGid
      * @return null|number|string
      */
-    public function offsetGet ($fieldGid) {
+    public function offsetGet($fieldGid) {
         return $this->data[$fieldGid]->getValue();
     }
 
@@ -103,7 +103,7 @@ 
      * @param string $fieldGid
      * @param null|number|string $value
      */
-    public function offsetSet ($fieldGid, $value) {
+    public function offsetSet($fieldGid, $value) {
         $this->data[$fieldGid]->setValue($value);
         $this->diff[$fieldGid] = true;
         $this->task->diff['custom_fields'] = true;
@@ -114,7 +114,7 @@ 
      *
      * @param string $fieldGid
      */
-    public function offsetUnset ($fieldGid) {
+    public function offsetUnset($fieldGid) {
         $this->offsetSet($fieldGid, null);
     }
 

src/Api/FileCache.php

Spacing +15 added lines, -15 removed lines

@@ -30,21 +30,21 @@ 
      */
     private $logger;
 
-    public function __construct (string $dir) {
+    public function __construct(string $dir) {
         $this->dir = $dir;
     }
 
-    private function _path ($key) {
+    private function _path($key) {
         $file = "{$this->dir}/{$key}~";
         clearstatcache(true, $file);
         return $file;
     }
 
-    public function clear () {
+    public function clear() {
         // unused. just delete the dir.
     }
 
-    public function delete ($key) {
+    public function delete($key) {
         $path = $this->_path($key);
         if (file_exists($path)) {
             $this->log('CACHE DELETE', $key);
@@ -52,11 +52,11 @@ 
         }
     }
 
-    public function deleteMultiple ($keys) {
+    public function deleteMultiple($keys) {
         // unused
     }
 
-    public function get ($key, $default = null) {
+    public function get($key, $default = null) {
         $path = $this->_path($key);
         if (file_exists($path)) {
             if (filemtime($path) > time()) {
@@ -75,32 +75,32 @@ 
     /**
      * @return LoggerInterface
      */
-    public function getLogger () {
+    public function getLogger() {
         return $this->logger;
     }
 
-    public function getMultiple ($keys, $default = null) {
+    public function getMultiple($keys, $default = null) {
         // unused
     }
 
-    public function has ($key) {
+    public function has($key) {
         // unused
     }
 
     /**
      * @return bool
      */
-    public function isGzEncoding (): bool {
+    public function isGzEncoding(): bool {
         return $this->gzEncoding;
     }
 
-    private function log (string $msg, string $key): void {
+    private function log(string $msg, string $key): void {
         if ($this->logger) {
             $this->logger->log($msg, $key);
         }
     }
 
-    public function set ($key, $value, $ttl = null) {
+    public function set($key, $value, $ttl = null) {
         assert($ttl instanceof DateInterval);
         $this->log('CACHE SET', $key);
         $path = $this->_path($key);
@@ -124,7 +124,7 @@ 
      * @param bool $gzEncoding
      * @return $this
      */
-    public function setGzEncoding (bool $gzEncoding) {
+    public function setGzEncoding(bool $gzEncoding) {
         $this->gzEncoding = $gzEncoding;
         return $this;
     }
@@ -133,12 +133,12 @@ 
      * @param null|LoggerInterface $logger
      * @return $this
      */
-    public function setLogger (?LoggerInterface $logger) {
+    public function setLogger(?LoggerInterface $logger) {
         $this->logger = $logger;
         return $this;
     }
 
-    public function setMultiple ($values, $ttl = null) {
+    public function setMultiple($values, $ttl = null) {
         // unused
     }
 }
\ No newline at end of file

Braces +1 added lines, -2 removed lines

@@ -65,8 +65,7 @@
             }
             $this->log('CACHE EXPIRE', $key);
             unlink($path);
-        }
-        else {
+        } else {
             $this->log('CACHE MISS', $key);
         }
         return $default;

src/User/TaskList.php

Spacing +5 added lines, -5 removed lines

@@ -30,11 +30,11 @@ 
         'workspace' => Workspace::class
     ];
 
-    final public function __toString (): string {
+    final public function __toString(): string {
         return "user_task_lists/{$this->getGid()}";
     }
 
-    public function getCacheKeys () {
+    public function getCacheKeys() {
         $keys = parent::getCacheKeys();
 
         /** @see User::getTaskList() */
@@ -47,7 +47,7 @@ 
      * @param array $filter
      * @return Traversable|Task[]
      */
-    public function getIterator (array $filter = []) {
+    public function getIterator(array $filter = []) {
         return $this->api->loadEach($this, Task::class, "{$this}/tasks");
     }
 
@@ -59,7 +59,7 @@ 
      * @param array $filter
      * @return Task[]
      */
-    public function getTasks (array $filter = []) {
+    public function getTasks(array $filter = []) {
         return iterator_to_array($this->getIterator($filter));
     }
 
@@ -68,7 +68,7 @@ 
      * @param array $apiFilter Given to the API to reduce network load.
      * @return Task[]
      */
-    public function selectTasks (callable $filter, array $apiFilter) {
+    public function selectTasks(callable $filter, array $apiFilter) {
         return $this->_select($this->getIterator($apiFilter), $filter);
     }
 }
\ No newline at end of file

src/Portfolio.php

Spacing +11 added lines, -11 removed lines

@@ -52,11 +52,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';
     }
 
@@ -65,7 +65,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;
     }
@@ -74,7 +74,7 @@ 
      * @param User $user
      * @return $this
      */
-    public function addMember (User $user) {
+    public function addMember(User $user) {
         return $this->addMembers([$user]);
     }
 
@@ -82,7 +82,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);
@@ -92,14 +92,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");
     }
 
@@ -108,7 +108,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;
     }
@@ -117,7 +117,7 @@ 
      * @param User $user
      * @return $this
      */
-    public function removeMember (User $user) {
+    public function removeMember(User $user) {
         return $this->removeMembers([$user]);
     }
 
@@ -125,7 +125,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);
@@ -135,7 +135,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