techdivision / import-attribute

src/Subjects/BunchSubject.php

Spacing +1 added lines, -1 removed lines

@@ -203,7 +203,7 @@
      */
     public function preLoadAttributeId(array $attribute)
     {
-        $this->preLoadedAttributeIds[$attribute[MemberNames::ATTRIBUTE_CODE]]= $attribute[MemberNames::ATTRIBUTE_ID];
+        $this->preLoadedAttributeIds[$attribute[MemberNames::ATTRIBUTE_CODE]] = $attribute[MemberNames::ATTRIBUTE_ID];
     }
 
     /**

src/Actions/Processors/CatalogAttributeUpdateProcessor.php

Spacing +1 added lines, -1 removed lines

@@ -76,7 +76,7 @@
             unset($keys[array_search(EntityStatus::MEMBER_NAME, $keys, true)]);
 
             // prepare the SET part of the SQL statement
-            array_walk($keys, function (&$value, $key) {
+            array_walk($keys, function(&$value, $key) {
                 $value = sprintf('%s=:%s', $value, $value);
             });
 

src/Observers/AttributeOptionValueExportObserver.php

Spacing +1 added lines, -1 removed lines

@@ -61,7 +61,7 @@
         $artefacts = array();
 
         // load the attribute option values for the custom store views
-        $attributeOptionValues = $this->getValue(ColumnKeys::ATTRIBUTE_OPTION_VALUES, array(), function ($value) {
+        $attributeOptionValues = $this->getValue(ColumnKeys::ATTRIBUTE_OPTION_VALUES, array(), function($value) {
             return $this->explode($value, $this->getMultipleFieldDelimiter());
         });
 

src/Observers/AttributeOptionExportObserver.php

Spacing +3 added lines, -3 removed lines

@@ -58,13 +58,13 @@
         $artefacts = array();
 
         // load the attribute option values/positions
-        $attributeOptionValues = $this->getValue(ColumnKeys::ATTRIBUTE_OPTION_VALUES, array(), function ($value) {
+        $attributeOptionValues = $this->getValue(ColumnKeys::ATTRIBUTE_OPTION_VALUES, array(), function($value) {
             return $this->explode($value, $this->getMultipleFieldDelimiter());
         });
-        $attributeOptionSortOrder = $this->getValue(ColumnKeys::ATTRIBUTE_OPTION_SORT_ORDER, array(), function ($value) {
+        $attributeOptionSortOrder = $this->getValue(ColumnKeys::ATTRIBUTE_OPTION_SORT_ORDER, array(), function($value) {
             return $this->explode($value, $this->getMultipleFieldDelimiter());
         });
-        $attributeOptionSwatch = $this->getValue(ColumnKeys::ATTRIBUTE_OPTION_SWATCH, array(), function ($value) {
+        $attributeOptionSwatch = $this->getValue(ColumnKeys::ATTRIBUTE_OPTION_SWATCH, array(), function($value) {
             return $this->explode($value, $this->getMultipleValueDelimiter());
         });
 

src/Observers/AttributeOptionSwatchFileUploadObserver.php

Spacing +1 added lines, -1 removed lines

@@ -47,7 +47,7 @@
         ));
 
         // skip this step for color swatches and text swatches
-        if (isset($attributeOptionSwatch[MemberNames::TYPE]) &&  $attributeOptionSwatch[MemberNames::TYPE] === SwatchTypes::IMAGE) {
+        if (isset($attributeOptionSwatch[MemberNames::TYPE]) && $attributeOptionSwatch[MemberNames::TYPE] === SwatchTypes::IMAGE) {
             // upload the file to the configured directory
             $imagePath = $this->getSubject()->uploadFile($attributeOptionSwatch[MemberNames::VALUE]);
 

src/Loaders/RawEntityLoader.php

Spacing +2 added lines, -2 removed lines

@@ -86,8 +86,8 @@
             // load the columns from the metadata
             $columns = array_filter(
                 $columnMetadataLoader->load($entityType),
-                function ($value) {
-                    return $value['Key'] !== 'PRI' && $value['Null'] === 'NO' ;
+                function($value) {
+                    return $value['Key'] !== 'PRI' && $value['Null'] === 'NO';
                 }
             );
             // initialize the raw entities and their default values, if available

tests/unit/Observers/CatalogAttributeUpdateObserverTest.php

Spacing +8 added lines, -8 removed lines

@@ -137,16 +137,16 @@ 
                     ->willReturn($row);
         $mockSubject->expects($this->any())
                     ->method('hasHeader')
-                    ->willReturnCallback(function ($key) use ($headers) {
+                    ->willReturnCallback(function($key) use ($headers) {
                         return array_key_exists($key, $headers);
                     });
         $mockSubject->expects($this->any())
                     ->method('getHeader')
-                    ->willReturnCallback(function ($key) use ($headers) {
+                    ->willReturnCallback(function($key) use ($headers) {
                         if (array_key_exists($key, $headers)) {
                             return $headers[$key];
                         }
-                        throw new \InvalidArgumentException(sprintf('Header %s is not available', $key));;
+                        throw new \InvalidArgumentException(sprintf('Header %s is not available', $key)); ;
                     });
         $mockSubject->expects($this->once())
                     ->method('hasBeenProcessed')
@@ -211,7 +211,7 @@ 
         // mock the entity merger method
         $this->mockEntityMerger->expects($this->any())
              ->method('merge')
-             ->willReturnCallback(function ($observer, $entity, $attr) use ($headers) {
+             ->willReturnCallback(function($observer, $entity, $attr) use ($headers) {
                  foreach (array_keys($attr) as $key) {
                      if (isset($headers[$key])) {
                          continue;
@@ -331,16 +331,16 @@ 
                     ->willReturn($lastAttributeId = 1001);
         $mockSubject->expects($this->any())
                     ->method('hasHeader')
-                    ->willReturnCallback(function ($key) use ($headers) {
+                    ->willReturnCallback(function($key) use ($headers) {
                         return array_key_exists($key, $headers);
                     });
         $mockSubject->expects($this->any())
                     ->method('getHeader')
-                    ->willReturnCallback(function ($key) use ($headers) {
+                    ->willReturnCallback(function($key) use ($headers) {
                         if (array_key_exists($key, $headers)) {
                             return $headers[$key];
                         }
-                        throw new \InvalidArgumentException(sprintf('Header %s is not available', $key));;
+                        throw new \InvalidArgumentException(sprintf('Header %s is not available', $key)); ;
                     });
 
         // initialize the existing entity
@@ -424,7 +424,7 @@ 
         // mock the entity merger method
         $this->mockEntityMerger->expects($this->any())
              ->method('merge')
-             ->willReturnCallback(function ($observer, $entity, $attr) use ($headers) {
+             ->willReturnCallback(function($observer, $entity, $attr) use ($headers) {
                  foreach (array_keys($attr) as $key) {
                      if (isset($headers[$key])) {
                          continue;