iamsaint / yml

src/components/Shop.php

Spacing +1 added lines, -1 removed lines

@@ -57,7 +57,7 @@
      */
     public function addCurrency(Currency $currency)
     {
-        if($is_valid = $currency->validate()) {
+        if ($is_valid = $currency->validate()) {
             $this->currencies[] = $currency;
         } else {
             $this->errors['currency'][] = $currency->errors;

src/components/Offer.php

Indentation +1 added lines, -1 removed lines

@@ -331,7 +331,7 @@
         $this->params = $params;
         return $this;
     }
-   /**
+    /**
      * @param mixed $params
      * @return Offer
      */

Spacing +20 added lines, -20 removed lines

@@ -41,13 +41,13 @@ 
     {
         $this->writer->startElement('offer');
         $this->writer->writeAttribute('id', $this->id);
-        if(null !== $this->type) $this->writer->writeAttribute('type', $this->type);
+        if (null !== $this->type) $this->writer->writeAttribute('type', $this->type);
         $this->writer->writeAttribute('available', $this->available ? 'true' : 'false');
 
-        if(null !== $this->url) $this->writer->writeElement('url', $this->url);
-        if(null !== $this->price) $this->writer->writeElement('price', $this->price);
-        if(null !== $this->currencyId) $this->writer->writeElement('currencyId', $this->currencyId);
-        if(null !== $this->categoryId) $this->writer->writeElement('categoryId', $this->categoryId);
+        if (null !== $this->url) $this->writer->writeElement('url', $this->url);
+        if (null !== $this->price) $this->writer->writeElement('price', $this->price);
+        if (null !== $this->currencyId) $this->writer->writeElement('currencyId', $this->currencyId);
+        if (null !== $this->categoryId) $this->writer->writeElement('categoryId', $this->categoryId);
 
         if (count($this->pictures) > 0) {
             foreach ($this->pictures as $picture) {
@@ -55,17 +55,17 @@ 
             }
         }
 
-        if($this->store) $this->writer->writeElement('store', $this->store ? 'true' : 'false');
-        if($this->delivery) $this->writer->writeElement('delivery', $this->delivery ? 'true' : 'false');
-        if(null !== $this->name) $this->writer->writeElement('name', $this->name);
-        if(null !== $this->vendor) $this->writer->writeElement('vendor', $this->vendor);
-        if(null !== $this->vendorCode) $this->writer->writeElement('vendorCode', $this->vendorCode);
-        if(null !== $this->model) $this->writer->writeElement('model', $this->model);
-        if(null !== $this->description) $this->writer->writeElement('description', $this->description);
-        if(null !== $this->salesNotes) $this->writer->writeElement('sales_notes', $this->salesNotes);
-        if(null !== $this->barcode) $this->writer->writeElement('barcode', $this->barcode);
-        if(null !== $this->age) $this->writer->writeElement('age', $this->age);
-        if($this->manufacturerWarranty) $this->writer->writeElement('manufacturer_warranty', $this->manufacturerWarranty ? 'true' : 'false');
+        if ($this->store) $this->writer->writeElement('store', $this->store ? 'true' : 'false');
+        if ($this->delivery) $this->writer->writeElement('delivery', $this->delivery ? 'true' : 'false');
+        if (null !== $this->name) $this->writer->writeElement('name', $this->name);
+        if (null !== $this->vendor) $this->writer->writeElement('vendor', $this->vendor);
+        if (null !== $this->vendorCode) $this->writer->writeElement('vendorCode', $this->vendorCode);
+        if (null !== $this->model) $this->writer->writeElement('model', $this->model);
+        if (null !== $this->description) $this->writer->writeElement('description', $this->description);
+        if (null !== $this->salesNotes) $this->writer->writeElement('sales_notes', $this->salesNotes);
+        if (null !== $this->barcode) $this->writer->writeElement('barcode', $this->barcode);
+        if (null !== $this->age) $this->writer->writeElement('age', $this->age);
+        if ($this->manufacturerWarranty) $this->writer->writeElement('manufacturer_warranty', $this->manufacturerWarranty ? 'true' : 'false');
 
         if (count($this->params) > 0) {
             foreach ($this->params as $param) {
@@ -139,11 +139,11 @@ 
      */
     public function addParam(OfferParam $param)
     {
-        if($param->validate()) {
+        if ($param->validate()) {
             $this->params[] = $param;
             return true;
         } else {
-            $this->errors['params'][]=$param->errors;
+            $this->errors['params'][] = $param->errors;
             return false;
         }
     }
@@ -154,11 +154,11 @@ 
      */
     public function addPicture(OfferPicture $picrure)
     {
-        if($picrure->validate()) {
+        if ($picrure->validate()) {
             $this->pictures[] = $picrure;
             return true;
         } else {
-            $this->errors['pictures'][]=$picrure->errors;
+            $this->errors['pictures'][] = $picrure->errors;
             return false;
         }
     }

Braces +48 added lines, -16 removed lines

@@ -41,13 +41,23 @@ 
     {
         $this->writer->startElement('offer');
         $this->writer->writeAttribute('id', $this->id);
-        if(null !== $this->type) $this->writer->writeAttribute('type', $this->type);
+        if(null !== $this->type) {
+            $this->writer->writeAttribute('type', $this->type);
+        }
         $this->writer->writeAttribute('available', $this->available ? 'true' : 'false');
 
-        if(null !== $this->url) $this->writer->writeElement('url', $this->url);
-        if(null !== $this->price) $this->writer->writeElement('price', $this->price);
-        if(null !== $this->currencyId) $this->writer->writeElement('currencyId', $this->currencyId);
-        if(null !== $this->categoryId) $this->writer->writeElement('categoryId', $this->categoryId);
+        if(null !== $this->url) {
+            $this->writer->writeElement('url', $this->url);
+        }
+        if(null !== $this->price) {
+            $this->writer->writeElement('price', $this->price);
+        }
+        if(null !== $this->currencyId) {
+            $this->writer->writeElement('currencyId', $this->currencyId);
+        }
+        if(null !== $this->categoryId) {
+            $this->writer->writeElement('categoryId', $this->categoryId);
+        }
 
         if (count($this->pictures) > 0) {
             foreach ($this->pictures as $picture) {
@@ -55,17 +65,39 @@ 
             }
         }
 
-        if($this->store) $this->writer->writeElement('store', $this->store ? 'true' : 'false');
-        if($this->delivery) $this->writer->writeElement('delivery', $this->delivery ? 'true' : 'false');
-        if(null !== $this->name) $this->writer->writeElement('name', $this->name);
-        if(null !== $this->vendor) $this->writer->writeElement('vendor', $this->vendor);
-        if(null !== $this->vendorCode) $this->writer->writeElement('vendorCode', $this->vendorCode);
-        if(null !== $this->model) $this->writer->writeElement('model', $this->model);
-        if(null !== $this->description) $this->writer->writeElement('description', $this->description);
-        if(null !== $this->salesNotes) $this->writer->writeElement('sales_notes', $this->salesNotes);
-        if(null !== $this->barcode) $this->writer->writeElement('barcode', $this->barcode);
-        if(null !== $this->age) $this->writer->writeElement('age', $this->age);
-        if($this->manufacturerWarranty) $this->writer->writeElement('manufacturer_warranty', $this->manufacturerWarranty ? 'true' : 'false');
+        if($this->store) {
+            $this->writer->writeElement('store', $this->store ? 'true' : 'false');
+        }
+        if($this->delivery) {
+            $this->writer->writeElement('delivery', $this->delivery ? 'true' : 'false');
+        }
+        if(null !== $this->name) {
+            $this->writer->writeElement('name', $this->name);
+        }
+        if(null !== $this->vendor) {
+            $this->writer->writeElement('vendor', $this->vendor);
+        }
+        if(null !== $this->vendorCode) {
+            $this->writer->writeElement('vendorCode', $this->vendorCode);
+        }
+        if(null !== $this->model) {
+            $this->writer->writeElement('model', $this->model);
+        }
+        if(null !== $this->description) {
+            $this->writer->writeElement('description', $this->description);
+        }
+        if(null !== $this->salesNotes) {
+            $this->writer->writeElement('sales_notes', $this->salesNotes);
+        }
+        if(null !== $this->barcode) {
+            $this->writer->writeElement('barcode', $this->barcode);
+        }
+        if(null !== $this->age) {
+            $this->writer->writeElement('age', $this->age);
+        }
+        if($this->manufacturerWarranty) {
+            $this->writer->writeElement('manufacturer_warranty', $this->manufacturerWarranty ? 'true' : 'false');
+        }
 
         if (count($this->params) > 0) {
             foreach ($this->params as $param) {

src/validators/one_of.php

Spacing +2 added lines, -2 removed lines

@@ -9,8 +9,8 @@
 
     public function validate(Object &$object, array $attributes, array $options = []):void
     {
-        foreach ($attributes as $attribute){
-            if( !in_array($object->$attribute, $options)) {
+        foreach ($attributes as $attribute) {
+            if (!in_array($object->$attribute, $options)) {
                 $object->addError($attribute, $attribute.' must be element of ['.implode(', ', $options).']');
             }
         }

src/validators/url.php

Spacing +2 added lines, -2 removed lines

@@ -9,8 +9,8 @@
 
     public function validate(Object &$object, array $attributes, array $options = []):void
     {
-        foreach ($attributes as $attribute){
-            if(!filter_var($object->$attribute, FILTER_VALIDATE_URL)) {
+        foreach ($attributes as $attribute) {
+            if (!filter_var($object->$attribute, FILTER_VALIDATE_URL)) {
                 $object->addError($attribute, $attribute.' must be valid url');
             }
         }

src/validators/required.php

Spacing +2 added lines, -2 removed lines

@@ -9,8 +9,8 @@
 
     public function validate(Object &$object, array $attributes, array $options = []):void
     {
-        foreach ($attributes as $attribute){
-            if( strlen(trim( (string) $object->$attribute )) === 0) {
+        foreach ($attributes as $attribute) {
+            if (strlen(trim((string) $object->$attribute)) === 0) {
                 $object->addError($attribute, $attribute.' cannot be empty');
             }
         }

src/Object.php

Spacing +2 added lines, -2 removed lines

@@ -72,14 +72,14 @@
                 throw new IncorrectRuleExceptin('Rule name must be a string');
             }
 
-            $class = '\\iamsaint\yml\\validators\\' . $rule[1];
+            $class = '\\iamsaint\yml\\validators\\'.$rule[1];
 
             if (!class_exists($class)) {
                 throw new IncorrectRuleExceptin('Validator not found');
             }
 
             $attributes = is_array($rule[0]) ? $rule[0] : [$rule[0]];
-            (new $class())->validate($this, $attributes, $rule[2] ? : []);
+            (new $class())->validate($this, $attributes, $rule[2] ?: []);
         }
 
         return count($this->errors) === 0;