Ticket #132 (closed enhancement: fixed)

Opened 2 years ago

Last modified 1 year ago

Répertoires pour plugin jTpl par défaut dans les applications

Reported by: bballizlife Assigned to: bballizlife
Priority: normal Milestone: Jelix 1.0beta3
Component: jelix-scripts Version: 1.0 beta1
Severity: major Keywords: jtpl plugins createapp
Cc: Php version:
Review: Hosting Provider:
Documentation needed: Blocking:

Description

Actuellement si on veut faire des plugins pour Jtpl personnalisés il faut créer des répertoires dans son application pour et déclarer y déclarer dans la configuration.

Sachant qu'on a déjà un répertoire plugins/ dans les applications, on pourrait générer de base plugins/jtpl/(common|html|text|xul)/ et que cela soit pris en compte.

Attachments

ticket_132_bballizlife-1.diff (0.5 kB) - added by bballizlife on 03/08/07 21:55:15.
Patch defaulconfig.ini.php.tpl pour prendre en compte app:plugins/jtpl/
ticket_132_bballizlife-2.diff (1.4 kB) - added by bballizlife on 03/08/07 21:56:04.
Patch pour createapp.cmd.php pour créer les répertoires de plugins jTpl à la création de l'application

Change History

03/03/07 00:25:10 changed by bballizlife

  • keywords set to jtpl plugins createapp.

03/03/07 00:29:25 changed by bballizlife

  • owner set to bballizlife.
  • status changed from new to assigned.

03/08/07 21:55:15 changed by bballizlife

  • attachment ticket_132_bballizlife-1.diff added.

Patch defaulconfig.ini.php.tpl pour prendre en compte app:plugins/jtpl/

03/08/07 21:56:04 changed by bballizlife

  • attachment ticket_132_bballizlife-2.diff added.

Patch pour createapp.cmd.php pour créer les répertoires de plugins jTpl à la création de l'application

03/09/07 14:08:51 changed by laurentj

je ne suis pas d'accord sur le nom du répertoire. Le répertoire plugins/ est réservé aux plugins du coordinateur, et ici, jelix va croire que le repertoire tpl est un plugin pour le controleur.

Il faut donc un autre répertoire : theapp/tplplugins par exemple et non theapp/plugins/tpl

À moins de créer un sous repertoire plugins/coord/ pour les plugins du coordinateur, mais il faut changer alors la conf par defaut etc.

03/09/07 14:29:20 changed by bballizlife

Effectivement je n'avais pas pensé à ce problème. Je serais plus intéressé à mettre en place les répertoires plugins/coord et plugins/tpl afin de regrouper sous le même répertoire tous les plugins utilisateurs d'une application et de limiter l'éparpillement des ressources. Si cette solution est retenue, je proposerais un nouveau patch en conséquence

03/09/07 18:15:31 changed by laurentj

  • blocking changed.

ok, dans ce cas, peut être faudrait il aussi réaménager lib/jelix-plugins, et y migrer les plugins jtpl (ce qui necessitera de modifier le build de jtpl standalone) ?

04/05/07 21:23:03 changed by laurentj

  • milestone changed from Jelix 1.0beta2 to Jelix 1.0beta3.

05/16/07 21:58:34 changed by bballizlife

Je m'interroge tout haut.

Est-ce bien pertinent de déplacer le répertoire de plugins du coordinateur de app/plugins vers app/plugins/coord/ ? Réaliser ce changement ne pose pas forcément de gros problème, mais qu'en est-il de la compatibilité ? On risque de casser pas mal d'applications utilisant des plugins coord ou celles-ci devront être modifiées pour déplacer tous leurs plugins coord.

On a donc 2 solutions :

  • app/plugins/coord et app/plugins/tpl

ou

  • app/plugins et app/tplplugins

Je crois que finalement je ne sais pas laquelle serait la mieux...

05/18/07 09:50:17 changed by laurentj

le problème à terme, est qu'il y a un risque qu'on ait pleins de répertoires type xxxxplugins.

Mais ce qu'on pourrait faire : xxxplugins jusqu'à la version 1.0. Et pour la version 1.1, on réorganiserait en rassemblant tout ça dans une repertoire plugins/

05/18/07 10:35:39 changed by bballizlife

Ok pour ce remaniement, ça me convient.

06/29/07 16:54:12 changed by laurentj

Le remaniement va se faire pour la beta3. En cours dans le trunk. :

plugins/
  coord/
    foo/
        foo.plugin.php
        foo.plugin.ini.php
  tpl/
    common/
        function.truc.php
    html/
        function.truc.php
  auth/
    foo/
        jAuthDriverFoo.class.php
  db/
    mysql/
        jDbConnection.mysql.php
        jDbResulset.mysql.php
        jDbTools.mysql.php
  wikirenderer/
    classicwr_to_text/
        classicwr_to_text.php
  url/
    significant/
        jUrlEngine.significant.class.php

Sachant que cela va être suivi d'un renommage

foo.plugin.php (fooPlugin = jIPlugin)
               --> foo.coord.php (fooCoordPlugin = jICoordPlugin)
function.truc.php (jtpl_function_truc)
               --> truc.function.php (truc_jtpl_function)
jAuthDriverFoo.class.php (jAuthDriverFoo = jIAuthDriver) 
               --> foo.auth.php (fooAuthDriver = jIAuthDriver)
jDbConnection.mysql.php (jDbConnectionMySQL = jDbConnection) 
               --> mysql.dbconnection.php (mysqlDbConnection = jDbConnection )
jDbResulset.mysql.php  (jDbResulsetMySQL = jDbResultSet) 
               --> mysql.dbresultset.php (mysqlDbresultset = jDbResultSet )
jDbTools.mysql.php (jDbToolsMysql = jDbTools) 
               --> mysql.dbtools.php (mysqlDbTools = jDbTools )
jUrlEngine.significant.class.php ( jUrlEngineSignificant = jIUrlEngine) 
               --> significant.url.php (significantUrlEngine = jIUrlEngine)

06/30/07 01:34:24 changed by laurentj

  • status changed from assigned to closed.
  • resolution set to fixed.
Download in other formats: Comma-delimited Text Tab-delimited Text RSS Feed