maksimru / multi-unit-models

src/Exceptions/NotSupportedMultiUnitField.php

Spacing +1 added lines, -1 removed lines

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

src/MultiUnitModelsServiceProvider.php

Spacing +2 added lines, -2 removed lines

@@ -20,7 +20,7 @@ 
         if ($this->app->runningInConsole()) {
             if (!str_contains($this->app->version(), 'Lumen')) {
                 $this->publishes([
-                    __DIR__.'/../config/multi-units.php' => config_path('multi-units.php'),
+                    __DIR__ . '/../config/multi-units.php' => config_path('multi-units.php'),
                 ], 'config');
             }
         }
@@ -50,6 +50,6 @@ 
      */
     public function register()
     {
-        $this->mergeConfigFrom(__DIR__.'/../config/multi-units.php', 'multi-units');
+        $this->mergeConfigFrom(__DIR__ . '/../config/multi-units.php', 'multi-units');
     }
 }

src/Traits/MultiUnitSupport.php

Doc Comments +1 added lines, -1 removed lines

@@ -166,7 +166,7 @@
     }
 
     /**
-     * @param                   $field
+     * @param                   string $field
      * @param AbstractUnit|null $unit
      *
      * @throws NotSupportedMultiUnitField

Spacing +14 added lines, -14 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, true),
@@ -86,7 +86,7 @@ 
              * @var Model|MultiUnitSupport $model
              */
             foreach (Arr::only($model->getMultiUnitColumns(), array_keys($model->getDirty())) as $unitBasedColumn => $options) {
-                $model->{$unitBasedColumn.$model->getUnitConversionDataPostfix()} = json_encode(
+                $model->{$unitBasedColumn . $model->getUnitConversionDataPostfix()} = json_encode(
                     $model->calculateMultiUnitConversionData(
                         $model->getDirty()[$unitBasedColumn],
                         $model->getMultiUnitFieldUnit($unitBasedColumn, true),
@@ -98,7 +98,7 @@ 
         });
     }
 
-    private function forgetUnitsInput(){
+    private function forgetUnitsInput() {
         //prevent saving of unit columns
         foreach ($this->getUnitConversionUnitColumns() as $unitColumn) {
             if (isset($this->attributes[$unitColumn])) {
@@ -116,7 +116,7 @@ 
      */
     private function calculateMultiUnitConversionData($value, AbstractUnit $unit, $requiredUnits)
     {
-        if(is_null($value))
+        if (is_null($value))
             return null;
 
         $conversionData = [];
@@ -133,7 +133,7 @@ 
 
     public function getMultiUnitExistingConversionData($field)
     {
-        return json_decode($this->{$field.$this->getUnitConversionDataPostfix()} ?? null);
+        return json_decode($this->{$field . $this->getUnitConversionDataPostfix()} ?? null);
     }
 
     /**
@@ -329,13 +329,13 @@ 
      */
     protected function getMultiUnitFieldUnit($field, $preferDefault = false)
     {
-        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;
                 }
             }
@@ -347,15 +347,15 @@ 
     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)
     {
-        if (isset($this->{$field.$this->getUnitAttributePostfix()})) {
-            $this->{$field.$this->getUnitAttributePostfix()} = $unit->getId();
+        if (isset($this->{$field . $this->getUnitAttributePostfix()})) {
+            $this->{$field . $this->getUnitAttributePostfix()} = $unit->getId();
         }
         $this->forgetMultiUnitFieldUnitInput($field);
     }
@@ -424,7 +424,7 @@ 
      */
     private function processMultiUnitFieldChanges($field, $value)
     {
-        if(!is_null($value)) {
+        if (!is_null($value)) {
             $existingConversionData = $this->getMultiUnitExistingConversionData($field);
             if (!is_null($existingConversionData)) {
                 $inputUnit = $this->getMultiUnitFieldUnit($field);

Braces +3 added lines, -2 removed lines

@@ -116,8 +116,9 @@
      */
     private function calculateMultiUnitConversionData($value, AbstractUnit $unit, $requiredUnits)
     {
-        if(is_null($value))
-            return null;
+        if(is_null($value)) {
+                    return null;
+        }
 
         $conversionData = [];
         foreach ($requiredUnits as $requiredUnitClass) {

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);
     }
 }