enhance: revamp transaction summary #640
phpcs.yml
on: pull_request
Run PHPCS inspection
2m 31s
Annotations
10 errors and 15 warnings
Run PHPCS inspection:
includes/Admin.php#L89
A function call to __() with texts containing placeholders was found, but was not accompanied by a "translators:" comment on the line above to clarify the meaning of the placeholders.
|
Run PHPCS inspection:
includes/Admin.php#L90
Multiple placeholders in translatable strings should be ordered. Expected "%1$s, %2$s, %3$s, %4$s", but got "%s, %s, %s, %s" in '%sThis post contains a sensitive short-code %s, that may allow others to sign-up with distinguished roles. If unsure, remove the short-code before publishing (recommended) %sas this may be exploited as a security vulnerability.%s'.
|
Run PHPCS inspection:
includes/Admin.php#L157
Not using strict comparison for in_array; supply true for $strict argument.
|
Run PHPCS inspection:
includes/Admin/Menu.php#L151
Scripts must be registered/enqueued via wp_enqueue_script()
|
Run PHPCS inspection:
includes/Admin/Menu.php#L366
Opening PHP tag must be on a line by itself
|
Run PHPCS inspection:
includes/Admin/Menu.php#L366
Opening PHP tag must be on a line by itself
|
Run PHPCS inspection:
includes/Admin/Menu.php#L367
Multi-line function call not indented correctly; expected 24 spaces but found 20
|
Run PHPCS inspection:
includes/Admin/Menu.php#L368
Line indented incorrectly; expected at least 16 tabs, found 5
|
Run PHPCS inspection:
includes/Admin/Menu.php#L368
Closing PHP tag must be on a line by itself
|
Run PHPCS inspection:
includes/Admin/Subscription.php#L409
Line indented incorrectly; expected at least 2 tabs, found 0
|
Run PHPCS inspection
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, pat-s/[email protected]. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
|
Run PHPCS inspection
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v2, pat-s/[email protected]. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|
Run PHPCS inspection
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
|
Run PHPCS inspection
The `save-state` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
|
Run PHPCS inspection
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
|
Run PHPCS inspection:
includes/Admin.php#L158
This comment is 54% valid code; is this commented out code?
|
Run PHPCS inspection:
includes/Admin.php#L159
wp_redirect() found. Using wp_safe_redirect(), along with the "allowed_redirect_hosts" filter if needed, can help avoid any chances of malicious redirects within code. It is also important to remember to call exit() after a redirect so that no other unwanted code is executed.
|
Run PHPCS inspection:
includes/Admin/Menu.php#L21
base64_encode() can be used to obfuscate code which is strongly discouraged. Please verify that the function is used for benign reasons.
|
Run PHPCS inspection:
includes/Admin/Menu.php#L44
This comment is 57% valid code; is this commented out code?
|
Run PHPCS inspection:
includes/Admin/Menu.php#L226
The method parameter $post_ID is never used
|
Run PHPCS inspection:
includes/Admin/Subscription.php#L88
Use placeholders and $wpdb->prepare(); found $sql
|
Run PHPCS inspection:
includes/Admin/Subscription.php#L149
The method parameter $userdata is never used
|
Run PHPCS inspection:
includes/Admin/Subscription.php#L149
The method parameter $form_id is never used
|
Run PHPCS inspection:
includes/Admin/Subscription.php#L149
The method parameter $form_settings is never used
|
Run PHPCS inspection:
includes/Admin/Subscription.php#L194
Processing form data without nonce verification.
|