Passed
Pull Request — master (#108)
by
unknown
07:14 queued 02:37
created
src/mailEclipse.php 1 patch
Spacing   +41 added lines, -41 removed lines patch added patch discarded remove patch
@@ -51,8 +51,8 @@  discard block
 block discarded – undo
51 51
         $template = self::getTemplates()
52 52
             ->where('template_slug', $templateSlug)->first();
53 53
 
54
-        if (! is_null($template)) {
55
-            self::saveTemplates(self::getTemplates()->reject(function ($value, $key) use ($template) {
54
+        if (!is_null($template)) {
55
+            self::saveTemplates(self::getTemplates()->reject(function($value, $key) use ($template) {
56 56
                 return $value->template_slug == $template->template_slug;
57 57
             }));
58 58
 
@@ -76,8 +76,8 @@  discard block
 block discarded – undo
76 76
     public static function getTemplatesFile()
77 77
     {
78 78
         $file = config('maileclipse.mailables_dir').'templates.json';
79
-        if (! file_exists($file)) {
80
-            if (! file_exists(config('maileclipse.mailables_dir'))) {
79
+        if (!file_exists($file)) {
80
+            if (!file_exists(config('maileclipse.mailables_dir'))) {
81 81
                 mkdir(config('maileclipse.mailables_dir'));
82 82
             }
83 83
             file_put_contents($file, '[]');
@@ -96,8 +96,8 @@  discard block
 block discarded – undo
96 96
         $template = self::getTemplates()
97 97
             ->where('template_slug', $request->templateslug)->first();
98 98
 
99
-        if (! is_null($template)) {
100
-            if (! preg_match("/^[a-zA-Z0-9-_\s]+$/", $request->title)) {
99
+        if (!is_null($template)) {
100
+            if (!preg_match("/^[a-zA-Z0-9-_\s]+$/", $request->title)) {
101 101
                 return response()->json([
102 102
                     'status' => 'failed',
103 103
                     'message' => 'Template name not valid',
@@ -121,7 +121,7 @@  discard block
 block discarded – undo
121 121
 
122 122
             // Update
123 123
             //
124
-            $oldForm = self::getTemplates()->reject(function ($value, $key) use ($template) {
124
+            $oldForm = self::getTemplates()->reject(function($value, $key) use ($template) {
125 125
                 return $value->template_slug == $template->template_slug;
126 126
             });
127 127
             $newForm = array_merge($oldForm->toArray(), [array_merge((array) $template, [
@@ -162,7 +162,7 @@  discard block
 block discarded – undo
162 162
         $template = self::getTemplates()
163 163
             ->where('template_slug', $templateSlug)->first();
164 164
 
165
-        if (! is_null($template)) {
165
+        if (!is_null($template)) {
166 166
             $template_view = self::$view_namespace.'::templates.'.$template->template_slug;
167 167
             $template_plaintext_view = $template_view.'_plain_text';
168 168
 
@@ -199,7 +199,7 @@  discard block
 block discarded – undo
199 199
 
200 200
     public static function createTemplate($request)
201 201
     {
202
-        if (! preg_match("/^[a-zA-Z0-9-_\s]+$/", $request->template_name)) {
202
+        if (!preg_match("/^[a-zA-Z0-9-_\s]+$/", $request->template_name)) {
203 203
             return response()->json([
204 204
 
205 205
                 'status' => 'error',
@@ -212,7 +212,7 @@  discard block
 block discarded – undo
212 212
 
213 213
         $templatename = Str::camel(preg_replace('/\s+/', '_', $request->template_name));
214 214
 
215
-        if (! view()->exists($view) && ! self::getTemplates()->contains('template_slug', '=', $templatename)) {
215
+        if (!view()->exists($view) && !self::getTemplates()->contains('template_slug', '=', $templatename)) {
216 216
             self::saveTemplates(self::getTemplates()
217 217
                 ->push([
218 218
                     'template_name' => $request->template_name,
@@ -225,7 +225,7 @@  discard block
 block discarded – undo
225 225
 
226 226
             $dir = resource_path('views/vendor/'.self::$view_namespace.'/templates');
227 227
 
228
-            if (! \File::isDirectory($dir)) {
228
+            if (!\File::isDirectory($dir)) {
229 229
                 \File::makeDirectory($dir, 0755, true);
230 230
             }
231 231
 
@@ -349,7 +349,7 @@  discard block
 block discarded – undo
349 349
 
350 350
         $replaced = self::templateComponentReplace($content);
351 351
 
352
-        if (! $save) {
352
+        if (!$save) {
353 353
             return $replaced;
354 354
         }
355 355
 
@@ -368,7 +368,7 @@  discard block
 block discarded – undo
368 368
             if ($template) {
369 369
                 $instance = null;
370 370
             } else {
371
-                if (! is_null(self::handleMailableViewDataArgs($namespace))) {
371
+                if (!is_null(self::handleMailableViewDataArgs($namespace))) {
372 372
                     $instance = self::handleMailableViewDataArgs($namespace);
373 373
                 } else {
374 374
                     $instance = new $namespace;
@@ -396,8 +396,8 @@  discard block
 block discarded – undo
396 396
 
397 397
         $templateData = collect($mailable->first())->only(['markdown', 'view_path', 'text_view_path', 'text_view', 'view_data', 'data', 'namespace'])->all();
398 398
 
399
-        $templateExists = ! is_null($templateData['view_path']);
400
-        $textTemplateExists = ! is_null($templateData['text_view_path']);
399
+        $templateExists = !is_null($templateData['view_path']);
400
+        $textTemplateExists = !is_null($templateData['text_view_path']);
401 401
 
402 402
         if ($templateExists) {
403 403
             $viewPathParams = collect($templateData)->union([
@@ -405,8 +405,8 @@  discard block
 block discarded – undo
405 405
                 'text_template' => $textTemplateExists ? file_get_contents($templateData['text_view_path']) : null,
406 406
                 'template' => file_get_contents($templateData['view_path']),
407 407
                 'markdowned_template' => self::markdownedTemplate($templateData['view_path']),
408
-                'template_name' => ! is_null($templateData['markdown']) ? $templateData['markdown'] : $templateData['data']->view,
409
-                'is_markdown' => ! is_null($templateData['markdown']) ? true : false,
408
+                'template_name' => !is_null($templateData['markdown']) ? $templateData['markdown'] : $templateData['data']->view,
409
+                'is_markdown' => !is_null($templateData['markdown']) ? true : false,
410 410
                 // 'text_template' => file_get_contents($templateData['text_view_path']),
411 411
 
412 412
             ])->all();
@@ -421,7 +421,7 @@  discard block
 block discarded – undo
421 421
     {
422 422
         $name = ucwords(Str::camel(preg_replace('/\s+/', '_', $request->input('name'))));
423 423
 
424
-        if (! self::getMailable('name', $name)->isEmpty() && ! $request->has('force')) {
424
+        if (!self::getMailable('name', $name)->isEmpty() && !$request->has('force')) {
425 425
             // return redirect()->route('createMailable')->with('error', 'mailable already exists! to overide it enable force option.');
426 426
             //
427 427
             return response()->json([
@@ -473,7 +473,7 @@  discard block
 block discarded – undo
473 473
     {
474 474
         $fqcns = [];
475 475
 
476
-        if (! file_exists(config('maileclipse.mailables_dir'))) {
476
+        if (!file_exists(config('maileclipse.mailables_dir'))) {
477 477
             return;
478 478
         } else {
479 479
             $allFiles = new RecursiveIteratorIterator(new RecursiveDirectoryIterator(config('maileclipse.mailables_dir')));
@@ -486,7 +486,7 @@  discard block
 block discarded – undo
486 486
                 $tokens = token_get_all($content);
487 487
                 $namespace = '';
488 488
                 for ($index = 0; isset($tokens[$index]); $index++) {
489
-                    if (! isset($tokens[$index][0])) {
489
+                    if (!isset($tokens[$index][0])) {
490 490
                         continue;
491 491
                     }
492 492
                     if (T_NAMESPACE === $tokens[$index][0]) {
@@ -502,7 +502,7 @@  discard block
 block discarded – undo
502 502
 
503 503
                         $mailableClass = $namespace.'\\'.$tokens[$index][1];
504 504
 
505
-                        if (! self::mailable_exists($mailableClass)) {
505
+                        if (!self::mailable_exists($mailableClass)) {
506 506
                             continue;
507 507
                         }
508 508
 
@@ -514,7 +514,7 @@  discard block
 block discarded – undo
514 514
 
515 515
                         $mailable_data = self::buildMailable($mailableClass);
516 516
 
517
-                        if (! is_null(self::handleMailableViewDataArgs($mailableClass))) {
517
+                        if (!is_null(self::handleMailableViewDataArgs($mailableClass))) {
518 518
                             $mailable_view_data = self::getMailableViewData(self::handleMailableViewDataArgs($mailableClass), $mailable_data);
519 519
                         } else {
520 520
                             $mailable_view_data = self::getMailableViewData(new $mailableClass, $mailable_data);
@@ -535,16 +535,16 @@  discard block
 block discarded – undo
535 535
                         $fqcns[$i]['view_path'] = null;
536 536
                         $fqcns[$i]['text_view_path'] = null;
537 537
 
538
-                        if (! is_null($fqcns[$i]['markdown']) && View::exists($fqcns[$i]['markdown'])) {
538
+                        if (!is_null($fqcns[$i]['markdown']) && View::exists($fqcns[$i]['markdown'])) {
539 539
                             $fqcns[$i]['view_path'] = View($fqcns[$i]['markdown'])->getPath();
540 540
                         }
541 541
 
542
-                        if (! is_null($fqcns[$i]['data'])) {
543
-                            if (! is_null($fqcns[$i]['data']->view) && View::exists($fqcns[$i]['data']->view)) {
542
+                        if (!is_null($fqcns[$i]['data'])) {
543
+                            if (!is_null($fqcns[$i]['data']->view) && View::exists($fqcns[$i]['data']->view)) {
544 544
                                 $fqcns[$i]['view_path'] = View($fqcns[$i]['data']->view)->getPath();
545 545
                             }
546 546
 
547
-                            if (! is_null($fqcns[$i]['data']->textView) && View::exists($fqcns[$i]['data']->textView)) {
547
+                            if (!is_null($fqcns[$i]['data']->textView) && View::exists($fqcns[$i]['data']->textView)) {
548 548
                                 $fqcns[$i]['text_view_path'] = View($fqcns[$i]['data']->textView)->getPath();
549 549
                                 $fqcns[$i]['text_view'] = $fqcns[$i]['data']->textView;
550 550
                             }
@@ -557,10 +557,10 @@  discard block
 block discarded – undo
557 557
                 }
558 558
             }
559 559
 
560
-            $collection = collect($fqcns)->map(function ($mailable) {
560
+            $collection = collect($fqcns)->map(function($mailable) {
561 561
                 return $mailable;
562
-            })->reject(function ($object) {
563
-                return ! method_exists($object['namespace'], 'build');
562
+            })->reject(function($object) {
563
+                return !method_exists($object['namespace'], 'build');
564 564
             });
565 565
 
566 566
             return $collection;
@@ -581,7 +581,7 @@  discard block
 block discarded – undo
581 581
 
582 582
             $eloquentFactory = app(EloquentFactory::class);
583 583
 
584
-            $args = collect($params)->map(function ($param) {
584
+            $args = collect($params)->map(function($param) {
585 585
                 if ($param->getType() !== null) {
586 586
                     if (class_exists($param->getType()->getName())) {
587 587
                         $parameters = [
@@ -627,7 +627,7 @@  discard block
 block discarded – undo
627 627
 
628 628
             $reflector = new ReflectionClass($mailable);
629 629
 
630
-            if (! $args->isEmpty()) {
630
+            if (!$args->isEmpty()) {
631 631
                 $foo = $reflector->newInstanceArgs($filteredparams);
632 632
 
633 633
                 return $foo;
@@ -659,17 +659,17 @@  discard block
 block discarded – undo
659 659
         $reflection = collect($params)->where('name', $arg)->first()->getType();
660 660
 
661 661
         if (version_compare(phpversion(), '7.1', '>=')) {
662
-            $type = ! is_null($reflection)
662
+            $type = !is_null($reflection)
663 663
                 ? self::TYPES[$reflection->getName()]
664 664
                 : null;
665 665
         } else {
666
-            $type = ! is_null($reflection)
666
+            $type = !is_null($reflection)
667 667
                 ? self::TYPES[/** @scrutinizer ignore-deprecated */ $reflection->__toString()]
668 668
                 : null;
669 669
         }
670 670
 
671 671
         try {
672
-            return ! is_null($type)
672
+            return !is_null($type)
673 673
                     ? $type
674 674
                     : new Mocked($arg, \ReeceM\Mocker\Utils\VarStore::singleton());
675 675
         } catch (\Exception $e) {
@@ -694,7 +694,7 @@  discard block
 block discarded – undo
694 694
 
695 695
         foreach ($properties as $prop) {
696 696
             if ($prop->class == $data->getName() || $prop->class == get_parent_class($data->getName()) &&
697
-                    get_parent_class($data->getName()) != 'Illuminate\Mail\Mailable' && ! $prop->isStatic()) {
697
+                    get_parent_class($data->getName()) != 'Illuminate\Mail\Mailable' && !$prop->isStatic()) {
698 698
                 $allProps[] = $prop->name;
699 699
             }
700 700
         }
@@ -713,7 +713,7 @@  discard block
 block discarded – undo
713 713
 
714 714
         $mailableData = collect($classProps)->merge($withFuncData);
715 715
 
716
-        $data = $mailableData->map(function ($parameter) use ($mailable_data) {
716
+        $data = $mailableData->map(function($parameter) use ($mailable_data) {
717 717
             return [
718 718
                 'key' => $parameter,
719 719
                 'value' => property_exists($mailable_data, $parameter) ? $mailable_data->$parameter : null,
@@ -746,7 +746,7 @@  discard block
 block discarded – undo
746 746
 
747 747
     protected static function mailable_exists($mailable)
748 748
     {
749
-        if (! class_exists($mailable)) {
749
+        if (!class_exists($mailable)) {
750 750
             return false;
751 751
         }
752 752
 
@@ -771,7 +771,7 @@  discard block
 block discarded – undo
771 771
     public static function buildMailable($instance, $type = 'call')
772 772
     {
773 773
         if ($type == 'call') {
774
-            if (! is_null(self::handleMailableViewDataArgs($instance))) {
774
+            if (!is_null(self::handleMailableViewDataArgs($instance))) {
775 775
                 return Container::getInstance()->call([self::handleMailableViewDataArgs($instance), 'build']);
776 776
             }
777 777
 
@@ -783,17 +783,17 @@  discard block
 block discarded – undo
783 783
 
784 784
     public static function renderPreview($simpleview, $view, $template = false, $instance = null)
785 785
     {
786
-        if (! View::exists($view)) {
786
+        if (!View::exists($view)) {
787 787
             return;
788 788
         }
789 789
 
790
-        if (! $template) {
790
+        if (!$template) {
791 791
             $obj = self::buildMailable($instance);
792 792
             $viewData = $obj->viewData;
793 793
             $_data = array_merge($instance->buildViewData(), $viewData);
794 794
 
795 795
             foreach ($_data as $key => $value) {
796
-                if (! is_object($value)) {
796
+                if (!is_object($value)) {
797 797
                     $_data[$key] = '<span class="maileclipse-key" title="Variable">'.$key.'</span>';
798 798
                 }
799 799
             }
Please login to merge, or discard this patch.