refactor(mail): Improve method call in MailChannel

- Refactored method call in MailChannel to improve readability and maintainability

Author: guanguans

config/exception-notify.php

@@ -73,21 +73,23 @@
      */
     'collectors' => [
         Guanguans\LaravelExceptionNotify\Collectors\ApplicationCollector::class,
-        Guanguans\LaravelExceptionNotify\Collectors\PhpInfoCollector::class,
         Guanguans\LaravelExceptionNotify\Collectors\ChoreCollector::class,
+        Guanguans\LaravelExceptionNotify\Collectors\PhpInfoCollector::class,
+        Guanguans\LaravelExceptionNotify\Collectors\RequestBasicCollector::class,
         Guanguans\LaravelExceptionNotify\Collectors\ExceptionBasicCollector::class,
         Guanguans\LaravelExceptionNotify\Collectors\ExceptionContextCollector::class,
         Guanguans\LaravelExceptionNotify\Collectors\ExceptionTraceCollector::class,
-        Guanguans\LaravelExceptionNotify\Collectors\RequestBasicCollector::class,
+
+        // Guanguans\LaravelExceptionNotify\Collectors\RequestCookieCollector::class,
+        // Guanguans\LaravelExceptionNotify\Collectors\RequestSessionCollector::class,
+        // Guanguans\LaravelExceptionNotify\Collectors\RequestMiddlewareCollector::class,
+        // Guanguans\LaravelExceptionNotify\Collectors\RequestServerCollector::class,
+
         // Guanguans\LaravelExceptionNotify\Collectors\RequestHeaderCollector::class,
         // Guanguans\LaravelExceptionNotify\Collectors\RequestQueryCollector::class,
         // Guanguans\LaravelExceptionNotify\Collectors\RequestPostCollector::class,
-        // Guanguans\LaravelExceptionNotify\Collectors\RequestRawFileCollector::class,
         // Guanguans\LaravelExceptionNotify\Collectors\RequestFileCollector::class,
-        // \Guanguans\LaravelExceptionNotify\Collectors\RequestMiddlewareCollector::class,
-        // \Guanguans\LaravelExceptionNotify\Collectors\RequestServerCollector::class,
-        // \Guanguans\LaravelExceptionNotify\Collectors\RequestCookieCollector::class,
-        // \Guanguans\LaravelExceptionNotify\Collectors\RequestSessionCollector::class,
+        // Guanguans\LaravelExceptionNotify\Collectors\RequestRawFileCollector::class,
     ],
 
     /**
@@ -117,7 +119,7 @@
         'mail' => [
             'driver' => 'mail',
             'mailer' => null,
-            'to' => env('EXCEPTION_NOTIFY_MAIL_TO', 'your@example.mail'),
+            'to' => ['users' => env('EXCEPTION_NOTIFY_MAIL_TO', 'your@example.mail')],
             'pipes' => [
                 SprintfHtmlPipe::class,
             ],

src/Channels/MailChannel.php

@@ -13,31 +13,45 @@
 
 namespace Guanguans\LaravelExceptionNotify\Channels;
 
+use Guanguans\LaravelExceptionNotify\Exceptions\InvalidArgumentException;
 use Guanguans\LaravelExceptionNotify\Mail\ReportExceptionMail;
+use Illuminate\Config\Repository;
 use Illuminate\Mail\Mailer;
 use Illuminate\Mail\PendingMail;
 use Illuminate\Support\Facades\Mail;
 use Illuminate\Support\Str;
 
 class MailChannel extends Channel
 {
+    public function __construct(Repository $config)
+    {
+        $validator = validator($config->all(), [
+            'mailer' => 'nullable|string',
+            'to' => 'required|array',
+            'extender' => static function (string $attribute, $value, \Closure $fail): void {
+                if (\is_string($value) || \is_callable($value)) {
+                    return;
+                }
+
+                $fail("The $attribute must be a callable or string.");
+            },
+        ]);
+
+        if ($validator->fails()) {
+            throw new InvalidArgumentException($validator->errors()->first());
+        }
+
+        parent::__construct($config);
+    }
+
     public function report(string $report): void
     {
         /** @var Mailer|PendingMail $mailerOrPendingMail */
         $mailerOrPendingMail = collect($this->config->all())
             ->except(['driver', 'mailer', 'extender', 'pipes'])
             ->reduce(
-                /**
-                 * @param array|mixed $parameters
-                 *
-                 * @throws \ReflectionException
-                 */
-                static function (object $mailerOrPendingMail, $parameters, string $method): object {
-                    $method = Str::camel($method);
-
-                    $object = 1 < (new \ReflectionObject($mailerOrPendingMail))->getMethod($method)->getNumberOfParameters()
-                        ? $mailerOrPendingMail->{$method}(...$parameters)
-                        : $mailerOrPendingMail->{$method}($parameters);
+                static function (object $mailerOrPendingMail, array $parameters, string $method): object {
+                    $object = app()->call([$mailerOrPendingMail, Str::camel($method)], $parameters);
 
                     return \is_object($object) ? $object : $mailerOrPendingMail;
                 },