speckcommerce / SpeckCatalog

src/SpeckCatalog/Mapper/Hydrator/Option.php

Spacing +1 added lines, -1 removed lines

@@ -13,7 +13,7 @@
         $data['name']           = $model->getName();
         $data['instruction']    = $model->getInstruction();
         $data['required']       = ($model->getRequired()) ? 1 : 0;
-        $data['builder']        = ($model->getBuilder())  ? 1 : 0;
+        $data['builder']        = ($model->getBuilder()) ? 1 : 0;
         $data['option_type_id'] = $model->getOptionTypeId();
 
         return $data;

src/SpeckCatalog/Model/Availability/Relational.php

Spacing +1 added lines, -1 removed lines

@@ -53,7 +53,7 @@
 
     public function __toString()
     {
-        $string = $this->has('distributor') ? $this->getDistributor()->getName(): '___' ;
+        $string = $this->has('distributor') ? $this->getDistributor()->getName() : '___';
         return $string;
     }
 }

Doc Comments +1 added lines, -1 removed lines

@@ -10,7 +10,7 @@
     protected $parent;
 
     /**
-     * @return parent
+     * @return AbstractModel
      */
     public function getParent()
     {

src/SpeckCatalog/Model/Choice/Relational.php

Spacing +1 added lines, -1 removed lines

@@ -68,7 +68,7 @@
         if ($this->getPriceDiscountFixed()) {
             return -$this->getPriceDiscountFixed();
         } elseif ($this->getPriceDiscountPercent()) {
-            return $parentPrice * -($this->getPriceDiscountPercent()/100);
+            return $parentPrice * -($this->getPriceDiscountPercent() / 100);
         } elseif ($this->getPriceNoCharge()) {
             return -$parentPrice;
         } elseif ($this->getPriceOverrideFixed()) {

Doc Comments +1 added lines, -1 removed lines

@@ -10,7 +10,7 @@
     protected $parent;
 
     /**
-     * @return parent
+     * @return AbstractModel
      */
     public function getParent()
     {

src/SpeckCatalog/Service/Category.php

Spacing +5 added lines, -5 removed lines

@@ -85,8 +85,8 @@ 
 
     public function addProduct($categoryOrId, $productOrId)
     {
-        $categoryId = ( is_numeric($categoryOrId) ? $categoryOrId : $categoryOrId->getCategoryId() );
-        $productId = ( is_numeric($productOrId) ? $productOrId  : $productOrId->getProductId() );
+        $categoryId = (is_numeric($categoryOrId) ? $categoryOrId : $categoryOrId->getCategoryId());
+        $productId = (is_numeric($productOrId) ? $productOrId : $productOrId->getProductId());
 
         return $this->getEntityMapper()->addProduct($categoryId, $productId);
     }
@@ -96,7 +96,7 @@ 
         if (null === $categoryOrId) {
             throw new \RuntimeException('categoryOrId cannot be null');
         }
-        $categoryId = ( is_numeric($categoryOrId) ? $categoryOrId : $categoryOrId->getCategoryId() );
+        $categoryId = (is_numeric($categoryOrId) ? $categoryOrId : $categoryOrId->getCategoryId());
 
         $parentCategoryId = (
             is_int($parentCategoryOrIdOrNull)
@@ -112,8 +112,8 @@ 
 
     public function setImage($categoryOrId, $imageOrId)
     {
-        $categoryId = ( is_int($categoryOrId) ? $categoryOrId : $categoryOrId->getCategoryId() );
-        $imageId    = ( is_int($imageOrId)    ? $imageOrId    : $imageOrId->getMediaId() );
+        $categoryId = (is_int($categoryOrId) ? $categoryOrId : $categoryOrId->getCategoryId());
+        $imageId    = (is_int($imageOrId) ? $imageOrId : $imageOrId->getMediaId());
 
         return $this->getImageService()->addLinker('category', $categoryId, $imageId);
     }

src/SpeckCatalog/Service/Choice.php

Spacing +5 added lines, -5 removed lines

@@ -42,8 +42,8 @@ 
 
     public function addOption($choiceOrId, $optionOrId)
     {
-        $choiceId = ( is_int($choiceOrId) ? $choiceOrId : $choiceOrId->getChoiceId() );
-        $optionId = ( is_int($optionOrId) ? $optionOrId : $optionOrId->getOptionId() );
+        $choiceId = (is_int($choiceOrId) ? $choiceOrId : $choiceOrId->getChoiceId());
+        $optionId = (is_int($optionOrId) ? $optionOrId : $optionOrId->getOptionId());
         $this->getEntityMapper()->addOption($choiceId, $optionId);
         return $this->getOptionService()->find(array('option_id' => $optionId));
     }
@@ -68,14 +68,14 @@ 
 
     public function removeOption($choiceOrId, $optionOrId)
     {
-        $productId = ( is_int($choiceOrId) ? $choiceOrId : $choiceOrId->getProductId() );
-        $optionId  = ( is_int($optionOrId) ? $optionOrId : $optionOrId->getOptionId() );
+        $productId = (is_int($choiceOrId) ? $choiceOrId : $choiceOrId->getProductId());
+        $optionId  = (is_int($optionOrId) ? $optionOrId : $optionOrId->getOptionId());
         $this->getEntityMapper()->removeOption($productId, $optionId);
     }
 
     public function choiceFromProduct($productOrId)
     {
-        $product = ( !is_int($productOrId) ? $productOrId : $this->getProductService()->find($productOrId) );
+        $product = (!is_int($productOrId) ? $productOrId : $this->getProductService()->find($productOrId));
         $choice = $this->getEntity()->setProductId($product->getProductId());
         return $this->persist($choice);
     }

src/SpeckCatalog/Service/Product.php

Spacing +2 added lines, -2 removed lines

@@ -185,8 +185,8 @@
 
     public function addOption($productOrId, $optionOrId)
     {
-        $productId = ( is_int($productOrId) ? $productOrId : $productOrId->getProductId() );
-        $optionId  = ( is_int($optionOrId)  ? $optionOrId  : $optionOrId->getOptionId() );
+        $productId = (is_int($productOrId) ? $productOrId : $productOrId->getProductId());
+        $optionId  = (is_int($optionOrId) ? $optionOrId : $optionOrId->getOptionId());
 
         $this->getEntityMapper()->addOption($productId, $optionId);
 

src/SpeckCatalog/View/Helper/Functions.php

Spacing +1 added lines, -1 removed lines

@@ -17,7 +17,7 @@
         if ($product->has('features')) {
             $str = '<ul>';
             $features = $product->getFeatures();
-            for ($i=0; $i < 3; $i++) {
+            for ($i = 0; $i < 3; $i++) {
                 if ($features[$i]) {
                     $str .= '<li><span>&nbsp;' . $features[$i] . '</span></li>';
                 }

src/SpeckCatalog/Controller/CatalogManagerController.php

Doc Comments +6 added lines

@@ -141,6 +141,9 @@ 
         return $this->getResponse()->setContent($html);
     }
 
+    /**
+     * @param string $helperName
+     */
     public function getViewHelper($helperName)
     {
         return $this->getServiceLocator()->get('viewhelpermanager')->get($helperName);
@@ -282,6 +285,9 @@ 
         return $dash->__invoke($name);
     }
 
+    /**
+     * @return string
+     */
     protected function camel($name)
     {
         $camel = new \Zend\Filter\Word\UnderscoreToCamelCase;

config/services/servicemanager.php

Spacing +8 added lines, -8 removed lines

@@ -4,37 +4,37 @@ 
         'speckcatalog_image_mapper' => false,
     ],
     'factories' => [
-        'speckcatalog_category_service'     => function ($sm) {
+        'speckcatalog_category_service'     => function($sm) {
             $service = new \SpeckCatalog\Service\Category;
             $domain  = $sm->get('multisite_domain_data');
             $service->setSiteId($domain['website_id']);
             return $service->setSiteId($domain['website_id']);
         },
-        'speckcatalog_product_image_service' => function ($sm) {
+        'speckcatalog_product_image_service' => function($sm) {
             $service = new \SpeckCatalog\Service\Image;
             $mapper  = $sm->get('speckcatalog_image_mapper')->setParentType('product');
             return $service->setEntityMapper($mapper);
         },
-        'speckcatalog_option_image_service' => function ($sm) {
+        'speckcatalog_option_image_service' => function($sm) {
             $service = new \SpeckCatalog\Service\Image;
             $mapper  = $sm->get('speckcatalog_image_mapper')->setParentType('option');
             return $service->setEntityMapper($mapper);
         },
-        'speckcatalog_module_options' => function ($sm) {
+        'speckcatalog_module_options' => function($sm) {
             $config = $sm->get('Config');
             return new \SpeckCatalog\Options\ModuleOptions(isset($config['speckcatalog']) ? $config['speckcatalog'] : []);
         },
-        'speckcatalog_availability_form' => function ($sm) {
+        'speckcatalog_availability_form' => function($sm) {
             $form = new \SpeckCatalog\Form\Availability;
             $form->setCompanyService($sm->get('speckcatalog_company_service'));
             return $form->init();
         },
-        'speckcatalog_product_form' => function ($sm) {
+        'speckcatalog_product_form' => function($sm) {
             $form = new \SpeckCatalog\Form\Product;
             $form->setCompanyService($sm->get('speckcatalog_company_service'));
             return $form->init();
         },
-        'speckcatalog_product_uom_form' => function ($sm) {
+        'speckcatalog_product_uom_form' => function($sm) {
             $form = new \SpeckCatalog\Form\ProductUom;
             $form->setUomService($sm->get('speckcatalog_uom_service'));
             return $form->init();
@@ -44,7 +44,7 @@ 
         'speckcatalog_db' => 'Zend\Db\Adapter\Adapter',
     ],
     'initializers' => [
-        function ($instance, $sm) {
+        function($instance, $sm) {
             if ($instance instanceof \SpeckCatalog\Mapper\DbAdapterAwareInterface) {
                 $dbAdapter = $sm->get('speckcatalog_db');
                 $instance->setDbAdapter($dbAdapter);