hfw / asana

src/Base/AbstractEntity/DeleteTrait.php

Spacing +1 added lines, -1 removed lines

@@ -10,7 +10,7 @@
     /**
      * `DELETE`
      */
-    public function delete (): void {
+    public function delete(): void {
         $this->{'_delete'}();
     }
 }
\ No newline at end of file

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

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;

Spacing +24 added lines, -24 removed lines

@@ -48,7 +48,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);
     }
@@ -64,9 +64,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) {
@@ -76,7 +76,7 @@ 
         return $this->{$call[1]}($call[2], ...$args);
     }
 
-    public function __debugInfo (): array {
+    public function __debugInfo(): array {
         return $this->data;
     }
 
@@ -86,7 +86,7 @@ 
      * @param $field
      * @return null|Data|mixed
      */
-    final public function __get ($field) {
+    final public function __get($field) {
         return $this->_get($field);
     }
 
@@ -98,7 +98,7 @@ 
      * @param $field
      * @return bool
      */
-    final public function __isset ($field) {
+    final public function __isset($field) {
         return true;
     }
 
@@ -109,7 +109,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);
     }
 
@@ -121,7 +121,7 @@ 
      * @param string $field
      * @return mixed
      */
-    protected function _get (string $field) {
+    protected function _get(string $field) {
         return $this->data[$field] ?? null;
     }
 
@@ -136,7 +136,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)) {
@@ -159,7 +159,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 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);
     }
 
@@ -188,7 +188,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);
     }
 
@@ -203,7 +203,7 @@ 
      * @param int $pages
      * @return Traversable|AbstractEntity[]
      */
-    final protected function _loadEach (string $class, string $path, array $query = [], int $pages = 0) {
+    final protected function _loadEach(string $class, string $path, array $query = [], int $pages = 0) {
         return $this->api->loadEach($class, $this, $path, $query, $pages);
     }
 
@@ -218,7 +218,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) ?? [];
         }
@@ -240,7 +240,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;
@@ -251,7 +251,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);
@@ -264,7 +264,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?
@@ -310,7 +310,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()) {
@@ -336,7 +336,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]);
         }
@@ -347,7 +347,7 @@ 
      * @see toArray()
      * @return array
      */
-    public function jsonSerialize (): array {
+    public function jsonSerialize(): array {
         $data = $this->toArray();
         ksort($data);
         return $data;
@@ -358,8 +358,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);
     }
 
     /**
@@ -367,7 +367,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();
@@ -387,7 +387,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/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/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) {
         return $this->{'_set'}('workspace', $workspace);
     }
 }
\ No newline at end of file

src/Task/External.php

Spacing +5 added lines, -5 removed lines

@@ -24,7 +24,7 @@ 
      */
     protected $task;
 
-    public function __construct (Task $task, array $data = []) {
+    public function __construct(Task $task, array $data = []) {
         parent::__construct($task, $data);
         $this->task = $task;
     }
@@ -36,7 +36,7 @@ 
      * @param mixed $value
      * @return $this
      */
-    protected function _set (string $field, $value) {
+    protected function _set(string $field, $value) {
         $this->task->diff['external'] = true;
         return parent::_set($field, $value);
     }
@@ -46,9 +46,9 @@ 
      *
      * @return null|bool|number|string|array
      */
-    public function getDataJsonDecoded () {
+    public function getDataJsonDecoded() {
         if (strlen($data = $this->getData())) {
-            return json_decode($data, true, 512, JSON_BIGINT_AS_STRING | JSON_THROW_ON_ERROR);
+            return json_decode($data, true, 512, JSON_BIGINT_AS_STRING|JSON_THROW_ON_ERROR);
         }
         return null;
     }
@@ -60,7 +60,7 @@ 
      * @param mixed $data
      * @return $this
      */
-    public function setDataJsonEncoded ($data) {
+    public function setDataJsonEncoded($data) {
         if (isset($data)) {
             return $this->setData(json_encode($data, JSON_THROW_ON_ERROR));
         }

src/Api/SimpleCache.php

Spacing +10 added lines, -10 removed lines

@@ -34,7 +34,7 @@ 
     /**
      * @param PSR16 $psr
      */
-    public function __construct (PSR16 $psr) {
+    public function __construct(PSR16 $psr) {
         $this->psr = $psr;
         $this->ttl = new DateInterval('PT1H');
     }
@@ -43,7 +43,7 @@ 
      * @param string $key
      * @throws InvalidArgumentException
      */
-    protected function _delete (string $key) {
+    protected function _delete(string $key) {
         $this->psr->delete('asana/' . $key);
     }
 
@@ -52,7 +52,7 @@ 
      * @return null|string|AbstractEntity
      * @throws InvalidArgumentException
      */
-    protected function _get (string $key) {
+    protected function _get(string $key) {
         return $this->psr->get('asana/' . $key);
     }
 
@@ -61,7 +61,7 @@ 
      * @param mixed $value
      * @throws InvalidArgumentException
      */
-    protected function _set (string $key, $value) {
+    protected function _set(string $key, $value) {
         $this->psr->set('asana/' . $key, $value, $this->ttl);
     }
 
@@ -70,7 +70,7 @@ 
      * @return bool
      * @throws InvalidArgumentException
      */
-    public function add (AbstractEntity $entity): bool {
+    public function add(AbstractEntity $entity): bool {
         if (parent::add($entity)) {
             $this->_set($entity->getGid(), $entity);
             return true;
@@ -83,7 +83,7 @@ 
      * @param string[] $keys
      * @throws InvalidArgumentException
      */
-    protected function addKeys (string $gid, array $keys): void {
+    protected function addKeys(string $gid, array $keys): void {
         parent::addKeys($gid, $keys);
         // stash gid refs
         foreach ($keys as $key) {
@@ -100,7 +100,7 @@ 
      * @return null|AbstractEntity
      * @throws InvalidArgumentException
      */
-    public function get (string $key, $caller, Closure $factory) {
+    public function get(string $key, $caller, Closure $factory) {
         // POOL MISS && CACHE HIT
         if (!isset($this->gids[$key]) and $entity = $this->_get($key)) {
             if ($entity instanceof AbstractEntity) {
@@ -123,7 +123,7 @@ 
     /**
      * @return DateInterval
      */
-    public function getTtl () {
+    public function getTtl() {
         return $this->ttl;
     }
 
@@ -131,7 +131,7 @@ 
      * @param AbstractEntity $entity
      * @throws InvalidArgumentException
      */
-    public function remove (AbstractEntity $entity): void {
+    public function remove(AbstractEntity $entity): void {
         parent::remove($entity);
         foreach ($entity->getCacheKeys() as $key) {
             $this->_delete($key);
@@ -142,7 +142,7 @@ 
      * @param DateInterval $ttl
      * @return $this
      */
-    public function setTtl (DateInterval $ttl) {
+    public function setTtl(DateInterval $ttl) {
         $this->ttl = $ttl;
         return $this;
     }

src/Base/AbstractEntity.php

Braces +5 added lines, -10 removed lines

@@ -109,8 +109,7 @@ 
     protected 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)));
@@ -126,11 +125,9 @@ 
     protected function _save (string $dir = null) {
         if (isset($dir)) {
             $remote = $this->api->post($dir, $this->getDiff(), ['expand' => 'this']);
-        }
-        elseif ($this->isDiff()) {
+        } elseif ($this->isDiff()) {
             $remote = $this->api->put($this, $this->getDiff(), ['expand' => 'this']);
-        }
-        else {
+        } else {
             return $this;
         }
         /** @var array $remote */
@@ -186,11 +183,9 @@ 
             $optField = static::$optFields[$field] ?? $field;
             $value = $this->api->get($this, [], ['fields' => $optField])[$field] ?? null;
             $this->_setMapped($field, $value);
-        }
-        elseif ($remote = $this->api->get($this, [], ['expand' => 'this'])) {
+        } elseif ($remote = $this->api->get($this, [], ['expand' => 'this'])) {
             $this->_setData($remote);
-        }
-        else { // deleted upstream.
+        } else { // deleted upstream.
             $this->api->getCache()->remove($this);
             throw new RuntimeException("{$this} was deleted upstream.");
         }

Spacing +15 added lines, -15 removed lines

@@ -22,7 +22,7 @@ 
      *
      * @return string
      */
-    abstract public function __toString (): string;
+    abstract public function __toString(): string;
 
     /**
      * Maps lazy-loaded / reloadable mapped {@see Data} to their proper expanded field expression.
@@ -42,7 +42,7 @@ 
      * @param AbstractEntity $entity
      * @internal
      */
-    public function __set (string $from, self $entity) {
+    public function __set(string $from, self $entity) {
         if ($from === Cache::class and $entity !== $this) {
             foreach ($entity->data as $field => $value) {
                 if (!isset($this->diff[$field])) {
@@ -60,7 +60,7 @@ 
      * @return $this
      * @internal
      */
-    protected function _addWithPost (string $addPath, array $data, string $field, array $diff) {
+    protected function _addWithPost(string $addPath, array $data, string $field, array $diff) {
         if ($this->hasGid()) {
             return $this->_setWithPost($addPath, $data, $field);
         }
@@ -72,7 +72,7 @@ 
      *
      * @internal
      */
-    protected function _cache (): void {
+    protected function _cache(): void {
         $this->api->getCache()->add($this);
     }
 
@@ -81,7 +81,7 @@ 
      *
      * @internal
      */
-    protected function _delete (): void {
+    protected function _delete(): void {
         $this->api->delete($this);
         $this->api->getCache()->remove($this);
     }
@@ -92,7 +92,7 @@ 
      * @param string $field
      * @return mixed
      */
-    protected function _get (string $field) {
+    protected function _get(string $field) {
         if (!array_key_exists($field, $this->data) and $this->hasGid()) {
             $this->reload($field);
         }
@@ -107,7 +107,7 @@ 
      * @return $this
      * @internal
      */
-    protected function _removeWithPost (string $rmPath, array $data, string $field, $diff) {
+    protected function _removeWithPost(string $rmPath, array $data, string $field, $diff) {
         if ($this->hasGid()) {
             return $this->_setWithPost($rmPath, $data, $field);
         }
@@ -124,7 +124,7 @@ 
      * @return $this
      * @internal
      */
-    protected function _save (string $dir = null) {
+    protected function _save(string $dir = null) {
         if (isset($dir)) {
             $remote = $this->api->post($dir, $this->getDiff(), ['expand' => 'this']);
         }
@@ -140,7 +140,7 @@ 
         return $this;
     }
 
-    protected function _setData (array $data): void {
+    protected function _setData(array $data): void {
         // make sure gid is consistently null|string across all entities.
         $data['gid'] = empty($data['gid']) ? null : (string)$data['gid'];
 
@@ -160,7 +160,7 @@ 
      * @return $this
      * @internal
      */
-    protected function _setWithPost (string $path, array $data, string $field, $value = null) {
+    protected function _setWithPost(string $path, array $data, string $field, $value = null) {
         if ($this->hasGid()) {
             /** @var array $remote */
             $remote = $this->api->post($path, $data, ['fields' => static::$optFields[$field] ?? $field]);
@@ -178,14 +178,14 @@ 
      *
      * @return string[]
      */
-    public function getCacheKeys () {
+    public function getCacheKeys() {
         return [$this->getGid(), (string)$this];
     }
 
     /**
      * @return null|string
      */
-    final public function getGid (): ?string {
+    final public function getGid(): ?string {
         return $this->data['gid'];
     }
 
@@ -194,14 +194,14 @@ 
      *
      * @return string
      */
-    final public function getResourceType (): string {
+    final public function getResourceType(): string {
         return $this::TYPE;
     }
 
     /**
      * @return bool
      */
-    final public function hasGid (): bool {
+    final public function hasGid(): bool {
         return isset($this->data['gid']);
     }
 
@@ -212,7 +212,7 @@ 
      * @param string $field
      * @return $this
      */
-    public function reload (string $field = null) {
+    public function reload(string $field = null) {
         if (!$this->hasGid()) {
             throw new LogicException(static::class . " has no GID, it can't be reloaded.");
         }

src/Task/Like.php

Spacing +2 added lines, -2 removed lines

@@ -18,13 +18,13 @@
         'user' => User::class
     ];
 
-    protected function _setData (array $data): void {
+    protected function _setData(array $data): void {
         // useless. likes aren't entities.
         unset($data['gid'], $data['resource_type']);
         parent::_setData($data);
     }
 
-    final public function getResourceType (): string {
+    final public function getResourceType(): string {
         return self::TYPE;
     }
 }
\ No newline at end of file