Re: Re: cvs: pear /Services_JSON JSON.php Test-JSON.php

From: Date: Fri, 24 Mar 2006 16:50:49 +0000
Subject: Re: Re: cvs: pear /Services_JSON JSON.php Test-JSON.php
References: 1 2  Groups: php.pear.cvs php.pear.dev php.pear.qa 
Request: Send a blank email to pear-cvs+get-39301@lists.php.net to get a copy of this message
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 -- toggg

« previous php.pear.cvs (#39301) next »