Fix the issues reported by phpcs

FILE: /Users/studenttrainees/New/drupalorgsite/docroot/modules/contrib/virustotal/virustotal.install --------------------------------------------------------------------------------------------------------------- FOUND 1 ERROR AFFECTING 1 LINE --------------------------------------------------------------------------------------------------------------- 34 | ERROR | The array declaration extends to column 84 (the limit is 80). The array content should be split | | up over multiple lines --------------------------------------------------------------------------------------------------------------- FILE: /Users/studenttrainees/New/drupalorgsite/docroot/modules/contrib/virustotal/README.md ------------------------------------------------------------------------------------------- FOUND 0 ERRORS AND 4 WARNINGS AFFECTING 4 LINES ------------------------------------------------------------------------------------------- 16 | WARNING | Line exceeds 80 characters; contains 86 characters 19 | WARNING | Line exceeds 80 characters; contains 81 characters 73 | WARNING | Line exceeds 80 characters; contains 83 characters 89 | WARNING | Line exceeds 80 characters; contains 83 characters ------------------------------------------------------------------------------------------- FILE: .../studenttrainees/New/drupalorgsite/docroot/modules/contrib/virustotal/src/Form/VirusTotalSettingsForm.php --------------------------------------------------------------------------------------------------------------- FOUND 2 ERRORS AND 1 WARNING AFFECTING 3 LINES --------------------------------------------------------------------------------------------------------------- 14 | WARNING | The class short comment should describe what the class does and not simply repeat the class | | name 17 | ERROR | Missing member variable doc comment 18 | ERROR | Missing member variable doc comment --------------------------------------------------------------------------------------------------------------- FILE: ...s/studenttrainees/New/drupalorgsite/docroot/modules/contrib/virustotal/src/Form/VirusTotalCommentForm.php --------------------------------------------------------------------------------------------------------------- FOUND 1 ERROR AND 1 WARNING AFFECTING 2 LINES --------------------------------------------------------------------------------------------------------------- 13 | WARNING | The class short comment should describe what the class does and not simply repeat the class | | name 16 | ERROR | Missing member variable doc comment --------------------------------------------------------------------------------------------------------------- FILE: ...s/studenttrainees/New/drupalorgsite/docroot/modules/contrib/virustotal/src/Form/VirusTotalScanURLForm.php --------------------------------------------------------------------------------------------------------------- FOUND 3 ERRORS AFFECTING 3 LINES --------------------------------------------------------------------------------------------------------------- 19 | ERROR | Missing member variable doc comment 20 | ERROR | Missing member variable doc comment 21 | ERROR | Missing member variable doc comment --------------------------------------------------------------------------------------------------------------- FILE: ...rs/studenttrainees/New/drupalorgsite/docroot/modules/contrib/virustotal/src/Form/VirusTotalReportForm.php --------------------------------------------------------------------------------------------------------------- FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE --------------------------------------------------------------------------------------------------------------- 62 | WARNING | [x] Inline @var declarations should use the /** */ delimiters --------------------------------------------------------------------------------------------------------------- PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY --------------------------------------------------------------------------------------------------------------- FILE: ...nttrainees/New/drupalorgsite/docroot/modules/contrib/virustotal/src/Form/VirusTotalReportSettingsForm.php --------------------------------------------------------------------------------------------------------------- FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE --------------------------------------------------------------------------------------------------------------- 12 | WARNING | The class short comment should describe what the class does and not simply repeat the class | | name --------------------------------------------------------------------------------------------------------------- FILE: ...trainees/New/drupalorgsite/docroot/modules/contrib/virustotal/src/Plugin/VirusTotal/VirusTotalScanner.php --------------------------------------------------------------------------------------------------------------- FOUND 1 ERROR AFFECTING 1 LINE --------------------------------------------------------------------------------------------------------------- 459 | ERROR | [x] Each PHP statement must be on a line by itself --------------------------------------------------------------------------------------------------------------- PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY --------------------------------------------------------------------------------------------------------------- FILE: ...ainees/New/drupalorgsite/docroot/modules/contrib/virustotal/src/Controller/VirusTotalReportController.php --------------------------------------------------------------------------------------------------------------- FOUND 1 ERROR AND 1 WARNING AFFECTING 2 LINES --------------------------------------------------------------------------------------------------------------- 18 | WARNING | The class short comment should describe what the class does and not simply repeat the class | | name 21 | ERROR | Missing member variable doc comment --------------------------------------------------------------------------------------------------------------- FILE: .../studenttrainees/New/drupalorgsite/docroot/modules/contrib/virustotal/src/VirusTotalReportListBuilder.php --------------------------------------------------------------------------------------------------------------- FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE --------------------------------------------------------------------------------------------------------------- 32 | WARNING | [x] Inline @var declarations should use the /** */ delimiters --------------------------------------------------------------------------------------------------------------- PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY --------------------------------------------------------------------------------------------------------------- FILE: ...w/drupalorgsite/docroot/modules/contrib/virustotal/src/EventSubscriber/VirusTotalFileInsertSubscriber.php --------------------------------------------------------------------------------------------------------------- FOUND 4 ERRORS AFFECTING 4 LINES --------------------------------------------------------------------------------------------------------------- 20 | ERROR | Missing member variable doc comment 21 | ERROR | Missing member variable doc comment 22 | ERROR | Missing member variable doc comment 23 | ERROR | Missing member variable doc comment --------------------------------------------------------------------------------------------------------------- FILE: ...w/drupalorgsite/docroot/modules/contrib/virustotal/src/EventSubscriber/VirusTotalFileDeleteSubscriber.php --------------------------------------------------------------------------------------------------------------- FOUND 4 ERRORS AND 1 WARNING AFFECTING 5 LINES --------------------------------------------------------------------------------------------------------------- 17 | ERROR | [ ] Missing member variable doc comment 18 | ERROR | [ ] Missing member variable doc comment 19 | ERROR | [ ] Missing member variable doc comment 20 | ERROR | [ ] Missing member variable doc comment 43 | WARNING | [x] 'TODO: Implement onEntityDelete() method.' should match the format '@todo Fix problem X | | here.' --------------------------------------------------------------------------------------------------------------- PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY --------------------------------------------------------------------------------------------------------------- Time: 813ms; Memory: 12MB

Comments & Activities

Issue created by @paraderojether
Issue was unassigned.
Status changed to Needs review over 1 year ago3:33am 17 May 2023
Comment over 1 year ago →
🇵🇭Philippines paraderojether
Created a patch to fix the issues reported by phpcs.

Please review.
Thank You.
Assigned to Kaustab_Roy
Comment over 1 year ago →
🇮🇳India Kaustab_Roy
i will review it
Issue was unassigned.
Status changed to RTBC over 1 year ago4:38am 17 May 2023
Comment over 1 year ago →
🇮🇳India Kaustab_Roy
patch #2 applied cleanly on branch 8.x-1.x
Both Drupal & DrupalPractice standard issues are resolved.
Status changed to Needs work over 1 year ago9:51am 17 May 2023
Comment over 1 year ago →
🇮🇹Italy apaderno Brescia, 🇮🇹
The report shows errors/warning for 24 files, but the MR just changes 12 files.
Comment over 1 year ago →
🇵🇭Philippines paraderojether
Hi apaderno,

The errors/and warnings only show in the 12 files:

- virustotal.install
- README.md
- VirusTotalSettingsForm.php
- VirusTotalCommentForm.php
- VirusTotalScanURLForm.php
- VirusTotalReportForm.php
- VirusTotalReportSettingsForm.php
- VirusTotal/VirusTotalScanner.php
- VirusTotalReportController.php
- VirusTotalReportListBuilder.php
- VirusTotalFileInsertSubscriber.php
- VirusTotalFileDeleteSubscriber.php

Thank You.
Status changed to Needs review over 1 year ago8:30am 18 May 2023
Comment over 1 year ago →
🇮🇹Italy apaderno Brescia, 🇮🇹
Status changed to RTBC over 1 year ago9:32am 13 June 2023
Comment over 1 year ago →
🇺🇦Ukraine ysamoylenko
Thank you for implementing and reviewing this patch. It will be committed soon.
Comment over 1 year ago →
System Message

ysamoylenko → committed f310493c on 8.x-1.x authored by paraderojether →
Issue #3360862 by paraderojether, Kaustab_Roy, ysamoylenko: Fix the...
Status changed to Fixed over 1 year ago9:33am 13 June 2023
Comment over 1 year ago →
🇺🇦Ukraine ysamoylenko
The patch has been committed.
Comment over 1 year ago →
System Message
Automatically closed - issue fixed for 2 weeks with no activity.

Fix the issues reported by phpcs

Problem/Motivation

Steps to reproduce

Proposed resolution

Remaining tasks

Comments & Activities