techdivision / import-product-url-rewrite

tests/unit/Observers/UrlRewriteObserverTest.php

Spacing +2 added lines, -2 removed lines

@@ -456,7 +456,7 @@ 
         $mockSubject->expects($this->any())
                     ->method('getCategory')
                     ->will(
-                        $this->returnCallback(function ($categoryId, $storeViewCode) use ($categories) {
+                        $this->returnCallback(function($categoryId, $storeViewCode) use ($categories) {
                             foreach ($categories as $category) {
                                 if ((int) $category[MemberNames::ENTITY_ID] === (int) $categoryId) {
                                     return $category;
@@ -483,7 +483,7 @@ 
         $mockSubject->expects($this->any())
                     ->method('getCoreConfigData')
                     ->will(
-                        $this->returnCallback(function ($arg1, $arg2) {
+                        $this->returnCallback(function($arg1, $arg2) {
                             return $arg2;
                         })
                     );

Indentation +11 added lines, -11 removed lines

@@ -73,8 +73,8 @@ 
 
         // initialize a mock processor instance
         $this->mockProductUrlRewriteProcessor = $this->getMockBuilder('TechDivision\Import\Product\UrlRewrite\Services\ProductUrlRewriteProcessorInterface')
-                                                     ->setMethods(get_class_methods('TechDivision\Import\Product\UrlRewrite\Services\ProductUrlRewriteProcessorInterface'))
-                                                     ->getMock();
+                                                        ->setMethods(get_class_methods('TechDivision\Import\Product\UrlRewrite\Services\ProductUrlRewriteProcessorInterface'))
+                                                        ->getMock();
 
         // initialize the observer
         $this->observer = new UrlRewriteObserver($this->mockProductUrlRewriteProcessor);
@@ -350,12 +350,12 @@ 
 
         // initialize the categories
         $categories = array(
-             $path1 = 'Default Category'                                => array(MemberNames::ENTITY_ID => 2, MemberNames::PARENT_ID => 1, MemberNames::IS_ANCHOR => null, MemberNames::URL_PATH => null, MemberNames::PATH => $path1),
-             $path2 = 'Default Category/Men'                            => array(MemberNames::ENTITY_ID => 3, MemberNames::PARENT_ID => 2, MemberNames::IS_ANCHOR => null, MemberNames::URL_PATH => 'men', MemberNames::PATH => $path2),
-             $path3 = 'Default Category/Men/Tops'                       => array(MemberNames::ENTITY_ID => 4, MemberNames::PARENT_ID => 3, MemberNames::IS_ANCHOR => null, MemberNames::URL_PATH => 'men/tops-men', MemberNames::PATH => $path3),
-             $path4 = 'Default Category/Men/Tops/Hoodies & Sweatshirts' => array(MemberNames::ENTITY_ID => 5, MemberNames::PARENT_ID => 4, MemberNames::IS_ANCHOR => null, MemberNames::URL_PATH => 'men/tops-men/hoodies-and-sweatshirts-men', MemberNames::PATH => $path4),
-             $path5 = 'Default Category/Collections'                    => array(MemberNames::ENTITY_ID => 6, MemberNames::PARENT_ID => 3, MemberNames::IS_ANCHOR => null, MemberNames::URL_PATH => 'collections', MemberNames::PATH => $path5),
-             $path6 = 'Default Category/Collections/Eco Friendly'       => array(MemberNames::ENTITY_ID => 7, MemberNames::PARENT_ID => 6, MemberNames::IS_ANCHOR => null, MemberNames::URL_PATH => 'collections/eco-friendly', MemberNames::PATH => $path6),
+                $path1 = 'Default Category'                                => array(MemberNames::ENTITY_ID => 2, MemberNames::PARENT_ID => 1, MemberNames::IS_ANCHOR => null, MemberNames::URL_PATH => null, MemberNames::PATH => $path1),
+                $path2 = 'Default Category/Men'                            => array(MemberNames::ENTITY_ID => 3, MemberNames::PARENT_ID => 2, MemberNames::IS_ANCHOR => null, MemberNames::URL_PATH => 'men', MemberNames::PATH => $path2),
+                $path3 = 'Default Category/Men/Tops'                       => array(MemberNames::ENTITY_ID => 4, MemberNames::PARENT_ID => 3, MemberNames::IS_ANCHOR => null, MemberNames::URL_PATH => 'men/tops-men', MemberNames::PATH => $path3),
+                $path4 = 'Default Category/Men/Tops/Hoodies & Sweatshirts' => array(MemberNames::ENTITY_ID => 5, MemberNames::PARENT_ID => 4, MemberNames::IS_ANCHOR => null, MemberNames::URL_PATH => 'men/tops-men/hoodies-and-sweatshirts-men', MemberNames::PATH => $path4),
+                $path5 = 'Default Category/Collections'                    => array(MemberNames::ENTITY_ID => 6, MemberNames::PARENT_ID => 3, MemberNames::IS_ANCHOR => null, MemberNames::URL_PATH => 'collections', MemberNames::PATH => $path5),
+                $path6 = 'Default Category/Collections/Eco Friendly'       => array(MemberNames::ENTITY_ID => 7, MemberNames::PARENT_ID => 6, MemberNames::IS_ANCHOR => null, MemberNames::URL_PATH => 'collections/eco-friendly', MemberNames::PATH => $path6),
         );
 
         // initialize a mock import adapter instance
@@ -377,8 +377,8 @@ 
 
         // mock the system logger
         $mockSystemLogger = $this->getMockBuilder('Psr\Log\LoggerInterface')
-                                 ->setMethods(get_class_methods('Psr\Log\LoggerInterface'))
-                                 ->getMock();
+                                    ->setMethods(get_class_methods('Psr\Log\LoggerInterface'))
+                                    ->getMock();
 
         // create a mock subject
         $mockSubject = $this->getMockBuilder('TechDivision\Import\Product\UrlRewrite\Subjects\UrlRewriteSubject')
@@ -487,7 +487,7 @@ 
                             return $arg2;
                         })
                     );
-       $mockSubject->expects(($this->exactly(1)))
+        $mockSubject->expects(($this->exactly(1)))
                     ->method('getImportAdapter')
                     ->willReturn($mockImportAdapter);
 

src/Observers/UrlRewriteObserver.php

Indentation +32 added lines, -32 removed lines

@@ -228,14 +228,14 @@ 
         if ($storeViewCode === StoreViewCodes::ADMIN) {
             // log a message and return
             $this->getSubject()
-                 ->getSystemLogger()
-                 ->debug(
-                     sprintf(
-                         'Store with code "%s" is not active, no URL rewrites will be created for product with SKU "%s"',
-                         $storeViewCode,
-                         $sku
-                     )
-                 );
+                    ->getSystemLogger()
+                    ->debug(
+                        sprintf(
+                            'Store with code "%s" is not active, no URL rewrites will be created for product with SKU "%s"',
+                            $storeViewCode,
+                            $sku
+                        )
+                    );
 
             // return without creating any rewrites
             return $this->getRow();
@@ -245,14 +245,14 @@ 
         if (!$this->getSubject()->storeIsActive($storeViewCode)) {
             // log a message and return
             $this->getSubject()
-                 ->getSystemLogger()
-                 ->debug(
-                     sprintf(
-                         'Store with code "%s" is not active, no URL rewrites will be created for product with SKU "%s"',
-                         $storeViewCode,
-                         $sku
-                     )
-                 );
+                    ->getSystemLogger()
+                    ->debug(
+                        sprintf(
+                            'Store with code "%s" is not active, no URL rewrites will be created for product with SKU "%s"',
+                            $storeViewCode,
+                            $sku
+                        )
+                    );
 
             // return without creating any rewrites
             return $this->getRow();
@@ -311,8 +311,8 @@ 
                     // query whether or not debug mode has been enabled
                     if ($this->getSubject()->isDebugMode()) {
                         $this->getSubject()
-                             ->getSystemLogger()
-                             ->warning($this->getSubject()->appendExceptionSuffix($e->getMessage()));
+                                ->getSystemLogger()
+                                ->warning($this->getSubject()->appendExceptionSuffix($e->getMessage()));
                     } else {
                         throw $e;
                     }
@@ -361,18 +361,18 @@ 
         // do NOT create new URL rewrites, if the product is NOT visible (any more), BUT
         // handle existing URL rewrites, e. g. to remove and clean up the URL rewrites
         if (!$this->isVisible()) {
-             // log a message
-             $this->getSubject()
-                 ->getSystemLogger()
-                 ->debug(
-                     sprintf(
-                         'Product with SKU "%s" is not visible, so no URL rewrites will be created',
-                         $this->getValue(ColumnKeys::SKU)
-                     )
-                 );
-
-             // return without creating any rewrites
-             return;
+                // log a message
+                $this->getSubject()
+                    ->getSystemLogger()
+                    ->debug(
+                        sprintf(
+                            'Product with SKU "%s" is not visible, so no URL rewrites will be created',
+                            $this->getValue(ColumnKeys::SKU)
+                        )
+                    );
+
+                // return without creating any rewrites
+                return;
         }
 
         // load the root category of the default store view (as we're in the
@@ -400,8 +400,8 @@ 
                 // query whether or not debug mode has been enabled
                 if ($this->getSubject()->isDebugMode()) {
                     $this->getSubject()
-                         ->getSystemLogger()
-                         ->warning($this->getSubject()->appendExceptionSuffix($e->getMessage()));
+                            ->getSystemLogger()
+                            ->warning($this->getSubject()->appendExceptionSuffix($e->getMessage()));
                 } else {
                     throw $e;
                 }

tests/unit/Observers/UrlRewriteUpdateObserverTest.php

Spacing +2 added lines, -2 removed lines

@@ -456,7 +456,7 @@ 
         $mockSubject->expects($this->any())
                     ->method('getCategory')
                     ->will(
-                        $this->returnCallback(function ($categoryId, $storeViewCode) use ($categories) {
+                        $this->returnCallback(function($categoryId, $storeViewCode) use ($categories) {
                             foreach ($categories as $category) {
                                 if ((int) $category[MemberNames::ENTITY_ID] === (int) $categoryId) {
                                     return $category;
@@ -483,7 +483,7 @@ 
         $mockSubject->expects($this->any())
                     ->method('getCoreConfigData')
                     ->will(
-                        $this->returnCallback(function ($arg1, $arg2) {
+                        $this->returnCallback(function($arg1, $arg2) {
                             return $arg2;
                         })
                     );