Re: Re: Proposal: IT[X] fork as a new package
| From: | Markus Wolff | Date: | Thu, 13 Feb 2003 12:38:24 +0000 |
| Subject: | Re: Re: Proposal: IT[X] fork as a new package | ||
| References: | 1 2 | Groups: | php.pear.dev |
| Request: | Send a blank email to pear-dev+get-13220@lists.php.net to get a copy of this message | ||
Am Thu, 13 Feb 2003 12:59:10 +0100 schrieb "Christian Dickmann" <chrisdicki@gmx.de>:
> > Well, as there is an obvious opposition to commiting my changes into the
> > IT[X] package, I'd rather finish banging my head against the wall. I can
> > understand the reasoning against my changes, though can't say that I
> > agree with it.
>
> You don't get it, do you?
> We are _not_ against your changes! str instead of preg would be
> nice, IF optional. global vars are nice too! I would _love_ to see this
> in IT. We should only keep BC in mind.
> The cache is a diffrent issue ... PA doesn't want it to be inside
> IT, but I have a question: how optional is it?
> Something like
> IT::loadTemplateFile($filename, $removeUnknownVariables = true,
> $removeEmptyBlocks = true, $cacheFile = false);
> would not break BC, wouldn't harm PA, but would add additional
> features for those who want it. How many lines of code are we talking
> about here?
I´m still thinking factory:
$tpl = HTML_Template_IT::factory(); // Classic version
$tpl = HTML_Template_IT::factory("cached"); // Cache-Enabled version
$tpl = HTML_Template_IT::factory("noregexp"); // Uses str_replace but
// no cache to enable
// <!-- INCLUDE ...
$tpl = HTML_Template_IT::factory("ext"); // Uses PECL extension, once
// it´s available
..I think this would be the most elegant solution...
Regards,
Markus
--
*21st Media* | Consulting, Konzeption, Produktion für die Bereiche:
Markus Wolff | Internet, Intranet, eCommerce, Content Management,
Hamburg,Germany | Softwareentwicklung, 3D-Animation, Videostreaming
http://21st.de | Tel. [+49](0)40/6887949-0, Fax: [+49](0)40/6887949-1