Le 17/11/2012 13:12, Régis Houssin a écrit :
Il y a plusieurs modules qui utilisent ceci, je ne peux pas l'inclure dans tout les modules !
Pourquoi pas ?
Chaque module doit être autosuffisant. Si on se met a mettre dans le
coeur des lib juste parcequ'il y a plusieurs modules externes qui
l'utilisent, on peut mettre des centaines de librairies et API dans le
include.
De plus Savant apporte une certaine structure même si on peut faire autrement, pas de lourdeur constatée.
Quelle structure ? Le PHP natif apporte la possibilité de faire la meme
structure, en plus simple.
Le site web meme de Savant n'apporte aucun argument en faveur de
l'utilisation de savant en lieu et place du include PHP. Leur propre
tutorial pourrait être remplacé par 1 seule ligne de code PHP pour le
même resultat et sans les inconvénients.
L'utilisation de Savant ne SERA DONC JAMAIS dans dolibarr. Par contre,
dans un fork, pourquoi pas.
-----------------------------------------
Régis Houssin
Tél. +33633020797
http://www.dolibarr.fr
http://www.dolibox.fr
Le 17 nov. 2012 à 13:06, "Laurent Destailleur (eldy)" <address@hidden> a écrit :
Non savant ne pas utilisé dans le coeur de Dolibarr. Cela n'a aucun interet.
On peut faire la meme chose que
$instancesavant.tpl("montemplate.tpl")
en faisant
include("montemplate.tpl")
Avec l'avantage de ne pas avoir a faire tous le set et sans dégradation
de performance.
Si des module sl'utilisent, il vaudrait mieux qu'ils l'inclu eux meme
dans leur package afin de ne pas alourdir dolibarr de dépendance inutile.
Le 17/11/2012 11:57, Régis Houssin a écrit :
Ne retire pas Savant des packages car il est utilisé dans des modules externes et sera utilisé dans le cœur
-----------------------------------------
Régis Houssin
Tél. +33633020797
http://www.dolibarr.fr
http://www.dolibox.fr
Le 17 nov. 2012 à 11:50, Laurent Destailleur <address@hidden> a écrit :
Branch: refs/heads/develop
Home: https://github.com/Dolibarr/dolibarr
Commit: 19b7cc66d58d815e39ac4dad3b20a7a8f02d48f1
https://github.com/Dolibarr/dolibarr/commit/19b7cc66d58d815e39ac4dad3b20a7a8f02d48f1
Author: Laurent Destailleur <address@hidden>
Date: 2012-11-17 (Sat, 17 Nov 2012)
Changed paths:
M build/makepack-dolibarr.pl
M htdocs/core/tpl/login.tpl.php
M htdocs/core/tpl/passwordforgotten.tpl.php
M htdocs/holiday/fiche.php
M htdocs/main.inc.php
M htdocs/paypal/lib/paypal.lib.php
M htdocs/theme/bureau2crea/tpl/login.tpl.php
M htdocs/theme/bureau2crea/tpl/passwordforgotten.tpl.php
Log Message:
-----------
Qual: Remove hidden option so is not efficient and not used files.
Qual: Uniformize code
_______________________________________________
Dolibarr-git mailing list
address@hidden
http://doliforge.org/mailman/listinfo/dolibarr-git
--
Eldy (Laurent Destailleur).
EMail: address@hidden
Web: http://www.destailleur.fr
Dolibarr (Project leader): http://www.dolibarr.org
To make a donation for Dolibarr project via Paypal: address@hidden
AWStats (Author) : http://awstats.sourceforge.net
To make a donation for AWStats project via Paypal: address@hidden
AWBot (Author) : http://awbot.sourceforge.net
CVSChangeLogBuilder (Author) : http://cvschangelogb.sourceforge.net