Changeset 113860 in spip-zone

Timestamp:

Feb 11, 2019, 8:10:03 PM (6 days ago)

Author:

eric@…

Message:

Il faut toujours utiliser sous_repertoire() lors de l'écriture.
On le vire donc de la config et on ajoute le dossier plugin à la config pour éviter de le recalculer.

Location:

_plugins_/cache/trunk

Files:

• cache/cache.php (modified) (1 diff)
• inc/cache.php (modified) (2 diffs)

_plugins_/cache/trunk/cache/cache.php

@@ -80 +80 @@
 
         // -- Sous-dossier spécifique au plugin
-        $sous_dossier = ($configuration['racine'] == _DIR_VAR) ? "cache-${plugin}" : "$plugin";
-        // -- Création et enregistrement du dossier de base
-        include_spip('inc/flock');
-        $configuration['dossier_base'] = sous_repertoire($configuration['racine'], $sous_dossier);
+        $configuration['dossier_plugin'] = ($configuration['racine'] == _DIR_VAR) ? "cache-${plugin}/" : "${plugin}/";
+        // -- Enregistrement du dossier de base
+        $configuration['dossier_base'] = $configuration['racine'] . $configuration['dossier_plugin'];
 
         // -- Construction du tableau des composants du nom : dans l'ordre on a toujours les composants obligatoires

_plugins_/cache/trunk/inc/cache.php

@@ -49 +49 @@
         include_spip('inc/flock');
         if (is_array($cache)) {
-                // Création du répertoire du cache à créer, si besoin.
-                $erreur_sous_dossier = false;
-                if ($configuration[$plugin]['sous_dossier']) {
-                        if (!empty($cache['sous_dossier'])) {
-                                // Si le cache nécessite un sous-dossier on crée le répertoire au besoin.
-                                sous_repertoire($configuration[$plugin]['dossier_base'], rtrim($cache['sous_dossier'], '/'));
-                        } else {
-                                // C'est une erreur, le sous-dossier n'a pas été fourni alors qu'il est requis. On arrête l'écriture.
-                                $erreur_sous_dossier = true;
-                        }
-                }
-
-                // Détermination du chemin du cache si pas d'erreur sur le sous-dossier :
-                // - le nom sans extension est construit à partir des éléments fournis sur le conteneur et
-                //   de la configuration du nom pour le plugin.
-                if (!$erreur_sous_dossier) {
+                // Vérification de la conformité entre la configuration et le sous-dossier du cache.
+                if (!$configuration[$plugin]['sous_dossier']
+                or ($configuration[$plugin]['sous_dossier'] and !empty($cache['sous_dossier']))) {
+                        // Détermination du chemin du cache si pas d'erreur sur le sous-dossier :
+                        // - le nom sans extension est construit à partir des éléments fournis sur le conteneur et
+                        //   de la configuration du nom pour le plugin.
                         $fichier_cache = cache_cache_composer($plugin, $cache, $configuration[$plugin]);
                 }
@@ -74 +64 @@
         
         if ($fichier_cache) {
+                // On crée les répertoires si besoin
+                sous_repertoire($configuration[$plugin]['racine'], rtrim($configuration[$plugin]['dossier_plugin'], '/'));
+                if ($configuration[$plugin]['sous_dossier']) {
+                        sous_repertoire($configuration[$plugin]['dossier_base'], rtrim($cache['sous_dossier'], '/'));
+                }
+
                 // Suivant que la configuration demande une sérialisation ou pas, on vérife le format du contenu
                 // de façon à toujours écrire une chaine.