Add performance related rules and no unlimited queries sniff - Githubissues

10up / phpcs-composer

Official 10up PHPCS rules.

MIT License

48 stars 8 forks source link

Add performance related rules and no unlimited queries sniff #41

Closed pabamato closed 1 year ago

pabamato commented 1 year ago

Description of the Change

Add missing DB and performance related sniffs
Add custom sniff to flag queries using -1 (unlimited) for posts_per_page
Closes # https://github.com/10up/phpcs-composer/issues/40

How to test the Change

Changelog Entry

Added - PostsPerPageNoUnlimited sniff. Detect usage of -1 on posts_per_page This feature has been removed from WordPress and VIP-go CS: https://github.com/WordPress/WordPress-Coding-Standards/pull/1531#pullrequestreview-175160708

Added - Import the following rules from WordPress CS:
WordPress.DB.PreparedSQL, WordPress.DB.DirectDatabaseQuery, WordPress.DB.SlowDBQuery

Credits

Checklist:
- [ x] I agree to follow this project's Code of Conduct.
- [ ] I have updated the documentation accordingly.
- [ ] I have added tests to cover my change.
- [ ] All new and existing tests pass.