Fix the issues reported by phpcs

FILE: /Users/roberttabigue/Projects/drupalorgissues/docroot/modules/contrib/spamspan/spamspan.install ----------------------------------------------------------------------------------------------------- FOUND 1 ERROR AFFECTING 1 LINE ----------------------------------------------------------------------------------------------------- 3 | ERROR | Missing short description in doc comment ----------------------------------------------------------------------------------------------------- FILE: /Users/roberttabigue/Projects/drupalorgissues/docroot/modules/contrib/spamspan/js/spamspan.js --------------------------------------------------------------------------------------------------- FOUND 14 ERRORS AFFECTING 3 LINES --------------------------------------------------------------------------------------------------- 33 | ERROR | [x] Expected 1 space before "?"; 0 found 33 | ERROR | [x] Expected 1 space after "?"; 0 found 59 | ERROR | [x] Expected 1 space before "="; 0 found 59 | ERROR | [x] Expected 1 space after "="; 0 found 66 | ERROR | [x] Expected 1 space before "<"; 0 found 66 | ERROR | [x] Expected 1 space after "<"; 0 found 66 | ERROR | [x] Expected 1 space before ">"; 0 found 66 | ERROR | [x] Expected 1 space after ">"; 0 found 66 | ERROR | [x] Expected 1 space before "<"; 0 found 66 | ERROR | [x] Expected 1 space after "<"; 0 found 66 | ERROR | [x] Expected 1 space before "/"; 0 found 66 | ERROR | [x] Expected 1 space after "/"; 0 found 66 | ERROR | [x] Expected 1 space before ">"; 0 found 66 | ERROR | [x] Expected 1 space after ">"; 0 found --------------------------------------------------------------------------------------------------- PHPCBF CAN FIX THE 14 MARKED SNIFF VIOLATIONS AUTOMATICALLY --------------------------------------------------------------------------------------------------- Time: 706ms; Memory: 14MB

Comments & Activities

Issue created by @roberttabigue
Status changed to Needs review over 1 year ago6:59am 24 July 2023
Open in Jenkins → Open on Drupal.org →
Core: 10.0.7 + Environment: PHP 8.2 & MySQL 8
last update over 1 year ago
16 pass, 2 fail
Comment over 1 year ago →
🇮🇳India mrinalini9 New Delhi
Added patch for the above phpcs issues, please review it.

Thanks!
Comment over 1 year ago →
System Message
The last submitted patch, 2: 3376155-2.patch, failed testing. View results →
Status changed to Needs work over 1 year ago7:44am 24 July 2023
Comment over 1 year ago →
Aswathy25
phpcbf --standard=Drupal --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml /path/to/drupal/example_module
Comment over 1 year ago →
🇩🇪Germany Anybody Porta Westfalica
Please fix in combination with: 📌 Fix JS code style errors RTBC

Provide a MR with all fixes to review.
Assigned to sourabhjain
Comment over 1 year ago →
🇮🇳India sourabhjain
Let me work on this.
Open on Drupal.org →
Core: 10.0.7 + Environment: PHP 8.2 & MySQL 8
last update over 1 year ago
Not currently mergeable.
@sourabhjain opened merge request.
Open in Jenkins → Open on Drupal.org →
Core: 10.0.7 + Environment: PHP 8.2 & MySQL 8
last update over 1 year ago
16 pass, 2 fail
Issue was unassigned.
Status changed to Needs review over 1 year ago6:36am 27 July 2023
Comment over 1 year ago →
🇮🇳India sourabhjain
@anybody created a PR including this issue changes https://www.drupal.org/project/spamspan/issues/3374657 📌 Fix JS code style errors RTBC
Please review.
Status changed to RTBC over 1 year ago4:30pm 27 July 2023
Comment over 1 year ago →
🇵🇭Philippines roberttabigue
Hi,

I applied the latest MR to the SpamSpan filter module against 3.x-dev on Drupal 9.5.10 and confirmed all PHPCS errors have been fixed.

I re-ran this command on the module:
phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,js,info,txt,md,yml,twig spamspan/

Please see the attached file for reference.

I'm moving this now to RTBC.

Thank you!

Fix the issues reported by phpcs

Problem/Motivation

Steps to reproduce

Comments & Activities