extdn / extdn-phpcs

PHP CodeSniffer rules
81 stars 10 forks source link

Rule: Do not use debugging statements #53

Closed jissereitsma closed 1 year ago

jissereitsma commented 6 years ago

Description

In a production-ready extension, there shouldn't be any functions like die(), print_r(), var_export() or var_dump(). This PR adds a new rule that checks upon this. This is a very simple straightforward PHPCS implementation.

One thing that I foresee as an issue already is that I sometimes log things away by using var_export($x, true). I'm not sure if that's a lack of quality or simply a nifty feature.

Related issues

None

Sniff checklist