scottbedard / oc-shop-plugin

models/Product.php

Spacing +13 added lines, -13 removed lines

@@ -159,12 +159,12 @@ 
      */
     protected function deleteRelatedInventories($inventories)
     {
-        return array_filter($inventories, function ($inventory) {
+        return array_filter($inventories, function($inventory) {
             if ($inventory['id'] !== null && $inventory['is_deleted']) {
                 Inventory::find($inventory['id'])->delete();
             }
 
-            return ! $inventory['is_deleted'];
+            return !$inventory['is_deleted'];
         });
     }
 
@@ -176,12 +176,12 @@ 
      */
     protected function deleteRelatedOptions($options)
     {
-        return array_filter($options, function ($option) {
+        return array_filter($options, function($option) {
             if ($option['id'] !== null && $option['is_deleted']) {
                 Option::find($option['id'])->delete();
             }
 
-            return ! $option['is_deleted'];
+            return !$option['is_deleted'];
         });
     }
 
@@ -193,12 +193,12 @@ 
      */
     protected function deleteRelatedOptionValues($optionValues)
     {
-        return array_filter($optionValues, function ($optionValue) {
+        return array_filter($optionValues, function($optionValue) {
             if ($optionValue['id'] !== null && $optionValue['is_deleted']) {
                 OptionValue::find($optionValue['id'])->delete();
             }
 
-            return ! $optionValue['is_deleted'];
+            return !$optionValue['is_deleted'];
         });
     }
 
@@ -330,7 +330,7 @@ 
      * @param  string $sessionKey
      * @return void
      */
-    protected function saveRelatedOptionValues(Option &$option, array $values, $sessionKey)
+    protected function saveRelatedOptionValues(Option & $option, array $values, $sessionKey)
     {
         foreach ($values as $index => $value) {
             $model = array_key_exists('id', $value) && $value['id'] !== null
@@ -383,7 +383,7 @@ 
      */
     public static function syncAllInheritedCategories()
     {
-        Queue::push(function ($job) {
+        Queue::push(function($job) {
             $data = [];
             $products = Product::with('categories')->get();
             $categoryTree = Category::getParentCategoryIds();
@@ -446,8 +446,8 @@ 
      */
     protected function validateInventories()
     {
-        if (! is_array($this->optionsInventories) ||
-            ! is_array($this->optionsInventories['options'])) {
+        if (!is_array($this->optionsInventories) ||
+            !is_array($this->optionsInventories['options'])) {
             return;
         }
 
@@ -461,7 +461,7 @@ 
             sort($inventory['valueIds']);
             $valueCombination = json_encode($inventory['valueIds']);
 
-            if (in_array($valueCombination, $takenValueCombinations) && ! $inventory['is_deleted']) {
+            if (in_array($valueCombination, $takenValueCombinations) && !$inventory['is_deleted']) {
                 Flash::error(Lang::get('bedard.shop::lang.products.form.duplicate_inventories_error'));
                 throw new ModelException($this);
             }
@@ -478,8 +478,8 @@ 
      */
     protected function validateOptions()
     {
-        if (! is_array($this->optionsInventories) ||
-            ! is_array($this->optionsInventories['options'])) {
+        if (!is_array($this->optionsInventories) ||
+            !is_array($this->optionsInventories['options'])) {
             return;
         }
 

models/Inventory.php

Spacing +2 added lines, -2 removed lines

@@ -91,11 +91,11 @@
      */
     public function scopeHasOptionValueIds($query, $ids)
     {
-        if (! is_array($ids)) {
+        if (!is_array($ids)) {
             $ids = [$ids];
         }
 
-        return $query->whereHas('optionValues', function ($optionValue) use ($ids) {
+        return $query->whereHas('optionValues', function($optionValue) use ($ids) {
             return $optionValue->whereIn('id', $ids);
         });
     }

updates/1.0/create_filters_table.php

Spacing +1 added lines, -1 removed lines

@@ -8,7 +8,7 @@
 {
     public function up()
     {
-        Schema::create('bedard_shop_filters', function (Blueprint $table) {
+        Schema::create('bedard_shop_filters', function(Blueprint $table) {
             $table->engine = 'InnoDB';
             $table->increments('id');
             $table->integer('category_id')->unsigned()->index();

controllers/Filters.php

Spacing +1 added lines, -1 removed lines

@@ -18,7 +18,7 @@
     public function validate()
     {
         $data = input('option');
-        if (! $data || ! is_array($data)) {
+        if (!$data || !is_array($data)) {
             return Response::make('Error', 422);
         }
 

models/Category.php

Doc Comments +3 added lines, -1 removed lines

@@ -141,7 +141,6 @@ 
     /**
      * Apply filters to a products query.
      *
-     * @param  \October\Rain\Database\Builder   $query
      * @return void
      */
     protected function applyProductFilters(Builder &$products)
@@ -239,6 +238,9 @@ 
         return $children;
     }
 
+    /**
+     * @param integer $children
+     */
     public static function getParentIds($children, \October\Rain\Database\Collection $categories = null)
     {
         if (! is_array($children)) {

Spacing +11 added lines, -11 removed lines

@@ -144,9 +144,9 @@ 
      * @param  \October\Rain\Database\Builder   $query
      * @return void
      */
-    protected function applyProductFilters(Builder &$products)
+    protected function applyProductFilters(Builder & $products)
     {
-        $products->where(function ($query) {
+        $products->where(function($query) {
             foreach ($this->filters as $filter) {
                 // price filters
                 if ($filter->left === 'base_price' || $filter->left === 'price') {
@@ -186,12 +186,12 @@ 
      */
     protected function deleteRelatedFilters(array $filters)
     {
-        return array_filter($filters, function ($filter) {
+        return array_filter($filters, function($filter) {
             if ($filter['id'] !== null && $filter['is_deleted']) {
                 Filter::find($filter['id'])->delete();
             }
 
-            return ! $filter['is_deleted'];
+            return !$filter['is_deleted'];
         });
     }
 
@@ -241,7 +241,7 @@ 
 
     public static function getParentIds($children, \October\Rain\Database\Collection $categories = null)
     {
-        if (! is_array($children)) {
+        if (!is_array($children)) {
             $children = [$children];
         }
 
@@ -251,13 +251,13 @@ 
 
         $parents = [];
         foreach ($children as $child) {
-            $category = $categories->filter(function ($model) use ($child) {
+            $category = $categories->filter(function($model) use ($child) {
                 return $model->id == $child;
             })->first();
 
             while ($category && $category->parent_id) {
                 $parents[] = $category->parent_id;
-                $category = $categories->filter(function ($model) use ($category) {
+                $category = $categories->filter(function($model) use ($category) {
                     return $model->id == $category->parent_id;
                 })->first();
             }
@@ -318,7 +318,7 @@ 
      */
     public function isCustomSorted()
     {
-        return ! $this->product_sort_column && ! $this->product_sort_direction;
+        return !$this->product_sort_column && !$this->product_sort_direction;
     }
 
     /**
@@ -340,7 +340,7 @@ 
     {
         $filters = $this->getOriginalPurgeValue('category_filters');
 
-        if (is_array($filters) && ! empty($filters)) {
+        if (is_array($filters) && !empty($filters)) {
             $filters = $this->deleteRelatedFilters($filters);
             $this->saveRelatedFilters($filters);
         }
@@ -442,7 +442,7 @@ 
      */
     protected function setNullParentId()
     {
-        if (! $this->parent_id) {
+        if (!$this->parent_id) {
             $this->parent_id = null;
         }
     }
@@ -483,7 +483,7 @@ 
      */
     public function syncInheritedProductsAfterDelete()
     {
-        if (! $this->dontSyncAfterDelete) {
+        if (!$this->dontSyncAfterDelete) {
             Product::syncAllInheritedCategories();
         }
     }

repositories/CategoryRepository.php

Spacing +1 added lines, -1 removed lines

@@ -14,7 +14,7 @@
     {
         $query = Category::isActive();
 
-        if (array_key_exists('select', $params) && is_array($params['select']) && ! empty($params['select'])) {
+        if (array_key_exists('select', $params) && is_array($params['select']) && !empty($params['select'])) {
             $query->select($params['select']);
         }