Merge pull request #1 from ManojKiranA/analysis-L39xMm

Apply fixes from StyleCI

app/Campaign.php

@@ -14,21 +14,22 @@ class Campaign extends Model
     const FINISHED = 'finished';
 
     /**
-     * Checks if Current Campaign status is as Specified
+     * Checks if Current Campaign status is as Specified.
      *
      * @param string $status
      * @return bool
      **/
-    public function isInStatus(string $status):bool
+    public function isInStatus(string $status): bool
     {
-        if($this->status === $status){
+        if ($this->status === $status) {
             return true;
         }
+
         return false;
     }
 
     /**
-     * Checks if Current Campaign status is Draft
+     * Checks if Current Campaign status is Draft.
      *
      * @return bool
      **/
@@ -38,7 +39,7 @@ public function isDraft()
     }
 
     /**
-     * Checks if Current Campaign status is Not Draft
+     * Checks if Current Campaign status is Not Draft.
      *
      * @return bool
      **/
@@ -48,7 +49,7 @@ public function isNotDraft()
     }
 
     /**
-     * Checks if Current Campaign status is Finished
+     * Checks if Current Campaign status is Finished.
      *
      * @return bool
      **/
@@ -58,7 +59,7 @@ public function isFinished()
     }
 
     /**
-     * Checks if Current Campaign status is Not FINISHED
+     * Checks if Current Campaign status is Not FINISHED.
      *
      * @return bool
      **/

app/Http/Controllers/CampaignController.php

@@ -128,12 +128,12 @@ public function update(CampaignStoreRequest $request, Campaign $campaign)
             return back();
         }
 
-        $updationArray = array_merge($request->only(['subject', 'sending_name', 'sending_email','content']),
+        $updationArray = array_merge($request->only(['subject', 'sending_name', 'sending_email', 'content']),
                     [
-                    'track_clicks' => $request->track_clicks ? 1 : 0,
-                    'track_opens' => $request->track_opens ? 1 : 0,
-                ]);
-                
+                        'track_clicks' => $request->track_clicks ? 1 : 0,
+                        'track_opens' => $request->track_opens ? 1 : 0,
+                    ]);
+
         $campaign->update($updationArray);
 
         return redirect()->route('campaigns.show', $campaign->id);

app/Http/Controllers/ListsController.php

@@ -49,7 +49,7 @@ public function create()
      */
     public function store(ListStoreRequest $request)
     {
-        if(! $request->exists('double_opt_in')){
+        if (! $request->exists('double_opt_in')) {
             $request->request->add(['double_opt_in' => 0]);
         }
 
@@ -66,7 +66,7 @@ public function store(ListStoreRequest $request)
      */
     public function show(Request $request, Lists $lists)
     {
-        $contacts = Contact::where('list_id','=',$lists->id)
+        $contacts = Contact::where('list_id', '=', $lists->id)
                             ->where('subscribed', ! $request->subscribed)
                             ->orderBy('id', 'desc')
                             ->paginate(10);
@@ -94,7 +94,7 @@ public function edit(Lists $lists)
      */
     public function update(ListUpdateRequest $request, Lists $lists)
     {
-        if(! $request->exists('double_opt_in')){
+        if (! $request->exists('double_opt_in')) {
             $request->request->add(['double_opt_in' => 0]);
         }
 

app/Http/Requests/CampaignStoreRequest.php

@@ -26,7 +26,7 @@ public function rules()
         return [
             'subject'   => ['required'],
             'sending_name'  => ['required'],
-            'sending_email' => ['required','email'],
+            'sending_email' => ['required', 'email'],
             'content' => ['required'],
         ];
     }

app/Http/Requests/ImportSaveRequest.php

@@ -24,7 +24,7 @@ public function authorize()
     public function rules()
     {
         return [
-            'file'  => ['required','mimes:csv,txt'],
+            'file'  => ['required', 'mimes:csv,txt'],
         ];
     }
 }

app/Http/Requests/SendTestMailRequest.php

@@ -24,7 +24,7 @@ public function authorize()
     public function rules()
     {
         return [
-            'email' => ['email','required'],
+            'email' => ['email', 'required'],
         ];
     }
 }

app/Http/Requests/SettingsUpdateRequest.php

@@ -25,7 +25,7 @@ public function rules()
     {
         return [
             'name' => ['required'],
-            'email' => ['required','email'],
+            'email' => ['required', 'email'],
         ];
     }
 }

app/Http/Requests/SmtpSettings.php

@@ -25,11 +25,11 @@ public function rules()
     {
         return [
             'host' => ['required'],
-            'port' => ['required','integer'],
+            'port' => ['required', 'integer'],
             'username' => ['required'],
             'password' => ['required'],
             'name' => ['required'],
-            'address' => ['required','email'],
+            'address' => ['required', 'email'],
         ];
     }
 }

app/Providers/AppServiceProvider.php

@@ -3,10 +3,10 @@
 namespace App\Providers;
 
 use App\View\Components\CheckErrorExistsComponent;
+use Illuminate\Support\Facades\Blade;
 use Illuminate\Support\Facades\Schema;
 use Illuminate\Support\ServiceProvider;
 use Laravel\Dusk\DuskServiceProvider;
-use Illuminate\Support\Facades\Blade;
 
 class AppServiceProvider extends ServiceProvider
 {
@@ -19,7 +19,7 @@ public function register()
     {
         if ($this->app->environment('local', 'testing')) {
             $this->app->register(DuskServiceProvider::class);
-        }   
+        }
     }
 
     /**

app/View/Components/CheckErrorExistsComponent.php

@@ -6,8 +6,7 @@
 
 class CheckErrorExistsComponent extends Component
 {
-    /** @var string $controlName */
-
+    /** @var string */
     public $controlName;
 
     /**

app/View/Components/ErrorAlertComponent.php

@@ -6,7 +6,7 @@
 
 class ErrorAlertComponent extends Component
 {
-    /** @var string $errorMessage */
+    /** @var string */
     public $errorMessage;
 
     /**