[9.x] Show Closure Location on the schedule:list (#41494)

* Show Closure Location on the `schedule:list`.

* style: fixes

* formatting

* formatting

* use laravel instance

Co-authored-by: Taylor Otwell <[email protected]>

Author: xiCO2k

src/Illuminate/Console/Scheduling/ScheduleListCommand.php

@@ -7,6 +7,8 @@
 use Illuminate\Console\Application;
 use Illuminate\Console\Command;
 use Illuminate\Support\Carbon;
+use ReflectionClass;
+use ReflectionFunction;
 use Symfony\Component\Console\Terminal;
 
 class ScheduleListCommand extends Command
@@ -59,6 +61,10 @@ public function handle(Schedule $schedule)
                 );
             }
 
+            if ($event instanceof CallbackEvent) {
+                $command = 'Closure at: '.$this->getClosureLocation($event);
+            }
+
             $command = mb_strlen($command) > 1 ? "{$command} " : '';
 
             $nextDueDateLabel = 'Next Due:';
@@ -135,6 +141,25 @@ private function formatCronExpression($expression, $spacing)
             ->implode(' ');
     }
 
+    /**
+     * Get the file and line number for the event closure.
+     *
+     * @param  \Illuminate\Console\Scheduling\CallbackEvent  $event
+     * @return string
+     */
+    private function getClosureLocation(CallbackEvent $event)
+    {
+        $function = new ReflectionFunction(tap((new ReflectionClass($event))->getProperty('callback'))
+                        ->setAccessible(true)
+                        ->getValue($event));
+
+        return sprintf(
+            '%s:%s',
+            str_replace($this->laravel->basePath().DIRECTORY_SEPARATOR, '', $function->getFileName() ?: ''),
+            $function->getStartLine()
+        );
+    }
+
     /**
      * Get the terminal width.
      *

tests/Integration/Console/Scheduling/ScheduleListCommandTest.php

@@ -23,17 +23,20 @@ protected function setUp(): void
 
     public function testDisplaySchedule()
     {
-        $this->schedule->call(fn () => '')->everyMinute();
         $this->schedule->command(FooCommand::class)->quarterly();
         $this->schedule->command('inspire')->twiceDaily(14, 18);
         $this->schedule->command('foobar', ['a' => 'b'])->everyMinute();
 
+        $this->schedule->call(fn () => '')->everyMinute();
+        $closureLineNumber = __LINE__ - 1;
+        $closureFilePath = __FILE__;
+
         $this->artisan(ScheduleListCommand::class)
             ->assertSuccessful()
-            ->expectsOutput('  * *     * *      *  ............................ Next Due: 1 minute from now')
             ->expectsOutput('  0 0     1 1-12/3 *  php artisan foo:command .... Next Due: 3 months from now')
             ->expectsOutput('  0 14,18 * *      *  php artisan inspire ........ Next Due: 14 hours from now')
-            ->expectsOutput('  * *     * *      *  php artisan foobar a='.ProcessUtils::escapeArgument('b').' ... Next Due: 1 minute from now');
+            ->expectsOutput('  * *     * *      *  php artisan foobar a='.ProcessUtils::escapeArgument('b').' ... Next Due: 1 minute from now')
+            ->expectsOutput('  * *     * *      *  Closure at: '.$closureFilePath.':'.$closureLineNumber.'  Next Due: 1 minute from now');
     }
 
     public function testDisplayScheduleInVerboseMode()