From 06a4f1216b8131f7cf66831297d14c58bfda83cf Mon Sep 17 00:00:00 2001
From: Prashant Singh <prashant.singh852@webkul.com>
Date: Fri, 9 Aug 2019 19:29:28 +0530
Subject: [PATCH 01/11] Dead code removal

---
 config/services.php                           | 20 +------------------
 packages/Webkul/Admin/src/Config/menu.php     | 16 +++++++--------
 .../views/checkout/onepage.blade.php          |  3 ---
 .../js/components/datagrid/datetime.vue       |  2 +-
 4 files changed, 9 insertions(+), 32 deletions(-)

diff --git a/config/services.php b/config/services.php
index 456421a1a..22a2286f8 100755
--- a/config/services.php
+++ b/config/services.php
@@ -42,23 +42,5 @@ return [
         'model' => App\User::class,
         'key' => env('STRIPE_KEY'),
         'secret' => env('STRIPE_SECRET'),
-    ],
-
-    'facebook' => [
-        'client_id'     => env('FACEBOOK_CLIENT_ID'),
-        'client_secret' => env('FACEBOOK_CLIENT_SECRET'),
-        'redirect'      => env('FACEBOOK_URL'),
-    ],
-
-    'twitter' => [
-        'client_id'     => env('TWITTER_CLIENT_ID'),
-        'client_secret' => env('TWITTER_CLIENT_SECRET'),
-        'redirect'      => env('TWITTER_URL'),
-    ],
-
-    'google' => [
-        'client_id'     => env('GOOGLE_CLIENT_ID'),
-        'client_secret' => env('GOOGLE_CLIENT_SECRET'),
-        'redirect'      => env('GOOGLE_URL'),
-    ],
+    ]
 ];
diff --git a/packages/Webkul/Admin/src/Config/menu.php b/packages/Webkul/Admin/src/Config/menu.php
index 1091a60fb..2c73ae44d 100755
--- a/packages/Webkul/Admin/src/Config/menu.php
+++ b/packages/Webkul/Admin/src/Config/menu.php
@@ -188,15 +188,13 @@ return [
         'route' => 'admin.cart-rule.index',
         'sort' => 1,
         'icon-class' => '',
-    ],
-    // [
-    //     'key' => 'promotions.catalog-rule',
-    //     'name' => 'admin::app.promotion.catalog-rule',
-    //     'route' => 'admin.catalog-rule.index',
-    //     'sort' => 1,
-    //     'icon-class' => '',
-    // ],
-    [
+    ], [
+        'key' => 'promotions.catalog-rule',
+        'name' => 'admin::app.promotion.catalog-rule',
+        'route' => 'admin.catalog-rule.index',
+        'sort' => 1,
+        'icon-class' => '',
+    ], [
         'key' => 'cms',
         'name' => 'admin::app.layouts.cms',
         'route' => 'admin.cms.index',
diff --git a/packages/Webkul/Shop/src/Resources/views/checkout/onepage.blade.php b/packages/Webkul/Shop/src/Resources/views/checkout/onepage.blade.php
index 2240c9a6c..0e7a4f16a 100755
--- a/packages/Webkul/Shop/src/Resources/views/checkout/onepage.blade.php
+++ b/packages/Webkul/Shop/src/Resources/views/checkout/onepage.blade.php
@@ -409,8 +409,6 @@
             mounted: function() {
                 this.templateRender = paymentHtml.render;
 
-                console.log(paymentHtml.render);
-
                 for (var i in paymentHtml.staticRenderFns) {
                     paymentTemplateRenderFns.push(paymentHtml.staticRenderFns[i]);
                 }
@@ -535,7 +533,6 @@
                 },
 
                 changeCoupon: function() {
-                    console.log('called');
                     if (this.couponChanged == true && this.changeCount == 0) {
                         this.changeCount++;
 
diff --git a/packages/Webkul/Ui/src/Resources/assets/js/components/datagrid/datetime.vue b/packages/Webkul/Ui/src/Resources/assets/js/components/datagrid/datetime.vue
index 60d67f16c..b4cf9df7a 100755
--- a/packages/Webkul/Ui/src/Resources/assets/js/components/datagrid/datetime.vue
+++ b/packages/Webkul/Ui/src/Resources/assets/js/components/datagrid/datetime.vue
@@ -11,7 +11,7 @@ Vue.use(VueFlatpickr);
 export default {
   data() {
     const now = new Date();
-    console.log(now);
+
     return {
       fpOptions: {
         utc: false,

From da093662dbfd7d07b37fe8b7f9dcb72f56ec1775 Mon Sep 17 00:00:00 2001
From: Prashant Singh <prashant.singh852@webkul.com>
Date: Mon, 12 Aug 2019 06:56:21 +0530
Subject: [PATCH 02/11] Catalog rule implementation in progress

---
 .../promotions/catalog-rule/create.blade.php  |  12 +-
 .../promotions/catalog-rule/edit.blade.php    |  43 ++-
 .../Http/Controllers/CustomerController.php   |   2 +-
 .../Discount/src/Config/rule-conditions.php   |  20 +-
 .../{ => Cart}/ConvertXToProductId.php        |  11 +-
 .../src/Helpers/{ => Cart}/CouponAbleRule.php |   4 +-
 .../src/Helpers/{ => Cart}/Discount.php       |  45 ++-
 .../Helpers/{ => Cart}/NonCouponAbleRule.php  |   4 +-
 .../Helpers/{ => Cart}/ValidatesDiscount.php  |   2 +-
 .../Discount/src/Helpers/Catalog/Apply.php    |  68 +++++
 .../Helpers/Catalog/ConvertXToProductId.php   | 272 ++++++++++++++++++
 .../Discount/src/Helpers/Catalog/Sale.php     | 107 +++++++
 .../src/Helpers/Catalog/SyncPreviousRules.php |   0
 .../Discount/src/Helpers/FindProducts.php     |  71 -----
 .../Http/Controllers/CartRuleController.php   |   2 +-
 .../Controllers/CatalogRuleController.php     |  38 +--
 .../src/Models/CatalogRuleCustomerGroups.php  |   6 +-
 .../Http/Controllers/OnepageController.php    |   9 +-
 .../assets/js/components/datetime.vue         |   4 +
 19 files changed, 562 insertions(+), 158 deletions(-)
 rename packages/Webkul/Discount/src/Helpers/{ => Cart}/ConvertXToProductId.php (96%)
 rename packages/Webkul/Discount/src/Helpers/{ => Cart}/CouponAbleRule.php (95%)
 rename packages/Webkul/Discount/src/Helpers/{ => Cart}/Discount.php (97%)
 rename packages/Webkul/Discount/src/Helpers/{ => Cart}/NonCouponAbleRule.php (88%)
 rename packages/Webkul/Discount/src/Helpers/{ => Cart}/ValidatesDiscount.php (97%)
 create mode 100644 packages/Webkul/Discount/src/Helpers/Catalog/Apply.php
 create mode 100644 packages/Webkul/Discount/src/Helpers/Catalog/ConvertXToProductId.php
 create mode 100644 packages/Webkul/Discount/src/Helpers/Catalog/Sale.php
 create mode 100644 packages/Webkul/Discount/src/Helpers/Catalog/SyncPreviousRules.php
 delete mode 100644 packages/Webkul/Discount/src/Helpers/FindProducts.php

diff --git a/packages/Webkul/Admin/src/Resources/views/promotions/catalog-rule/create.blade.php b/packages/Webkul/Admin/src/Resources/views/promotions/catalog-rule/create.blade.php
index e6032fd8b..1e9df1516 100644
--- a/packages/Webkul/Admin/src/Resources/views/promotions/catalog-rule/create.blade.php
+++ b/packages/Webkul/Admin/src/Resources/views/promotions/catalog-rule/create.blade.php
@@ -136,7 +136,11 @@
                                             <option v-for="(attr_ip, index1) in attribute_input" :value="attr_ip.code" :key="index1">@{{ attr_ip.name }}</option>
                                         </select>
 
-                                        <select class="control" v-model="attribute_values[index].condition" style="margin-right: 15px;">
+                                        <select v-show='attribute_values[index].type == "select" || attribute_values[index].type == "multiselect"' class="control" v-model="attribute_values[index].condition" style="margin-right: 15px;">
+                                            <option v-for="(condition, index) in conditions.select" :value="index" :key="index">@{{ condition }}</option>
+                                        </select>
+
+                                        <select v-show='attribute_values[index].type == "text" || attribute_values[index].type == "textarea" || attribute_values[index].type == "price"' class="control" v-model="attribute_values[index].condition" style="margin-right: 15px;">
                                             <option v-for="(condition, index) in conditions.string" :value="index" :key="index">@{{ condition }}</option>
                                         </select>
 
@@ -159,7 +163,7 @@
                                         <span class="icon trash-icon" v-on:click="removeAttr(index)"></span>
                                     </div>
 
-                                    <span class="btn btn-primary btn-lg mt-20" v-on:click="addAttributeCondition">Add Attribute Condition</span>
+                                    <span class="btn btn-primary btn-lg mt-20" v-on:click="addAttributeCondition">{{ __('admin::app.promotion.add-attr-condition') }}</span>
                                 </div>
                             </accordian>
 
@@ -256,6 +260,7 @@
                 },
 
                 mounted () {
+                    console.log(this.attribute_input);
                 },
 
                 methods: {
@@ -336,6 +341,7 @@
                         for(i in this.attribute_input) {
                             if (i == selectedIndex) {
                                 if (this.attribute_input[i].has_options == true) {
+                                    this.selectedIndex
                                     this.attribute_values[index].options = this.attribute_input[i].options;
                                 }
 
@@ -355,7 +361,7 @@
                     },
 
                     removeAttr(index) {
-                        this.conditions_list.splice(index, 1);
+                        this.attribute_values.splice(index, 1);
                     },
 
                     removeCat(index) {
diff --git a/packages/Webkul/Admin/src/Resources/views/promotions/catalog-rule/edit.blade.php b/packages/Webkul/Admin/src/Resources/views/promotions/catalog-rule/edit.blade.php
index 3bd7b26d7..91db3e8be 100644
--- a/packages/Webkul/Admin/src/Resources/views/promotions/catalog-rule/edit.blade.php
+++ b/packages/Webkul/Admin/src/Resources/views/promotions/catalog-rule/edit.blade.php
@@ -12,7 +12,7 @@
     @push('scripts')
         <script type="text/x-template" id="catalog-rule-form-template">
             <div>
-                <form method="POST" action="{{ route('admin.catalog-rule.store') }}" @submit.prevent="onSubmit">
+                <form method="POST" action="{{ route('admin.catalog-rule.update', $catalog_rule[5]->id) }}" @submit.prevent="onSubmit">
                     @csrf
 
                     <div class="page-header">
@@ -91,7 +91,11 @@
                                         <span class="control-error" v-if="errors.has('status')">@{{ errors.first('status') }}</span>
                                     </div>
 
-                                    <datetime :name="starts_from">
+                                    @php
+                                        $now = new \Carbon\Carbon();
+                                    @endphp
+
+                                    <date :name="starts_from">
                                         <div class="control-group" :class="[errors.has('starts_from') ? 'has-error' : '']">
                                             <label for="starts_from" class="required">{{ __('admin::app.promotion.general-info.starts-from') }}</label>
 
@@ -99,9 +103,9 @@
 
                                             <span class="control-error" v-if="errors.has('starts_from')">@{{ errors.first('starts_from') }}</span>
                                         </div>
-                                    </datetime>
+                                    </date>
 
-                                    <datetime :name="starts_from">
+                                    <date :name="starts_from">
                                         <div class="control-group" :class="[errors.has('ends_till') ? 'has-error' : '']">
                                             <label for="ends_till" class="required">{{ __('admin::app.promotion.general-info.ends-till') }}</label>
 
@@ -109,7 +113,7 @@
 
                                             <span class="control-error" v-if="errors.has('ends_till')">@{{ errors.first('ends_till') }}</span>
                                         </div>
-                                    </datetime>
+                                    </date>
                                 </div>
                             </accordian>
 
@@ -136,7 +140,11 @@
                                             <option v-for="(attr_ip, index1) in attribute_input" :value="attr_ip.code" :key="index1">@{{ attr_ip.name }}</option>
                                         </select>
 
-                                        <select class="control" v-model="attribute_values[index].condition" style="margin-right: 15px;">
+                                        <select v-show='attribute_values[index].type == "select" || attribute_values[index].type == "multiselect"' class="control" v-model="attribute_values[index].condition" style="margin-right: 15px;">
+                                            <option v-for="(condition, index) in conditions.select" :value="index" :key="index">@{{ condition }}</option>
+                                        </select>
+
+                                        <select v-show='attribute_values[index].type == "text" || attribute_values[index].type == "textarea" || attribute_values[index].type == "price"' class="control" v-model="attribute_values[index].condition" style="margin-right: 15px;">
                                             <option v-for="(condition, index) in conditions.string" :value="index" :key="index">@{{ condition }}</option>
                                         </select>
 
@@ -159,7 +167,7 @@
                                         <span class="icon trash-icon" v-on:click="removeAttr(index)"></span>
                                     </div>
 
-                                    <span class="btn btn-primary btn-lg mt-20" v-on:click="addAttributeCondition">Add Attribute Condition</span>
+                                    <span class="btn btn-primary btn-lg mt-20" v-on:click="addAttributeCondition">{{ __('admin::app.promotion.add-attr-condition') }}</span>
                                 </div>
                             </accordian>
 
@@ -235,7 +243,7 @@
 
                         category_options: @json($catalog_rule[1]),
                         category_values: [],
-                        conditions: @json($catalog_rule[5]).conditions,
+                        conditions: @json($catalog_rule[3]).conditions,
                         attribute_values: [],
                         attr_object: {
                             attribute: null,
@@ -243,11 +251,12 @@
                             value: [],
                             options: []
                         },
-                        attribute_input: @json($catalog_rule[3])
+                        attribute_input: @json($catalog_rule[0])
                     }
                 },
 
                 mounted () {
+                    console.log(this.conditions)
                     channels = @json($catalog_rule[5]->channels);
 
                     this.channels = [];
@@ -292,6 +301,18 @@
                 },
 
                 methods: {
+                    created() {
+                        VeeValidate.Validator.extend('is_time', {
+                            getMessage: field => `The format must be HH:MM:SS`,
+                            validate: (value) => new Promise(resolve => {
+                                let regex = new RegExp("([0-1][0-9]|2[0-3]):([0-5][0-9]):([0-5][0-9])");
+                                resolve({
+                                    valid: value && regex.test(value)
+                                });
+                            })
+                        });
+                    },
+
                     categoryLabel (option) {
                         return option.name + ' [ ' + option.slug + ' ]';
                     },
@@ -321,6 +342,8 @@
                                 }
 
                                 this.attribute_values[index].type = this.attribute_input[i].type;
+
+                                debugger
                             }
                         }
                     },
@@ -336,7 +359,7 @@
                     },
 
                     removeAttr(index) {
-                        this.conditions_list.splice(index, 1);
+                        this.attribute_values.splice(index, 1);
                     },
 
                     removeCat(index) {
diff --git a/packages/Webkul/Customer/src/Http/Controllers/CustomerController.php b/packages/Webkul/Customer/src/Http/Controllers/CustomerController.php
index a2270025d..4e362c5eb 100755
--- a/packages/Webkul/Customer/src/Http/Controllers/CustomerController.php
+++ b/packages/Webkul/Customer/src/Http/Controllers/CustomerController.php
@@ -13,7 +13,7 @@ use Hash;
 /**
  * Customer controlller for the customer basically for the tasks of customers which will be done after customer authentication.
  *
- * @author    Prashant Singh <prashant.singh852@webkul.com>
+ * @author  Prashant Singh <prashant.singh852@webkul.com>
  * @copyright 2018 Webkul Software Pvt Ltd (http://www.webkul.com)
  */
 class CustomerController extends Controller
diff --git a/packages/Webkul/Discount/src/Config/rule-conditions.php b/packages/Webkul/Discount/src/Config/rule-conditions.php
index 4f68508a2..20c371df9 100644
--- a/packages/Webkul/Discount/src/Config/rule-conditions.php
+++ b/packages/Webkul/Discount/src/Config/rule-conditions.php
@@ -133,10 +133,22 @@ return [
 
             'string' => [
                 '=' => 'Equals',
-                // '>=' => 'Greater or equals',
-                // '<=' => 'Lesser or equals',
-                // '>' => 'Greater than',
-                // '<' => 'Lesser than',
+                '>=' => 'Greater or equals',
+                '<=' => 'Lesser or equals',
+                '>' => 'Greater than',
+                '<' => 'Lesser than',
+                '{}' => 'Contains',
+                '!{}' => 'Does not contains'
+            ],
+
+            'select' => [
+                '=' => 'Equals',
+                '{}' => 'Contains',
+                '!{}' => 'Does not contains'
+            ],
+
+            'multiselect' => [
+                '=' => 'Equals',
                 '{}' => 'Contains',
                 '!{}' => 'Does not contains'
             ],
diff --git a/packages/Webkul/Discount/src/Helpers/ConvertXToProductId.php b/packages/Webkul/Discount/src/Helpers/Cart/ConvertXToProductId.php
similarity index 96%
rename from packages/Webkul/Discount/src/Helpers/ConvertXToProductId.php
rename to packages/Webkul/Discount/src/Helpers/Cart/ConvertXToProductId.php
index ce7114867..0ac03a8cc 100644
--- a/packages/Webkul/Discount/src/Helpers/ConvertXToProductId.php
+++ b/packages/Webkul/Discount/src/Helpers/Cart/ConvertXToProductId.php
@@ -1,6 +1,6 @@
 <?php
 
-namespace Webkul\Discount\Helpers;
+namespace Webkul\Discount\Helpers\Cart;
 
 use Webkul\Discount\Repositories\CartRuleRepository as CartRule;
 use Webkul\Attribute\Repositories\AttributeRepository as Attribute;
@@ -8,7 +8,6 @@ use Webkul\Attribute\Repositories\AttributeOptionRepository as AttributeOption;
 use Webkul\Category\Repositories\CategoryRepository as Category;
 use Webkul\Product\Repositories\ProductRepository as Product;
 use Webkul\Product\Models\ProductAttributeValue as ProductAttributeValue;
-use DB;
 
 class ConvertXToProductId
 {
@@ -284,13 +283,9 @@ class ConvertXToProductId
                 $qb->where('product_categories.category_id', $categoryId);
             }
 
-            if (is_null(request()->input('status'))) {
-                $qb->where('product_flat.status', 1);
-            }
+            $qb->where('product_flat.status', 1);
 
-            if (is_null(request()->input('visible_individually'))) {
-                $qb->where('product_flat.visible_individually', 1);
-            }
+            $qb->where('product_flat.visible_individually', 1);
 
             $queryBuilder = $qb->leftJoin('product_flat as flat_variants', function($qb) use($channel, $locale) {
                 $qb->on('product_flat.id', '=', 'flat_variants.parent_id');
diff --git a/packages/Webkul/Discount/src/Helpers/CouponAbleRule.php b/packages/Webkul/Discount/src/Helpers/Cart/CouponAbleRule.php
similarity index 95%
rename from packages/Webkul/Discount/src/Helpers/CouponAbleRule.php
rename to packages/Webkul/Discount/src/Helpers/Cart/CouponAbleRule.php
index da6f32888..2d4572dda 100644
--- a/packages/Webkul/Discount/src/Helpers/CouponAbleRule.php
+++ b/packages/Webkul/Discount/src/Helpers/Cart/CouponAbleRule.php
@@ -1,10 +1,8 @@
 <?php
 
-namespace Webkul\Discount\Helpers;
+namespace Webkul\Discount\Helpers\Cart;
 
 use Webkul\Discount\Helpers\Discount;
-
-use Carbon\Carbon;
 use Cart;
 
 class CouponAbleRule extends Discount
diff --git a/packages/Webkul/Discount/src/Helpers/Discount.php b/packages/Webkul/Discount/src/Helpers/Cart/Discount.php
similarity index 97%
rename from packages/Webkul/Discount/src/Helpers/Discount.php
rename to packages/Webkul/Discount/src/Helpers/Cart/Discount.php
index 591964dd7..e1334315d 100644
--- a/packages/Webkul/Discount/src/Helpers/Discount.php
+++ b/packages/Webkul/Discount/src/Helpers/Cart/Discount.php
@@ -1,6 +1,6 @@
 <?php
 
-namespace Webkul\Discount\Helpers;
+namespace Webkul\Discount\Helpers\Cart;
 
 use Webkul\Discount\Repositories\CartRuleRepository as CartRule;
 use Webkul\Checkout\Repositories\CartItemRepository as CartItem;
@@ -283,30 +283,6 @@ abstract class Discount
         return $maxImpact;
     }
 
-    /**
-     * To find oldest rule
-     *
-     * @param Collection $rules
-     *
-     * @return Collection $rule
-     */
-    public function findOldestRule($rules)
-    {
-        $leastId = 0;
-
-        if ($rules->count()) {
-            $leastId = $rules->min('id');
-
-            foreach ($rules as $rule) {
-                if ($rule->id == $leastId) {
-                    return $rule;
-                }
-            }
-        } else {
-            return collect();
-        }
-    }
-
     /**
      * To calculate the impact of the rule
      *
@@ -350,6 +326,23 @@ abstract class Discount
 
         $channelBased = false;
 
+        // time based constraints
+        if ($rule->starts_from != null && $rule->ends_till == null) {
+            if (Carbon::parse($rule->starts_from) < now()) {
+                $timeBased = true;
+            }
+        } else if ($rule->starts_from == null && $rule->ends_till != null) {
+            if (Carbon::parse($rule->ends_till) > now()) {
+                $timeBased = true;
+            }
+        } else if ($rule->starts_from != null && $rule->ends_till != null) {
+            if (Carbon::parse($rule->starts_from) < now() && now() < Carbon::parse($rule->ends_till)) {
+                $timeBased = true;
+            }
+        } else {
+            $timeBased = true;
+        }
+
         // channel based constraints
         foreach ($rule->channels as $channel) {
             if ($channel->channel_id == core()->getCurrentChannel()->id) {
@@ -407,7 +400,7 @@ abstract class Discount
             }
         }
 
-        if ($channelBased && $customerGroupBased && $conditionsBased) {
+        if ($channelBased && $customerGroupBased && $timeBased && $conditionsBased) {
             if ($rule->uses_attribute_conditions == 1 && $partialMatch) {
                 return true;
             } else if ($rule->uses_attribute_conditions == 0) {
diff --git a/packages/Webkul/Discount/src/Helpers/NonCouponAbleRule.php b/packages/Webkul/Discount/src/Helpers/Cart/NonCouponAbleRule.php
similarity index 88%
rename from packages/Webkul/Discount/src/Helpers/NonCouponAbleRule.php
rename to packages/Webkul/Discount/src/Helpers/Cart/NonCouponAbleRule.php
index 554ae2b74..ad445cef3 100644
--- a/packages/Webkul/Discount/src/Helpers/NonCouponAbleRule.php
+++ b/packages/Webkul/Discount/src/Helpers/Cart/NonCouponAbleRule.php
@@ -1,10 +1,8 @@
 <?php
 
-namespace Webkul\Discount\Helpers;
+namespace Webkul\Discount\Helpers\Cart;
 
 use Webkul\Discount\Helpers\Discount;
-use Webkul\Discount\Repositories\CartRuleCartRepository as CartRuleCart;
-use Cart;
 
 class NonCouponAbleRule extends Discount
 {
diff --git a/packages/Webkul/Discount/src/Helpers/ValidatesDiscount.php b/packages/Webkul/Discount/src/Helpers/Cart/ValidatesDiscount.php
similarity index 97%
rename from packages/Webkul/Discount/src/Helpers/ValidatesDiscount.php
rename to packages/Webkul/Discount/src/Helpers/Cart/ValidatesDiscount.php
index c4d4d0976..31e382cd4 100644
--- a/packages/Webkul/Discount/src/Helpers/ValidatesDiscount.php
+++ b/packages/Webkul/Discount/src/Helpers/Cart/ValidatesDiscount.php
@@ -1,6 +1,6 @@
 <?php
 
-namespace Webkul\Discount\Helpers;
+namespace Webkul\Discount\Helpers\Cart;
 
 use Webkul\Discount\Helpers\Discount;
 use Webkul\Discount\Repositories\CartRuleCartRepository as CartRuleCart;
diff --git a/packages/Webkul/Discount/src/Helpers/Catalog/Apply.php b/packages/Webkul/Discount/src/Helpers/Catalog/Apply.php
new file mode 100644
index 000000000..049ce9884
--- /dev/null
+++ b/packages/Webkul/Discount/src/Helpers/Catalog/Apply.php
@@ -0,0 +1,68 @@
+<?php
+
+namespace Webkul\Discount\Helpers\Catalog;
+
+use Webkul\Discount\Repositories\CatalogRuleRepository as CatalogRule;
+use Webkul\Discount\Helpers\Catalog\Sale;
+
+class Apply extends Sale
+{
+    /**
+     * To hold the catalog rule repository instance
+     */
+    protected $catalogRule;
+
+    /**
+     * To maintain the list of deceased rules and remove their records if present
+     */
+    protected $activeRules;
+
+    /**
+     * To maintain the list of deceased rules and remove their records if present
+     */
+    protected $deceased;
+
+    /**
+     * To hold the rule classes
+     */
+    protected $rules;
+
+    /**
+     * @param CatalogRule $catalogRule
+     */
+    public function __construct(CatalogRule $catalogRule)
+    {
+        $this->catalogRule = $catalogRule;
+
+        $this->active = collect();
+
+        $this->deceased = collect();
+
+        $this->rules = config('discount-rules.catalog');
+    }
+
+    /**
+     * To apply the new catalog rules and sync previous rules changes,
+     */
+    public function apply()
+    {
+        $rules = $this->catalogRule->all();
+
+        foreach ($rules as $rule) {
+            $validated = $this->checkApplicability($rule);
+
+            if ($validated) {
+                $this->active->push($rule->id);
+
+                // Job execution for active rules
+                $products = $this->getProductIds($rule);
+
+                $this->setSalePrice($products);
+            } else {
+                $this->deceased->push($rule->id);
+
+                // Job execution for deceased rules
+            }
+        }
+    }
+}
\ No newline at end of file
diff --git a/packages/Webkul/Discount/src/Helpers/Catalog/ConvertXToProductId.php b/packages/Webkul/Discount/src/Helpers/Catalog/ConvertXToProductId.php
new file mode 100644
index 000000000..52755c3cd
--- /dev/null
+++ b/packages/Webkul/Discount/src/Helpers/Catalog/ConvertXToProductId.php
@@ -0,0 +1,272 @@
+<?php
+
+namespace Webkul\Discount\Helpers\Catalog;
+
+use Webkul\Discount\Repositories\CatalogRuleRepository as CatalogRule;
+use Webkul\Attribute\Repositories\AttributeRepository as Attribute;
+use Webkul\Attribute\Repositories\AttributeOptionRepository as AttributeOption;
+use Webkul\Category\Repositories\CategoryRepository as Category;
+use Webkul\Product\Repositories\ProductRepository as Product;
+use Webkul\Product\Models\ProductAttributeValue as ProductAttributeValue;
+
+class ConvertXToProductId
+{
+    /**
+     * CategoryRepository instance
+     */
+    protected $category;
+
+    /**
+     * AttributeRepository instance
+     */
+    protected $attribute;
+
+    /**
+     * ProductRepository instance
+     */
+    protected $product;
+
+    /**
+     * AttributeOptionRepository instance
+     */
+    protected $attributeOption;
+
+    /**
+     * CatalogRuleRepository instance
+     */
+    protected $catalogRule;
+
+    /**
+     * ProductAttributeValueRepository instance
+     */
+    protected $pav;
+
+    /**
+     * Condition symbols for matching the criteria with attributes selected
+     */
+    protected $symbols;
+
+    public function __construct(
+        Category $category,
+        Attribute $attribute,
+        Product $product,
+        AttributeOption $attributeOption,
+        CatalogRule $catalogRule,
+        ProductAttributeValue $pav
+    ) {
+        $this->category = $category;
+
+        $this->attribute = $attribute;
+
+        $this->product = $product;
+
+        $this->attributeOption = $attributeOption;
+
+        $this->catalogRule = $catalogRule;
+
+        $this->pav = $pav;
+
+        $this->conditionSymbols = config('pricerules.cart.conditions.symbols');
+    }
+
+    public function convertX($attribute_conditions)
+    {
+        $attributeConditions = json_decode($attribute_conditions);
+
+        $categoryValues = $attributeConditions->categories;
+
+        $attributeValues = $attributeConditions->attributes;
+
+        if (!isset($categoryValues) && !isset($attributeValues)) {
+            return false;
+        }
+
+        $categoryResult = collect();
+
+        if (isset($categoryValues) && count($categoryValues)) {
+            $categoryResult = $this->convertFromCategories($categoryValues);
+        }
+
+        $attributeResult = collect();
+        if (isset($attributeValues) && count($attributeValues)) {
+            $attributeResult = $this->convertFromAttributes($attributeValues);
+        }
+
+        // now call the function that will find all the unique product ids
+        $productIDs = $this->findAllUniqueIds($attributeResult, $categoryResult);
+
+        return $productIDs;
+    }
+
+    /**
+     * This method will return product id from the attribute and attribute option params
+     *
+     * @return array
+     */
+    public function convertFromAttributes($attributeOptions)
+    {
+        $products = collect();
+
+        foreach ($attributeOptions as $attributeOption) {
+            $selectedOptions = $attributeOption->value;
+
+            if ($attributeOption->type == 'select' || $attributeOption->type == 'multiselect') {
+                $attribute = $this->attribute->findWhere([
+                    'code' => $attributeOption->attribute
+                ]);
+
+                $attributeOptions = $attribute->first()->options;
+
+                $selectedAttributeOptions = collect();
+
+                foreach ($attributeOptions as $attributeOption) {
+                    foreach ($selectedOptions as $key => $value) {
+                        if ($attributeOption->id == $value) {
+                            $selectedAttributeOptions->push($attributeOption);
+                        }
+                    }
+                }
+
+                foreach ($selectedAttributeOptions as $selectedAttributeOption) {
+                    $typeColumn = $this->pav::$attributeTypeFields[$attribute->first()->type];
+
+                    $pavResults = $this->pav->where(
+                        "{$typeColumn}",
+                        $selectedAttributeOption->id
+                    )->get();
+
+                    foreach ($pavResults as $pavResult) {
+                        if ($pavResult->product->type == 'simple')
+                            $products->push($pavResult->product);
+                    }
+                }
+            } else {
+                $attribute = $this->attribute->findWhere([
+                    'code' => $attributeOption->attribute
+                ]);
+
+                $pavValues = $attribute->first();
+
+                $selectedAttributeValues = collect();
+
+                if ($attributeOption->attribute == 'sku') {
+                    $testValue = $attributeOption->value;
+                    $testCondition = $attributeOption->condition;
+
+                    if ($testCondition == '{}') {
+                        $foundProducts = $this->product->findWhere([
+                            ['sku', 'like', '%' . $testValue . '%'],
+                            ['type', '!=', 'configurable']
+                        ])->flatten()->all();
+                    } else if ($testCondition == '!{}') {
+                        $foundProducts = $this->product->findWhere([
+                            ['sku', 'not like', '%' . $testValue . '%'],
+                            ['type', '!=', 'configurable']
+                        ])->flatten()->all();
+                    } else if ($testCondition == '=') {
+                        $foundProducts = $this->product->findWhere([
+                            ['sku', '=', '%' . $testValue . '%'],
+                            ['type', '!=', 'configurable']
+                        ])->flatten()->all();
+                    }
+                }
+
+                foreach ($foundProducts as $foundProduct) {
+                    $products->push($foundProduct);
+                }
+            }
+        }
+
+        return $products;
+    }
+
+    /**
+     * This method will return product id from the attribute and attribute option params
+     *
+     * @param Collection $categories
+     *
+     * @return array
+     */
+    public function convertFromCategories($categories)
+    {
+        $products = collect();
+
+        foreach ($categories as $category) {
+            $data = $this->getAll($category->id);
+            if ($data->count()) {
+                $products->push($data);
+
+                unset($data);
+            }
+        }
+
+        return $products->flatten()->all();
+    }
+
+    /**
+     * This method will remove the duplicates from the array and merge all the items into single array
+     *
+     * @param array
+     *
+     * @return array
+     */
+    public function findAllUniqueIds(...$data)
+    {
+        $attributeResult = $data[0] ?? collect();
+        $categoryResult = $data[1] ?? collect();
+
+        // find matched attribute options product ids
+        $mergedCollection = $attributeResult->merge($categoryResult);
+
+        $productIDs = collect();
+        foreach ($mergedCollection as $merged) {
+            $productIDs->push($merged->id);
+        }
+
+        // find all the unique product ids
+        $productIDs = $productIDs->unique();
+
+        return $productIDs->flatten()->all();
+    }
+
+    /**
+     * To get all product ids lying in the criteria
+     *
+     * @param integer $categoryId
+     *
+     * @return Collection
+     */
+    public function getAll($categoryId = null)
+    {
+
+        $results = app('Webkul\Product\Repositories\ProductFlatRepository')->scopeQuery(function ($query) use ($categoryId) {
+
+            $channel = request()->get('channel') ?: (core()->getCurrentChannelCode() ?: core()->getDefaultChannelCode());
+
+            $locale = request()->get('locale') ?: app()->getLocale();
+
+            $qb = $query->distinct()
+                ->select('products.id')
+                ->leftJoin('products', 'product_flat.product_id', '=', 'products.id')
+                ->leftJoin('product_categories', 'products.id', '=', 'product_categories.product_id')
+                ->where('products.type', '!=', 'configurable')
+                ->whereNotNull('product_flat.url_key');
+
+            if ($categoryId) {
+                $qb->where('product_categories.category_id', $categoryId);
+            }
+
+            $qb->where('product_flat.status', 1);
+
+            $qb->where('product_flat.visible_individually', 1);
+
+            $queryBuilder = $qb->leftJoin('product_flat as flat_variants', function ($qb) use ($channel, $locale) {
+                $qb->on('product_flat.id', '=', 'flat_variants.parent_id');
+            });
+
+            return $qb->groupBy('product_flat.id');
+        })->get();
+
+        return $results;
+    }
+}
diff --git a/packages/Webkul/Discount/src/Helpers/Catalog/Sale.php b/packages/Webkul/Discount/src/Helpers/Catalog/Sale.php
new file mode 100644
index 000000000..ca0f02703
--- /dev/null
+++ b/packages/Webkul/Discount/src/Helpers/Catalog/Sale.php
@@ -0,0 +1,107 @@
+<?php
+
+namespace Webkul\Discount\Helpers\Catalog;
+
+use Webkul\Discount\Repositories\CatalogRepository as CatalogRule;
+use Webkul\Discount\Helpers\Catalog\ConvertXToProductId;
+
+abstract class Sale
+{
+    /**
+     * To hold the catalog rule repository instance
+     */
+    protected $catalogRule;
+
+    /**
+     * ConvertXToProductId instance
+     */
+    protected $convertX;
+
+    /**
+     * To hold the rule classes
+     */
+    protected $rules;
+
+    /**
+     * @param CatalogRule $catalogRule
+     */
+    public function __construct(CatalogRule $catalogRule, ConvertXToProductId $convertX)
+    {
+        $this->catalogRule = $catalogRule;
+
+        $this->convertX = $convertX;
+
+        $this->rules = config('discount-rules.catalog');
+    }
+
+    abstract function apply();
+
+    /**
+     * To validate rule
+     *
+     * @param CatalogRule $rule
+     *
+     * @return Boolean
+     */
+    public function checkApplicability($rule)
+    {
+        $cart = \Cart::getCart();
+
+        $timeBased = false;
+
+        $status = false;
+
+        // time based constraints
+        if ($rule->starts_from != null && $rule->ends_till == null) {
+            if (Carbon::parse($rule->starts_from) < now()) {
+                $timeBased = true;
+            }
+        } else if ($rule->starts_from == null && $rule->ends_till != null) {
+            if (Carbon::parse($rule->ends_till) > now()) {
+                $timeBased = true;
+            }
+        } else if ($rule->starts_from != null && $rule->ends_till != null) {
+            if (Carbon::parse($rule->starts_from) < now() && now() < Carbon::parse($rule->ends_till)) {
+                $timeBased = true;
+            }
+        } else {
+            $timeBased = true;
+        }
+
+        if ($rule->status) {
+            $status = true;
+        }
+
+        if ($timeBased && $status) {
+            return true;
+        } else {
+            return false;
+        }
+    }
+
+    /**
+     * Function to maintain products and catalog rule
+     */
+    public function getProductIds($rule)
+    {
+        $productIDs = $rule->conditions;
+
+        if ($rule->conditions) {
+            $conditions = $rule->conditions;
+
+            // $convertX = new ConvertXToProductId();
+
+            dd($this->convertX);
+            $productIDs = $this->convertX->convertX($rule->conditions);
+        } else {
+            // apply on all products
+        }
+
+        dd($productIDs);
+    }
+
+     /**
+      * Function to maintain products, catalog rule, price
+      */
+
+}
\ No newline at end of file
diff --git a/packages/Webkul/Discount/src/Helpers/Catalog/SyncPreviousRules.php b/packages/Webkul/Discount/src/Helpers/Catalog/SyncPreviousRules.php
new file mode 100644
index 000000000..e69de29bb
diff --git a/packages/Webkul/Discount/src/Helpers/FindProducts.php b/packages/Webkul/Discount/src/Helpers/FindProducts.php
deleted file mode 100644
index 7a7de3370..000000000
--- a/packages/Webkul/Discount/src/Helpers/FindProducts.php
+++ /dev/null
@@ -1,71 +0,0 @@
-<?php
-
-namespace Webkul\Discount\Helpers;
-
-use Webkul\Product\Repositories\ProductRepository as Product;
-use Webkul\Product\Repositories\ProductFlatRepository as ProductFlat;
-use Webkul\Attribute\Repositories\AttributeRepository as Attribute;
-use Webkul\Core\Repositories\LocaleRepository as Locale;
-
-class FindProducts
-{
-    /**
-     * To hold the product repository instance
-     */
-    protected $product;
-
-    /**
-     * To hold the attribute repository instance
-     */
-    protected $attribute;
-
-    public function __construct(Product $product, Attribute $attribute, ProductFlat $productFlat, Locale $locale)
-    {
-        $this->product = $product;
-
-        $this->attribute = $attribute;
-
-        $this->productFlat = $productFlat;
-
-        $this->locale = $locale;
-    }
-
-    public function findByConditions($conditions)
-    {
-        /**
-         * Empty collection instance to hold all the products that satisfy the conditions
-         */
-        $products = array();
-
-        foreach ($conditions as $condition) {
-            $attribute = $this->attribute->findOneByField('code', $condition->attribute);
-            dump($attribute->type);
-            if ($condition->type == 'select' || $condition->type == 'multiselect') {
-                $values = $condition->value;
-                $attributeValues = array();
-
-                foreach ($condition->options as $option) {
-                    foreach ($values as $value) {
-                        if ($value == $option->id) {
-                            array_push($attributeValues, $option);
-                        }
-                    }
-                }
-
-                $defaultChannelCode = core()->getDefaultChannel()->code;
-                $defaultLocaleCode = $this->locale->find(core()->getDefaultChannel()->default_locale_id)->code;
-
-                foreach ($attributeValues as $attributeValue) {
-                    $productFound = $this->productFlat->findByField([$attribute->code => $attributeValue->id, 'locale' => $defaultLocaleCode, 'channel' => $defaultChannelCode]);
-
-                    if ($productFound->count()) {
-                        array_push($products, $productFound->toArray());
-                    }
-                }
-            } else {
-            }
-        }
-
-        dd(array_flatten($products, 1));
-    }
-}
\ No newline at end of file
diff --git a/packages/Webkul/Discount/src/Http/Controllers/CartRuleController.php b/packages/Webkul/Discount/src/Http/Controllers/CartRuleController.php
index ee7c84ba2..7504d3188 100644
--- a/packages/Webkul/Discount/src/Http/Controllers/CartRuleController.php
+++ b/packages/Webkul/Discount/src/Http/Controllers/CartRuleController.php
@@ -5,7 +5,7 @@ namespace Webkul\Discount\Http\Controllers;
 use Webkul\Discount\Repositories\CartRuleRepository as CartRule;
 use Webkul\Category\Repositories\CategoryRepository as Category;
 use Webkul\Attribute\Repositories\AttributeRepository as Attribute;
-use Webkul\Discount\Helpers\ConvertXToProductId as ConvertX;
+use Webkul\Discount\Helpers\Cart\ConvertXToProductId as ConvertX;
 use Webkul\Discount\Repositories\CartRuleLabelsRepository as CartRuleLabels;
 use Webkul\Discount\Repositories\CartRuleCouponsRepository as CartRuleCoupons;
 
diff --git a/packages/Webkul/Discount/src/Http/Controllers/CatalogRuleController.php b/packages/Webkul/Discount/src/Http/Controllers/CatalogRuleController.php
index def9d21ea..c1fb2fd95 100644
--- a/packages/Webkul/Discount/src/Http/Controllers/CatalogRuleController.php
+++ b/packages/Webkul/Discount/src/Http/Controllers/CatalogRuleController.php
@@ -12,7 +12,7 @@ use Webkul\Product\Repositories\ProductFlatRepository as Product;
 use Webkul\Discount\Repositories\CatalogRuleRepository as CatalogRule;
 use Webkul\Discount\Repositories\CatalogRuleChannelsRepository as CatalogRuleChannels;
 use Webkul\Discount\Repositories\CatalogRuleCustomerGroupsRepository as CatalogRuleCustomerGroups;
-use Webkul\Discount\Helpers\FindProducts;
+use Webkul\Discount\Helpers\Catalog\Apply;
 
 /**
  * Catalog Rule controller
@@ -68,28 +68,35 @@ class CatalogRuleController extends Controller
     protected $catalogRuleCustomerGroups;
 
     /**
-     * To hold the catalog repository instance
+     * To hold catalog repository instance
      */
     protected $catalogRule;
 
     /**
-     * Find products using conditions helper instance
+     * To hold Sale instance
      */
-    protected $findProducts;
+    protected $apply;
 
-    public function __construct(Attribute $attribute, AttributeFamily $attributeFamily, Category $category, Product $product, CatalogRule $catalogRule, CatalogRuleChannels $catalogRuleChannels, CatalogRuleCustomerGroups $catalogRuleCustomerGroups, FindProducts $findProducts)
-    {
+    public function __construct(
+        Attribute $attribute,
+        AttributeFamily $attributeFamily,
+        Category $category, Product $product,
+        CatalogRule $catalogRule,
+        CatalogRuleChannels $catalogRuleChannels,
+        CatalogRuleCustomerGroups $catalogRuleCustomerGroups,
+        Apply $sale
+    ) {
         $this->_config = request('_config');
         $this->attribute = $attribute;
         $this->attributeFamily = $attributeFamily;
         $this->category = $category;
         $this->product = $product;
+        $this->sale = $sale;
         $this->catalogRule = $catalogRule;
         $this->catalogRuleChannels = $catalogRuleChannels;
         $this->catalogRuleCustomerGroups = $catalogRuleCustomerGroups;
         $this->appliedConfig = config('pricerules.catalog');
         $this->appliedConditions = config('pricerules.conditions');
-        $this->findProducts = $findProducts;
     }
 
     public function index()
@@ -242,7 +249,7 @@ class CatalogRuleController extends Controller
         // }
 
         $this->validate(request(), [
-            'name' => 'required|stringunique:catalog_rule,name,'.$id,
+            'name' => 'required|string|unique:catalog_rules,name,' . $id,
             'starts_from' => 'present|nullable|date',
             'ends_till' => 'present|nullable|date',
             'description' => 'string',
@@ -280,15 +287,15 @@ class CatalogRuleController extends Controller
         unset($catalog_rule['all_conditions']);
 
         if (isset($catalog_rule['disc_amount'])) {
-            $catalog_rule['action_type'] = $catalog_rule['apply'];
+            $catalog_rule['action_code'] = $catalog_rule['action_type'];
             $catalog_rule['actions'] = [
-                'action_type' => $catalog_rule['apply'],
+                'action_code' => $catalog_rule['action_type'],
                 'disc_amount' => $catalog_rule['disc_amount']
             ];
         } else if (isset($catalog_rule['disc_percent'])) {
-            $catalog_rule['action_type'] = $catalog_rule['apply'];
+            $catalog_rule['action_code'] = $catalog_rule['action_type'];
             $catalog_rule['actions'] = [
-                'action_type' => $catalog_rule['apply'],
+                'action_code' => $catalog_rule['action'],
                 'disc_percent' => $catalog_rule['disc_percent'],
             ];
         }
@@ -319,12 +326,7 @@ class CatalogRuleController extends Controller
 
     public function applyRules()
     {
-        $catalogRules = $this->catalogRule->all();
-        $decoded = json_decode($catalogRules->first()->conditions);
-        $conditions = json_decode($decoded[0]);
-        $optionableAttributes = $this->fetchOptionableAttributes();
-
-        $results = $this->findProducts->findByConditions($conditions);
+        $this->sale->apply();
     }
 
     public function fetchOptionableAttributes()
diff --git a/packages/Webkul/Discount/src/Models/CatalogRuleCustomerGroups.php b/packages/Webkul/Discount/src/Models/CatalogRuleCustomerGroups.php
index 9a6d56d4f..d34e4541f 100644
--- a/packages/Webkul/Discount/src/Models/CatalogRuleCustomerGroups.php
+++ b/packages/Webkul/Discount/src/Models/CatalogRuleCustomerGroups.php
@@ -3,8 +3,8 @@
 namespace Webkul\Discount\Models;
 
 use Illuminate\Database\Eloquent\Model;
+use Webkul\Customer\Models\CustomerGroupProxy;
 use Webkul\Discount\Contracts\CatalogRuleCustomerGroups as CatalogRuleCustomerGroupsContract;
-use Webkul\Discount\Models\CatalogRuleProxy as CatalogRule;
 
 class CatalogRuleCustomerGroups extends Model implements CatalogRuleCustomerGroupsContract
 {
@@ -12,8 +12,8 @@ class CatalogRuleCustomerGroups extends Model implements CatalogRuleCustomerGrou
 
     protected $guarded = ['created_at', 'updated_at'];
 
-    public function catalog_rule()
+    public function customer_group()
     {
-        return $this->belongsTo(CatalogRule::modelClass(), 'catalog_rule_id');
+        return $this->hasOne(CustomerGroupProxy::modelClass(), 'id', 'customer_group_id');
     }
 }
\ No newline at end of file
diff --git a/packages/Webkul/Shop/src/Http/Controllers/OnepageController.php b/packages/Webkul/Shop/src/Http/Controllers/OnepageController.php
index a3e76d33e..99d6cb2f4 100755
--- a/packages/Webkul/Shop/src/Http/Controllers/OnepageController.php
+++ b/packages/Webkul/Shop/src/Http/Controllers/OnepageController.php
@@ -8,12 +8,9 @@ use Webkul\Shipping\Facades\Shipping;
 use Webkul\Payment\Facades\Payment;
 use Webkul\Checkout\Http\Requests\CustomerAddressForm;
 use Webkul\Sales\Repositories\OrderRepository;
-use Webkul\Discount\Helpers\CouponAbleRule as Coupon;
-use Webkul\Discount\Helpers\NonCouponAbleRule as NonCoupon;
-use Webkul\Discount\Helpers\ValidatesDiscount;
-use Illuminate\Http\Request;
-use Illuminate\Http\Response;
-use Auth;
+use Webkul\Discount\Helpers\Cart\CouponAbleRule as Coupon;
+use Webkul\Discount\Helpers\Cart\NonCouponAbleRule as NonCoupon;
+use Webkul\Discount\Helpers\Cart\ValidatesDiscount;
 
 /**
  * Chekout controller for the customer and guest for placing order
diff --git a/packages/Webkul/Ui/src/Resources/assets/js/components/datetime.vue b/packages/Webkul/Ui/src/Resources/assets/js/components/datetime.vue
index ddb47ed78..6070a8b11 100755
--- a/packages/Webkul/Ui/src/Resources/assets/js/components/datetime.vue
+++ b/packages/Webkul/Ui/src/Resources/assets/js/components/datetime.vue
@@ -21,6 +21,10 @@ export default {
 		};
 	},
 
+	created() {
+
+	},
+
 	mounted() {
 		var this_this = this;
 

From 2a36c6bd0ec5686b40a48441ddb0dc8acd5e84ac Mon Sep 17 00:00:00 2001
From: Prashant Singh <prashant.singh852@webkul.com>
Date: Mon, 12 Aug 2019 19:23:33 +0530
Subject: [PATCH 03/11] Catalog rule implementation in progress

---
 .../views/promotions/cart-rule/edit.blade.php |  2 +-
 .../promotions/catalog-rule/edit.blade.php    |  6 +-
 .../Http/Controllers/CustomerController.php   |  3 +-
 ...ate_catalog_rule_products_price_table.php} |  4 +-
 .../Discount/src/Helpers/Catalog/Apply.php    | 61 +++++++++++-
 .../Helpers/Catalog/ConvertXToProductId.php   |  2 +-
 .../Discount/src/Helpers/Catalog/Sale.php     | 40 ++------
 .../Http/Controllers/CartRuleController.php   |  5 +-
 .../Controllers/CatalogRuleController.php     | 50 +++++++++-
 .../src/Models/CatalogRuleProducts.php        |  8 +-
 .../src/Models/CatalogRuleProductsPrice.php   |  6 +-
 .../src/Providers/ModuleServiceProvider.php   |  1 +
 .../src/Repositories/CatalogRuleProducts.php  | 24 -----
 .../Repositories/CatalogRuleProductsPrice.php | 24 -----
 .../CatalogRuleProductsPriceRepository.php    | 51 ++++++++++
 .../CatalogRuleProductsRepository.php         | 96 +++++++++++++++++++
 16 files changed, 275 insertions(+), 108 deletions(-)
 rename packages/Webkul/Discount/src/Database/Migrations/{2019_08_08_130583_create_catalog_rule_product_price_table.php => 2019_08_08_130583_create_catalog_rule_products_price_table.php} (90%)
 delete mode 100644 packages/Webkul/Discount/src/Repositories/CatalogRuleProducts.php
 delete mode 100644 packages/Webkul/Discount/src/Repositories/CatalogRuleProductsPrice.php
 create mode 100644 packages/Webkul/Discount/src/Repositories/CatalogRuleProductsPriceRepository.php
 create mode 100644 packages/Webkul/Discount/src/Repositories/CatalogRuleProductsRepository.php

diff --git a/packages/Webkul/Admin/src/Resources/views/promotions/cart-rule/edit.blade.php b/packages/Webkul/Admin/src/Resources/views/promotions/cart-rule/edit.blade.php
index f671427b1..c49279b14 100644
--- a/packages/Webkul/Admin/src/Resources/views/promotions/cart-rule/edit.blade.php
+++ b/packages/Webkul/Admin/src/Resources/views/promotions/cart-rule/edit.blade.php
@@ -689,7 +689,7 @@
                                 delete this.attribute_values[i].options;
                             }
 
-                            if (this.category_values.length > 0) {
+                            if (this.category_values != null && this.category_values.length > 0) {
                                 this.all_attributes.categories = this.category_values;
                             }
 
diff --git a/packages/Webkul/Admin/src/Resources/views/promotions/catalog-rule/edit.blade.php b/packages/Webkul/Admin/src/Resources/views/promotions/catalog-rule/edit.blade.php
index 91db3e8be..b2de8d170 100644
--- a/packages/Webkul/Admin/src/Resources/views/promotions/catalog-rule/edit.blade.php
+++ b/packages/Webkul/Admin/src/Resources/views/promotions/catalog-rule/edit.blade.php
@@ -1,7 +1,7 @@
 @extends('admin::layouts.content')
 
 @section('page_title')
-    {{ __('admin::app.promotion.add-catalog-rule') }}
+    {{ __('admin::app.promotion.edit-catalog-rule') }}
 @stop
 
 @section('content')
@@ -20,13 +20,13 @@
                             <h1>
                                 <i class="icon angle-left-icon back-link" onclick="history.length > 1 ? history.go(-1) : window.location = '{{ url('/admin/dashboard') }}';"></i>
 
-                                {{ __('admin::app.promotion.add-catalog-rule') }}
+                                {{ __('admin::app.promotion.edit-catalog-rule') }}
                             </h1>
                         </div>
 
                         <div class="page-action fixed-action">
                             <button type="submit" class="btn btn-lg btn-primary">
-                                {{ __('admin::app.promotion.save-btn-title') }}
+                                {{ __('admin::app.promotion.edit-btn-title') }}
                             </button>
                         </div>
                     </div>
diff --git a/packages/Webkul/Customer/src/Http/Controllers/CustomerController.php b/packages/Webkul/Customer/src/Http/Controllers/CustomerController.php
index 4e362c5eb..14c36e819 100755
--- a/packages/Webkul/Customer/src/Http/Controllers/CustomerController.php
+++ b/packages/Webkul/Customer/src/Http/Controllers/CustomerController.php
@@ -11,7 +11,8 @@ use Auth;
 use Hash;
 
 /**
- * Customer controlller for the customer basically for the tasks of customers which will be done after customer authentication.
+ * Customer controlller for the customer basically for the tasks of customers which will be
+ * done after customer authentication.
  *
  * @author  Prashant Singh <prashant.singh852@webkul.com>
  * @copyright 2018 Webkul Software Pvt Ltd (http://www.webkul.com)
diff --git a/packages/Webkul/Discount/src/Database/Migrations/2019_08_08_130583_create_catalog_rule_product_price_table.php b/packages/Webkul/Discount/src/Database/Migrations/2019_08_08_130583_create_catalog_rule_products_price_table.php
similarity index 90%
rename from packages/Webkul/Discount/src/Database/Migrations/2019_08_08_130583_create_catalog_rule_product_price_table.php
rename to packages/Webkul/Discount/src/Database/Migrations/2019_08_08_130583_create_catalog_rule_products_price_table.php
index 740ce6f3c..b9d9eff19 100644
--- a/packages/Webkul/Discount/src/Database/Migrations/2019_08_08_130583_create_catalog_rule_product_price_table.php
+++ b/packages/Webkul/Discount/src/Database/Migrations/2019_08_08_130583_create_catalog_rule_products_price_table.php
@@ -4,7 +4,7 @@ use Illuminate\Support\Facades\Schema;
 use Illuminate\Database\Schema\Blueprint;
 use Illuminate\Database\Migrations\Migration;
 
-class CreateCatalogRuleProductPriceTable extends Migration
+class CreateCatalogRuleProductsPriceTable extends Migration
 {
     /**
      * Run the migrations.
@@ -13,7 +13,7 @@ class CreateCatalogRuleProductPriceTable extends Migration
      */
     public function up()
     {
-        Schema::create('catalog_rule_product_price', function (Blueprint $table) {
+        Schema::create('catalog_rule_products_price', function (Blueprint $table) {
             $table->bigIncrements('id');
             $table->integer('channel_id')->unsigned();
             $table->foreign('channel_id')->references('id')->on('channels')->onDelete('cascade');
diff --git a/packages/Webkul/Discount/src/Helpers/Catalog/Apply.php b/packages/Webkul/Discount/src/Helpers/Catalog/Apply.php
index 049ce9884..7f7b4e6e8 100644
--- a/packages/Webkul/Discount/src/Helpers/Catalog/Apply.php
+++ b/packages/Webkul/Discount/src/Helpers/Catalog/Apply.php
@@ -3,6 +3,9 @@
 namespace Webkul\Discount\Helpers\Catalog;
 
 use Webkul\Discount\Repositories\CatalogRuleRepository as CatalogRule;
+use Webkul\Discount\Repositories\CatalogRuleProductsRepository as CatalogRuleProducts;
+use Webkul\Discount\Repositories\CatalogRuleProductsPriceRepository as CatalogRuleProductsPrice;
+use Webkul\Discount\Helpers\Catalog\ConvertXToProductId as ConvertX;
 use Webkul\Discount\Helpers\Catalog\Sale;
 
 class Apply extends Sale
@@ -22,6 +25,21 @@ class Apply extends Sale
      */
     protected $deceased;
 
+    /**
+     * Holds convertXToProductId instance
+     */
+    protected $convertX;
+
+    /**
+     * Hold catalog rule products repository instance
+     */
+    protected $catalogRuleProduct;
+
+    /**
+     * Hold catalog rule products price repository instance
+     */
+    protected $catalogRuleProductPrice;
+
     /**
      * To hold the rule classes
      */
@@ -30,10 +48,20 @@ class Apply extends Sale
     /**
      * @param CatalogRule $catalogRule
      */
-    public function __construct(CatalogRule $catalogRule)
-    {
+    public function __construct(
+        CatalogRule $catalogRule,
+        ConvertX $convertX,
+        CatalogRuleProducts $catalogRuleProduct,
+        CatalogRuleProductsPrice $catalogRuleProductPrice
+    ) {
         $this->catalogRule = $catalogRule;
 
+        $this->convertX = $convertX;
+
+        $this->catalogRuleProduct = $catalogRuleProduct;
+
+        $this->catalogRuleProductPrice = $catalogRuleProductPrice;
+
         $this->active = collect();
 
         $this->deceased = collect();
@@ -55,14 +83,39 @@ class Apply extends Sale
                 $this->active->push($rule->id);
 
                 // Job execution for active rules
-                $products = $this->getProductIds($rule);
+                $productIDs = $this->getProductIds($rule);
 
-                $this->setSalePrice($products);
+                $this->setSale($rule, $productIDs);
             } else {
                 $this->deceased->push($rule->id);
 
                 // Job execution for deceased rules
             }
         }
+
+        dd('done');
+    }
+
+    /**
+     * To set the sale price for products falling catalog rule criteria
+     *
+     * @param CatalogRule $catalogRule
+     *
+     * @return Void
+     */
+    public function setSale($rule, $productIDs)
+    {
+        if (is_array($productIDs)) {
+            // apply on selected products
+            foreach ($productIDs as $productID) {
+                $this->catalogRuleProduct->createOrUpdate($rule, $productID);
+
+                $this->catalogRuleProductPrice->createOrUpdate($rule, $productID);
+            }
+        } else if ($productIDs == '*') {
+            $this->catalogRuleProduct->createOrUpdate($rule, $productIDs);
+
+            $this->catalogRuleProductPrice->createOrUpdate($rule, $productIDs);
+        }
     }
 }
\ No newline at end of file
diff --git a/packages/Webkul/Discount/src/Helpers/Catalog/ConvertXToProductId.php b/packages/Webkul/Discount/src/Helpers/Catalog/ConvertXToProductId.php
index 52755c3cd..ff346b2db 100644
--- a/packages/Webkul/Discount/src/Helpers/Catalog/ConvertXToProductId.php
+++ b/packages/Webkul/Discount/src/Helpers/Catalog/ConvertXToProductId.php
@@ -71,7 +71,7 @@ class ConvertXToProductId
 
     public function convertX($attribute_conditions)
     {
-        $attributeConditions = json_decode($attribute_conditions);
+        $attributeConditions = json_decode(json_decode($attribute_conditions));
 
         $categoryValues = $attributeConditions->categories;
 
diff --git a/packages/Webkul/Discount/src/Helpers/Catalog/Sale.php b/packages/Webkul/Discount/src/Helpers/Catalog/Sale.php
index ca0f02703..8fbb1cce0 100644
--- a/packages/Webkul/Discount/src/Helpers/Catalog/Sale.php
+++ b/packages/Webkul/Discount/src/Helpers/Catalog/Sale.php
@@ -7,33 +7,6 @@ use Webkul\Discount\Helpers\Catalog\ConvertXToProductId;
 
 abstract class Sale
 {
-    /**
-     * To hold the catalog rule repository instance
-     */
-    protected $catalogRule;
-
-    /**
-     * ConvertXToProductId instance
-     */
-    protected $convertX;
-
-    /**
-     * To hold the rule classes
-     */
-    protected $rules;
-
-    /**
-     * @param CatalogRule $catalogRule
-     */
-    public function __construct(CatalogRule $catalogRule, ConvertXToProductId $convertX)
-    {
-        $this->catalogRule = $catalogRule;
-
-        $this->convertX = $convertX;
-
-        $this->rules = config('discount-rules.catalog');
-    }
-
     abstract function apply();
 
     /**
@@ -81,23 +54,22 @@ abstract class Sale
 
     /**
      * Function to maintain products and catalog rule
+     *
+     * @param CatalogRule $rule
+     *
+     * @return String|NULL
      */
     public function getProductIds($rule)
     {
-        $productIDs = $rule->conditions;
-
         if ($rule->conditions) {
             $conditions = $rule->conditions;
 
-            // $convertX = new ConvertXToProductId();
-
-            dd($this->convertX);
             $productIDs = $this->convertX->convertX($rule->conditions);
         } else {
-            // apply on all products
+            $productIDs = '*';
         }
 
-        dd($productIDs);
+        return $productIDs;
     }
 
      /**
diff --git a/packages/Webkul/Discount/src/Http/Controllers/CartRuleController.php b/packages/Webkul/Discount/src/Http/Controllers/CartRuleController.php
index 7504d3188..72666418d 100644
--- a/packages/Webkul/Discount/src/Http/Controllers/CartRuleController.php
+++ b/packages/Webkul/Discount/src/Http/Controllers/CartRuleController.php
@@ -117,7 +117,7 @@ class CartRuleController extends Controller
     public function store()
     {
         $validated = $this->validate(request(), [
-            'name' => 'required|string',
+            'name' => 'required|string|unique:cart_rules,name',
             'description' => 'string',
             'customer_groups' => 'required|array',
             'channels' => 'required|array',
@@ -349,8 +349,9 @@ class CartRuleController extends Controller
      */
     public function update($id)
     {
+        dd(request()->all());
         $this->validate(request(), [
-            'name' => 'required|string',
+            'name' => 'required|string|unique:cart_rules,name,'.$id,
             'description' => 'string',
             'customer_groups' => 'required|array',
             'channels' => 'required|array',
diff --git a/packages/Webkul/Discount/src/Http/Controllers/CatalogRuleController.php b/packages/Webkul/Discount/src/Http/Controllers/CatalogRuleController.php
index c1fb2fd95..31db687f2 100644
--- a/packages/Webkul/Discount/src/Http/Controllers/CatalogRuleController.php
+++ b/packages/Webkul/Discount/src/Http/Controllers/CatalogRuleController.php
@@ -104,12 +104,21 @@ class CatalogRuleController extends Controller
         return view($this->_config['view']);
     }
 
+    /**
+     * To load create form for catalog rule
+     *
+     * @return View
+     */
     public function create()
     {
-        // dd($this->appliedConfig);
         return view($this->_config['view'])->with('catalog_rule', [$this->appliedConfig, $this->category->getPartial(), $this->getStatesAndCountries(), $this->attribute->getPartial()]);
     }
 
+    /**
+     * To store newly created catalog rule and store it
+     *
+     * @return Redirect
+     */
     public function store()
     {
         $this->validate(request(), [
@@ -148,8 +157,11 @@ class CatalogRuleController extends Controller
         unset($catalog_rule['criteria']);
 
         $catalog_rule['conditions'] = $catalog_rule['all_conditions'];
+
         unset($catalog_rule['all_conditions']);
 
+        $catalog_rule['conditions'] = json_encode($catalog_rule['conditions']);
+
         if (isset($catalog_rule['disc_amount'])) {
             $catalog_rule['actions'] = [
                 'action_code' => $catalog_rule['action_type'],
@@ -173,7 +185,6 @@ class CatalogRuleController extends Controller
         unset($catalog_rule['all_actions']);
 
         $catalog_rule['actions'] = json_encode($catalog_rule['actions']);
-        $catalog_rule['conditions'] = json_encode($catalog_rule['conditions']);
 
         $catalogRule = $this->catalogRule->create($catalog_rule);
 
@@ -203,6 +214,13 @@ class CatalogRuleController extends Controller
         }
     }
 
+    /**
+     * To load edit for previously created catalog rule
+     *
+     * @param $id
+     *
+     * @return View
+     */
     public function edit($id)
     {
         $catalog_rule = $this->catalogRule->find($id);
@@ -224,12 +242,19 @@ class CatalogRuleController extends Controller
             ]);
     }
 
+    /**
+     * To update previously created catalog rule
+     *
+     * @param $id
+     *
+     * @return Redirect
+     */
     public function update($id)
     {
         $data = request()->input();
 
         // $validated = \Validator::make($data, [
-        //     'name' => 'required|stringunique:catalog_rule,name,' . $id,
+        //     'name' => 'required|string|unique:catalog_rules,name,' . $id,
         //     'starts_from' => 'present|nullable|date',
         //     'ends_till' => 'present|nullable|date',
         //     'description' => 'string',
@@ -249,7 +274,8 @@ class CatalogRuleController extends Controller
         // }
 
         $this->validate(request(), [
-            'name' => 'required|string|unique:catalog_rules,name,' . $id,
+            'name' => 'required|string|unique:catalog_rules,name,'.$id,
+            // 'name' => 'required|string',
             'starts_from' => 'present|nullable|date',
             'ends_till' => 'present|nullable|date',
             'description' => 'string',
@@ -300,6 +326,9 @@ class CatalogRuleController extends Controller
             ];
         }
 
+        $catalog_rule['discount_amount'] = $catalog_rule['disc_amount'];
+
+        unset($catalog_rule['disc_amount']);
         unset($catalog_rule['apply']);
         unset($catalog_rule['attributes']);
         unset($catalog_rule['_token']);
@@ -324,6 +353,11 @@ class CatalogRuleController extends Controller
         }
     }
 
+    /**
+     * To apply the catalog rules
+     *
+     * @return Redirect
+     */
     public function applyRules()
     {
         $this->sale->apply();
@@ -342,6 +376,13 @@ class CatalogRuleController extends Controller
         return $attributesWithOptions;
     }
 
+    /**
+     * To delete existing catalog rule
+     *
+     * @param Integer $id
+     *
+     * @return Redirect
+     */
     public function destroy($id)
     {
         $catalogRule = $this->catalogRule->findOrFail($id);
@@ -357,7 +398,6 @@ class CatalogRuleController extends Controller
         }
     }
 
-
     /**
      * Get Countries and states list from core helpers
      *
diff --git a/packages/Webkul/Discount/src/Models/CatalogRuleProducts.php b/packages/Webkul/Discount/src/Models/CatalogRuleProducts.php
index 93ed7f637..ba8e1468f 100644
--- a/packages/Webkul/Discount/src/Models/CatalogRuleProducts.php
+++ b/packages/Webkul/Discount/src/Models/CatalogRuleProducts.php
@@ -3,13 +3,13 @@
 namespace Webkul\Discount\Models;
 
 use Illuminate\Database\Eloquent\Model;
-use Webkul\Discount\Contracts\CatalogRuleProducts as CatalogRuleProductContract;
+use Webkul\Discount\Contracts\CatalogRuleProducts as CatalogRuleProductsContract;
 use Webkul\Discount\Models\CatalogRuleChannelsProxy as CatalogRuleChannels;
 use Webkul\Discount\Models\CatalogRuleCustomerGroupsProxy as CatalogRuleCustomerGroups;
 
-class CatalogRuleProducts extends Model implements CatalogRuleProductContract
+class CatalogRuleProducts extends Model implements CatalogRuleProductsContract
 {
-    protected $table = 'catalog_rules_products';
+    protected $table = 'catalog_rule_products';
 
-    protected $guarded = ['created_at', 'updated_at'];
+    protected $fillable = ['catalog_rule_id', 'starts_from', 'ends_till', 'customer_group_id', 'channel_id', 'product_id', 'action_code', 'action_amount'];
 }
\ No newline at end of file
diff --git a/packages/Webkul/Discount/src/Models/CatalogRuleProductsPrice.php b/packages/Webkul/Discount/src/Models/CatalogRuleProductsPrice.php
index e09d6149b..61f439e26 100644
--- a/packages/Webkul/Discount/src/Models/CatalogRuleProductsPrice.php
+++ b/packages/Webkul/Discount/src/Models/CatalogRuleProductsPrice.php
@@ -3,11 +3,11 @@
 namespace Webkul\Discount\Models;
 
 use Illuminate\Database\Eloquent\Model;
-use Webkul\Discount\Contracts\CatalogRuleProductsPrice as CatalogRuleProductPriceContract;
+use Webkul\Discount\Contracts\CatalogRuleProductsPrice as CatalogRuleProductsPriceContract;
 
-class CatalogRuleProductsPrice extends Model implements CatalogRuleProductContract
+class CatalogRuleProductsPrice extends Model implements CatalogRuleProductsPriceContract
 {
     protected $table = 'catalog_rules_products_price';
 
-    protected $guarded = ['created_at', 'updated_at'];
+    protected $fillable = ['catalog_rule_id', 'starts_from', 'ends_till', 'customer_group_id', 'channel_id', 'product_id', 'rule_price'];
 }
\ No newline at end of file
diff --git a/packages/Webkul/Discount/src/Providers/ModuleServiceProvider.php b/packages/Webkul/Discount/src/Providers/ModuleServiceProvider.php
index ca41e9a4f..4332e00dc 100644
--- a/packages/Webkul/Discount/src/Providers/ModuleServiceProvider.php
+++ b/packages/Webkul/Discount/src/Providers/ModuleServiceProvider.php
@@ -11,6 +11,7 @@ class ModuleServiceProvider extends BaseModuleServiceProvider
         \Webkul\Discount\Models\CatalogRuleChannels::class,
         \Webkul\Discount\Models\CatalogRuleCustomerGroups::class,
         \Webkul\Discount\Models\CatalogRuleProducts::class,
+        \Webkul\Discount\Models\CatalogRuleProductsPrice::class,
         \Webkul\Discount\Models\CartRule::class,
         \Webkul\Discount\Models\CartRuleChannels::class,
         \Webkul\Discount\Models\CartRuleCustomerGroups::class,
diff --git a/packages/Webkul/Discount/src/Repositories/CatalogRuleProducts.php b/packages/Webkul/Discount/src/Repositories/CatalogRuleProducts.php
deleted file mode 100644
index 142179ddc..000000000
--- a/packages/Webkul/Discount/src/Repositories/CatalogRuleProducts.php
+++ /dev/null
@@ -1,24 +0,0 @@
-<?php
-
-namespace Webkul\Discount\Repositories;
-
-use Webkul\Core\Eloquent\Repository;
-
-/**
- * Catalog Rule Products Reposotory
- *
- * @author  Prashant Singh <prashant.singh852@webkul.com>
- * @copyright 2018 Webkul Software Pvt Ltd (http://www.webkul.com)
- */
-class CatalogRuleProductsRepository extends Repository
-{
-    /**
-     * Specify Model class name
-     *
-     * @return mixed
-     */
-    function model()
-    {
-        return 'Webkul\Discount\Contracts\CatalogRuleProducts';
-    }
-}
diff --git a/packages/Webkul/Discount/src/Repositories/CatalogRuleProductsPrice.php b/packages/Webkul/Discount/src/Repositories/CatalogRuleProductsPrice.php
deleted file mode 100644
index f330516dc..000000000
--- a/packages/Webkul/Discount/src/Repositories/CatalogRuleProductsPrice.php
+++ /dev/null
@@ -1,24 +0,0 @@
-<?php
-
-namespace Webkul\Discount\Repositories;
-
-use Webkul\Core\Eloquent\Repository;
-
-/**
- * Catalog Rule Products Price Reposotory
- *
- * @author  Prashant Singh <prashant.singh852@webkul.com>
- * @copyright 2018 Webkul Software Pvt Ltd (http://www.webkul.com)
- */
-class CatalogRuleProductsPriceRepository extends Repository
-{
-    /**
-     * Specify Model class name
-     *
-     * @return mixed
-     */
-    function model()
-    {
-        return 'Webkul\Discount\Contracts\CatalogRuleProductsPrice';
-    }
-}
\ No newline at end of file
diff --git a/packages/Webkul/Discount/src/Repositories/CatalogRuleProductsPriceRepository.php b/packages/Webkul/Discount/src/Repositories/CatalogRuleProductsPriceRepository.php
new file mode 100644
index 000000000..99a146db6
--- /dev/null
+++ b/packages/Webkul/Discount/src/Repositories/CatalogRuleProductsPriceRepository.php
@@ -0,0 +1,51 @@
+<?php
+
+namespace Webkul\Discount\Repositories;
+
+use Webkul\Core\Eloquent\Repository;
+use Webkul\Product\Repositories\ProductRepository as Product;
+
+/**
+ * Catalog Rule Products Price Reposotory
+ *
+ * @author  Prashant Singh <prashant.singh852@webkul.com>
+ * @copyright 2018 Webkul Software Pvt Ltd (http://www.webkul.com)
+ */
+class CatalogRuleProductsPriceRepository extends Repository
+{
+    /**
+     * ProductRepository instance
+     */
+    protected $product;
+
+    public function __construct(Product $product)
+    {
+        $this->product = $product;
+    }
+
+    /**
+     * Specify Model class name
+     *
+     * @return mixed
+     */
+    function model()
+    {
+        return 'Webkul\Discount\Contracts\CatalogRuleProductsPrice';
+    }
+
+    /**
+     * Create or update resource
+     */
+    public function createOrUpdate($rule, $productID)
+    {
+        if ($productID == '*') {
+
+        } else {
+            $products = $this->product->all('id');
+
+            foreach ($products as $product) {
+
+            }
+        }
+    }
+}
\ No newline at end of file
diff --git a/packages/Webkul/Discount/src/Repositories/CatalogRuleProductsRepository.php b/packages/Webkul/Discount/src/Repositories/CatalogRuleProductsRepository.php
new file mode 100644
index 000000000..959569d1a
--- /dev/null
+++ b/packages/Webkul/Discount/src/Repositories/CatalogRuleProductsRepository.php
@@ -0,0 +1,96 @@
+<?php
+
+namespace Webkul\Discount\Repositories;
+
+use Webkul\Core\Eloquent\Repository;
+use Webkul\Product\Repositories\ProductRepository as Product;
+use Illuminate\Container\Container as App;
+
+/**
+ * Catalog Rule Products Reposotory
+ *
+ * @author  Prashant Singh <prashant.singh852@webkul.com>
+ * @copyright 2018 Webkul Software Pvt Ltd (http://www.webkul.com)
+ */
+class CatalogRuleProductsRepository extends Repository
+{
+    /**
+     * ProductRepository instance
+     */
+    protected $product;
+
+    public function __construct(Product $product, App $app)
+    {
+        $this->product = $product;
+
+        parent::__construct($app);
+    }
+
+    /**
+     * Specify Model class name
+     *
+     * @return Mixed
+     */
+    function model()
+    {
+        return 'Webkul\Discount\Contracts\CatalogRuleProducts';
+    }
+
+    /**
+     * Create or update catalog rule product resource
+     */
+    public function createOrUpdate($rule, $productID)
+    {
+        $channels = $rule->channels;
+        $customerGroups = $rule->customer_groups;
+
+        $channelsGroupsCross = $channels->crossJoin($customerGroups);
+
+        if ($productID == '*') {
+            $products = $this->product->all('id');
+
+            // for all products do that thing later
+        } else {
+            $product = $this->product->find($productID);
+
+            foreach ($channelsGroupsCross as $channelGroup) {
+                $channelId = $channelGroup[0]->channel_id;
+                $groupId = $channelGroup[1]->customer_group_id;
+
+                $model = new $this->model();
+
+                $catalogRuleProduct = $model->where([
+                    'channel_id' => $channelId,
+                    'customer_group_id' => $groupId,
+                    'product_id' => $productID
+                ])->get();
+
+                if ($catalogRuleProduct->count()) {
+                    // update
+                    $catalogRuleProduct->first()->update([
+                        'catalog_rule_id' => $rule->id,
+                        'starts_from' => $rule->starts_from,
+                        'ends_till' => $rule->ends_till,
+                        'customer_group_id' => $groupId,
+                        'channel_id' => $channelId,
+                        'product_id' => $productID,
+                        'action_code' => $rule->action_code,
+                        'action_amount' => $rule->discount_amount
+                    ]);
+                } else {
+                    // create
+                    $this->create([
+                        'catalog_rule_id' => $rule->id,
+                        'starts_from' => $rule->starts_from,
+                        'ends_till' => $rule->ends_till,
+                        'customer_group_id' => $groupId,
+                        'channel_id' => $channelId,
+                        'product_id' => $productID,
+                        'action_code' => $rule->action_code,
+                        'action_amount' => $rule->discount_amount
+                    ]);
+                }
+            }
+        }
+    }
+}

From dd9ff1629da2998c91354d7d5f5f02362cc7c7ca Mon Sep 17 00:00:00 2001
From: Prashant Singh <prashant.singh852@webkul.com>
Date: Tue, 13 Aug 2019 23:31:14 +0530
Subject: [PATCH 04/11] Fixed issue #1287

---
 packages/Webkul/Admin/src/Providers/EventServiceProvider.php  | 2 --
 .../Webkul/Attribute/src/Repositories/AttributeRepository.php | 4 ++--
 2 files changed, 2 insertions(+), 4 deletions(-)

diff --git a/packages/Webkul/Admin/src/Providers/EventServiceProvider.php b/packages/Webkul/Admin/src/Providers/EventServiceProvider.php
index 0651d4859..ac1cf5ac1 100755
--- a/packages/Webkul/Admin/src/Providers/EventServiceProvider.php
+++ b/packages/Webkul/Admin/src/Providers/EventServiceProvider.php
@@ -21,7 +21,5 @@ class EventServiceProvider extends ServiceProvider
         Event::listen('sales.shipment.save.after', 'Webkul\Admin\Listeners\Order@sendNewShipmentMail');
 
         Event::listen('checkout.order.save.after', 'Webkul\Admin\Listeners\Order@updateProductInventory');
-
-        Event::listen('products.datagrid.sync', 'Webkul\Admin\Listeners\Product@sync');
     }
 }
\ No newline at end of file
diff --git a/packages/Webkul/Attribute/src/Repositories/AttributeRepository.php b/packages/Webkul/Attribute/src/Repositories/AttributeRepository.php
index 4c7c921a9..96d43a944 100755
--- a/packages/Webkul/Attribute/src/Repositories/AttributeRepository.php
+++ b/packages/Webkul/Attribute/src/Repositories/AttributeRepository.php
@@ -224,7 +224,7 @@ class AttributeRepository extends Repository
                 if ($attribute->options()->exists()) {
                     array_push($trimmed, [
                         'id' => $attribute->id,
-                        'name' => $attribute->name,
+                        'name' => $attribute->admin_name,
                         'type' => $attribute->type,
                         'code' => $attribute->code,
                         'has_options' => true,
@@ -233,7 +233,7 @@ class AttributeRepository extends Repository
                 } else {
                     array_push($trimmed, [
                         'id' => $attribute->id,
-                        'name' => $attribute->name,
+                        'name' => $attribute->admin_name,
                         'type' => $attribute->type,
                         'code' => $attribute->code,
                         'has_options' => false,

From ce56fb3bb1fc18eaef5a8b07422b55615bb000cf Mon Sep 17 00:00:00 2001
From: Prashant Singh <prashant.singh852@webkul.com>
Date: Wed, 14 Aug 2019 02:35:00 +0530
Subject: [PATCH 05/11] Added default attribute brand

---
 .../src/Database/Seeders/AttributeGroupTableSeeder.php       | 3 ++-
 .../Attribute/src/Database/Seeders/AttributeTableSeeder.php  | 5 ++++-
 2 files changed, 6 insertions(+), 2 deletions(-)

diff --git a/packages/Webkul/Attribute/src/Database/Seeders/AttributeGroupTableSeeder.php b/packages/Webkul/Attribute/src/Database/Seeders/AttributeGroupTableSeeder.php
index 78d719819..92fa5db6c 100755
--- a/packages/Webkul/Attribute/src/Database/Seeders/AttributeGroupTableSeeder.php
+++ b/packages/Webkul/Attribute/src/Database/Seeders/AttributeGroupTableSeeder.php
@@ -43,7 +43,8 @@ class AttributeGroupTableSeeder extends Seeder
             ['attribute_id' => '21','attribute_group_id' => '5','position' => '3'],
             ['attribute_id' => '22','attribute_group_id' => '5','position' => '4'],
             ['attribute_id' => '23','attribute_group_id' => '1','position' => '9'],
-            ['attribute_id' => '24','attribute_group_id' => '1','position' => '10']
+            ['attribute_id' => '24','attribute_group_id' => '1','position' => '10'],
+            ['attribute_id' => '25','attribute_group_id' => '1','position' => '11']
         ]);
     }
 }
\ No newline at end of file
diff --git a/packages/Webkul/Attribute/src/Database/Seeders/AttributeTableSeeder.php b/packages/Webkul/Attribute/src/Database/Seeders/AttributeTableSeeder.php
index 380710c38..6d0eac76c 100755
--- a/packages/Webkul/Attribute/src/Database/Seeders/AttributeTableSeeder.php
+++ b/packages/Webkul/Attribute/src/Database/Seeders/AttributeTableSeeder.php
@@ -59,6 +59,8 @@ class AttributeTableSeeder extends Seeder
             ['id' => '23','code' => 'color','admin_name' => 'Color','type' => 'select','validation' => NULL,'position' => '23','is_required' => '0','is_unique' => '0','value_per_locale' => '0','value_per_channel' => '0','is_filterable' => '1','is_configurable' => '1','is_user_defined' => '1','is_visible_on_front' => '0',
             'use_in_flat' => '1','created_at' => $now,'updated_at' => $now],
             ['id' => '24','code' => 'size','admin_name' => 'Size','type' => 'select','validation' => NULL,'position' => '24','is_required' => '0','is_unique' => '0','value_per_locale' => '0','value_per_channel' => '0','is_filterable' => '1','is_configurable' => '1','is_user_defined' => '1','is_visible_on_front' => '0',
+            'use_in_flat' => '1','created_at' => $now,'updated_at' => $now],
+            ['id' => '25','code' => 'brand','admin_name' => 'Brand','type' => 'text','validation' => NULL,'position' => '25','is_required' => '0','is_unique' => '0','value_per_locale' => '0','value_per_channel' => '0','is_filterable' => '1','is_configurable' => '0','is_user_defined' => '0','is_visible_on_front' => '1',
             'use_in_flat' => '1','created_at' => $now,'updated_at' => $now]
         ]);
 
@@ -87,7 +89,8 @@ class AttributeTableSeeder extends Seeder
            ['id' => '21','locale' => 'en','name' => 'Depth','attribute_id' => '21'],
            ['id' => '22','locale' => 'en','name' => 'Weight','attribute_id' => '22'],
            ['id' => '23','locale' => 'en','name' => 'Color','attribute_id' => '23'],
-           ['id' => '24','locale' => 'en','name' => 'Size','attribute_id' => '24']
+           ['id' => '24','locale' => 'en','name' => 'Size','attribute_id' => '24'],
+           ['id' => '25','locale' => 'en','name' => 'Brand','attribute_id' => '25']
         ]);
     }
 }
\ No newline at end of file

From 9425b39b788fbf848ea68a8e3b5dc5e1f0ca80f4 Mon Sep 17 00:00:00 2001
From: Prashant Singh <prashant.singh852@webkul.com>
Date: Wed, 14 Aug 2019 13:42:03 +0530
Subject: [PATCH 06/11] Fixed issue #1294

---
 packages/Webkul/Admin/src/Config/menu.php        | 16 +++++++++-------
 .../Http/Controllers/Admin/PageController.php    |  4 ++--
 2 files changed, 11 insertions(+), 9 deletions(-)

diff --git a/packages/Webkul/Admin/src/Config/menu.php b/packages/Webkul/Admin/src/Config/menu.php
index 2c73ae44d..1091a60fb 100755
--- a/packages/Webkul/Admin/src/Config/menu.php
+++ b/packages/Webkul/Admin/src/Config/menu.php
@@ -188,13 +188,15 @@ return [
         'route' => 'admin.cart-rule.index',
         'sort' => 1,
         'icon-class' => '',
-    ], [
-        'key' => 'promotions.catalog-rule',
-        'name' => 'admin::app.promotion.catalog-rule',
-        'route' => 'admin.catalog-rule.index',
-        'sort' => 1,
-        'icon-class' => '',
-    ], [
+    ],
+    // [
+    //     'key' => 'promotions.catalog-rule',
+    //     'name' => 'admin::app.promotion.catalog-rule',
+    //     'route' => 'admin.catalog-rule.index',
+    //     'sort' => 1,
+    //     'icon-class' => '',
+    // ],
+    [
         'key' => 'cms',
         'name' => 'admin::app.layouts.cms',
         'route' => 'admin.cms.index',
diff --git a/packages/Webkul/CMS/src/Http/Controllers/Admin/PageController.php b/packages/Webkul/CMS/src/Http/Controllers/Admin/PageController.php
index fd05c87b0..070a7d26d 100644
--- a/packages/Webkul/CMS/src/Http/Controllers/Admin/PageController.php
+++ b/packages/Webkul/CMS/src/Http/Controllers/Admin/PageController.php
@@ -194,11 +194,11 @@ use Webkul\Core\Repositories\LocaleRepository as Locale;
         $page = $this->cms->findOrFail($id);
 
         if ($page->delete()) {
-            session()->flash('success', trans('admin::app.cms.delete-success'));
+            session()->flash('success', trans('admin::app.cms.pages.delete-success'));
 
             return response()->json(['message' => true], 200);
         } else {
-            session()->flash('success', trans('admin::app.cms.delete-failure'));
+            session()->flash('success', trans('admin::app.cms.pages.delete-failure'));
 
             return response()->json(['message' => false], 200);
         }

From 54e54918a7ee5c6521d5871d02c7db517faab2f4 Mon Sep 17 00:00:00 2001
From: Prashant Singh <prashant.singh852@webkul.com>
Date: Wed, 14 Aug 2019 13:44:21 +0530
Subject: [PATCH 07/11] Fixed issue #1293

---
 packages/Webkul/Admin/src/Resources/lang/en/app.php | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/packages/Webkul/Admin/src/Resources/lang/en/app.php b/packages/Webkul/Admin/src/Resources/lang/en/app.php
index 3aefe0d71..fd879b7f5 100755
--- a/packages/Webkul/Admin/src/Resources/lang/en/app.php
+++ b/packages/Webkul/Admin/src/Resources/lang/en/app.php
@@ -989,7 +989,7 @@ return [
             'url-key' => 'URL Key',
             'create-btn-title' => 'Add Page',
             'edit-title' => 'Edit Page',
-            'edit-btn-title' => 'Edit Page',
+            'edit-btn-title' => 'Save Page',
             'create-success' => 'Page created successfully',
             'create-failure' => 'Page cannot be created',
             'update-success' => 'Page updated successfully',

From de1c00f15eb184279b4bc25b98fd4965238d07f2 Mon Sep 17 00:00:00 2001
From: Prashant Singh <prashant.singh852@webkul.com>
Date: Wed, 14 Aug 2019 13:49:04 +0530
Subject: [PATCH 08/11] Fixed issue #1295

---
 .../Webkul/Admin/src/Resources/views/cms/create.blade.php  | 4 ++--
 .../src/Http/Controllers/Shop/PagePresenterController.php  | 7 ++++++-
 2 files changed, 8 insertions(+), 3 deletions(-)

diff --git a/packages/Webkul/Admin/src/Resources/views/cms/create.blade.php b/packages/Webkul/Admin/src/Resources/views/cms/create.blade.php
index 768dd4a5e..52dede384 100644
--- a/packages/Webkul/Admin/src/Resources/views/cms/create.blade.php
+++ b/packages/Webkul/Admin/src/Resources/views/cms/create.blade.php
@@ -77,9 +77,9 @@
                         {!! __('admin::app.cms.pages.three-col') !!}
 
                         <div class="mt-10 mb-10" @submit.prevent="showModal('showHelpers')">
-                            <button class="btn btn-sm btn-primary">
+                            <a class="btn btn-sm btn-primary">
                                 {{ __('admin::app.cms.pages.helper-classes') }}
-                            </button>
+                            </a>
                         </div>
 
                         <span class="control-error" v-if="errors.has('html_content')">@{{ errors.first('html_content') }}</span>
diff --git a/packages/Webkul/CMS/src/Http/Controllers/Shop/PagePresenterController.php b/packages/Webkul/CMS/src/Http/Controllers/Shop/PagePresenterController.php
index b02dc360a..b7e54981a 100644
--- a/packages/Webkul/CMS/src/Http/Controllers/Shop/PagePresenterController.php
+++ b/packages/Webkul/CMS/src/Http/Controllers/Shop/PagePresenterController.php
@@ -75,6 +75,11 @@ use Webkul\Core\Repositories\LocaleRepository as Locale;
             'channel_id' => $currentChannel->id
         ]);
 
-        return view('shop::cms.page')->with('page', $page);
+        if ($page) {
+            return view('shop::cms.page')->with('page', $page);
+        } else {
+            abort(404);
+        }
+
     }
 }
\ No newline at end of file

From c56a7910694e5456ade2bf873ab305362a8337df Mon Sep 17 00:00:00 2001
From: Prashant Singh <prashant.singh852@webkul.com>
Date: Wed, 14 Aug 2019 13:49:04 +0530
Subject: [PATCH 09/11] Fixed issue #1295

---
 .../Webkul/Admin/src/Resources/views/cms/create.blade.php  | 4 ++--
 .../src/Http/Controllers/Shop/PagePresenterController.php  | 7 ++++++-
 2 files changed, 8 insertions(+), 3 deletions(-)

diff --git a/packages/Webkul/Admin/src/Resources/views/cms/create.blade.php b/packages/Webkul/Admin/src/Resources/views/cms/create.blade.php
index 768dd4a5e..52dede384 100644
--- a/packages/Webkul/Admin/src/Resources/views/cms/create.blade.php
+++ b/packages/Webkul/Admin/src/Resources/views/cms/create.blade.php
@@ -77,9 +77,9 @@
                         {!! __('admin::app.cms.pages.three-col') !!}
 
                         <div class="mt-10 mb-10" @submit.prevent="showModal('showHelpers')">
-                            <button class="btn btn-sm btn-primary">
+                            <a class="btn btn-sm btn-primary">
                                 {{ __('admin::app.cms.pages.helper-classes') }}
-                            </button>
+                            </a>
                         </div>
 
                         <span class="control-error" v-if="errors.has('html_content')">@{{ errors.first('html_content') }}</span>
diff --git a/packages/Webkul/CMS/src/Http/Controllers/Shop/PagePresenterController.php b/packages/Webkul/CMS/src/Http/Controllers/Shop/PagePresenterController.php
index b02dc360a..b7e54981a 100644
--- a/packages/Webkul/CMS/src/Http/Controllers/Shop/PagePresenterController.php
+++ b/packages/Webkul/CMS/src/Http/Controllers/Shop/PagePresenterController.php
@@ -75,6 +75,11 @@ use Webkul\Core\Repositories\LocaleRepository as Locale;
             'channel_id' => $currentChannel->id
         ]);
 
-        return view('shop::cms.page')->with('page', $page);
+        if ($page) {
+            return view('shop::cms.page')->with('page', $page);
+        } else {
+            abort(404);
+        }
+
     }
 }
\ No newline at end of file

From da5ad23c3be4e16ea3518ccb95734ff31cb8967a Mon Sep 17 00:00:00 2001
From: Prashant Singh <prashant.singh852@webkul.com>
Date: Wed, 14 Aug 2019 13:50:32 +0530
Subject: [PATCH 10/11] Fixed issue #1289

---
 packages/Webkul/Admin/src/Resources/views/cms/create.blade.php | 2 +-
 packages/Webkul/Admin/src/Resources/views/cms/edit.blade.php   | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/packages/Webkul/Admin/src/Resources/views/cms/create.blade.php b/packages/Webkul/Admin/src/Resources/views/cms/create.blade.php
index 52dede384..c6106638d 100644
--- a/packages/Webkul/Admin/src/Resources/views/cms/create.blade.php
+++ b/packages/Webkul/Admin/src/Resources/views/cms/create.blade.php
@@ -110,7 +110,7 @@
                     </div>
 
                     <div class="control-group" :class="[errors.has('meta_description') ? 'has-error' : '']">
-                        <label for="meta_description" class="required">{{ __('admin::app.cms.pages.meta_description') }}</label>
+                        <label for="meta_description">{{ __('admin::app.cms.pages.meta_description') }}</label>
 
                         <textarea type="text" class="control" name="meta_description" value="{{ old('meta_description') }}" data-vv-as="&quot;{{ __('admin::app.cms.pages.meta_description') }}&quot;"></textarea>
 
diff --git a/packages/Webkul/Admin/src/Resources/views/cms/edit.blade.php b/packages/Webkul/Admin/src/Resources/views/cms/edit.blade.php
index d1907e53e..4798350f6 100644
--- a/packages/Webkul/Admin/src/Resources/views/cms/edit.blade.php
+++ b/packages/Webkul/Admin/src/Resources/views/cms/edit.blade.php
@@ -114,7 +114,7 @@
                     </div>
 
                     <div class="control-group" :class="[errors.has('meta_description') ? 'has-error' : '']">
-                        <label for="meta_description" class="required">{{ __('admin::app.cms.pages.meta_description') }}</label>
+                        <label for="meta_description">{{ __('admin::app.cms.pages.meta_description') }}</label>
 
                         <textarea type="text" class="control" name="meta_description" data-vv-as="&quot;{{ __('admin::app.cms.pages.meta_description') }}&quot;">{{ $page->meta_description ?? old('meta_description') }}</textarea>
 

From de24f9a39c149228fd6610305ae2b962f8a3f7d7 Mon Sep 17 00:00:00 2001
From: Prashant Singh <prashant.singh852@webkul.com>
Date: Wed, 14 Aug 2019 14:01:49 +0530
Subject: [PATCH 11/11] Fixed issue #1292 for CMS pages by making a separate
 view for helper classes and leaving future scope for things as well

---
 .../Admin/src/Resources/views/cms/create.blade.php |  8 ++++----
 .../Admin/src/Resources/views/cms/edit.blade.php   | 14 +++-----------
 packages/Webkul/Ui/src/Http/routes.php             |  2 ++
 ...helper-classes.php => helper-classes.blade.php} |  0
 4 files changed, 9 insertions(+), 15 deletions(-)
 rename packages/Webkul/Ui/src/Resources/views/partials/{helper-classes.php => helper-classes.blade.php} (100%)

diff --git a/packages/Webkul/Admin/src/Resources/views/cms/create.blade.php b/packages/Webkul/Admin/src/Resources/views/cms/create.blade.php
index c6106638d..9542e2467 100644
--- a/packages/Webkul/Admin/src/Resources/views/cms/create.blade.php
+++ b/packages/Webkul/Admin/src/Resources/views/cms/create.blade.php
@@ -76,8 +76,8 @@
                         {!! __('admin::app.cms.pages.two-col') !!}
                         {!! __('admin::app.cms.pages.three-col') !!}
 
-                        <div class="mt-10 mb-10" @submit.prevent="showModal('showHelpers')">
-                            <a class="btn btn-sm btn-primary">
+                        <div class="mt-10 mb-10">
+                            <a target="_blank" href="{{ route('ui.helper.classes') }}" class="btn btn-sm btn-primary">
                                 {{ __('admin::app.cms.pages.helper-classes') }}
                             </a>
                         </div>
@@ -121,13 +121,13 @@
         </form>
     </div>
 
-    <modal id="showHelpers" :is-open="modalIds.showHelpers">
+    {{-- <modal id="showHelpers" :is-open="modalIds.showHelpers">
         <h3 slot="header">{{ __('admin::app.cms.pages.helper-classes') }}</h3>
 
         <div slot="body">
             @include('ui::partials.helper-classes')
         </div>
-    </modal>
+    </modal> --}}
 @stop
 
 @push('scripts')
diff --git a/packages/Webkul/Admin/src/Resources/views/cms/edit.blade.php b/packages/Webkul/Admin/src/Resources/views/cms/edit.blade.php
index 4798350f6..ac837c951 100644
--- a/packages/Webkul/Admin/src/Resources/views/cms/edit.blade.php
+++ b/packages/Webkul/Admin/src/Resources/views/cms/edit.blade.php
@@ -80,10 +80,10 @@
                         {!! __('admin::app.cms.pages.two-col') !!}
                         {!! __('admin::app.cms.pages.three-col') !!}
 
-                        <div class="mt-10 mb-10" @submit.prevent="showModal('showHelpers')">
-                            <button class="btn btn-sm btn-primary">
+                        <div class="mt-10 mb-10">
+                            <a target="_blank" href="{{ route('ui.helper.classes') }}" class="btn btn-sm btn-primary">
                                 {{ __('admin::app.cms.pages.helper-classes') }}
-                            </button>
+                            </a>
                         </div>
 
                         <span class="control-error" v-if="errors.has('html_content')">@{{ errors.first('html_content') }}</span>
@@ -124,14 +124,6 @@
             </div>
         </form>
     </div>
-
-    <modal id="showHelpers" :is-open="modalIds.showHelpers">
-        <h3 slot="header">{{ __('admin::app.cms.pages.helper-classes') }}</h3>
-
-        <div slot="body">
-            @include('ui::partials.helper-classes')
-        </div>
-    </modal>
 @stop
 
 @push('scripts')
diff --git a/packages/Webkul/Ui/src/Http/routes.php b/packages/Webkul/Ui/src/Http/routes.php
index ab6e594f6..55db43b66 100755
--- a/packages/Webkul/Ui/src/Http/routes.php
+++ b/packages/Webkul/Ui/src/Http/routes.php
@@ -2,6 +2,8 @@
 
 Route::view('/ui-kit', 'ui::partials.ui-kit');
 
+Route::view('/helper-classess', 'ui::partials.helper-classes')->name('ui.helper.classes');
+
 // Route::get('/users', function () {
 //     $users = \Webkul\User\Models\Admin::paginate(1);
 //     echo $users->links();
diff --git a/packages/Webkul/Ui/src/Resources/views/partials/helper-classes.php b/packages/Webkul/Ui/src/Resources/views/partials/helper-classes.blade.php
similarity index 100%
rename from packages/Webkul/Ui/src/Resources/views/partials/helper-classes.php
rename to packages/Webkul/Ui/src/Resources/views/partials/helper-classes.blade.php