From 55afb815a373596e1947743826bd9c7c9535c4ba Mon Sep 17 00:00:00 2001
From: Tim Bond <cookieguru@gmail.com>
Date: Fri, 22 Dec 2023 12:31:37 -0800
Subject: [PATCH] Remove example trailing slash middleware since it is not
 compatible

The example middleware to remove the trailing slash from routes in the
Slim V4 documentation is not PSR-15 compatible, and thus does not work
in Slim 4.  Thankfully, the linked middleweare is, so update the docs
to point solely to that middleware.

Additionally, clarify that rewriting a request of a non-GET operation
only affects the in-memory copy of the Request object, and not the
actual user called by the consumer.
---
 docs/v4/cookbook/route-patterns.md | 49 +++---------------------------
 1 file changed, 5 insertions(+), 44 deletions(-)

diff --git a/docs/v4/cookbook/route-patterns.md b/docs/v4/cookbook/route-patterns.md
index 45afbab4..c579dfe9 100644
--- a/docs/v4/cookbook/route-patterns.md
+++ b/docs/v4/cookbook/route-patterns.md
@@ -5,53 +5,14 @@ title: Trailing / in route patterns
 Slim treats a URL pattern with a trailing slash as different to one without. 
 That is, `/user` and `/user/` are different and so can have different callbacks attached.
 
-For GET requests a permanent redirect is fine, but for other request methods like POST or PUT the browser will send the second request with the GET method. 
-To avoid this you simply need to remove the trailing slash and pass the manipulated url to the next middleware.
+For GET requests a permanent redirect is fine, but for other request methods like POST or PUT the browser will send the second request with the GET method.
+To avoid this you simply need to remove the trailing slash in the `Request` object and pass the manipulated url to the next middleware.
 
-If you want to redirect/rewrite all URLs that end in a `/` to the non-trailing `/` equivalent, then you can add this middleware:
-
-```php
-<?php
-
-use Psr\Http\Message\RequestInterface as Request;
-use Psr\Http\Server\RequestHandlerInterface as RequestHandler;
-use Slim\Factory\AppFactory;
-use Slim\Psr7\Response;
-
-require __DIR__ . '/../vendor/autoload.php';
-
-$app = AppFactory::create();
-
-$app->add(function (Request $request, RequestHandler $handler) {
-    $uri = $request->getUri();
-    $path = $uri->getPath();
-    
-    if ($path != '/' && substr($path, -1) == '/') {
-        // recursively remove slashes when its more than 1 slash
-        $path = rtrim($path, '/');
-
-        // permanently redirect paths with a trailing slash
-        // to their non-trailing counterpart
-        $uri = $uri->withPath($path);
-        
-        if ($request->getMethod() == 'GET') {
-            $response = new Response();
-            return $response
-                ->withHeader('Location', (string) $uri)
-                ->withStatus(301);
-        } else {
-            $request = $request->withUri($uri);
-        }
-    }
-
-    return $handler->handle($request);
-});
-```
-
-Alternatively, consider [middlewares/trailing-slash](//github.com/middlewares/trailing-slash) middleware which also allows you to force a trailing slash to be appended to all URLs:
+If you want to redirect/rewrite all URLs that end in a `/` to the non-trailing `/` equivalent, consider [middlewares/trailing-slash](//github.com/middlewares/trailing-slash) middleware.
+Alternatively, the middlware also allows you to force a trailing slash to be appended to all URLs.
 
 ```php
 use Middlewares\TrailingSlash;
 
-$app->add(new TrailingSlash(true)); // true adds the trailing slash (false removes it)
+$app->add(new TrailingSlash(trailingSlash: true)); // true adds the trailing slash (false removes it)
 ```