From faab7173bd373ed33eeb84c9062ef30d6b7ab1e0 Mon Sep 17 00:00:00 2001
From: Bertrand Benjamin <benjamin.bertrand@opytex.org>
Date: Sat, 22 Aug 2026 18:27:13 +0200
Subject: [PATCH] =?UTF-8?q?feat:=20ram=C3=A8ne=20=C3=A0=20la=20vue=20d'o?=
 =?UTF-8?q?=C3=B9=20l'on=20vient=20une=20fois=20le=20document=20enregistr?=
 =?UTF-8?q?=C3=A9?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Éditer une ligne depuis Dépenses ou Recettes ouvrait le document, puis
relâchait sur la liste des documents : il fallait revenir à la page
d'analyse et refaire ses filtres à la main pour corriger la ligne suivante.

Les deux tables passent maintenant à l'éditeur la vue d'où l'on part,
filtres compris, et il y ramène à ses trois sorties : enregistrement
terminé, abandon, et document introuvable. Sans cette destination — depuis
la liste des documents, ou un lien direct — le repli reste la liste.

La destination arrive par l'URL, donc fabricable par n'importe qui : seule
une page de l'application est suivie (utils/retour.js). Une URL absolue, un
chemin protocol-relatif que le navigateur lirait comme un domaine, ou des
contre-slashs retombent sur le repli.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
---
 .../components/analytics/DepensesTable.vue    | 19 +++++++++-
 .../src/components/revenus/RevenusTable.vue   | 19 +++++++++-
 frontend/src/pages/EditDocumentPage.vue       | 11 ++++--
 frontend/src/utils/retour.js                  | 22 ++++++++++++
 frontend/tests/retour.test.js                 | 36 +++++++++++++++++++
 5 files changed, 102 insertions(+), 5 deletions(-)
 create mode 100644 frontend/src/utils/retour.js
 create mode 100644 frontend/tests/retour.test.js

diff --git a/frontend/src/components/analytics/DepensesTable.vue b/frontend/src/components/analytics/DepensesTable.vue
index 729f769..b70d7c1 100644
--- a/frontend/src/components/analytics/DepensesTable.vue
+++ b/frontend/src/components/analytics/DepensesTable.vue
@@ -84,7 +84,7 @@
             </td>
             <td class="text-center">
               <router-link
-                :to="{ path: '/documents/' + dep.document_id + '/edit', query: { highlight: 'operation', fournisseur: dep.fournisseur, description: dep.description } }"
+                :to="lienEdition(dep)"
                 class="text-gray-500 hover:text-blue-400 transition-colors"
                 title="Editer le document source"
               >
@@ -142,6 +142,9 @@
 
 <script setup>
 import { ref, computed, watch } from 'vue'
+import { useRoute } from 'vue-router'
+
+const route = useRoute()
 
 const props = defineProps({
   depenses: {
@@ -169,6 +172,20 @@ const paginatedDepenses = computed(() =>
   props.depenses.slice(startIndex.value, endIndex.value)
 )
 
+/** L'editeur recoit la vue d'ou l'on part, filtres compris : une fois
+    l'enregistrement termine, il y ramene au lieu de la liste des documents. */
+function lienEdition(depense) {
+  return {
+    path: `/documents/${depense.document_id}/edit`,
+    query: {
+      highlight: 'operation',
+      fournisseur: depense.fournisseur,
+      description: depense.description,
+      retour: route.fullPath
+    }
+  }
+}
+
 function formatDate(dateStr) {
   if (!dateStr) return '-'
   const d = new Date(dateStr)
diff --git a/frontend/src/components/revenus/RevenusTable.vue b/frontend/src/components/revenus/RevenusTable.vue
index 9534da9..9dfae12 100644
--- a/frontend/src/components/revenus/RevenusTable.vue
+++ b/frontend/src/components/revenus/RevenusTable.vue
@@ -77,7 +77,7 @@
             </td>
             <td class="text-center">
               <router-link
-                :to="{ path: '/documents/' + revenu.document_id + '/edit', query: { highlight: 'locataire', lot_numero: revenu.lot_numero, locataire_nom: revenu.locataire_nom } }"
+                :to="lienEdition(revenu)"
                 class="text-gray-500 hover:text-blue-400 transition-colors"
                 title="Editer le document source"
               >
@@ -140,6 +140,23 @@
 
 <script setup>
 import { ref, computed } from 'vue'
+import { useRoute } from 'vue-router'
+
+const route = useRoute()
+
+/** L'editeur recoit la vue d'ou l'on part, filtres compris : une fois
+    l'enregistrement termine, il y ramene au lieu de la liste des documents. */
+function lienEdition(revenu) {
+  return {
+    path: `/documents/${revenu.document_id}/edit`,
+    query: {
+      highlight: 'locataire',
+      lot_numero: revenu.lot_numero,
+      locataire_nom: revenu.locataire_nom,
+      retour: route.fullPath
+    }
+  }
+}
 
 const props = defineProps({
   revenus: {
diff --git a/frontend/src/pages/EditDocumentPage.vue b/frontend/src/pages/EditDocumentPage.vue
index 1c85d45..2dc3a5c 100644
--- a/frontend/src/pages/EditDocumentPage.vue
+++ b/frontend/src/pages/EditDocumentPage.vue
@@ -151,6 +151,7 @@ import JsonViewer from '../components/JsonViewer.vue'
 import TaggingStep from '../components/TaggingStep.vue'
 import ReExtractionDiff from '../components/ReExtractionDiff.vue'
 import { computeExtractionDiff } from '../utils/diffExtraction.js'
+import { cheminRetour } from '../utils/retour.js'
 
 const router = useRouter()
 const route = useRoute()
@@ -167,6 +168,10 @@ const reExtractError = ref(null)
 const diff = ref(null)
 const previousData = ref(null)
 
+// D'ou l'on vient, et donc ou repartir une fois l'edition terminee : la table
+// d'analyse avec ses filtres si elle l'a dit, la liste des documents sinon.
+const destinationRetour = computed(() => cheminRetour(route.query.retour, '/documents'))
+
 const highlightInfo = computed(() => {
   const q = route.query
   if (q.highlight === 'locataire') return { type: 'locataire', lot_numero: q.lot_numero, locataire_nom: q.locataire_nom }
@@ -196,7 +201,7 @@ async function loadDocument() {
   } catch (err) {
     console.error('Error loading document:', err)
     alert('Erreur lors du chargement du document: ' + err.message)
-    router.push('/documents')
+    router.push(destinationRetour.value)
   } finally {
     isLoading.value = false
   }
@@ -214,7 +219,7 @@ function cancel() {
     const confirmed = confirm('Abandonner les modifications ?')
     if (!confirmed) return
   }
-  router.push('/documents')
+  router.push(destinationRetour.value)
 }
 
 function goToTagging() {
@@ -284,7 +289,7 @@ async function handleSave(depensesTags, shouldOverwrite) {
     
     if (result.success) {
       // Succès - redirection vers la liste des documents
-      router.push('/documents')
+      router.push(destinationRetour.value)
     } else {
       throw new Error(result.message || 'Échec de la sauvegarde')
     }
diff --git a/frontend/src/utils/retour.js b/frontend/src/utils/retour.js
new file mode 100644
index 0000000..344b1b6
--- /dev/null
+++ b/frontend/src/utils/retour.js
@@ -0,0 +1,22 @@
+// Une table d'analyse envoie vers l'editeur de document la page ou revenir une
+// fois l'enregistrement termine, filtres compris. Cette destination arrive par
+// l'URL : elle est donc fabricable par n'importe qui, et n'est suivie que si
+// elle designe une page de l'application.
+
+/**
+ * La destination de retour portee par une query, ou `repli` si elle manque ou
+ * ne designe pas un chemin interne.
+ *
+ * Sont refuses tout ce qui peut sortir du site : une URL absolue
+ * (`https://ailleurs`), un chemin protocol-relatif (`//ailleurs`) que le
+ * navigateur traite comme un domaine, et les contre-slashs dont certains
+ * navigateurs font des slashs.
+ */
+export function cheminRetour(brut, repli = '/documents') {
+  const valeur = Array.isArray(brut) ? brut[0] : brut
+  if (typeof valeur !== 'string' || valeur === '') return repli
+  if (!valeur.startsWith('/')) return repli
+  if (valeur.startsWith('//')) return repli
+  if (valeur.includes('\\')) return repli
+  return valeur
+}
diff --git a/frontend/tests/retour.test.js b/frontend/tests/retour.test.js
new file mode 100644
index 0000000..0013871
--- /dev/null
+++ b/frontend/tests/retour.test.js
@@ -0,0 +1,36 @@
+// La destination de retour arrive par l'URL : elle est fabricable par
+// n'importe qui, et un lien pieges ne doit pas pouvoir renvoyer l'utilisateur
+// hors du site apres un enregistrement.
+import { describe, expect, it } from 'vitest'
+
+import { cheminRetour } from '../src/utils/retour.js'
+
+describe('cheminRetour', () => {
+  it('rend la page d analyse avec ses filtres', () => {
+    const vue = '/analytics?immeuble_id=3&fournisseur=PPR&tag_id=0'
+
+    expect(cheminRetour(vue)).toBe(vue)
+  })
+
+  it('retombe sur la liste des documents quand rien n est demande', () => {
+    expect(cheminRetour(undefined)).toBe('/documents')
+    expect(cheminRetour('')).toBe('/documents')
+  })
+
+  it('refuse ce qui sortirait du site', () => {
+    // `//ailleurs.example` est un chemin protocol-relatif : le navigateur y lit
+    // un domaine, pas une page de l'application.
+    expect(cheminRetour('https://ailleurs.example')).toBe('/documents')
+    expect(cheminRetour('//ailleurs.example')).toBe('/documents')
+    expect(cheminRetour('/\\ailleurs.example')).toBe('/documents')
+    expect(cheminRetour('analytics')).toBe('/documents')
+  })
+
+  it('accepte le repli que l appelant lui donne', () => {
+    expect(cheminRetour(null, '/revenus')).toBe('/revenus')
+  })
+
+  it('ne retient que la premiere valeur quand le parametre est repete', () => {
+    expect(cheminRetour(['/analytics', 'https://ailleurs.example'])).toBe('/analytics')
+  })
+})