hfw / shopify

src/Customer/Invite.php

Spacing +2 added lines, -2 removed lines

@@ -23,12 +23,12 @@
      */
     protected $customer;
 
-    public function __construct (Customer $customer) {
+    public function __construct(Customer $customer) {
         $this->customer = $customer;
         parent::__construct($customer);
     }
 
-    public function send (): void {
+    public function send(): void {
         $this->api->post("{$this->customer}/send_invite", ['customer_invite' => $this->getData()]);
     }
 }
\ No newline at end of file

src/Theme.php

Spacing +6 added lines, -6 removed lines

@@ -34,7 +34,7 @@ 
      */
     protected $assets;
 
-    final protected static function getType (): string {
+    final protected static function getType(): string {
         return 'theme';
     }
 
@@ -42,18 +42,18 @@ 
      * @param Api $api
      * @return Theme[]
      */
-    public static function loadAll (Api $api) {
+    public static function loadAll(Api $api) {
         return static::toList(new static($api), $api->get('themes')['themes']);
     }
 
-    final public function __toString (): string {
+    final public function __toString(): string {
         return "themes/{$this->getId()}";
     }
 
     /**
      * @return Asset[]
      */
-    public function getAssets () {
+    public function getAssets() {
         if (!isset($this->assets)) {
             /**
              * Asset values are not returned by Shopify for this query.
@@ -69,14 +69,14 @@ 
         return $this->assets->getItems();
     }
 
-    final protected function getDir (): string {
+    final protected function getDir(): string {
         return 'themes';
     }
 
     /**
      * @return Asset
      */
-    public function newAsset () {
+    public function newAsset() {
         $this->getAssets();
         return $this->assets->add(Asset::toData($this, ['theme_id' => $this->getId()]));
     }

src/Product.php

Spacing +9 added lines, -9 removed lines

@@ -61,34 +61,34 @@ 
         'variants' => [Variant::class, 'toDataSet']
     ];
 
-    final protected static function getMetafieldType (): string {
+    final protected static function getMetafieldType(): string {
         return 'product';
     }
 
-    final protected static function getSearchPath (): string {
+    final protected static function getSearchPath(): string {
         return 'products';
     }
 
-    final protected static function getSearchType (): string {
+    final protected static function getSearchType(): string {
         return 'products';
     }
 
-    final protected static function getType (): string {
+    final protected static function getType(): string {
         return 'product';
     }
 
-    final public function __toString (): string {
+    final public function __toString(): string {
         return "products/{$this->getId()}";
     }
 
-    final protected function getDir (): string {
+    final protected function getDir(): string {
         return 'products';
     }
 
     /**
      * @return Image
      */
-    public function newImage () {
+    public function newImage() {
         return $this->data['images']->add(Image::toData($this, ['product_id' => $this->getId()]));
     }
 
@@ -97,7 +97,7 @@ 
      * @param string[] $values
      * @return Option
      */
-    public function newOption (string $name, array $values) {
+    public function newOption(string $name, array $values) {
         $option = Option::toData($this, ['product_id' => $this->getId()]);
         $option->setName($name)->setValues($values);
         return $this->data['options']->add($option);
@@ -106,7 +106,7 @@ 
     /**
      * @return Variant
      */
-    public function newVariant () {
+    public function newVariant() {
         return $this->data['variants']->add(Variant::toData($this, ['product_id' => $this->getId()]));
     }
 

src/Report.php

Spacing +5 added lines, -5 removed lines

@@ -21,23 +21,23 @@
 
     use SearchTrait;
 
-    final protected static function getSearchPath (): string {
+    final protected static function getSearchPath(): string {
         return 'reports';
     }
 
-    final protected static function getSearchType (): string {
+    final protected static function getSearchType(): string {
         return 'reports';
     }
 
-    final protected static function getType (): string {
+    final protected static function getType(): string {
         return 'report';
     }
 
-    final public function __toString (): string {
+    final public function __toString(): string {
         return "reports/{$this->getId()}";
     }
 
-    final protected function getDir (): string {
+    final protected function getDir(): string {
         return 'reports';
     }
 }
\ No newline at end of file

src/Base/EntityTrait.php

Spacing +4 added lines, -4 removed lines

@@ -21,14 +21,14 @@ 
      *
      * @return string
      */
-    abstract protected static function getType (): string;
+    abstract protected static function getType(): string;
 
     /**
      * The entity's unique path for GET/PUT/DELETE.
      *
      * @return string
      */
-    abstract public function __toString (): string;
+    abstract public function __toString(): string;
 
     /**
      * @var Api
@@ -41,7 +41,7 @@ 
      * @param mixed ...$extra Other arguments needed by the constructor.
      * @return null|static
      */
-    public static function load (Api $api, array $data, ...$extra) {
+    public static function load(Api $api, array $data, ...$extra) {
         $class = static::class;
         /** @var Data $proto */
         $proto = new $class($api, $data, ...$extra);
@@ -53,7 +53,7 @@ 
         return null;
     }
 
-    public function delete (): void {
+    public function delete(): void {
         /** @var Data $this */
         $this->api->delete($this);
         $this->notify('delete');

src/Base/AbstractEntity.php

Spacing +2 added lines, -2 removed lines

@@ -17,7 +17,7 @@ 
      *
      * @return string
      */
-    abstract protected function getDir (): string;
+    abstract protected function getDir(): string;
 
     /**
      * @var bool
@@ -36,7 +36,7 @@ 
      *
      * @return $this
      */
-    public function save () {
+    public function save() {
         if ($this->isDiff()) {
             $type = static::getType();
             if (!$this->hasId()) {

Braces +1 added lines, -2 removed lines

@@ -41,8 +41,7 @@
             $type = static::getType();
             if (!$this->hasId()) {
                 $remote = $this->api->post($this->getDir(), [$type => $this->getPatch()]);
-            }
-            else {
+            } else {
                 assert($this->canUpdate);
                 $remote = $this->api->put($this, [$type => $this->getPatch()]);
             }

src/Base/SearchTrait.php

Spacing +3 added lines, -3 removed lines

@@ -13,21 +13,21 @@
      * @param array $query
      * @return string
      */
-    abstract protected static function getSearchPath (array $query): string;
+    abstract protected static function getSearchPath(array $query): string;
 
     /**
      * The pluralized key that results are wrapped in.
      *
      * @return string
      */
-    abstract protected static function getSearchType (): string;
+    abstract protected static function getSearchType(): string;
 
     /**
      * @param Api $api
      * @param array $query
      * @return Traversable|static[]
      */
-    public static function search (Api $api, array $query) {
+    public static function search(Api $api, array $query) {
         $continue = !isset($query['limit']);
         $query += ['limit' => 250];
         $class = static::class;

src/Base/DataSet.php

Spacing +14 added lines, -14 removed lines

@@ -31,7 +31,7 @@ 
     /**
      * @param Data[] $items
      */
-    public function __construct (array $items = []) {
+    public function __construct(array $items = []) {
         foreach ($items as $item) {
             $this->add($item);
         }
@@ -44,7 +44,7 @@ 
      * @param Data $item
      * @return Data|mixed
      */
-    public function add (Data $item) {
+    public function add(Data $item) {
         $key = spl_object_hash($item);
         $this->items[$key] = $item;
         $this->diff[$key] = $item;
@@ -68,13 +68,13 @@ 
      *
      * @return $this
      */
-    public function clear () {
+    public function clear() {
         $this->items = [];
         $this->diff = [];
         return $this;
     }
 
-    public function count () {
+    public function count() {
         return count($this->items);
     }
 
@@ -84,7 +84,7 @@ 
      * @param Closure $keep
      * @return $this
      */
-    public function filter (Closure $keep) {
+    public function filter(Closure $keep) {
         $oldItems = $this->items;
         $this->items = array_filter($this->items, $keep);
         $this->diff += array_diff_key($oldItems, $this->items);
@@ -95,7 +95,7 @@ 
      * @param Closure $selector
      * @return null|Data
      */
-    public function first (Closure $selector) {
+    public function first(Closure $selector) {
         foreach ($this->items as $item) {
             if ($selector->__invoke($item)) {
                 return $item;
@@ -104,7 +104,7 @@ 
         return null;
     }
 
-    final public function getData (): array {
+    final public function getData(): array {
         return array_map(function(Data $item) {
             return $item->getData();
         }, $this->getItems());
@@ -113,24 +113,24 @@ 
     /**
      * @return Data[]
      */
-    final public function getItems (): array {
+    final public function getItems(): array {
         return array_values($this->items);
     }
 
     /**
      * @return Traversable|Data[]
      */
-    final public function getIterator () {
+    final public function getIterator() {
         yield from $this->getItems();
     }
 
-    protected function getPatch (): array {
+    protected function getPatch(): array {
         return array_map(function(Data $item) {
             return $item->getPatch();
         }, $this->getItems());
     }
 
-    protected function isDiff (): bool {
+    protected function isDiff(): bool {
         if ($this->diff) {
             return true;
         }
@@ -147,7 +147,7 @@ 
      *
      * @return $this
      */
-    public function save () {
+    public function save() {
         foreach ($this->items as $item) {
             assert($item instanceof EntityInterface);
             $item->save(); // onSave() clears the diff for the item.
@@ -165,7 +165,7 @@ 
      * @param Closure $selector
      * @return Data[]
      */
-    public function select (Closure $selector) {
+    public function select(Closure $selector) {
         return array_filter($this->items, $selector);
     }
 
@@ -173,7 +173,7 @@ 
      * @param Closure $cmp
      * @return $this
      */
-    public function sort (Closure $cmp) {
+    public function sort(Closure $cmp) {
         $oldKeys = array_keys($this->items);
         uasort($this->items, $cmp);
         $newKeys = array_keys($this->items);

src/Base/Data.php

Braces +1 added lines, -2 removed lines

@@ -128,8 +128,7 @@
     public function __construct ($caller, array $data = []) {
         if ($caller instanceof Api) {
             $this->api = $caller;
-        }
-        elseif ($caller instanceof Data) {
+        } elseif ($caller instanceof Data) {
             $this->api = $caller->api;
         }
         $this->import($data);

Spacing +22 added lines, -22 removed lines

@@ -82,7 +82,7 @@ 
      * @param null|array $data
      * @return null|static|mixed
      */
-    protected static function toData ($caller, ?array $data) {
+    protected static function toData($caller, ?array $data) {
         return isset($data) ? new static($caller, $data) : null;
     }
 
@@ -93,7 +93,7 @@ 
      * @param null|array $table
      * @return DataSet
      */
-    protected static function toDataSet ($caller, array $table = null) {
+    protected static function toDataSet($caller, array $table = null) {
         return new DataSet(static::toList($caller, $table));
     }
 
@@ -104,7 +104,7 @@ 
      * @param null|array[] $table
      * @return Hash
      */
-    protected static function toHash ($caller, array $table = null) {
+    protected static function toHash($caller, array $table = null) {
         unset($caller);
         return new Hash($table ?? []);
     }
@@ -116,7 +116,7 @@ 
      * @param null|array[] $table
      * @return array|static[]
      */
-    protected static function toList ($caller, array $table = null) {
+    protected static function toList($caller, array $table = null) {
         return array_map(function(array $row) use ($caller) {
             return static::toData($caller, $row);
         }, $table ?? []);
@@ -126,7 +126,7 @@ 
      * @param Api|Data $caller
      * @param array $data
      */
-    public function __construct ($caller, array $data = []) {
+    public function __construct($caller, array $data = []) {
         if ($caller instanceof Api) {
             $this->api = $caller;
         }
@@ -141,9 +141,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('/^(filter|get|has|is|on|select|set|sort)(.+)$/', $method, $call);
             $call[2] = preg_replace_callback('/[A-Z]/', function(array $match) {
                 return '_' . strtolower($match[0]);
@@ -156,7 +156,7 @@ 
      * @param array $data
      * @return $this
      */
-    protected function apply (array $data) {
+    protected function apply(array $data) {
         $this->data = $data + $this->data;
         $this->diff += array_fill_keys(array_keys($data), true);
         return $this;
@@ -166,7 +166,7 @@ 
      * @param string $key
      * @return bool
      */
-    final protected function exists (string $key): bool {
+    final protected function exists(string $key): bool {
         return array_key_exists($key, $this->data);
     }
 
@@ -179,7 +179,7 @@ 
      * @param callable $filter
      * @return $this
      */
-    protected function filter (string $key, callable $filter) {
+    protected function filter(string $key, callable $filter) {
         assert($this->data[$key] instanceof DataSet);
         $this->data[$key]->filter($filter);
         return $this;
@@ -195,7 +195,7 @@ 
      * @param null $default
      * @return mixed
      */
-    protected function get (string $key, $default = null) {
+    protected function get(string $key, $default = null) {
         $value = $this->data[$key] ?? $default;
         return $value instanceof DataSet ? $value->getItems() : $value;
     }
@@ -203,7 +203,7 @@ 
     /**
      * @return array
      */
-    public function getData (): array {
+    public function getData(): array {
         $data = array_map(function($value) {
             if ($value instanceof AbstractData) {
                 return $value->getData();
@@ -217,7 +217,7 @@ 
     /**
      * @return array
      */
-    protected function getPatch (): array {
+    protected function getPatch(): array {
         $patch = [];
         foreach ($this->data as $key => $value) {
             // mapped structures
@@ -247,7 +247,7 @@ 
      * @param string $key
      * @return bool
      */
-    protected function has (string $key) {
+    protected function has(string $key) {
         $value = $this->data[$key] ?? null;
         if (isset($value)) {
             if (is_countable($value)) {
@@ -264,7 +264,7 @@ 
      * @param array $data
      * @return $this
      */
-    protected function import (array $data) {
+    protected function import(array $data) {
         foreach ($this->map as $key => $callable) {
             $data[$key] = call_user_func($callable, $this, $data[$key] ?? null);
         }
@@ -284,14 +284,14 @@ 
      * @param string $key
      * @return bool
      */
-    protected function is (string $key): bool {
+    protected function is(string $key): bool {
         return !empty($this->data[$key]);
     }
 
     /**
      * @return bool
      */
-    protected function isDiff (): bool {
+    protected function isDiff(): bool {
         if ($this->diff) {
             return true;
         }
@@ -317,7 +317,7 @@ 
      * @param mixed ...$args
      * @return $this
      */
-    protected function notify (string $event, ...$args) {
+    protected function notify(string $event, ...$args) {
         $args[] = $this;
         if (in_array($event, $this->_events)) {
             throw new LogicException('Circular event: ' . implode(' -> ', $this->_events) . " -> **{$event}**");
@@ -345,7 +345,7 @@ 
      * @param Closure $observer
      * @return $this
      */
-    protected function on (string $event, Closure $observer) {
+    protected function on(string $event, Closure $observer) {
         $this->_observers[$event][] = $observer;
         return $this;
     }
@@ -359,7 +359,7 @@ 
      * @param Closure $selector
      * @return Data[]
      */
-    protected function select (string $key, Closure $selector) {
+    protected function select(string $key, Closure $selector) {
         assert($this->data[$key] instanceof DataSet);
         return $this->data[$key]->select($selector);
     }
@@ -373,7 +373,7 @@ 
      * @param mixed $value
      * @return $this
      */
-    protected function set (string $key, $value) {
+    protected function set(string $key, $value) {
         $this->data[$key] = is_array($value) ? array_values($value) : $value;
         $this->diff[$key] = true;
         return $this;
@@ -388,7 +388,7 @@ 
      * @param Closure $cmp
      * @return $this
      */
-    protected function sort (string $key, Closure $cmp) {
+    protected function sort(string $key, Closure $cmp) {
         $dataSet = $this->data[$key];
         assert($dataSet instanceof DataSet);
         $dataSet->sort($cmp);