syndication icon indicating copy to clipboard operation
syndication copied to clipboard

Published 20 hours ago • verified publisher icon Automattic

PHPCS Fixes

Open vaurdan opened this issue 3 years ago • 1 comments

Initial extensive PHPCS clean-up on the codebase. I tried to address all the WordPress standards where possible and ignored the ones that are valid use cases.

I didn't invest much time in the documentation warnings.

This PR is forked from #159, so it includes the OpenSSL implementation for encryption.

I also added a few TODOs along with the code where we should invest some time to improve (mostly missing nonces and class naming).

(WIP, class-wp-push-syndication-server.php is still untouched, and I need to pull the changes from #157)

vaurdan avatar Jul 06 '21 15:07 vaurdan

Thanks for the feedback, I have applied your suggestions.

I also added the missing PHPCS changes on the class-wp-push-syndication-server.php file, feel free to take a look!

vaurdan avatar Jul 09 '21 15:07 vaurdan