Re: Net_Dict unmaintained

From: Date: Mon, 14 Dec 2009 04:52:50 +0000
Subject: Re: Net_Dict unmaintained
References: 1 2 3 4 5 6  Groups: php.pear.qa 
Request: Send a blank email to pear-qa+get-5542@lists.php.net to get a copy of this message
On Sun, Dec 13, 2009 at 09:40:39AM +0000, Ken Guest wrote: > There is something to do with Net_Dict, you could bring the code for > it up to Standard: > > > http://pear.php.net/bugs/search.php?cmd=display&package_name[]=Net_Dict&status=OpenFeedback&bug_type=Bugs > > k. Those are easy enough - find the diff to http://svn.php.net/repository/pear/packages/Net_Dict/trunk/Net attached. There were two I didn't "fix": 350 | ERROR | Method name "Net_Dict::SASLAuth" is not in camel caps format 363 | ERROR | Method name "Net_Dict::SASLResp" is not in camel caps format I think the code sniffer is wrong here - SASLFoo is the right name for these methods IMO, because they're implementing SASL. Probably the sweetest irony in this process is that PHP_CodeSniffer itself does not pass its own test suite (currently showing 20 errors and an impressive 1,428 warnings on the PEAR QA Statuss overview).

Attachment: [text/x-diff] dict.patch
« previous php.pear.qa (#5542) next »