Add php codesniffer with wordpress standards #39

carlHandy · 2024-07-27T04:24:35Z

Ensures the code is following recommended coding practices

github-actions · 2024-07-27T05:31:09Z

PHP CodeSniffer Summary

......WW.... 12 / 12 (100%)



FILE: ...r/work/mmg-wp-plugin/mmg-wp-plugin/includes/class-mmg-checkout-payment.php
--------------------------------------------------------------------------------
FOUND 0 ERRORS AND 4 WARNINGS AFFECTING 3 LINES
--------------------------------------------------------------------------------
 251 | WARNING | base64_encode() can be used to obfuscate code which is
     |         | strongly discouraged. Please verify that the function is used

PHP CodeSniffer Results

Click to expand full report

......WW.... 12 / 12 (100%)



FILE: ...r/work/mmg-wp-plugin/mmg-wp-plugin/includes/class-mmg-checkout-payment.php
--------------------------------------------------------------------------------
FOUND 0 ERRORS AND 4 WARNINGS AFFECTING 3 LINES
--------------------------------------------------------------------------------
 251 | WARNING | base64_encode() can be used to obfuscate code which is
     |         | strongly discouraged. Please verify that the function is used
     |         | for benign reasons.
 349 | WARNING | base64_decode() can be used to obfuscate code which is
     |         | strongly discouraged. Please verify that the function is used
     |         | for benign reasons.
 405 | WARNING | Processing form data without nonce verification.
 405 | WARNING | Processing form data without nonce verification.
--------------------------------------------------------------------------------


FILE: /home/runner/work/mmg-wp-plugin/mmg-wp-plugin/tests/bootstrap.php
--------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
--------------------------------------------------------------------------------
 26 | WARNING | putenv() found. Changing configuration values at runtime is
    |         | strongly discouraged.
--------------------------------------------------------------------------------

Time: 1.42 secs; Memory: 18MB

github-actions · 2024-07-27T05:34:05Z

PHP CodeSniffer Summary

......WW.... 12 / 12 (100%)



FILE: ...r/work/mmg-wp-plugin/mmg-wp-plugin/includes/class-mmg-checkout-payment.php
--------------------------------------------------------------------------------
FOUND 0 ERRORS AND 4 WARNINGS AFFECTING 3 LINES
--------------------------------------------------------------------------------
 251 | WARNING | base64_encode() can be used to obfuscate code which is
     |         | strongly discouraged. Please verify that the function is used

PHP CodeSniffer Results

Click to expand full report

......WW.... 12 / 12 (100%)



FILE: ...r/work/mmg-wp-plugin/mmg-wp-plugin/includes/class-mmg-checkout-payment.php
--------------------------------------------------------------------------------
FOUND 0 ERRORS AND 4 WARNINGS AFFECTING 3 LINES
--------------------------------------------------------------------------------
 251 | WARNING | base64_encode() can be used to obfuscate code which is
     |         | strongly discouraged. Please verify that the function is used
     |         | for benign reasons.
 349 | WARNING | base64_decode() can be used to obfuscate code which is
     |         | strongly discouraged. Please verify that the function is used
     |         | for benign reasons.
 405 | WARNING | Processing form data without nonce verification.
 405 | WARNING | Processing form data without nonce verification.
--------------------------------------------------------------------------------


FILE: /home/runner/work/mmg-wp-plugin/mmg-wp-plugin/tests/bootstrap.php
--------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
--------------------------------------------------------------------------------
 26 | WARNING | putenv() found. Changing configuration values at runtime is
    |         | strongly discouraged.
--------------------------------------------------------------------------------

Time: 1.41 secs; Memory: 18MB

github-actions · 2024-07-27T05:34:41Z

PHPUnit Test Summary

Installing...
Running as single site... To run multisite, use -c tests/phpunit/multisite.xml
Not running ajax tests. To execute these, use --group ajax.
Not running ms-files tests. To execute these, use --group ms-files.
Not running external-http tests. To execute these, use --group external-http.
<div id="error"><p class="wpdberror"><strong>WordPress database error:</strong> [Table &#039;wordpress_test.wptests_wc_webhooks&#039; doesn&#039;t exist]<br /><code>SELECT webhook_id
				FROM wptests_wc_webhooks
				WHERE 1=1
				AND `status` = &#039;active&#039;

PHPUnit Test Results

Click to expand full report

Installing...
Running as single site... To run multisite, use -c tests/phpunit/multisite.xml
Not running ajax tests. To execute these, use --group ajax.
Not running ms-files tests. To execute these, use --group ms-files.
Not running external-http tests. To execute these, use --group external-http.
<div id="error"><p class="wpdberror"><strong>WordPress database error:</strong> [Table &#039;wordpress_test.wptests_wc_webhooks&#039; doesn&#039;t exist]<br /><code>SELECT webhook_id
				FROM wptests_wc_webhooks
				WHERE 1=1
				AND `status` = &#039;active&#039;
				
				
				
				
				
				ORDER BY webhook_id DESC</code></p></div>PHPUnit 9.6.20 by Sebastian Bergmann and contributors.

Runtime:       PHP 7.4.33
Configuration: phpunit.xml

Test 'Test_MMG_Checkout_Payment_Confirmation::test_handle_payment_confirmation' started
Test 'Test_MMG_Checkout_Payment_Confirmation::test_handle_payment_confirmation' ended
Test 'Test_MMG_Checkout_Payment::test_generate_checkout_url' started
{"success":false,"data":"Error generating checkout URL: Invalid security token"}

github-actions · 2024-07-27T05:38:59Z

PHP CodeSniffer Summary

......WW.... 12 / 12 (100%)



FILE: ...r/work/mmg-wp-plugin/mmg-wp-plugin/includes/class-mmg-checkout-payment.php
--------------------------------------------------------------------------------
FOUND 0 ERRORS AND 4 WARNINGS AFFECTING 3 LINES
--------------------------------------------------------------------------------
 251 | WARNING | base64_encode() can be used to obfuscate code which is
     |         | strongly discouraged. Please verify that the function is used

PHP CodeSniffer Results

Click to expand full report

......WW.... 12 / 12 (100%)



FILE: ...r/work/mmg-wp-plugin/mmg-wp-plugin/includes/class-mmg-checkout-payment.php
--------------------------------------------------------------------------------
FOUND 0 ERRORS AND 4 WARNINGS AFFECTING 3 LINES
--------------------------------------------------------------------------------
 251 | WARNING | base64_encode() can be used to obfuscate code which is
     |         | strongly discouraged. Please verify that the function is used
     |         | for benign reasons.
 349 | WARNING | base64_decode() can be used to obfuscate code which is
     |         | strongly discouraged. Please verify that the function is used
     |         | for benign reasons.
 405 | WARNING | Processing form data without nonce verification.
 405 | WARNING | Processing form data without nonce verification.
--------------------------------------------------------------------------------


FILE: /home/runner/work/mmg-wp-plugin/mmg-wp-plugin/tests/bootstrap.php
--------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
--------------------------------------------------------------------------------
 26 | WARNING | putenv() found. Changing configuration values at runtime is
    |         | strongly discouraged.
--------------------------------------------------------------------------------

Time: 1.48 secs; Memory: 18MB

github-actions · 2024-07-27T05:39:31Z

PHPUnit Test Summary

Installing...
Running as single site... To run multisite, use -c tests/phpunit/multisite.xml
Not running ajax tests. To execute these, use --group ajax.
Not running ms-files tests. To execute these, use --group ms-files.
Not running external-http tests. To execute these, use --group external-http.
<div id="error"><p class="wpdberror"><strong>WordPress database error:</strong> [Table &#039;wordpress_test.wptests_wc_webhooks&#039; doesn&#039;t exist]<br /><code>SELECT webhook_id
				FROM wptests_wc_webhooks
				WHERE 1=1
				AND `status` = &#039;active&#039;

PHPUnit Test Results

Click to expand full report

Installing...
Running as single site... To run multisite, use -c tests/phpunit/multisite.xml
Not running ajax tests. To execute these, use --group ajax.
Not running ms-files tests. To execute these, use --group ms-files.
Not running external-http tests. To execute these, use --group external-http.
<div id="error"><p class="wpdberror"><strong>WordPress database error:</strong> [Table &#039;wordpress_test.wptests_wc_webhooks&#039; doesn&#039;t exist]<br /><code>SELECT webhook_id
				FROM wptests_wc_webhooks
				WHERE 1=1
				AND `status` = &#039;active&#039;
				
				
				
				
				
				ORDER BY webhook_id DESC</code></p></div>PHPUnit 9.6.20 by Sebastian Bergmann and contributors.

Runtime:       PHP 7.4.33
Configuration: phpunit.xml

Test 'Test_MMG_Checkout_Payment_Confirmation::test_handle_payment_confirmation' started
Test 'Test_MMG_Checkout_Payment_Confirmation::test_handle_payment_confirmation' ended
Test 'Test_MMG_Checkout_Payment::test_generate_checkout_url' started
{"success":false,"data":"Error generating checkout URL: Invalid security token"}

github-actions · 2024-07-27T06:43:15Z

PHP CodeSniffer Summary

......WW.... 12 / 12 (100%)



FILE: ...r/work/mmg-wp-plugin/mmg-wp-plugin/includes/class-mmg-checkout-payment.php
--------------------------------------------------------------------------------
FOUND 0 ERRORS AND 4 WARNINGS AFFECTING 3 LINES
--------------------------------------------------------------------------------
 251 | WARNING | base64_encode() can be used to obfuscate code which is
     |         | strongly discouraged. Please verify that the function is used

PHP CodeSniffer Results

Click to expand full report

......WW.... 12 / 12 (100%)



FILE: ...r/work/mmg-wp-plugin/mmg-wp-plugin/includes/class-mmg-checkout-payment.php
--------------------------------------------------------------------------------
FOUND 0 ERRORS AND 4 WARNINGS AFFECTING 3 LINES
--------------------------------------------------------------------------------
 251 | WARNING | base64_encode() can be used to obfuscate code which is
     |         | strongly discouraged. Please verify that the function is used
     |         | for benign reasons.
 349 | WARNING | base64_decode() can be used to obfuscate code which is
     |         | strongly discouraged. Please verify that the function is used
     |         | for benign reasons.
 405 | WARNING | Processing form data without nonce verification.
 405 | WARNING | Processing form data without nonce verification.
--------------------------------------------------------------------------------


FILE: /home/runner/work/mmg-wp-plugin/mmg-wp-plugin/tests/bootstrap.php
--------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
--------------------------------------------------------------------------------
 26 | WARNING | putenv() found. Changing configuration values at runtime is
    |         | strongly discouraged.
--------------------------------------------------------------------------------

Time: 1.46 secs; Memory: 18MB

github-actions · 2024-07-27T06:43:48Z

PHPUnit Test Summary

Installing...
Running as single site... To run multisite, use -c tests/phpunit/multisite.xml
Not running ajax tests. To execute these, use --group ajax.
Not running ms-files tests. To execute these, use --group ms-files.
Not running external-http tests. To execute these, use --group external-http.
<div id="error"><p class="wpdberror"><strong>WordPress database error:</strong> [Table &#039;wordpress_test.wptests_wc_webhooks&#039; doesn&#039;t exist]<br /><code>SELECT webhook_id
				FROM wptests_wc_webhooks
				WHERE 1=1
				AND `status` = &#039;active&#039;

PHPUnit Test Results

Click to expand full report

Installing...
Running as single site... To run multisite, use -c tests/phpunit/multisite.xml
Not running ajax tests. To execute these, use --group ajax.
Not running ms-files tests. To execute these, use --group ms-files.
Not running external-http tests. To execute these, use --group external-http.
<div id="error"><p class="wpdberror"><strong>WordPress database error:</strong> [Table &#039;wordpress_test.wptests_wc_webhooks&#039; doesn&#039;t exist]<br /><code>SELECT webhook_id
				FROM wptests_wc_webhooks
				WHERE 1=1
				AND `status` = &#039;active&#039;
				
				
				
				
				
				ORDER BY webhook_id DESC</code></p></div>PHPUnit 9.6.20 by Sebastian Bergmann and contributors.

Runtime:       PHP 7.4.33
Configuration: phpunit.xml

Test 'Test_MMG_Checkout_Payment_Confirmation::test_handle_payment_confirmation' started
Test 'Test_MMG_Checkout_Payment_Confirmation::test_handle_payment_confirmation' ended
Test 'Test_MMG_Checkout_Payment::test_generate_checkout_url' started
{"success":false,"data":"Error generating checkout URL: Invalid security token"}

github-actions · 2024-07-27T06:47:57Z

PHP CodeSniffer Summary

......WW.... 12 / 12 (100%)



FILE: ...r/work/mmg-wp-plugin/mmg-wp-plugin/includes/class-mmg-checkout-payment.php
--------------------------------------------------------------------------------
FOUND 0 ERRORS AND 4 WARNINGS AFFECTING 3 LINES
--------------------------------------------------------------------------------
 251 | WARNING | base64_encode() can be used to obfuscate code which is
     |         | strongly discouraged. Please verify that the function is used

PHP CodeSniffer Results

Click to expand full report

......WW.... 12 / 12 (100%)



FILE: ...r/work/mmg-wp-plugin/mmg-wp-plugin/includes/class-mmg-checkout-payment.php
--------------------------------------------------------------------------------
FOUND 0 ERRORS AND 4 WARNINGS AFFECTING 3 LINES
--------------------------------------------------------------------------------
 251 | WARNING | base64_encode() can be used to obfuscate code which is
     |         | strongly discouraged. Please verify that the function is used
     |         | for benign reasons.
 349 | WARNING | base64_decode() can be used to obfuscate code which is
     |         | strongly discouraged. Please verify that the function is used
     |         | for benign reasons.
 405 | WARNING | Processing form data without nonce verification.
 405 | WARNING | Processing form data without nonce verification.
--------------------------------------------------------------------------------


FILE: /home/runner/work/mmg-wp-plugin/mmg-wp-plugin/tests/bootstrap.php
--------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
--------------------------------------------------------------------------------
 26 | WARNING | putenv() found. Changing configuration values at runtime is
    |         | strongly discouraged.
--------------------------------------------------------------------------------

Time: 1.42 secs; Memory: 18MB

github-actions · 2024-07-27T06:48:41Z

PHPUnit Test Summary

Installing...
Running as single site... To run multisite, use -c tests/phpunit/multisite.xml
Not running ajax tests. To execute these, use --group ajax.
Not running ms-files tests. To execute these, use --group ms-files.
Not running external-http tests. To execute these, use --group external-http.
<div id="error"><p class="wpdberror"><strong>WordPress database error:</strong> [Table &#039;wordpress_test.wptests_wc_webhooks&#039; doesn&#039;t exist]<br /><code>SELECT webhook_id
				FROM wptests_wc_webhooks
				WHERE 1=1
				AND `status` = &#039;active&#039;

PHPUnit Test Results

Click to expand full report

Installing...
Running as single site... To run multisite, use -c tests/phpunit/multisite.xml
Not running ajax tests. To execute these, use --group ajax.
Not running ms-files tests. To execute these, use --group ms-files.
Not running external-http tests. To execute these, use --group external-http.
<div id="error"><p class="wpdberror"><strong>WordPress database error:</strong> [Table &#039;wordpress_test.wptests_wc_webhooks&#039; doesn&#039;t exist]<br /><code>SELECT webhook_id
				FROM wptests_wc_webhooks
				WHERE 1=1
				AND `status` = &#039;active&#039;
				
				
				
				
				
				ORDER BY webhook_id DESC</code></p></div>PHPUnit 9.6.20 by Sebastian Bergmann and contributors.

Runtime:       PHP 7.4.33
Configuration: phpunit.xml

Test 'Test_MMG_Checkout_Payment_Confirmation::test_handle_payment_confirmation' started
Test 'Test_MMG_Checkout_Payment_Confirmation::test_handle_payment_confirmation' ended
Test 'Test_MMG_Checkout_Payment::test_generate_checkout_url' started
{"success":false,"data":"Error generating checkout URL: Invalid security token"}

carlHandy added 15 commits July 27, 2024 00:21

add code sniffer

e2d47d5

lint: fix formatting

48b4eb0

lint: fix violations in main.php and uninstall.php

1f105d4

lint: fix violations in tests dir

962f376

lint: fix violations with settings class

4f9414a

phpcs: class-wc-mmg-gateway

8c4d68a

phpcs: activator

d410cd6

phpcs: deactivator

5a74ba6

phpcs: checkout-payment class

400eb88

phpcs: fix all errors

85bf943

only fail phpcs on errors

c12f0a6

post phpcs results in PR comment

b5de9f4

install cspr

43e6c67

print results to txt

bbe1399

update job permissions

00ce97a

post unit test results to comments

d2cf81f

remove wp-content from final build

84a198a

carlHandy added 5 commits July 27, 2024 01:54

fail job if tests fail

2b72ab0

fail test if checkout url is empty

bddeeed

add rewrite rules to activator

9ffcf44

phpcs: fix errors

a6df41b

fix merge conflict

f56672c

explicitly zip

d39ee2a

carlHandy merged commit 560d99b into main Jul 27, 2024
4 checks passed

carlHandy deleted the add-code-sniffer branch July 27, 2024 06:50

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Add php codesniffer with wordpress standards #39

Add php codesniffer with wordpress standards #39

carlHandy commented Jul 27, 2024

github-actions bot commented Jul 27, 2024

github-actions bot commented Jul 27, 2024

github-actions bot commented Jul 27, 2024

github-actions bot commented Jul 27, 2024

github-actions bot commented Jul 27, 2024

github-actions bot commented Jul 27, 2024

github-actions bot commented Jul 27, 2024

github-actions bot commented Jul 27, 2024

github-actions bot commented Jul 27, 2024

Add php codesniffer with wordpress standards #39

Add php codesniffer with wordpress standards #39

Conversation

carlHandy commented Jul 27, 2024

github-actions bot commented Jul 27, 2024

PHP CodeSniffer Summary

PHP CodeSniffer Results

github-actions bot commented Jul 27, 2024

PHP CodeSniffer Summary

PHP CodeSniffer Results

github-actions bot commented Jul 27, 2024

PHPUnit Test Summary

PHPUnit Test Results

github-actions bot commented Jul 27, 2024

PHP CodeSniffer Summary

PHP CodeSniffer Results

github-actions bot commented Jul 27, 2024

PHPUnit Test Summary

PHPUnit Test Results

github-actions bot commented Jul 27, 2024

PHP CodeSniffer Summary

PHP CodeSniffer Results

github-actions bot commented Jul 27, 2024

PHPUnit Test Summary

PHPUnit Test Results

github-actions bot commented Jul 27, 2024

PHP CodeSniffer Summary

PHP CodeSniffer Results

github-actions bot commented Jul 27, 2024

PHPUnit Test Summary

PHPUnit Test Results