techdivision / import-product-url-rewrite

src/Services/ProductUrlRewriteProcessor.php

Doc Comments +1 added lines, -1 removed lines

@@ -151,7 +151,7 @@
      * Commits a transaction, returning the database connection to autocommit mode until the next call to
      * ProductProcessor::beginTransaction() starts a new transaction.
      *
-     * @return boolean Returns TRUE on success or FALSE on failure
+     * @return boolean|null Returns TRUE on success or FALSE on failure
      * @link http://php.net/manual/en/pdo.commit.php
      */
     public function commit()

src/Observers/UrlRewriteObserver.php

Doc Comments +2 added lines, -2 removed lines

@@ -117,7 +117,7 @@ 
     /**
      * Return's the product bunch processor instance.
      *
-     * @return \TechDivision\Import\Product\Services\ProductBunchProcessorInterface The product bunch processor instance
+     * @return ProductUrlRewriteProcessorInterface The product bunch processor instance
      */
     protected function getProductUrlRewriteProcessor()
     {
@@ -622,7 +622,7 @@ 
      * @param string $path          The path of the category to return
      * @param string $storeViewCode The store view code of the category to return
      *
-     * @return array The category
+     * @return string The category
      */
     protected function getCategoryByPath($path, $storeViewCode = StoreViewCodes::ADMIN)
     {

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

RoboFile.php

Unused Use Statements -4 removed lines

@@ -18,10 +18,6 @@
  * @link      http://www.techdivision.com
  */
 
-use Lurker\Event\FilesystemEvent;
-
-use Symfony\Component\Finder\Finder;
-
 /**
  * Defines the available build tasks.
  *

Indentation +12 added lines, -12 removed lines

@@ -58,9 +58,9 @@ 
     {
         // optimize autoloader with custom path
         return $this->taskComposerInstall()
-             ->preferDist()
-             ->optimizeAutoloader()
-             ->run();
+                ->preferDist()
+                ->optimizeAutoloader()
+                ->run();
     }
 
     /**
@@ -72,9 +72,9 @@ 
     {
         // optimize autoloader with custom path
         return $this->taskComposerUpdate()
-             ->preferDist()
-             ->optimizeAutoloader()
-             ->run();
+                ->preferDist()
+                ->optimizeAutoloader()
+                ->run();
     }
 
     /**
@@ -95,10 +95,10 @@ 
     public function prepare()
     {
         return $this->taskFileSystemStack()
-             ->mkdir($this->properties['dist.dir'])
-             ->mkdir($this->properties['target.dir'])
-             ->mkdir(sprintf('%s/reports', $this->properties['target.dir']))
-             ->run();
+                ->mkdir($this->properties['dist.dir'])
+                ->mkdir($this->properties['target.dir'])
+                ->mkdir(sprintf('%s/reports', $this->properties['target.dir']))
+                ->run();
     }
 
     /**
@@ -168,8 +168,8 @@ 
 
         // run PHPUnit
         return $this->taskPHPUnit(sprintf('%s/bin/phpunit', $this->properties['vendor.dir']))
-             ->configFile('phpunit.xml')
-             ->run();
+                ->configFile('phpunit.xml')
+                ->run();
     }
 
     /**

src/Observers/UrlRewriteUpdateObserver.php

Indentation +24 added lines, -24 removed lines

@@ -100,14 +100,14 @@ 
 
                             // finally log a warning that the old category is not available ony more
                             $this->getSubject()
-                                 ->getSystemLogger()
-                                 ->warning(
-                                     sprintf(
-                                         'Category with ID "%d" is not longer available for URL rewrite with ID "%d"',
-                                         $metadata[UrlRewriteObserver::CATEGORY_ID],
-                                         $existingUrlRewrite[MemberNames::URL_REWRITE_ID]
-                                     )
-                                 );
+                                    ->getSystemLogger()
+                                    ->warning(
+                                        sprintf(
+                                            'Category with ID "%d" is not longer available for URL rewrite with ID "%d"',
+                                            $metadata[UrlRewriteObserver::CATEGORY_ID],
+                                            $existingUrlRewrite[MemberNames::URL_REWRITE_ID]
+                                        )
+                                    );
                         }
                     }
                 }
@@ -119,14 +119,14 @@ 
                 if ($targetPath === $existingUrlRewrite[MemberNames::REQUEST_PATH]) {
                     // finally log a warning that the old category is not available ony more
                     $this->getSubject()
-                         ->getSystemLogger()
-                         ->warning(
-                             sprintf(
-                                 'New target path "%s" eqals request path for URL rewrite with ID "%d"',
-                                 $existingUrlRewrite[MemberNames::REQUEST_PATH],
-                                 $existingUrlRewrite[MemberNames::URL_REWRITE_ID]
-                             )
-                         );
+                            ->getSystemLogger()
+                            ->warning(
+                                sprintf(
+                                    'New target path "%s" eqals request path for URL rewrite with ID "%d"',
+                                    $existingUrlRewrite[MemberNames::REQUEST_PATH],
+                                    $existingUrlRewrite[MemberNames::URL_REWRITE_ID]
+                                )
+                            );
 
                     // stop processing the URL rewrite
                     continue;
@@ -153,14 +153,14 @@ 
 
                     // log a message, that old URL rewrites have been cleaned-up
                     $this->getSubject()
-                         ->getSystemLogger()
-                         ->warning(
-                             sprintf(
-                                 'Cleaned-up URL rewrite "%s" for product with SKU "%s"',
-                                 $existingUrlRewrite[MemberNames::REQUEST_PATH],
-                                 $this->getValue(ColumnKeys::SKU)
-                             )
-                         );
+                            ->getSystemLogger()
+                            ->warning(
+                                sprintf(
+                                    'Cleaned-up URL rewrite "%s" for product with SKU "%s"',
+                                    $existingUrlRewrite[MemberNames::REQUEST_PATH],
+                                    $this->getValue(ColumnKeys::SKU)
+                                )
+                            );
                 }
             }
         }

Spacing +1 added lines, -1 removed lines

@@ -105,7 +105,7 @@
 
                 // skip update of URL rewrite, if nothing to change
                 if ($targetPath === $existingUrlRewrite[MemberNames::TARGET_PATH] &&
-                    301 === (int)$existingUrlRewrite[MemberNames::REDIRECT_TYPE]) {
+                    301 === (int) $existingUrlRewrite[MemberNames::REDIRECT_TYPE]) {
                     // stop processing the URL rewrite
                     continue;
                 }

tests/unit/Observers/UrlRewriteObserverTest.php

Doc Comments +1 added lines, -1 removed lines

@@ -80,7 +80,7 @@
      * Create's and invokes the UrlRewriteObserver instance.
      *
      * @param boolean $generateCategoryProductRewrites Whether or not the configuration to generare catagory/product URL rewrites has been activated
-     * @param array   $productCategoryIds              The array with the category IDs the product has been related with
+     * @param string[]   $productCategoryIds              The array with the category IDs the product has been related with
      * @param array   $category                        The array with the categories
      *
      * @return void

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 +5 added lines, -5 removed lines

@@ -69,8 +69,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);
@@ -370,8 +370,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')
@@ -480,7 +480,7 @@ 
                             return $arg2;
                         })
                     );
-       $mockSubject->expects(($this->any()))
+        $mockSubject->expects(($this->any()))
                     ->method('getImportAdapter')
                     ->willReturn($mockImportAdapter);
 

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

Indentation +5 added lines, -5 removed lines

@@ -66,8 +66,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 UrlRewriteUpdateObserver($this->mockProductUrlRewriteProcessor);
@@ -133,8 +133,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();
 
         // the found URL rewrites
         $urlRewrites = array(
@@ -296,7 +296,7 @@ 
                     ->method('getUrlRewritesByEntityTypeAndEntityIdAndStoreId')
                     ->with(UrlRewriteObserver::ENTITY_TYPE, $entityId, $storeId)
                     ->willReturn($urlRewrites);
-       $this->mockProductUrlRewriteProcessor->expects($this->exactly(6))
+        $this->mockProductUrlRewriteProcessor->expects($this->exactly(6))
                     ->method('persistUrlRewrite')
                     ->withConsecutive(
                         array(