[Phpmyadmin-devel] New PHPCS rule

[Michal Čihař]

Hi

Dne Sun, 15 Dec 2013 20:36:04 +0100
Hugues Peccatte <hugues.peccatte at gmail.com> napsal(a):

> What would you think about using the "nesting level" sniff please?
> This sniff permits to detect when there are too much nested levels like:
> if (...) {
>     if (...) {
>         if (...) {
>             if (...) {
>                 if (...) {
>                     ....
>                 }
>             }
>         }
>     }
> }
> 
> This kind of syntax often means that a refactoring is needed or a rewrite.

Lot of things which need refactoring are already detected by phpmd
(though it's only executed nightly as it takes quite a lot of time):

http://ci.phpmyadmin.net/job/phpMyAdmin/891/pmdResult/

-- 
	Michal Čihař | http://cihar.com | http://phpmyadmin.net
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: not available
URL: <http://lists.phpmyadmin.net/pipermail/developers/attachments/20131219/016b3f4d/attachment.sig>