Plugin coord de cache

[hadrien]

Je vous propose mon plugin de cache fichier consistant en un plugin coord et une classe jCache pour cleaner des parties du cache.

[hadrien]

jCache plugin files

[hadrien]

celui là est mieux

[hadrien]

Voilà un diff. Je sais pas si j'ai mis les fichiers au bon endroit, mais je sais pas quoi faire de plus sans de plus amples informations… :/

[laurentj]

Je ferais une revue du code plus tard. Voir si on ne peut pas rapprocher ça avec le jCache proposé dans le ticket #540.

[hadrien]

fix création du dossier de cache

[nuks]

Je me permet de faire un début de review :) Dans ton sélecteur, tu utilises le séparateur "_". Tu devrais permettre l'utilisation du séparateur ":" aussi.

Les fichiers: Le répertoire jelix-modules est utilisé quand tu veux proposés à l'utilisateur des contrôleurs et modules de base pour gérer ton plugin. Ce n'est pas le cas :)

Ces fichiers sont inutiles:

lib/jelix-modules/jcache/controllers/default.classic.php
lib/jelix-modules/jcache/module.xml

Ceux si sont à déplacer et à renommer:

lib/jelix-modules/jcache/install/var/cache.plugin.ini.php
-> lib/jelix/plugins/coord/jcache.plugin.ini.php.dist

lib/jelix-plugins/jcache/classes/jCache.class.php
->  lib/jelix/plugins/jcache/classes/jcache.class.php

lib/jelix-plugins/coord/cache/cache.coord.php
-> lib/jelix/plugins/coord/cache/jcache.coord.php

Tu dois créer un fichier:

lib/jelix/plugins/coord/plugin.xml

qui ressemble à ça

<?xml version="1.0" encoding="UTF-8"?>
<plugin name="auth">
   <version major="" minor="" status="" copixversion=""  date="" />
   <author name="Gerald Croes" email="gcroes@aston.fr" website="http://www.phpside.org" role="creator" />
   <contributor name="Laurent Jouanneau" />
   <description></description>
   <infos />
</plugin>

utilises ça si tu veux inclure ta classe cache dans ton plugin coord

require(JELIX_LIB_PATH.'jcache/jcache.class.php');

A confirmer par laurentj :)

[nuks]

Je me permet de retirer ce que j'ai dit: les séparateurs ne devraient pas poser de problèmes, j'ai lus trop vite :/

[hadrien]

Avec les dernières modifications de nuks`

[hadrien]

On va y arriver…

[nuks]

J'ai dis quelques idioties (corrigées par bastnic sur IRC). Ce que j'ai dis n'est pas à prendre en exemple.

[laurentj]

+	/**
+	 * Get request params and check file cached
+	 * @param {array} $params Params array
+	 * @return void
+	 **/

Description of parameter and the return value don't really correspond to the definition of this values in the jICoordPlugin interface ;-)

+			else
+			{
+				// Include cached file
+				include(
+					$this->path .
+					$this->_localPath .
+					$this->_fileKey
+				);
+				exit();
+			}
+		}

I really don't like this exit, because it doesn't execute other plugins, it doesn't finished correctly all the process... Perhaps we should hack jCoordinator to handle this case of kind of output...

We also really should see if we can reuse the future jCache class of ticket #540..