developer.jelix.org n'est plus utilisée, et existe uniquement pour son historique. Postez les nouveaux tickets sur le compte github.
Opened 13 years ago
Closed 9 years ago
#617 closed new feature (wontfix)
Plugin coord de cache
Reported by: | hadrien | Owned by: | |
---|---|---|---|
Priority: | normal | Milestone: | |
Component: | jelix:plugins:coord | Version: | 1.0.4 |
Severity: | normal | Keywords: | |
Cc: | Blocked By: | ||
Blocking: | Documentation needed: | no | |
Hosting Provider: | Php version: |
Description
Je vous propose mon plugin de cache fichier consistant en un plugin coord et une classe jCache pour cleaner des parties du cache.
Attachments (6)
Change History (14)
Changed 13 years ago by hadrien
comment:1 Changed 13 years ago by hadrien
- Type changed from bug to new feature
Changed 13 years ago by 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… :/
comment:2 Changed 13 years ago by laurentj
- review set to review?
Je ferais une revue du code plus tard. Voir si on ne peut pas rapprocher ça avec le jCache proposé dans le ticket #540.
comment:3 Changed 13 years ago by 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 :)
comment:4 Changed 13 years ago by 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 :/
comment:5 Changed 13 years ago by nuks
J'ai dis quelques idioties (corrigées par bastnic sur IRC). Ce que j'ai dis n'est pas à prendre en exemple.
comment:6 Changed 12 years ago by laurentj
- review changed from review? to review-
+ /** + * 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..
comment:7 Changed 11 years ago by laurentj
- Component changed from jelix:plugins to jelix:plugins:coord
comment:8 Changed 9 years ago by laurentj
- Resolution set to wontfix
- review review- deleted
- Status changed from new to closed
jCache plugin files