Contrib.social

Fix the issues reported by phpcs

Open on Drupal.org →
Created on 3 July 2024, 6 months ago
Updated 17 July 2024, 5 months ago

Problem/Motivation

There is a files need to fixed with coding standards.
Here is the command : -
phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,js,info,txt,md,yml,twig web/modules/contrib/admin_status_report/

FILE: /var/www/html/testd10/web/modules/contrib/admin_status_report/admin_status_report.info.yml
-------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 3 WARNINGS AFFECTING 1 LINE
-------------------------------------------------------------------------------------------------------------
 1 | WARNING | Remove "project" from the info file, it will be added by drupal.org packaging automatically
 1 | WARNING | Remove "datestamp" from the info file, it will be added by drupal.org packaging automatically
 1 | WARNING | Remove "version" from the info file, it will be added by drupal.org packaging automatically
-------------------------------------------------------------------------------------------------------------


FILE: /var/www/html/testd10/web/modules/contrib/admin_status_report/src/Form/AdminStatusReportForm.php
------------------------------------------------------------------------------------------------------
FOUND 2 ERRORS AND 1 WARNING AFFECTING 3 LINES
------------------------------------------------------------------------------------------------------
 109 | ERROR   | [x] Expected newline after closing brace
 201 | WARNING | [ ] Unused variable $values.
 228 | ERROR   | [x] Expected 1 newline at end of file; 0 found
------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 2 MARKED SNIFF VIOLATIONS AUTOMATICALLY
------------------------------------------------------------------------------------------------------


FILE: /var/www/html/testd10/web/modules/contrib/admin_status_report/src/EventSubscriber/AdminStatusEventSubscriber.php
----------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
----------------------------------------------------------------------------------------------------------------------
 85 | ERROR | [x] Line indented incorrectly; expected 6 spaces, found 8
----------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
----------------------------------------------------------------------------------------------------------------------


FILE: /var/www/html/testd10/web/modules/contrib/admin_status_report/src/Plugin/AdminStatus/AdminStatusPluginBase.php
--------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
--------------------------------------------------------------------------------------------------------------------
 10 | WARNING | Line exceeds 80 characters; contains 86 characters
--------------------------------------------------------------------------------------------------------------------


FILE: /var/www/html/testd10/web/modules/contrib/admin_status_report/src/Plugin/AdminStatus/CoreStatusReport.php
---------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
---------------------------------------------------------------------------------------------------------------
 18 | ERROR | Missing member variable doc comment
---------------------------------------------------------------------------------------------------------------


FILE: /var/www/html/testd10/web/modules/contrib/admin_status_report/src/AdminStatusInterface.php
------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
------------------------------------------------------------------------------------------------
 85 | WARNING | Line exceeds 80 characters; contains 82 characters
------------------------------------------------------------------------------------------------

Time: 217ms; Memory: 10MB

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

📌 Task
Status

Fixed

Version

1.0

Component

Code

Created by

🇮🇳India chandansha

Live updates comments and jobs are added and updated live.
  • Coding standards

    It involves compliance with, or the content of coding standards. Requires broad community agreement.

Sign in to follow issues

Merge Requests

Comments & Activities

  • Issue created by @chandansha
  • Merge request !33458758-fix-the-issues: fix all issue except 2 issue. → (Merged) created by chandansha
  • Status changed to Needs work 6 months ago
  • Comment 6 months ago →
    🇮🇳India chandansha

    except below two issue fix all.

    FILE: /var/www/html/testd10/web/modules/custom/admin_status_report-3458758/src/Form/AdminStatusReportForm.php
-------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
-------------------------------------------------------------------------------------------------------------
 202 | WARNING | Unused variable $values.
-------------------------------------------------------------------------------------------------------------


FILE: /var/www/html/testd10/web/modules/custom/admin_status_report-3458758/src/Plugin/AdminStatus/CoreStatusReport.php
----------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
----------------------------------------------------------------------------------------------------------------------
 18 | ERROR | Missing member variable doc comment
----------------------------------------------------------------------------------------------------------------------

Time: 136ms; Memory: 10MB
  • Status changed to Fixed 6 months ago
  • Comment 6 months ago →
    🇮🇳India chaitanyadessai Goa

    Merged thanks.

  • Comment 5 months ago →
    System Message

    Automatically closed - issue fixed for 2 weeks with no activity.

contrib.social Blog FAQ Discussions
Production build 0.71.5 2024