remove support for the -h option in routes command

Author: paulbalandan

system/Commands/Utilities/Routes.php

@@ -84,15 +84,6 @@ public function run(array $params)
     {
         $sortByHandler = array_key_exists('sort-by-handler', $params);
 
-        if (! $sortByHandler && array_key_exists('h', $params)) {
-            // @todo to remove support in v4.8.0
-            // Support -h as a shortcut but print a warning that it is not the intended use of -h.
-            CLI::write('Warning: -h will be used as shortcut for --help in v4.8.0. Please use --sort-by-handler to sort by handler.', 'yellow');
-            CLI::newLine();
-
-            $sortByHandler = true;
-        }
-
         $host = $params['host'] ?? null;
 
         // Set HTTP_HOST

tests/system/Commands/Utilities/RoutesTest.php

@@ -117,41 +117,6 @@ public function testRoutesCommandSortByHandler(): void
         $this->assertStringContainsString($expected, $this->getBuffer());
     }
 
-    /**
-     * @todo To remove this test and the backward compatibility for -h in v4.8.0.
-     */
-    public function testRoutesCommandSortByHandlerUsingShortcutForBc(): void
-    {
-        Services::resetSingle('routes');
-
-        command('routes -h');
-
-        $expected = <<<'EOL'
-            Warning: -h will be used as shortcut for --help in v4.8.0. Please use --sort-by-handler to sort by handler.
-
-            +---------+---------+---------------+----------------------------------------+----------------+---------------+
-            | Method  | Route   | Name          | Handler ↓                              | Before Filters | After Filters |
-            +---------+---------+---------------+----------------------------------------+----------------+---------------+
-            | GET     | closure | »             | (Closure)                              |                |               |
-            | GET     | /       | »             | \App\Controllers\Home::index           |                |               |
-            | GET     | testing | testing-index | \App\Controllers\TestController::index |                |               |
-            | HEAD    | testing | testing-index | \App\Controllers\TestController::index |                |               |
-            | POST    | testing | testing-index | \App\Controllers\TestController::index |                |               |
-            | PATCH   | testing | testing-index | \App\Controllers\TestController::index |                |               |
-            | PUT     | testing | testing-index | \App\Controllers\TestController::index |                |               |
-            | DELETE  | testing | testing-index | \App\Controllers\TestController::index |                |               |
-            | OPTIONS | testing | testing-index | \App\Controllers\TestController::index |                |               |
-            | TRACE   | testing | testing-index | \App\Controllers\TestController::index |                |               |
-            | CONNECT | testing | testing-index | \App\Controllers\TestController::index |                |               |
-            | CLI     | testing | testing-index | \App\Controllers\TestController::index |                |               |
-            +---------+---------+---------------+----------------------------------------+----------------+---------------+
-            EOL;
-        $this->assertStringContainsString(
-            $expected,
-            (string) preg_replace('/\e\[[^m]+m/u', '', $this->getBuffer()),
-        );
-    }
-
     public function testRoutesCommandHostHostname(): void
     {
         Services::resetSingle('routes');