scottbedard / oc-shop-plugin

models/Price.php

Spacing +1 added lines, -1 removed lines

@@ -15,7 +15,7 @@
     /**
      * @var array Guarded fields
      */
-    protected $guarded = ['*'];
+    protected $guarded = [ '*' ];
 
     /**
      * @var array Fillable fields

updates/1.0/create_prices_table.php

Spacing +1 added lines, -1 removed lines

@@ -8,7 +8,7 @@
 {
     public function up()
     {
-        Schema::create('bedard_shop_prices', function (Blueprint $table) {
+        Schema::create('bedard_shop_prices', function(Blueprint $table) {
             $table->engine = 'InnoDB';
             $table->increments('id');
             $table->decimal('price', 10, 2)->unsigned();

traits/Timeable.php

Spacing +8 added lines, -8 removed lines

@@ -12,7 +12,7 @@ 
      */
     public static function bootTimeable()
     {
-        static::extend(function ($model) {
+        static::extend(function($model) {
             $model->addFillable('start_at', 'end_at');
             $model->addDateAttribute('start_at');
             $model->addDateAttribute('end_at');
@@ -27,7 +27,7 @@ 
      */
     public function scopeIsActive(Builder $query)
     {
-        return $query->where(function ($model) {
+        return $query->where(function($model) {
             return $model->isNotExpired()->isNotUpcoming();
         });
     }
@@ -40,7 +40,7 @@ 
      */
     public function scopeIsExpired(Builder $query)
     {
-        return $query->where(function ($model) {
+        return $query->where(function($model) {
             return $model->whereNotNull('end_at')
                 ->where('end_at', '<=', (string) Carbon::now());
         });
@@ -54,7 +54,7 @@ 
      */
     public function scopeIsUpcoming(Builder $query)
     {
-        return $query->where(function ($model) {
+        return $query->where(function($model) {
             return $model->whereNotNull('start_at')
                 ->where('start_at', '>', (string) Carbon::now());
         });
@@ -68,8 +68,8 @@ 
      */
     public function scopeIsNotActive(Builder $query)
     {
-        return $query->where(function ($model) {
-            return $model->isExpired()->orWhere(function ($q) {
+        return $query->where(function($model) {
+            return $model->isExpired()->orWhere(function($q) {
                 $q->isUpcoming();
             });
         });
@@ -83,7 +83,7 @@ 
      */
     public function scopeIsNotExpired(Builder $query)
     {
-        return $query->where(function ($model) {
+        return $query->where(function($model) {
             return $model->whereNull('end_at')
                 ->orWhere('end_at', '>', (string) Carbon::now());
         });
@@ -97,7 +97,7 @@ 
      */
     public function scopeIsNotUpcoming(Builder $query)
     {
-        return $query->where(function ($model) {
+        return $query->where(function($model) {
             return $model->whereNull('start_at')
                 ->orWhere('start_at', '<=', (string) Carbon::now());
         });

models/Product.php

Spacing +10 added lines, -10 removed lines

@@ -29,7 +29,7 @@ 
     /**
      * @var array Guarded fields
      */
-    protected $guarded = ['*'];
+    protected $guarded = [ '*' ];
 
     /**
      * @var array Fillable fields
@@ -125,8 +125,8 @@ 
     public function saveBasePrice()
     {
         Price::updateOrCreate(
-            ['product_id' => $this->id, 'discount_id' => null],
-            ['price' => $this->base_price]
+            [ 'product_id' => $this->id, 'discount_id' => null ],
+            [ 'price' => $this->base_price ]
         );
     }
 
@@ -173,21 +173,21 @@ 
      */
     public static function syncAllInheritedCategories()
     {
-        Queue::push(function ($job) {
-            $data = [];
+        Queue::push(function($job) {
+            $data = [ ];
             $products = Product::with('categories')->get();
             $categoryTree = Category::getParentCategoryIds();
 
             foreach ($products as $product) {
-                $inheritedCategoryIds = [];
+                $inheritedCategoryIds = [ ];
                 foreach ($product->categories as $category) {
                     if (array_key_exists($category->id, $categoryTree)) {
-                        $inheritedCategoryIds = array_merge($inheritedCategoryIds, $categoryTree[$category->id]);
+                        $inheritedCategoryIds = array_merge($inheritedCategoryIds, $categoryTree[ $category->id ]);
                     }
                 }
 
                 foreach (array_unique($inheritedCategoryIds) as $categoryId) {
-                    $data[] = [
+                    $data[ ] = [
                         'category_id' => $categoryId,
                         'product_id' => $product->id,
                         'is_inherited' => 1,
@@ -210,11 +210,11 @@ 
      */
     public function syncInheritedCategories(array $categoryIds = null)
     {
-        $data = [];
+        $data = [ ];
         $categoryIds = $this->categories()->lists('id');
         $parentIds = Category::isParentOf($categoryIds)->lists('id');
         foreach ($parentIds as $parentId) {
-            $data[] = [
+            $data[ ] = [
                 'category_id' => $parentId,
                 'product_id' => $this->id,
                 'is_inherited' => true,

controllers/Categories.php

Doc Comments -1 removed lines

@@ -65,7 +65,6 @@
     /**
      * Override the default list delete behavior
      *
-     * @param  \Bedard\Shop\Models\Category $record
      * @return void
      */
     public function overrideListDelete(Category $catgory)

models/Category.php

Spacing +18 added lines, -18 removed lines

@@ -35,7 +35,7 @@ 
     /**
      * @var array Guarded fields
      */
-    protected $guarded = ['*'];
+    protected $guarded = [ '*' ];
 
     /**
      * @var array Fillable fields
@@ -65,7 +65,7 @@ 
         'products' => [
             'Bedard\Shop\Models\Product',
             'table' => 'bedard_shop_category_product',
-            'pivot' => ['is_inherited'],
+            'pivot' => [ 'is_inherited' ],
         ],
     ];
 
@@ -117,9 +117,9 @@ 
     {
         $categories = self::select('id', 'parent_id')->get();
 
-        $tree = [];
+        $tree = [ ];
         foreach ($categories as $category) {
-            $tree[$category->id] = self::walkParentCategories($categories, $category);
+            $tree[ $category->id ] = self::walkParentCategories($categories, $category);
         }
 
         return $tree;
@@ -141,10 +141,10 @@ 
                 ->get();
         }
 
-        $children = [];
+        $children = [ ];
         foreach ($categories as $category) {
             if ($category->parent_id == $parent) {
-                $children[] = $category->id;
+                $children[ ] = $category->id;
                 $children = array_merge($children, self::getChildIds($category->id, $categories));
             }
         }
@@ -154,23 +154,23 @@ 
 
     public static function getParentIds($children, \October\Rain\Database\Collection $categories = null)
     {
-        if (! is_array($children)) {
-            $children = [$children];
+        if (!is_array($children)) {
+            $children = [ $children ];
         }
 
         if ($categories === null) {
             $categories = self::select('id', 'parent_id')->get();
         }
 
-        $parents = [];
+        $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) {
+                $parents[ ] = $category->parent_id;
+                $category = $categories->filter(function($model) use ($category) {
                     return $model->id == $category->parent_id;
                 })->first();
             }
@@ -250,7 +250,7 @@ 
      */
     public function setNullParentId()
     {
-        if (! $this->parent_id) {
+        if (!$this->parent_id) {
             $this->parent_id = null;
         }
     }
@@ -272,7 +272,7 @@ 
      */
     public function syncInheritedProductsAfterDelete()
     {
-        if (! $this->dontSyncAfterDelete) {
+        if (!$this->dontSyncAfterDelete) {
             Product::syncAllInheritedCategories();
         }
     }
@@ -298,8 +298,8 @@ 
     public static function updateMany(array $categories)
     {
         foreach ($categories as $category) {
-            $id = $category['id'];
-            unset($category['id']);
+            $id = $category[ 'id' ];
+            unset($category[ 'id' ]);
             self::whereId($id)->update($category);
         }
     }
@@ -312,10 +312,10 @@ 
      * @param  array                                $tree           Tree of parent IDs
      * @return arrau
      */
-    public static function walkParentCategories($categories, $category, $tree = [])
+    public static function walkParentCategories($categories, $category, $tree = [ ])
     {
         if ($category && $category->parent_id !== null) {
-            $tree[] = $category->parent_id;
+            $tree[ ] = $category->parent_id;
             $tree = array_merge($tree, self::walkParentCategories($categories, $categories->find($category->parent_id)));
         }
 

models/Discount.php

Spacing +14 added lines, -14 removed lines

@@ -43,7 +43,7 @@ 
     /**
      * @var array Guarded fields
      */
-    protected $guarded = ['*'];
+    protected $guarded = [ '*' ];
 
     /**
      * @var array Fillable fields
@@ -156,7 +156,7 @@ 
     public function filterFields($fields)
     {
         $fields->amount_exact->hidden = $this->is_percentage;
-        $fields->amount_percentage->hidden = ! $this->is_percentage;
+        $fields->amount_percentage->hidden = !$this->is_percentage;
     }
 
     /**
@@ -168,14 +168,14 @@ 
     {
         $productIds = $this->products()->lists('id');
         $categories = $this->categories()
-            ->with(['products' => function ($product) {
+            ->with([ 'products' => function($product) {
                 return $product->select('id');
-            }])
-            ->get(['id']);
+            } ])
+            ->get([ 'id' ]);
 
         foreach ($categories as $category) {
             foreach ($category->products as $product) {
-                $productIds[] = $product->id;
+                $productIds[ ] = $product->id;
             }
         }
 
@@ -189,7 +189,7 @@ 
      */
     public function getAmountExactAttribute()
     {
-        return ! $this->is_percentage ? $this->amount : 0;
+        return !$this->is_percentage ? $this->amount : 0;
     }
 
     /**
@@ -210,7 +210,7 @@ 
     public function savePrices()
     {
         $id = $this->id;
-        Queue::push(function ($job) use ($id) {
+        Queue::push(function($job) use ($id) {
             $discount = Discount::findOrFail($id);
             $productIds = $discount->getAllProductIds();
             $products = Product::whereIn('id', $productIds)->select('id', 'base_price')->get();
@@ -240,7 +240,7 @@ 
      */
     public static function syncAllPrices()
     {
-        $data = [];
+        $data = [ ];
         $discounts = self::isNotExpired()->with('categories.products', 'products')->get();
         foreach ($discounts as $discount) {
             $products = $discount->products;
@@ -249,7 +249,7 @@ 
             }
 
             foreach ($products as $product) {
-                $data[] = [
+                $data[ ] = [
                     'discount_id' => $discount->id,
                     'end_at' => $discount->end_at,
                     'price' => $discount->calculatePrice($product->base_price),
@@ -266,17 +266,17 @@ 
     public static function syncProductPrice(Product $product, array $categoryIds)
     {
         $discounts = self::isNotExpired()
-            ->whereHas('products', function ($query) use ($product) {
+            ->whereHas('products', function($query) use ($product) {
                 return $query->where('id', $product->id);
             })
-            ->orWhereHas('categories', function ($query) use ($categoryIds) {
+            ->orWhereHas('categories', function($query) use ($categoryIds) {
                 return $query->whereIn('id', $categoryIds);
             })
             ->get();
 
-        $data = [];
+        $data = [ ];
         foreach ($discounts as $discount) {
-            $data[] = [
+            $data[ ] = [
                 'discount_id' => $discount->id,
                 'end_at' => $discount->end_at,
                 'price' => $discount->calculatePrice($product->base_price),

controllers/Discounts.php

Spacing +2 added lines, -2 removed lines

@@ -39,10 +39,10 @@
     public function listExtendQuery($query)
     {
         $query->with([
-            'categories' => function ($category) {
+            'categories' => function($category) {
                 return $category->select('name');
             },
-            'products' => function ($product) {
+            'products' => function($product) {
                 return $product->select('name');
             },
         ])->selectStatus();