Re: [PEPr] Proposal for HTML::HTML_QuickForm_advmultiselect

On 6/10/05, Laurent Laville <pear@laurent-laville.org> wrote: > Hi Ian, > > First, thanks to your comment, it is the first after one week since a > opened (officially) the proposal ! > > > Ian P. Christian wrote: > > > > Notice: Undefined property: HTML_QuickForm_advmultselect::$_jsPrefix > > in /var/www/ag5/lib/PEAR/HTML/QuickForm/advmultselect.php on line 89 > > > > Notice: Undefined property: HTML_QuickForm_advmultselect::$_jsPrefix > > in /var/www/ag5/lib/PEAR/HTML/QuickForm/advmultselect.php on line 67 > > > > Notice: Undefined property: HTML_QuickForm_advmultselect::$_jsPrefix > > in /var/www/ag5/lib/PEAR/HTML/QuickForm/advmultselect.php on line 68 > > > > Notice: Undefined property: HTML_QuickForm_advmultselect::$_jsPrefix > > in /var/www/ag5/lib/PEAR/HTML/QuickForm/advmultselect.php on line 67 > > > > Notice: Undefined property: HTML_QuickForm_advmultselect::$_jsPrefix > > in /var/www/ag5/lib/PEAR/HTML/QuickForm/advmultselect.php on line 68 > > > > This is whilst testing with your example code, I suggest you up your error > > reporting level :) > > Second, i'm surprised you got these errors notice. > Why ? > Because, when i write in development my code, i always fix it to E_ALL, > to avoid such surprises. > And to be honest, i think you have the wrong version of the source > class. Look the name (the i is missing) in the latest version, i've put > online the friday 3rd. Lines 67,68, 89 there are no references to > $_jsPrefix. And are you setting a js prefix in your test code? If you don't thes enotices should come up as (I believe) $this->_jsPrefix = null is the same as not setting it at all. > > Check the source online at: > > http://laurent-laville.org/?mod=pear&des=pepr&act=advmultiselect&show=source_cls > > > > > > Looks good though when this is fixed, will get my vote. > > > > Kind Regards, > > > > Thanks again for your participation. > -- Justin Patrin