bricoles

9 files changed, 39 insertions, 60 deletions

diff --git a/controller/admin.php b/controller/admin.php
index f3eeeca..038d4c3 100644
--- a/controller/admin.php
+++ b/controller/admin.php
@@ -31,7 +31,7 @@ function pageArticlesSimplesAdmin(string $page, string $title, string $headerPai
         {

                 // nom de l'article ciblé (objet et session)

                 $Articles->fileCode = $fileCode;

-                $_SESSION['target'] = $Articles->fileCode;

+                $_SESSION['file_code'] = $Articles->fileCode;

 

                 if($suppression)

                 {

@@ -212,7 +212,7 @@ function albumEdit($fileCode, int $suppression)
                 {

                         $page = 'album';

                         $Albums->readOne();

-                        $_SESSION['target'] = $fileCode;

+                        $_SESSION['file_code'] = $fileCode;

                         $title = "Modifier un album";

 

                         for($i = 0; $i < $Albums->fileListCount; $i++)

diff --git a/controller/ajax.php b/controller/ajax.php
index 26aa9c6..ed840cd 100644
--- a/controller/ajax.php
+++ b/controller/ajax.php
@@ -38,48 +38,6 @@ if(isset($_GET['action']) && isset($_GET['page']) && $_GET['action'] == 'upload_
 }
 
 
-// inversion des positions de deux éléments d'une page
-/*if(isset($_GET['action']) && isset($_GET['page']) && ($_GET['action'] == 'monter' || $_GET['action'] == 'descendre') && isset($_GET['file_code']) && !empty($_GET['file_code']))
-{
-    // sécurité !!
-    if(!isset($_SESSION['admin']) || $_SESSION['admin'] != 1)
-    {
-        header('Location: index.php?erreur=interdit');
-    }
-    else
-    {
-        require('controller/admin.php');
-        require('model/Article.php');
-
-        $pagesArticlesSimples = ['menu', 'melaine', 'musique', 'presse', 'livres', 'jaime', 'peinture', 'archives', 'legal'];
-        $sens = 0;
-
-        if($_GET['action'] == 'monter')
-        {
-            $sens = 1;
-        }
-        if($_GET['action'] == 'descendre')
-        {
-            $sens = 0;
-        }
-
-        if(in_array($_GET['page'], $pagesArticlesSimples))
-        {
-            // 1 pour monter
-            inversionPositions($_GET['page'], $_GET['file_code'], 'Article', $sens);
-        }
-        elseif($_GET['page'] == 'discographie')
-        {
-            require('model/Album.php');
-            
-            // 1 pour monter
-            inversionPositions($_GET['page'], $_GET['file_code'], 'Album', $sens);
-        }
-    }
-    exit(); // stop !!
-}*/
-
-
 // page restauration quand le fichier zip est lourd
 // -> input file onchange
 if(isset($_GET['action']) && $_GET['action'] == 'restauration'
@@ -114,4 +72,17 @@ if(isset($_GET['action']) && $_GET['action'] == 'restauration'
         echo('file send');
         exit(); // stop !!
     }
-}
\ No newline at end of file
+}
+
+// nettoyage
+if(!isset($_GET['action']) || $_GET['action'] != 'restauration')
+{
+    if(isset($_SESSION['fileName']))
+    {
+        unset($_SESSION['fileName']);
+    }
+    if(isset($_SESSION['fileSize']))
+    {
+        unset($_SESSION['fileSize']);
+    }
+}
diff --git a/controller/backup.php b/controller/backup.php
index df0cf86..73438b4 100644
--- a/controller/backup.php
+++ b/controller/backup.php
@@ -165,7 +165,8 @@ function pageRestauration($from)
         global $maxWeight;

         

         // recharger la même page en écrivant les données

-        if(isset($_SESSION['fileName']) && isset($_SESSION['fileSize']) && $_SESSION['fileSize'] > $maxWeight)

+        if(isset($_SESSION['fileName']) && isset($_SESSION['fileSize']) && $_SESSION['fileSize'] > $maxWeight

+                && file_exists('data/tmp/' . $_SESSION['fileName']))

         {

                 // une copie du zip est conservée dans data/ au cas où

                 $path = 'data/' . $_SESSION['fileName'];

diff --git a/controller/ckeditor.php b/controller/ckeditor.php
index f73107d..4d4639d 100644
--- a/controller/ckeditor.php
+++ b/controller/ckeditor.php
@@ -68,9 +68,9 @@ function submitCKeditor()
         $Album = new Album($_GET['page'], 'discographie');

 

         // on retrouve les données si nécessaire

-        if(isset($_SESSION['target']) && $_SESSION['target'] != '')

+        if(isset($_SESSION['file_code']) && $_SESSION['file_code'] != '')

         {

-            $Album->fileCode = $_SESSION['target'];

+            $Album->fileCode = $_SESSION['file_code'];

             $Album->readOne();

         }

 

@@ -79,7 +79,7 @@ function submitCKeditor()
         if(!empty($_FILES['upload']['name']))

         {

             $nouvellePochette = $_FILES['upload']['name'];

-            $nouvellePochetteMini = pathinfo($_FILES['upload']['name'], PATHINFO_FILENAME)  . '-mini.' . pathinfo($_FILES['upload']['name'], PATHINFO_EXTENSION);

+            $nouvellePochetteMini = pathinfo($_FILES['upload']['name'], PATHINFO_FILENAME) . '-mini.' . pathinfo($_FILES['upload']['name'], PATHINFO_EXTENSION);

         }

         else

         {

@@ -90,7 +90,7 @@ function submitCKeditor()
         //var_dump($nouvellePochetteMini); die();

 

         // enregistrement nouvel album

-        if(!isset($_SESSION['target']) || $_SESSION['target'] == '')

+        if(!isset($_SESSION['file_code']) || $_SESSION['file_code'] == '')

         {

             // page disco

             $Album->createVignette($nouveauTitre, $nouvelleAnnee, $nouvellePochette, $nouvellePochetteMini);

@@ -105,7 +105,7 @@ function submitCKeditor()
         // modification

         else

         {

-            $Album->albumCode = $_SESSION['target'];

+            $Album->albumCode = $_SESSION['file_code'];

             

             // page disco

             $Album->format = 'json';

@@ -124,7 +124,7 @@ function submitCKeditor()
     {

         $Article = new Article($_GET['page'], $_GET['page']);

         

-        if(!isset($_SESSION['target']) || $_SESSION['target'] === '')

+        if(!isset($_SESSION['file_code']) || $_SESSION['file_code'] === '')

         {

             $Article->create($html);

 

@@ -132,7 +132,7 @@ function submitCKeditor()
         }

         else

         {

-            $Article->fileCode = $_SESSION['target']; // erreur ici?

+            $Article->fileCode = $_SESSION['file_code']; // erreur ici?

             $Article->update($html);

 

             // update html-mini

@@ -140,7 +140,7 @@ function submitCKeditor()
     }

 

     // nettoyage

-    unset($_SESSION['target']);

+    unset($_SESSION['file_code']);

 

     if($_GET['page'] === 'discographie' || $_GET['page'] === 'album')

     {

diff --git a/index.php b/index.php
index 14371e5..4e2de59 100644
--- a/index.php
+++ b/index.php
@@ -49,7 +49,7 @@ if(isset($_SESSION['admin']) && $_SESSION['admin'] == 1
 }

 else

 {

-    unset($_SESSION['target']);

+    unset($_SESSION['file_code']);

 }

 

 

diff --git a/model/Article.php b/model/Article.php
index 2ea80f8..0e8db61 100644
--- a/model/Article.php
+++ b/model/Article.php
@@ -18,7 +18,7 @@ class Article
 
         // pour un article (ou album) spécifique
         //public $fileName = ''; // = $_SESSION['nomFichier']
-        public $fileCode = ''; // $_SESSION['target']
+        public $fileCode = ''; // = $_SESSION['file_code']
         protected $time; // timestamp pour noms des fichiers créés
 
         public function __construct(string $page, string $folder)
diff --git a/public/file_upload.js b/public/file_upload.js
index dd47803..88a61a8 100644
--- a/public/file_upload.js
+++ b/public/file_upload.js
@@ -25,7 +25,7 @@ function uploadDespiteServerMaxWeightLimit(maxPHPiniWeight, archiveFormat)
         //var fileInfos = getFileInfo(tagId);
         var file = document.getElementById(tagId).files[0];
 
-        // si le le fichier est assez léger, javascript s'arrête ici
+        // si le fichier est assez léger, javascript s'arrête ici
         if(file.size > maxPHPiniWeight)
         {
                 // découpage
diff --git a/view/template-formulaires.php b/view/template-formulaires.php
index e7c7470..8c1d161 100644
--- a/view/template-formulaires.php
+++ b/view/template-formulaires.php
@@ -54,10 +54,15 @@ if($page === 'discographie' || $page === 'album')
         <textarea id="editor" name="contenu" ></textarea>

         <input class="boutonSubmitEditeur" type="submit" value="Valider" />

         <a class="boutonAnnuler" href="index.php?page=<?= $lienBoutonAnnuler ?>" >

-            <input type="button" value="Annuler" />

-        </a>

+            <input type="button" value="Annuler" /></a>

+<?php

+if($page === 'album')

+{

+?>

         <button><a href="index.php?page=discographie" >Retour page discographie</a></button>

-        

+<?php

+}

+?>   

         <script>

             let editor; // variable utilisable depuis une fonction

             

diff --git a/à faire après livraison.txt b/à faire après livraison.txt
index fa78d62..36997ee 100644
--- a/à faire après livraison.txt
+++ b/à faire après livraison.txt
@@ -1,3 +1,5 @@
+écrire Security::fixLinks()
+
 barre de progression au téléchargement d'un melaineDATA
 -> permettra de savoir aussi si la limite de 30s d'un script PHP a une importance