refactor(routes): use RESTful URL for text print-plain

Change /text/print-plain?text={id} to /text/{id}/print-plain

- Add new RESTful route with legacy fallback
- Update TextPrintController::printPlain to accept route param
- Update views and TypeScript to use new URL pattern

Author: HugoFara

src/Modules/Review/Views/header.php

@@ -50,7 +50,7 @@
         <a href="/text/read?start=<?php echo $textId; ?>" target="_top">
             <?php echo \Lwt\Shared\UI\Helpers\IconHelper::render('book-open', ['title' => 'Read', 'alt' => 'Read']); ?>
         </a>
-        <a href="/text/print-plain?text=<?php echo $textId; ?>" target="_top">
+        <a href="/text/<?php echo $textId; ?>/print-plain" target="_top">
             <?php echo \Lwt\Shared\UI\Helpers\IconHelper::render('printer', ['title' => 'Print', 'alt' => 'Print']); ?>
         </a>
         <?php echo (new AnnotationService())->getAnnotationLink($textId); ?>

src/Modules/Text/Http/TextPrintController.php

@@ -82,15 +82,19 @@ public function getPrintService(): TextPrintService
     /**
      * Print plain text with annotations (replaces text_print_plain.php).
      *
-     * Route: /text/print-plain?text=[textid]&ann=[annotationcode]&status=[statuscode]
+     * Routes:
+     * - GET /text/{text:int}/print-plain (new RESTful route)
+     * - GET /text/print-plain?text=[textid] (legacy route)
      *
-     * @param array $params Route parameters
+     * Optional query params: ann=[annotationcode]&status=[statuscode]
+     *
+     * @param int|null $text Text ID (injected from route parameter)
      *
      * @return void
      *
      * @psalm-suppress UnusedVariable Variables are used in included view files
      */
-    public function printPlain(array $params): void
+    public function printPlain(?int $text = null): void
     {
         include_once self::BACKEND_PATH . '/Core/Bootstrap/db_bootstrap.php';
         include_once dirname(__DIR__) . '/Application/Services/TextStatisticsService.php';
@@ -101,7 +105,8 @@ public function printPlain(array $params): void
         include_once dirname(__DIR__, 2) . '/Vocabulary/Application/Services/ExpressionService.php';
         include_once __DIR__ . '/../../../Shared/Infrastructure/Database/Restore.php';
 
-        $textId = (int) $this->param('text', '0');
+        // Support both new route param injection and legacy query param
+        $textId = $text ?? (int) $this->param('text', '0');
 
         if ($textId === 0) {
             $this->redirect('/text/edit');
@@ -289,7 +294,7 @@ public function deleteAnnotation(array $params): void
 
         $deleted = $this->printService->deleteAnnotation($textId);
         if ($deleted) {
-            $this->redirect('/text/print-plain?text=' . $textId);
+            $this->redirect('/text/' . $textId . '/print-plain');
         }
 
         // If deletion failed, redirect back to print view

src/Modules/Text/Views/edit_list.php

@@ -304,7 +304,7 @@ class="dropdown-menu card-dropdown"
                                             @click.outside="open = false"
                                             x-cloak>
                                             <div class="dropdown-content">
-                                                <a :href="'/text/print-plain?text=' + text.id" class="dropdown-item">
+                                                <a :href="'/text/' + text.id + '/print-plain'" class="dropdown-item">
                                                     <?php echo IconHelper::render('printer', ['size' => 14]); ?>
                                                     <span>Print</span>
                                                 </a>

src/Modules/Text/Views/print_alpine.php

@@ -78,7 +78,7 @@
 $hasAnnotation = $viewData['hasAnnotation'];
 
 // Determine URLs based on mode
-$printUrl = $mode === 'plain' ? '/text/print-plain?text=' : '/text/{id}/print';
+$printUrl = $mode === 'plain' ? '/text/{id}/print-plain' : '/text/{id}/print';
 ?>
 <!-- Alpine.js container -->
 <div x-data="textPrintApp()" x-init="init()" x-cloak>

src/Modules/Text/Views/read_desktop.php

@@ -102,7 +102,7 @@
               <a href="/review?text=<?php echo $textId; ?>" class="button is-small">Review</a>
             </div>
             <div class="control">
-              <a href="/text/print-plain?text=<?php echo $textId; ?>" class="button is-small">Print</a>
+              <a href="/text/<?php echo $textId; ?>/print-plain" class="button is-small">Print</a>
             </div>
             <div class="control">
               <a href="/texts/<?php echo $textId; ?>/edit" class="button is-small">Edit</a>

src/Modules/Text/Views/read_header.php

@@ -90,7 +90,7 @@
             echo \Lwt\Shared\UI\Helpers\IconHelper::render('circle-help', ['title' => 'Review', 'alt' => 'Review']);
             ?>
         </a>
-        <a href="/text/print-plain?text=<?php echo $textId; ?>" target="_top">
+        <a href="/text/<?php echo $textId; ?>/print-plain" target="_top">
             <?php echo \Lwt\Shared\UI\Helpers\IconHelper::render('printer', ['title' => 'Print', 'alt' => 'Print']); ?>
         </a>
         <?php echo (new AnnotationService())->getAnnotationLink($textId); ?>

src/backend/Router/routes.php

@@ -99,6 +99,9 @@ function registerRoutes(Router $router): void
     $router->get('/text/{text:int}/print/edit', 'Lwt\\Modules\\Text\\Http\\TextPrintController@editAnnotation', AUTH_MIDDLEWARE);
     // RESTful route: DELETE /text/123/annotation
     $router->delete('/text/{text:int}/annotation', 'Lwt\\Modules\\Text\\Http\\TextPrintController@deleteAnnotation', AUTH_MIDDLEWARE);
+    // RESTful route: /text/123/print-plain
+    $router->get('/text/{text:int}/print-plain', 'Lwt\\Modules\\Text\\Http\\TextPrintController@printPlain', AUTH_MIDDLEWARE);
+    // Legacy route: /text/print-plain?text=123
     $router->registerWithMiddleware(
         '/text/print-plain',
         'Lwt\\Modules\\Text\\Http\\TextPrintController@printPlain',

src/frontend/js/modules/text/pages/text_print_app.ts

@@ -155,18 +155,19 @@ function getPageConfig(): PageConfig {
   // Fallback: try to get text ID from URL
   const pathname = window.location.pathname;
 
-  // RESTful URL pattern: /text/{id}/print or /text/{id}/print/edit
-  const printMatch = pathname.match(/\/text\/(\d+)\/print(?:\/edit)?$/);
+  // RESTful URL pattern: /text/{id}/print, /text/{id}/print/edit, or /text/{id}/print-plain
+  const printMatch = pathname.match(/\/text\/(\d+)\/print(?:-plain|\/edit)?$/);
   if (printMatch) {
     const textId = parseInt(printMatch[1], 10);
     const isEdit = pathname.endsWith('/print/edit');
+    const isPlain = pathname.endsWith('/print-plain');
     return {
       textId,
-      mode: isEdit ? 'edit' : 'annotated'
+      mode: isEdit ? 'edit' : isPlain ? 'plain' : 'annotated'
     };
   }
 
-  // Legacy/plain print URL pattern: /text/print-plain?text={id}
+  // Legacy plain print URL pattern: /text/print-plain?text={id}
   const params = new URLSearchParams(window.location.search);
   const textId = parseInt(params.get('text') || '0', 10);
 
@@ -529,7 +530,7 @@ export function textPrintAppData(): TextPrintAppData {
             if (response.redirected) {
               window.location.href = response.url;
             } else if (response.ok) {
-              window.location.href = `/text/print-plain?text=${textId}`;
+              window.location.href = `/text/${textId}/print-plain`;
             }
           })
           .catch((error) => {