New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Update Code Obfuscation Check to display multiple errors #366

Open

ernilambar wants to merge 2 commits into WordPress:trunk from ernilambar:363-fix-obfuscation

Member

ernilambar commented Dec 29, 2023

Fixes #363

Added new static methods in Abstract_File_Check
- files_preg_match() - Returns multiple files unlike file_preg_match()
- Similarly files_str_contains
Updated methods in Code_Obfuscation_Check to handle array of files instead of a single file
files_preg_match() returns array of file and match. This data could be useful in the future enhancements.
Eg:

Screen Shot 2023-12-29 at 1 41 55 PM

CLI output:

Screen Shot 2023-12-29 at 1 32 26 PM

ernilambar force-pushed the 363-fix-obfuscation branch 4 times, most recently from fb65781 to 7373989 Compare

December 29, 2023 09:39

davidperezgar added the [Team] Plugin Review label

ernilambar added 2 commits

July 2, 2024 10:33


          Update Code Obfuscation Check to display multiple errors

c56a2fb


          Update tests for Code Obfuscation

bd5b13c

ernilambar force-pushed the 363-fix-obfuscation branch from 7373989 to bd5b13c Compare

July 2, 2024 04:48

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment