maksimru / multi-unit-models

src/Exceptions/NotSupportedMultiUnitFieldUnit.php

Spacing +1 added lines, -1 removed lines

@@ -8,6 +8,6 @@
 {
     public function __construct($field, $unit, $code = 0, Throwable $previous = null)
     {
-        parent::__construct('Requested field "'.$field.'" doesn\'t support ' . $unit, $code, $previous);
+        parent::__construct('Requested field "' . $field . '" doesn\'t support ' . $unit, $code, $previous);
     }
 }

src/Traits/MultiUnitSupport.php

Spacing +11 added lines, -11 removed lines

@@ -18,14 +18,14 @@ 
     private function getUnitConversionDataColumns()
     {
         return array_map(function ($column) {
-            return $column.$this->getUnitConversionDataPostfix();
+            return $column . $this->getUnitConversionDataPostfix();
         }, array_keys($this->getMultiUnitColumns()));
     }
 
     private function getUnitConversionUnitColumns()
     {
         return array_map(function ($column) {
-            return $column.$this->getUnitAttributePostfix();
+            return $column . $this->getUnitAttributePostfix();
         }, array_keys($this->getMultiUnitColumns()));
     }
 
@@ -66,7 +66,7 @@ 
              */
             foreach ($model->getMultiUnitColumns() as $unitBasedColumn => $options) {
                 if (isset($model->attributes[$unitBasedColumn])) {
-                    $model->{$unitBasedColumn.$model->getUnitConversionDataPostfix()} = json_encode(
+                    $model->{$unitBasedColumn . $model->getUnitConversionDataPostfix()} = json_encode(
                         $model->calculateMultiUnitConversionData(
                             $model->attributes[$unitBasedColumn],
                             $model->getMultiUnitFieldUnit($unitBasedColumn),
@@ -91,7 +91,7 @@ 
              * @var Model|MultiUnitSupport $model
              */
             foreach (Arr::only($model->getMultiUnitColumns(), array_keys($model->getDirty())) as $unitBasedColumn => $options) {
-                $model->{$unitBasedColumn.$model->getUnitConversionDataPostfix()} = json_encode($model->calculateMultiUnitConversionData($model->getDirty()[$unitBasedColumn], new $options['default_unit'](), $options['supported_units']));
+                $model->{$unitBasedColumn . $model->getUnitConversionDataPostfix()} = json_encode($model->calculateMultiUnitConversionData($model->getDirty()[$unitBasedColumn], new $options['default_unit'](), $options['supported_units']));
             }
         });
     }
@@ -119,7 +119,7 @@ 
 
     public function getMultiUnitExistingConversionData($field)
     {
-        return json_decode($this->{$field.$this->getUnitConversionDataPostfix()} ?? null);
+        return json_decode($this->{$field . $this->getUnitConversionDataPostfix()} ?? null);
     }
 
     /**
@@ -219,7 +219,7 @@ 
                     break;
                 }
             }
-            if($found)
+            if ($found)
                 $this->multiUnitSelectedUnits[$field] = $unitClass;
             else
                 throw new NotSupportedMultiUnitFieldUnit($field, $unit);
@@ -315,13 +315,13 @@ 
      */
     protected function getMultiUnitFieldUnit($field)
     {
-        if (isset($this->{$field.$this->getUnitAttributePostfix()})) {
+        if (isset($this->{$field . $this->getUnitAttributePostfix()})) {
             foreach ($this->getMultiUnitFieldSupportedUnits($field) as $unitClass) {
                 /**
                  * @var AbstractUnit $unit
                  */
                 $unit = new $unitClass();
-                if (strtolower($unit->getId()) == strtolower($this->{$field.$this->getUnitAttributePostfix()})) {
+                if (strtolower($unit->getId()) == strtolower($this->{$field . $this->getUnitAttributePostfix()})) {
                     return $unit;
                 }
             }
@@ -333,14 +333,14 @@ 
     protected function forgetMultiUnitFieldUnitInput($field)
     {
         //prevent column_units to by saved to DB
-        if (isset($this->attributes[$field.$this->getUnitAttributePostfix()])) {
-            $this->syncOriginalAttribute($field.$this->getUnitAttributePostfix());
+        if (isset($this->attributes[$field . $this->getUnitAttributePostfix()])) {
+            $this->syncOriginalAttribute($field . $this->getUnitAttributePostfix());
         }
     }
 
     protected function setMultiUnitFieldUnit($field, AbstractUnit $unit)
     {
-        $this->{$field.$this->getUnitAttributePostfix()} = $unit->getId();
+        $this->{$field . $this->getUnitAttributePostfix()} = $unit->getId();
         $this->forgetMultiUnitFieldUnitInput($field);
     }
 

Braces +7 added lines, -6 removed lines

@@ -219,13 +219,14 @@
                     break;
                 }
             }
-            if($found)
-                $this->multiUnitSelectedUnits[$field] = $unitClass;
-            else
-                throw new NotSupportedMultiUnitFieldUnit($field, $unit);
+            if($found) {
+                            $this->multiUnitSelectedUnits[$field] = $unitClass;
+            } else {
+                            throw new NotSupportedMultiUnitFieldUnit($field, $unit);
+            }
+        } else {
+                    throw new NotSupportedMultiUnitField($field);
         }
-        else
-            throw new NotSupportedMultiUnitField($field);
     }