hfw / asana

src/Color.php

Spacing +1 added lines, -1 removed lines

@@ -35,7 +35,7 @@
     /**
      * @return string
      */
-    public static function random () {
+    public static function random() {
         try {
             $colors = (new ReflectionClass(self::class))->getConstants();
             return $colors[array_rand($colors)];

Braces +1 added lines, -2 removed lines

@@ -39,8 +39,7 @@
         try {
             $colors = (new ReflectionClass(self::class))->getConstants();
             return $colors[array_rand($colors)];
-        }
-        catch (ReflectionException $exception) {
+        } catch (ReflectionException $exception) {
             return 'none'; // unreachable
         }
     }

src/CustomField/EnumOption.php

Spacing +6 added lines, -6 removed lines

@@ -37,23 +37,23 @@ 
      * @param CustomField $customField
      * @param array $data
      */
-    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;
     }
 
@@ -61,7 +61,7 @@ 
      * @param EnumOption $option
      * @return $this
      */
-    public function insertAfter (EnumOption $option) {
+    public function insertAfter(EnumOption $option) {
         assert($option->hasGid());
         if ($this->hasGid()) {
             $this->api->post("{$this->_getDir()}/insert", [
@@ -80,7 +80,7 @@ 
      * @param EnumOption $option
      * @return $this
      */
-    public function insertBefore (EnumOption $option) {
+    public function insertBefore(EnumOption $option) {
         assert($option->hasGid());
         if ($this->hasGid()) {
             $this->api->post("{$this->_getDir()}/insert", [

Braces +2 added lines, -4 removed lines

@@ -68,8 +68,7 @@ 
                 'enum_option' => $this->getGid(),
                 'after_enum_option' => $option->getGid()
             ]);
-        }
-        else {
+        } else {
             unset($this->data['insert_before'], $this->diff['insert_before']);
             $this->_set('insert_after', $option->getGid());
         }
@@ -87,8 +86,7 @@ 
                 'enum_option' => $this->getGid(),
                 'before_enum_option' => $option->getGid()
             ]);
-        }
-        else {
+        } else {
             unset($this->data['insert_after'], $this->diff['insert_after']);
             $this->_set('insert_before', $option->getGid());
         }

src/Webhook/AbstractWebhook.php

Spacing +3 added lines, -3 removed lines

@@ -30,11 +30,11 @@ 
     }
     use DeleteTrait;
 
-    final public function __toString (): string {
+    final public function __toString(): string {
         return "webhooks/{$this->getGid()}";
     }
 
-    final protected function _getDir (): string {
+    final protected function _getDir(): string {
         return 'webhooks';
     }
 
@@ -43,7 +43,7 @@ 
      * @param string $target
      * @return $this
      */
-    public function create ($resource, string $target) {
+    public function create($resource, string $target) {
         $this->_set('resource', $resource);
         $this->_set('target', $target);
         return $this->_create();

src/CustomField.php

Braces +1 added lines, -2 removed lines

@@ -62,8 +62,7 @@
         $option->setName($name);
         if ($this->hasGid()) {
             $option->create();
-        }
-        else {
+        } else {
             $this->diff['enum_options'] = true;
         }
         $this->data['enum_options'][] = $option;

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/Api.php

Braces +1 added lines, -2 removed lines

@@ -460,8 +460,7 @@
                 'sync' => $token,
                 'opt_expand' => 'this'
             ]));
-        }
-        catch (Error $error) {
+        } catch (Error $error) {
             if ($error->getCode() !== 412) {
                 throw $error;
             }

Spacing +37 added lines, -37 removed lines

@@ -52,24 +52,24 @@ 
     /**
      * @return Cache
      */
-    public static function getCache () {
+    public static function getCache() {
         return static::$cache ?? static::$cache = new Cache();
     }
 
     /**
      * @return Api
      */
-    public static function getDefault () {
+    public static function getDefault() {
         return self::$default;
     }
 
     /**
      * @return LoggerInterface
      */
-    public static function getLogger () {
+    public static function getLogger() {
         return static::$logger ?? static::$logger = new class implements LoggerInterface {
 
-                public function log (string $info, string $path, ?array $payload): void {
+                public function log(string $info, string $path, ?array $payload): void {
                     // stub
                 }
 
@@ -79,28 +79,28 @@ 
     /**
      * @param Cache $cache
      */
-    public static function setCache (Cache $cache) {
+    public static function setCache(Cache $cache) {
         static::$cache = $cache;
     }
 
     /**
      * @param Api $default
      */
-    public static function setDefault (Api $default) {
+    public static function setDefault(Api $default) {
         self::$default = $default;
     }
 
     /**
      * @param LoggerInterface $logger
      */
-    public static function setLogger (LoggerInterface $logger) {
+    public static function setLogger(LoggerInterface $logger) {
         static::$logger = $logger;
     }
 
     /**
      * @param string $token
      */
-    public function __construct (string $token) {
+    public function __construct(string $token) {
         $this->token = $token;
         if (!static::$default) {
             static::$default = $this;
@@ -112,7 +112,7 @@ 
      *
      * @param string $path
      */
-    public function delete (string $path): void {
+    public function delete(string $path): void {
         $this->exec('DELETE', $path);
     }
 
@@ -123,7 +123,7 @@ 
      * @return null|array
      * @internal
      */
-    protected function exec (string $method, string $path, array $opts = null) {
+    protected function exec(string $method, string $path, array $opts = null) {
         $ch = curl_init();
         curl_setopt_array($ch, [
             CURLOPT_CUSTOMREQUEST => $method,
@@ -174,7 +174,7 @@ 
      * @param array $data
      * @return mixed
      */
-    public function factory (string $class, $caller, array $data = []) {
+    public function factory(string $class, $caller, array $data = []) {
         return new $class($caller, $data);
     }
 
@@ -186,7 +186,7 @@ 
      * @param array $options
      * @return null|array
      */
-    public function get (string $path, array $query = [], array $options = []) {
+    public function get(string $path, array $query = [], array $options = []) {
         foreach ($options as $name => $value) {
             $query["opt_{$name}"] = $value;
         }
@@ -200,7 +200,7 @@ 
      * @param string $gid
      * @return null|Attachment
      */
-    public function getAttachment (string $gid) {
+    public function getAttachment(string $gid) {
         return $this->load(Attachment::class, $this, "attachments/{$gid}");
     }
 
@@ -210,7 +210,7 @@ 
      * @param string $gid
      * @return null|CustomField
      */
-    public function getCustomField (string $gid) {
+    public function getCustomField(string $gid) {
         return $this->load(CustomField::class, $this, "custom_fields/{$gid}");
     }
 
@@ -219,7 +219,7 @@ 
      *
      * @return User
      */
-    public function getMe () {
+    public function getMe() {
         return $this->getUser('me');
     }
 
@@ -230,7 +230,7 @@ 
      * @param array $query
      * @return string
      */
-    protected function getPath (string $path, array $query): string {
+    protected function getPath(string $path, array $query): string {
         return $query ? $path . '?' . http_build_query($query) : $path;
     }
 
@@ -240,7 +240,7 @@ 
      * @param string $gid
      * @return null|Portfolio
      */
-    public function getPortfolio (string $gid) {
+    public function getPortfolio(string $gid) {
         return $this->load(Portfolio::class, $this, "portfolios/{$gid}");
     }
 
@@ -250,7 +250,7 @@ 
      * @param string $gid
      * @return null|Project
      */
-    public function getProject (string $gid) {
+    public function getProject(string $gid) {
         return $this->load(Project::class, $this, "projects/{$gid}");
     }
 
@@ -260,7 +260,7 @@ 
      * @param string $gid
      * @return null|Section
      */
-    public function getSection (string $gid) {
+    public function getSection(string $gid) {
         return $this->load(Section::class, $this, "sections/{$gid}");
     }
 
@@ -270,7 +270,7 @@ 
      * @param string $gid
      * @return null|Story
      */
-    public function getStory (string $gid) {
+    public function getStory(string $gid) {
         return $this->load(Story::class, $this, "stories/{$gid}");
     }
 
@@ -280,7 +280,7 @@ 
      * @param string $gid
      * @return null|Tag
      */
-    public function getTag (string $gid) {
+    public function getTag(string $gid) {
         return $this->load(Tag::class, $this, "tags/{$gid}");
     }
 
@@ -290,7 +290,7 @@ 
      * @param string $gid
      * @return null|Task
      */
-    public function getTask (string $gid) {
+    public function getTask(string $gid) {
         return $this->load(Task::class, $this, "tasks/{$gid}");
     }
 
@@ -300,7 +300,7 @@ 
      * @param string $gid
      * @return null|Team
      */
-    public function getTeam (string $gid) {
+    public function getTeam(string $gid) {
         return $this->load(Team::class, $this, "teams/{$gid}");
     }
 
@@ -310,7 +310,7 @@ 
      * @param string $gid
      * @return null|User
      */
-    public function getUser (string $gid) {
+    public function getUser(string $gid) {
         return $this->load(User::class, $this, "users/{$gid}");
     }
 
@@ -322,7 +322,7 @@ 
      * @param array $data
      * @return ProjectEvent|TaskEvent|StoryEvent
      */
-    public function getWebhookEvent (array $data) {
+    public function getWebhookEvent(array $data) {
         static $classes = [
             Project::TYPE => ProjectEvent::class,
             Task::TYPE => TaskEvent::class,
@@ -335,7 +335,7 @@ 
      * @param string $gid
      * @return null|Workspace
      */
-    public function getWorkspace (string $gid) {
+    public function getWorkspace(string $gid) {
         return $this->load(Workspace::class, $this, "workspaces/{$gid}");
     }
 
@@ -343,7 +343,7 @@ 
      * @param string $name
      * @return null|Workspace
      */
-    public function getWorkspaceByName (string $name) {
+    public function getWorkspaceByName(string $name) {
         foreach ($this->getMe()->getWorkspaces() as $workspace) {
             if ($workspace->getName() === $name) {
                 return $workspace;
@@ -356,16 +356,16 @@ 
      * @param string $json
      * @return null|array
      */
-    protected function jsonDecode (string $json) {
-        return json_decode($json, true, JSON_BIGINT_AS_STRING | JSON_THROW_ON_ERROR);
+    protected function jsonDecode(string $json) {
+        return json_decode($json, true, JSON_BIGINT_AS_STRING|JSON_THROW_ON_ERROR);
     }
 
     /**
      * @param array $data
      * @return string
      */
-    protected function jsonEncode (array $data): string {
-        return json_encode($data, JSON_PRETTY_PRINT | JSON_THROW_ON_ERROR);
+    protected function jsonEncode(array $data): string {
+        return json_encode($data, JSON_PRETTY_PRINT|JSON_THROW_ON_ERROR);
     }
 
     /**
@@ -379,7 +379,7 @@ 
      * @param array $query
      * @return null|mixed|AbstractEntity
      */
-    public function load (string $class, $caller, string $path, array $query = []) {
+    public function load(string $class, $caller, string $path, array $query = []) {
         $key = $this->getPath($path, $query);
         return $this->getCache()->get($key, $caller, function($caller) use ($class, $path, $query) {
             $data = $this->get($path, $query, ['expand' => 'this']);
@@ -399,7 +399,7 @@ 
      * @param int $pages If positive, stops after this many pages have been fetched.
      * @return array|AbstractEntity[]
      */
-    public function loadAll (string $class, $caller, string $path, array $query = [], int $pages = 0) {
+    public function loadAll(string $class, $caller, string $path, array $query = [], int $pages = 0) {
         $query['opt_expand'] = 'this';
         $query += ['limit' => 100];
         $path = $this->getPath($path, $query);
@@ -425,7 +425,7 @@ 
      * @param array $options
      * @return null|array
      */
-    public function post (string $path, array $data = [], array $options = []) {
+    public function post(string $path, array $data = [], array $options = []) {
         $response = $this->exec('POST', $path, [
             CURLOPT_HTTPHEADER => ['Content-Type: application/json'],
             CURLOPT_POSTFIELDS => $this->jsonEncode(['options' => $options, 'data' => $data])
@@ -441,7 +441,7 @@ 
      * @param array $options
      * @return null|array
      */
-    public function put (string $path, array $data = [], array $options = []) {
+    public function put(string $path, array $data = [], array $options = []) {
         $response = $this->exec('PUT', $path, [
             CURLOPT_HTTPHEADER => ['Content-Type: application/json'],
             CURLOPT_POSTFIELDS => $this->jsonEncode(['options' => $options, 'data' => $data])
@@ -460,7 +460,7 @@ 
      * @param null|string $token
      * @return ProjectEvent[]|TaskEvent[]|StoryEvent[]
      */
-    public function sync ($entity, ?string &$token) {
+    public function sync($entity, ?string &$token) {
         try {
             $response = $this->exec('GET', $this->getPath('events', [
                 'resource' => $entity->getGid(),
@@ -495,7 +495,7 @@ 
      * @param string $file
      * @return null|array
      */
-    public function upload (string $path, string $file) {
+    public function upload(string $path, string $file) {
         $response = $this->exec('POST', $path, [
             CURLOPT_POSTFIELDS => ['file' => new CURLFile(realpath($file))] // multipart/form-data
         ]);

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/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 +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/Api/Cache.php

Spacing +4 added lines, -4 removed lines

@@ -32,7 +32,7 @@ 
      *
      * @param AbstractEntity $entity
      */
-    public function add (AbstractEntity $entity): void {
+    public function add(AbstractEntity $entity): void {
         if ($gid = $entity->getGid() and !$entity->isDiff()) {
             $this->entities[$gid] = $entity;
             foreach ($this->getKeys($entity) as $key) {
@@ -52,7 +52,7 @@ 
      * @param Closure $factory `fn($caller): null|AbstractEntity`
      * @return null|mixed|AbstractEntity
      */
-    public function get (string $key, $caller, Closure $factory) {
+    public function get(string $key, $caller, Closure $factory) {
         if ($gid = $this->gids[$key] ?? null) {
             return $this->entities[$gid];
         }
@@ -81,7 +81,7 @@ 
      * @param AbstractEntity $entity
      * @return string[]
      */
-    protected function getKeys (AbstractEntity $entity) {
+    protected function getKeys(AbstractEntity $entity) {
         if (!$gid = $entity->getGid() or $entity->isDiff()) {
             return [];
         }
@@ -97,7 +97,7 @@ 
      *
      * @param AbstractEntity $entity
      */
-    public function remove (AbstractEntity $entity): void {
+    public function remove(AbstractEntity $entity): void {
         unset($this->entities[$entity->getGid()]);
         foreach ($this->getKeys($entity) as $key) {
             unset($this->gids[$key]);

src/Api/LoggerInterface.php

Spacing +1 added lines, -1 removed lines

@@ -12,5 +12,5 @@
      * @param string $path
      * @param null|array $payload
      */
-    public function log (string $info, string $path, ?array $payload): void;
+    public function log(string $info, string $path, ?array $payload): void;
 }
\ No newline at end of file