>>> [!note] Migrated issue <!-- Drupal.org comment --> <!-- Migrated from issue #3458455. --> Reported by: [hetal.solanki](https://www.drupal.org/user/3702323) Related to !2 >>> <h3 id="summary-problem-motivation">Problem/Motivation</h3> <p>command -:<br> phpcs --standard=Drupal --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml web/modules/contrib/vpn_login/</p> <p>FILE: /var/www/html/d10/web/modules/contrib/vpn_login/src/EventSubscriber/LoginSubscriber.php<br> --------------------------------------------------------------------------------------------------------------------------------------------<br> FOUND 11 ERRORS AND 1 WARNING AFFECTING 12 LINES<br> --------------------------------------------------------------------------------------------------------------------------------------------<br> 7 | ERROR | [x] Use statements should be sorted alphabetically. The first wrong one is Symfony\Component\HttpKernel\Event\RequestEvent.<br> 8 | WARNING | [x] Unused use statement<br> 14 | ERROR | [x] Missing class doc comment<br> 16 | ERROR | [ ] Missing member variable doc comment<br> 18 | ERROR | [x] Missing function doc comment<br> 22 | ERROR | [x] Missing function doc comment<br> 27 | ERROR | [x] Missing function doc comment<br> 52 | ERROR | [x] Expected newline after closing brace<br> 67 | ERROR | [x] Separate the @param and @return sections by a blank line.<br> 80 | ERROR | [x] Expected newline after closing brace<br> 83 | ERROR | [x] Expected 1 blank line after function; 0 found<br> 84 | ERROR | [x] The closing brace for the class must have an empty line before it<br> --------------------------------------------------------------------------------------------------------------------------------------------<br> PHPCBF CAN FIX THE 11 MARKED SNIFF VIOLATIONS AUTOMATICALLY<br> --------------------------------------------------------------------------------------------------------------------------------------------</p> <p>FILE: /var/www/html/d10/web/modules/contrib/vpn_login/src/EventSubscriber/RouteSubscriber.php<br> ---------------------------------------------------------------------------------------------<br> FOUND 4 ERRORS AFFECTING 3 LINES<br> ---------------------------------------------------------------------------------------------<br> 10 | ERROR | [x] Missing class doc comment<br> 52 | ERROR | [x] Expected 1 blank line after function; 0 found<br> 53 | ERROR | [x] Expected 1 newline at end of file; 2 found<br> 53 | ERROR | [x] The closing brace for the class must have an empty line before it<br> ---------------------------------------------------------------------------------------------<br> PHPCBF CAN FIX THE 4 MARKED SNIFF VIOLATIONS AUTOMATICALLY<br> ---------------------------------------------------------------------------------------------</p> <p>FILE: /var/www/html/d10/web/modules/contrib/vpn_login/src/Form/VpnLoginConfigForm.php<br> -------------------------------------------------------------------------------------<br> FOUND 4 ERRORS AFFECTING 4 LINES<br> -------------------------------------------------------------------------------------<br> 8 | ERROR | [x] Missing class doc comment<br> 99 | ERROR | [x] Expected newline after closing brace<br> 116 | ERROR | [x] Expected 1 blank line after function; 0 found<br> 117 | ERROR | [x] The closing brace for the class must have an empty line before it<br> -------------------------------------------------------------------------------------<br> PHPCBF CAN FIX THE 4 MARKED SNIFF VIOLATIONS AUTOMATICALLY<br> -------------------------------------------------------------------------------------</p> <p>FILE: /var/www/html/d10/web/modules/contrib/twig_tweak/twig_tweak.info.yml<br> -------------------------------------------------------------------------------------------------------------<br> FOUND 0 ERRORS AND 3 WARNINGS AFFECTING 1 LINE<br> -------------------------------------------------------------------------------------------------------------<br> 1 | WARNING | Remove "project" from the info file, it will be added by drupal.org packaging automatically<br> 1 | WARNING | Remove "datestamp" from the info file, it will be added by drupal.org packaging automatically<br> 1 | WARNING | Remove "version" from the info file, it will be added by drupal.org packaging automatically<br> -------------------------------------------------------------------------------------------------------------</p> <p>FILE: /var/www/html/d10/web/modules/contrib/twig_tweak/tests/twig_tweak_test/twig_tweak_test.info.yml<br> -------------------------------------------------------------------------------------------------------------<br> FOUND 0 ERRORS AND 3 WARNINGS AFFECTING 1 LINE<br> -------------------------------------------------------------------------------------------------------------<br> 1 | WARNING | Remove "project" from the info file, it will be added by drupal.org packaging automatically<br> 1 | WARNING | Remove "datestamp" from the info file, it will be added by drupal.org packaging automatically<br> 1 | WARNING | Remove "version" from the info file, it will be added by drupal.org packaging automatically<br> -------------------------------------------------------------------------------------------------------------</p> <p>Time: 1.09 secs; Memory: 14MB</p> <h4 id="summary-steps-reproduce">Steps to reproduce</h4> <p>Run PHPCS on Config Pages module.</p> <h3 id="summary-proposed-resolution">Proposed resolution</h3> <p>Fixed the coding standards.</p> <h3 id="summary-remaining-tasks">Remaining tasks</h3> <p>Submit the patch either MR.</p>