From 687df3dbddc123c61a4b4b32bfed0adc16438999 Mon Sep 17 00:00:00 2001
From: Jitendra Singh <jitendra@webkul.com>
Date: Wed, 19 Feb 2020 17:45:43 +0530
Subject: [PATCH] Improved cms package code style

---
 .../src/Http/Controllers/Admin/PageController.php | 15 ++++++++-------
 .../Webkul/CMS/src/Repositories/CmsRepository.php |  6 ++++--
 2 files changed, 12 insertions(+), 9 deletions(-)

diff --git a/packages/Webkul/CMS/src/Http/Controllers/Admin/PageController.php b/packages/Webkul/CMS/src/Http/Controllers/Admin/PageController.php
index 4f1189a49..32617b285 100644
--- a/packages/Webkul/CMS/src/Http/Controllers/Admin/PageController.php
+++ b/packages/Webkul/CMS/src/Http/Controllers/Admin/PageController.php
@@ -72,9 +72,9 @@ use Webkul\CMS\Repositories\CmsRepository;
         $data = request()->all();
 
         $this->validate(request(), [
-            'url_key' => ['required', 'unique:cms_page_translations,url_key', new \Webkul\Core\Contracts\Validations\Slug],
-            'page_title' => 'required',
-            'channels' => 'required',
+            'url_key'      => ['required', 'unique:cms_page_translations,url_key', new \Webkul\Core\Contracts\Validations\Slug],
+            'page_title'   => 'required',
+            'channels'     => 'required',
             'html_content' => 'required'
         ]);
         
@@ -109,13 +109,14 @@ use Webkul\CMS\Repositories\CmsRepository;
         $locale = request()->get('locale') ?: app()->getLocale();
 
         $this->validate(request(), [
-            $locale . '.url_key' => ['required', new \Webkul\Core\Contracts\Validations\Slug, function ($attribute, $value, $fail) use ($id) {
-                if (! $this->cmsRepository->isUrlKeyUnique($id, $value))
+            $locale . '.url_key'      => ['required', new \Webkul\Core\Contracts\Validations\Slug, function ($attribute, $value, $fail) use ($id) {
+                if (! $this->cmsRepository->isUrlKeyUnique($id, $value)) {
                     $fail(trans('admin::app.response.already-taken', ['name' => 'Page']));
+                }
             }],
-            $locale . '.page_title' => 'required',
+            $locale . '.page_title'   => 'required',
             $locale . '.html_content' => 'required',
-            'channels' => 'required'
+            'channels'                => 'required'
         ]);
 
         $this->cmsRepository->update(request()->all(), $id);
diff --git a/packages/Webkul/CMS/src/Repositories/CmsRepository.php b/packages/Webkul/CMS/src/Repositories/CmsRepository.php
index 0c5e0812f..898bd48bb 100644
--- a/packages/Webkul/CMS/src/Repositories/CmsRepository.php
+++ b/packages/Webkul/CMS/src/Repositories/CmsRepository.php
@@ -37,8 +37,9 @@ class CmsRepository extends Repository
 
         foreach (core()->getAllLocales() as $locale) {
             foreach ($model->translatedAttributes as $attribute) {
-                if (isset($data[$attribute]))
+                if (isset($data[$attribute])) {
                     $data[$locale->code][$attribute] = $data[$attribute];
+                }
             }
         }
 
@@ -100,8 +101,9 @@ class CmsRepository extends Repository
     {
         $page = $this->model->whereTranslation('url_key', $urlKey)->first();
 
-        if ($page)
+        if ($page) {
             return $page;
+        }
 
         throw (new ModelNotFoundException)->setModel(
             get_class($this->model), $urlKey