hfw / asana

src/Task/FieldValues.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(FieldValue $value) {
             return $value->getValue();
         }, array_intersect_key($this->data, $this->diff));
@@ -70,14 +70,14 @@ 
      * @param $fieldGid
      * @return FieldValue
      */
-    public function getField ($fieldGid) {
+    public function getField($fieldGid) {
         return $this->data[$fieldGid];
     }
 
     /**
      * @return Generator
      */
-    public function getIterator () {
+    public function getIterator() {
         foreach ($this->data as $field) {
             yield $this[$field['gid']];
         }
@@ -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/Task/FieldValue.php

Spacing +11 added lines, -11 removed lines

@@ -26,7 +26,7 @@ 
      */
     protected $owner;
 
-    public function __construct (Data $owner, array $data = []) {
+    public function __construct(Data $owner, array $data = []) {
         $this->owner = $owner;
         parent::__construct($owner, $data);
     }
@@ -34,14 +34,14 @@ 
     /**
      * @return string
      */
-    public function getCurrentOptionName () {
+    public function getCurrentOptionName() {
         return $this->getEnumOptionNames()[$this->getValue()];
     }
 
     /**
      * @return mixed
      */
-    public function getCustomFieldGid () {
+    public function getCustomFieldGid() {
         return $this->data['gid'];
     }
 
@@ -50,7 +50,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');
@@ -61,7 +61,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');
@@ -70,7 +70,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;
@@ -78,15 +78,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;
     }
 
@@ -94,7 +94,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) {
@@ -110,7 +110,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/Api/FileCache.php

Spacing +10 added lines, -10 removed lines

@@ -25,31 +25,31 @@ 
      */
     private $logger;
 
-    public function __construct (string $dir, LoggerInterface $logger) {
+    public function __construct(string $dir, LoggerInterface $logger) {
         $this->dir = $dir;
         $this->logger = $logger;
     }
 
-    private function _path ($key) {
+    private function _path($key) {
         $file = "{$this->dir}/{$key}~";
         clearstatcache(true, $file);
         return $file;
     }
 
-    public function clear () {
+    public function clear() {
         // unused
     }
 
-    public function delete ($key) {
+    public function delete($key) {
         $this->logger->log('CACHE DELETE', $key);
         unlink($this->_path($key));
     }
 
-    public function deleteMultiple ($keys) {
+    public function deleteMultiple($keys) {
         // unused
     }
 
-    public function get ($key, $default = null) {
+    public function get($key, $default = null) {
         $file = $this->_path($key);
         if (file_exists($file)) {
             if (filemtime($file) > time()) {
@@ -65,15 +65,15 @@ 
         return $default;
     }
 
-    public function getMultiple ($keys, $default = null) {
+    public function getMultiple($keys, $default = null) {
         // unused
     }
 
-    public function has ($key) {
+    public function has($key) {
         // unused
     }
 
-    public function set ($key, $value, $ttl = null) {
+    public function set($key, $value, $ttl = null) {
         assert($ttl instanceof DateInterval);
         $this->logger->log('CACHE SET', $key);
         $path = $this->_path($key);
@@ -93,7 +93,7 @@ 
         touch($path, (new DateTime())->add($ttl)->getTimeStamp());
     }
 
-    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

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

src/CustomField.php

Spacing +12 added lines, -12 removed lines

@@ -40,15 +40,15 @@ 
         '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';
     }
 
-    protected function _setData (array $data): void {
+    protected function _setData(array $data): void {
         // deprecated for resource_subtype. also removing to conceptually separate from FieldValue
         unset($data['type']);
         parent::_setData($data);
@@ -62,7 +62,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);
@@ -78,35 +78,35 @@ 
     /**
      * @return bool
      */
-    public function hasNotificationsEnabled (): bool {
+    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
      */
-    public function isGlobalToWorkspace (): bool {
+    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;
     }
 
@@ -114,7 +114,7 @@ 
      * @param bool $flag
      * @return $this
      */
-    public function setGlobalToWorkspace (bool $flag) {
+    public function setGlobalToWorkspace(bool $flag) {
         return $this->_set('is_global_to_workspace', $flag);
     }
 
@@ -122,7 +122,7 @@ 
      * @param bool $flag
      * @return $this
      */
-    public function setNotificationsEnabled (bool $flag) {
+    public function setNotificationsEnabled(bool $flag) {
         return $this->_set('has_notifications_enabled', $flag);
     }
 
@@ -130,7 +130,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);