Passed
Pull Request — master (#108)
by
unknown
04:04
created
src/mailEclipse.php 1 patch
Spacing   +34 added lines, -34 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
 
@@ -501,7 +501,7 @@  discard block
 block discarded – undo
501 501
 
502 502
                         [$name, $extension] = explode('.', $phpFile->getFilename());
503 503
 
504
-                        $mailableClass = $namespace . '\\' . $tokens[$index][1];
504
+                        $mailableClass = $namespace.'\\'.$tokens[$index][1];
505 505
 
506 506
                         if (!self::mailable_exists($mailableClass)) {
507 507
                             continue;
@@ -558,9 +558,9 @@  discard block
 block discarded – undo
558 558
                 }
559 559
             }
560 560
 
561
-            $collection = collect($fqcns)->map(function ($mailable) {
561
+            $collection = collect($fqcns)->map(function($mailable) {
562 562
                 return $mailable;
563
-            })->reject(function ($object) {
563
+            })->reject(function($object) {
564 564
                 return !method_exists($object['namespace'], 'build');
565 565
             });
566 566
 
@@ -584,7 +584,7 @@  discard block
 block discarded – undo
584 584
 
585 585
             $eloquentFactory = app(EloquentFactory::class);
586 586
 
587
-            $args = collect($params)->map(function ($param) {
587
+            $args = collect($params)->map(function($param) {
588 588
                 if ($param->getType() !== null) {
589 589
                     if (class_exists($param->getType()->getName())) {
590 590
                         $parameters = [
@@ -630,7 +630,7 @@  discard block
 block discarded – undo
630 630
 
631 631
             $reflector = new ReflectionClass($mailable);
632 632
 
633
-            if (! $args->isEmpty()) {
633
+            if (!$args->isEmpty()) {
634 634
                 $foo = $reflector->newInstanceArgs($filteredparams);
635 635
 
636 636
                 return $foo;
@@ -662,17 +662,17 @@  discard block
 block discarded – undo
662 662
         $reflection = collect($params)->where('name', $arg)->first()->getType();
663 663
 
664 664
         if (version_compare(phpversion(), '7.1', '>=')) {
665
-            $type = ! is_null($reflection)
665
+            $type = !is_null($reflection)
666 666
                 ? self::TYPES[$reflection->getName()]
667 667
                 : null;
668 668
         } else {
669
-            $type = ! is_null($reflection)
669
+            $type = !is_null($reflection)
670 670
                 ? self::TYPES[/** @scrutinizer ignore-deprecated */ $reflection->__toString()]
671 671
                 : null;
672 672
         }
673 673
 
674 674
         try {
675
-            return ! is_null($type)
675
+            return !is_null($type)
676 676
                     ? $type
677 677
                     : new Mocked($arg, \ReeceM\Mocker\Utils\VarStore::singleton());
678 678
         } catch (\Exception $e) {
@@ -697,7 +697,7 @@  discard block
 block discarded – undo
697 697
 
698 698
         foreach ($properties as $prop) {
699 699
             if ($prop->class == $data->getName() || $prop->class == get_parent_class($data->getName()) &&
700
-                    get_parent_class($data->getName()) != 'Illuminate\Mail\Mailable' && ! $prop->isStatic()) {
700
+                    get_parent_class($data->getName()) != 'Illuminate\Mail\Mailable' && !$prop->isStatic()) {
701 701
                 $allProps[] = $prop->name;
702 702
             }
703 703
         }
@@ -716,7 +716,7 @@  discard block
 block discarded – undo
716 716
 
717 717
         $mailableData = collect($classProps)->merge($withFuncData);
718 718
 
719
-        $data = $mailableData->map(function ($parameter) use ($mailable_data) {
719
+        $data = $mailableData->map(function($parameter) use ($mailable_data) {
720 720
             return [
721 721
                 'key' => $parameter,
722 722
                 'value' => property_exists($mailable_data, $parameter) ? $mailable_data->$parameter : null,
@@ -749,7 +749,7 @@  discard block
 block discarded – undo
749 749
 
750 750
     protected static function mailable_exists($mailable)
751 751
     {
752
-        if (! class_exists($mailable)) {
752
+        if (!class_exists($mailable)) {
753 753
             return false;
754 754
         }
755 755
 
@@ -774,7 +774,7 @@  discard block
 block discarded – undo
774 774
     public static function buildMailable($instance, $type = 'call')
775 775
     {
776 776
         if ($type == 'call') {
777
-            if (! is_null(self::handleMailableViewDataArgs($instance))) {
777
+            if (!is_null(self::handleMailableViewDataArgs($instance))) {
778 778
                 return Container::getInstance()->call([self::handleMailableViewDataArgs($instance), 'build']);
779 779
             }
780 780
 
@@ -786,17 +786,17 @@  discard block
 block discarded – undo
786 786
 
787 787
     public static function renderPreview($simpleview, $view, $template = false, $instance = null)
788 788
     {
789
-        if (! View::exists($view)) {
789
+        if (!View::exists($view)) {
790 790
             return;
791 791
         }
792 792
 
793
-        if (! $template) {
793
+        if (!$template) {
794 794
             $obj = self::buildMailable($instance);
795 795
             $viewData = $obj->viewData;
796 796
             $_data = array_merge($instance->buildViewData(), $viewData);
797 797
 
798 798
             foreach ($_data as $key => $value) {
799
-                if (! is_object($value)) {
799
+                if (!is_object($value)) {
800 800
                     $_data[$key] = '<span class="maileclipse-key" title="Variable">'.$key.'</span>';
801 801
                 }
802 802
             }
Please login to merge, or discard this patch.