Re: [Fwd: Re: [PEAR-DEV] Re: [PEAR-CVS] cvs: pear /Services_JSON JSON.php Test-JSON.php]

And before Pierre stops me: I voted +1 , we were 8 to do so , 3 were with conditions , not obviously about the name. Dan and I comented the commit. Pierre controled. Regards -- toggg bertrand Gugger wrote: > How you can present things > > -------- Original Message -------- > Subject: Re: [PEAR-DEV] Re: [PEAR-CVS] cvs: pear /Services_JSON > JSON.php Test-JSON.php > Date: Fri, 24 Mar 2006 17:50:49 +0100 > From: toggg@php.net (bertrand Gugger) > Reply-To: toggg@php.net > To: Daniel Convissor <danielc@analysisandsolutions.com> > CC: Michal Migurski <migurski@php.net>, pear-cvs@lists.php.net, > PEAR Dev List <pear-dev@lists.php.net>, PEAR QA List > <pear-qa@lists.php.net> > Newsgroups: php.pear.cvs,php.pear.dev,php.pear.qa > References: <cvsmigurski1143179864@cvsserver> > <20060324154010.GA1212@panix.com> > > Bonjour, > Daniel Convissor wrote: > >> On Fri, Mar 24, 2006 at 05:57:44AM -0000, Michal Migurski wrote: >> >> >>> migurski Fri Mar 24 05:57:44 2006 UTC >>> >>> Added files: /pear/Services_JSON JSON.php >>> Test-JSON.php >> >> >> Cool. Glad to see this getting into CVS! >> >> Three of the votes for this package expressed reservations with >> putting this package being in the Services category: >> >> http://pear.php.net/pepr/pepr-vote-show.php?id=198&handle=wenz >> http://pear.php.net/pepr/pepr-vote-show.php?id=198&handle=justinpatrin >> http://pear.php.net/pepr/pepr-vote-show.php?id=198&handle=bolk >> >> I share those reservations. Perhaps put it in the Text or HTML >> categories? Please, a consensus on where to put this should be >> reached before the first release. >> >> Thanks, >> >> --Dan >> >> >> > I was also happy to see this commit. > Congrats Michal ! > > As for other classes, > I really like to see client executable code included. > > For the category, > if javascript belongs to html , then HTML category. > That should be, as javascript belongs to public. > Just not the server of it. > > I would add a call for a new role by pear installer , client script , as > javascript > > Should be in some public area. > > Regards