[PEPr] +1 for Payment::PagamentoCerto

From: Date: Tue, 06 Jan 2009 11:20:13 +0000
Subject: [PEPr] +1 for Payment::PagamentoCerto
References: 1  Groups: php.pear.dev 
Request: Send a blank email to pear-dev+get-51398@lists.php.net to get a copy of this message
Kazuyoshi Tlacaelel (http://pear.php.net/user/ktlacaelel) has voted +1 on the proposal for Payment::PagamentoCerto. Proposal information: http://pear.php.net/pepr/pepr-proposal-show.php?id=583 Vote information: http://pear.php.net/pepr/pepr-vote-show.php?id=583&handle=ktlacaelel This vote is conditional. The condition is: Hello Pedro! 1. ran codesniffer and got the following results thought you may want to take a look at those! $ phpcs Payment/ | grep -i '^\w' FILE: ...u/Desktop/Payment_PagamentoCerto-0.2.0/Payment/PagamentoCerto/Order.php FOUND 10 ERROR(S) AND 0 WARNING(S) AFFECTING 10 LINE(S) FILE: .../Payment_PagamentoCerto-0.2.0/Payment/PagamentoCerto/PagamentoCerto.php FOUND 2 ERROR(S) AND 2 WARNING(S) AFFECTING 4 LINE(S) FILE: ...top/Payment_PagamentoCerto-0.2.0/Payment/PagamentoCerto/Transaction.php FOUND 2 ERROR(S) AND 0 WARNING(S) AFFECTING 2 LINE(S) FILE: ...ktop/Payment_PagamentoCerto-0.2.0/Payment/PagamentoCerto/XmlBuilder.php FOUND 9 ERROR(S) AND 4 WARNING(S) AFFECTING 11 LINE(S) ----------------- 2. properties are camelcased ( maybe create dto data access objects ) with getters and setters? 3. code is readable, easy to understand and has tests. that is great! -- Sent by PEPr, the automatic proposal system at http://pear.php.net

« previous php.pear.dev (#51398) next »