Gestion des esperluettes dans les titres/noms de fichiers (#193)

Scribouilli · Mar 25, 2024 · fe485f0 · fe485f0
1 parent c345420
commit fe485f0
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 4 deletions.
diff --git a/assets/scripts/components/screens/intern/ListContenu.svelte b/assets/scripts/components/screens/intern/ListContenu.svelte
@@ -120,7 +120,7 @@
                 </div>
               {:else}
                 <a
-                  href="{atelierPrefix}?path={contenu.path}&repoName={repoName}&account={account}"
+                  href="{atelierPrefix}?path={encodeURIComponent(contenu.path)}&repoName={repoName}&account={account}"
                 >
                   Modifier</a
                 >

diff --git a/assets/scripts/routes/atelier-articles.js b/assets/scripts/routes/atelier-articles.js
@@ -86,7 +86,7 @@ export default async ({ querystring }) => {
   }
 
   const state = store.state
-  const fileName = new URLSearchParams(querystring).get('path') ?? ''
+  const fileName = decodeURIComponent(new URLSearchParams(querystring).get('path') ?? '')
   const mapStateToProps = makeMapStateToProps(fileName)
 
   const articleContenu = new ArticleContenu({

diff --git a/assets/scripts/routes/atelier-pages.js b/assets/scripts/routes/atelier-pages.js
@@ -86,7 +86,7 @@ export default async ({ querystring }) => {
   await setCurrentRepositoryFromQuerystring(querystring)
 
   const state = store.state
-  const fileName = new URLSearchParams(querystring).get('path') ?? ''
+  const fileName = decodeURIComponent(new URLSearchParams(querystring).get('path') ?? '')
   const mapStateToProps = makeMapStateToProps(fileName)
 
   const currentRepository = store.state.currentRepository

diff --git a/assets/scripts/utils.js b/assets/scripts/utils.js
@@ -45,7 +45,7 @@ export const handleErrors = errorMessage => {
  */
 function makeFilenameCompatibleString(string) {
   return string
-    .replace(/\/|#|\?|:/g, '-') // replace url confusing characters
+    .replace(/\/|#|\?|:|&|\(|\)/g, '-') // replace url confusing characters
     .normalize('NFD')
     .replace(/[\u0300-\u036f]/g, '') // remove accent because GH pages triggers file download
     .split('.')
-Original file line number
+Diff line change
@@ Expand Up / @@ -120,7 +120,7 @@ @@
                     </div>
                   {:else}
                     <a
-                      href="{atelierPrefix}?path={contenu.path}&repoName={repoName}&account={account}"
+                      href="{atelierPrefix}?path={encodeURIComponent(contenu.path)}&repoName={repoName}&account={account}"
                     >
                       Modifier</a
                     >
@@ Expand Down @@