Re: Re: cvs: pear /HTML_BBCodeParser BBCodeParser.php /HTML_BBCodeParser/BBCodeParser/Filter Basic.php Extended.php Images.php Links.php Lists.php
| From: | Arnaud Limbourg | Date: | Fri, 14 Oct 2005 13:36:07 +0000 |
| Subject: | Re: Re: cvs: pear /HTML_BBCodeParser BBCodeParser.php /HTML_BBCodeParser/BBCodeParser/Filter Basic.php Extended.php Images.php Links.php Lists.php | ||
| References: | 1 2 | Groups: | php.pear.cvs php.pear.qa |
| Request: | Send a blank email to pear-qa+get-3129@lists.php.net to get a copy of this message | ||
Lorenzo Alberton wrote:
> Hi guys,
>
> I have an old patch that I once submitted to Stijn
> but never heard back from him.
>
> It basically adds these methods:
>
> - addFilter()
> - removeFilter()
> - addFilters()
> - setOption()
>
> they allow dynamically loading/unloading of filters
> and options (ATM you can only set them in the constructor).
> The addFilters() method accepts a CSV string list or an array.
>
> The patch (that also includes a few CS issues)
> is available for review here:
>
> http://www.archaeogate.org/tmp/pear/BBCodeParser.patch.txt
>
> Can I commit it?
Hi Lorenzo,
I'd like Seth to review this patch and tell us if the testcases still
pass :)
It looks harmless AFAICS.
Arnaud.