Crud routes are not cached

Distinguish between get and post cache
tina4stack · Nov 29, 2023 · c261075 · c261075
1 parent 25d49e2
commit c261075
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 3 deletions.
diff --git a/Tina4/Routing/Crud.php b/Tina4/Routing/Crud.php
@@ -186,6 +186,7 @@ public static function route($path, ORM $object, $function, $secure = false): vo
          * @description  {$path} CRUD
          * @tags CRUD
          * @secure
+         * @no-cache
          */
         Route::get(
             $path . "/form",
@@ -198,6 +199,7 @@ function (Response $response, Request $request) use ($object, $function) {
 
         /**
          * @description  {$path} CRUD
+         * @no-cache
          */
         Route::post(
             $path,
@@ -223,6 +225,7 @@ function (Response $response, Request $request) use ($object, $function) {
          * @description  {$path} CRUD
          * @tags CRUD
          * @secure
+         * @no-cache
          */
         Route::get(
             $path,
@@ -239,6 +242,7 @@ function (Response $response, Request $request) use ($object, $function) {
          * @description  {$path} CRUD
          * @tags CRUD
          * @secure
+         * @no-cache
          */
         Route::get(
             $path . "/{id}",
@@ -258,6 +262,7 @@ function (Response $response, Request $request) use ($object, $function) {
          * @description  {$path} CRUD
          * @tags CRUD
          * @secure
+         * @no-cache
          */
         Route::post(
             $path . "/{id}",
@@ -281,6 +286,7 @@ function (Response $response, Request $request) use ($object, $function) {
          * @description  {$path} CRUD
          * @tags CRUD
          * @secure
+         * @no-cache
          */
         Route::delete(
             $path . "/{id}",

diff --git a/Tina4/Routing/Router.php b/Tina4/Routing/Router.php
@@ -109,7 +109,7 @@ final public function resolveRoute(?string $method, ?string $url, ?Config $confi
         if ($routerResponse = $this->handleRoutes($method, $url, $customHeaders, $customRequest)) {
             if ($routerResponse->cached) {
                 $this->createCacheResponse(
-                    $url,
+                    $url.$method,
                     $routerResponse->httpCode,
                     $routerResponse->content,
                     $this->addCORS($routerResponse->headers),
@@ -514,7 +514,7 @@ public function handleRoutes($method, $url, array $customHeaders=[], $customRequ
                 }
                   else
                 if ($route["cached"]) {
-                    $cacheResult = $this->getCacheResponse($url);
+                    $cacheResult = $this->getCacheResponse($url.$method);
                     if ($cacheResult !== null && $url !== "/cache/clear" && $url !== "/migrate" && $url !== "/migrate/create") {
                         Debug::message("$this->GUID Got cached result for $url", TINA4_LOG_DEBUG);
                         return new RouterResponse(
@@ -538,7 +538,7 @@ public function handleRoutes($method, $url, array $customHeaders=[], $customRequ
                             return new RouterResponse("", HTTP_FORBIDDEN, $headers);
                         } else {
                             if ($route["cached"]) {
-                                $cacheResult = $this->getCacheResponse($url);
+                                $cacheResult = $this->getCacheResponse($url.$method);
                                 if ($cacheResult !== null && $url !== "/cache/clear" && $url !== "/migrate" && $url !== "/migrate/create") {
                                     Debug::message("$this->GUID Got cached result for $url", TINA4_LOG_DEBUG);
                                     return new RouterResponse(