Contrib.social

Fix the warnings/errors reported by PHP_CodeSniffer

Open on Drupal.org →
Created on 10 July 2023, about 2 years ago

Problem/Motivation

FILE: /home/lenovo/d7to9/web/modules/contrib/facets_pretty_paths/config/schema/facets_pretty_paths.schema.yml
-------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
-------------------------------------------------------------------------------------------------------------
 7 | ERROR | [x] Expected 1 newline at end of file; 0 found
-------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-------------------------------------------------------------------------------------------------------------


FILE: /home/lenovo/d7to9/web/modules/contrib/facets_pretty_paths/src/RouteSubscriber.php
---------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
---------------------------------------------------------------------------------------------
 41 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
---------------------------------------------------------------------------------------------


FILE: /home/lenovo/d7to9/web/modules/contrib/facets_pretty_paths/src/PrettyPathsActiveFilters.php
-----------------------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AND 1 WARNING AFFECTING 2 LINES
-----------------------------------------------------------------------------------------------------------------------------------------
 100 | WARNING | Unused variable $url_alias.
 144 | ERROR   | The array declaration extends to column 116 (the limit is 80). The array content should be split up over multiple lines
-----------------------------------------------------------------------------------------------------------------------------------------


FILE: /home/lenovo/d7to9/web/modules/contrib/facets_pretty_paths/src/PrettyPathBreadcrumb.php
------------------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AND 2 WARNINGS AFFECTING 3 LINES
------------------------------------------------------------------------------------------------------------------------------------
 10 | ERROR   | [x] Use statements should be sorted alphabetically. The first wrong one is Drupal\Core\Routing\RouteMatchInterface.
 39 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency injection instead
 40 | WARNING | [ ] \Drupal calls should be avoided in classes, use dependency injection instead
------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
------------------------------------------------------------------------------------------------------------------------------------


FILE: /home/lenovo/d7to9/web/modules/contrib/facets_pretty_paths/facets_pretty_paths.module
-------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
-------------------------------------------------------------------------------------------
 22 | WARNING | [x] Inline @var declarations should use the /** */ delimiters
-------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-------------------------------------------------------------------------------------------


FILE: /home/lenovo/d7to9/web/modules/contrib/facets_pretty_paths/README.md
--------------------------------------------------------------------------
FOUND 0 ERRORS AND 4 WARNINGS AFFECTING 4 LINES
--------------------------------------------------------------------------
  3 | WARNING | Line exceeds 80 characters; contains 98 characters
  9 | WARNING | Line exceeds 80 characters; contains 83 characters
 33 | WARNING | Line exceeds 80 characters; contains 81 characters
 41 | WARNING | Line exceeds 80 characters; contains 88 characters
--------------------------------------------------------------------------

Time: 178ms; Memory: 10MB

Steps to reproduce

Execute the command: phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig facets_pretty_paths/

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

📌 Task
Status

Active

Version

1.0

Component

Code

Created by

🇮🇳India roshni27

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

Comments & Activities

  • Issue created by @roshni27
  • Issue was unassigned.
  • Status changed to Needs review about 2 years ago
  • Open in Jenkins → Open on Drupal.org →
    Core: 9.5.x + Environment: PHP 7.3 & MySQL 5.7
    last update about 2 years ago
    3 pass, 4 fail
  • Comment about 2 years ago →
    🇮🇳India roshni27

    Some issues which are better know to maintainer are pending, for reading purpose character limits issues remain and all remaining issues solve by me. Please review the MR.

  • Comment about 2 years ago →
    System Message

    The last submitted patch, 2: fixes-error-phpcs-3373652-2.patch, failed testing. View results →

  • Comment about 2 years ago →
    🇮🇳India roshni27

    Please check on PHP Version PHP 8.1

  • Status changed to Needs work about 2 years ago
  • Comment about 2 years ago →
    🇵🇭Philippines kenyoOwen

    Hi roshni27

    After I applied your patch and run the command there are still warnings present:
    phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml facets_pretty_paths

    FILE: /Users/studenttrainees/Project10/drupalorg10/docroot/modules/contrib/facets_pretty_paths/README.md
    --------------------------------------------------------------------------------------------------------
    FOUND 0 ERRORS AND 4 WARNINGS AFFECTING 4 LINES
    --------------------------------------------------------------------------------------------------------
    3 | WARNING | Line exceeds 80 characters; contains 98 characters
    9 | WARNING | Line exceeds 80 characters; contains 83 characters
    33 | WARNING | Line exceeds 80 characters; contains 81 characters
    41 | WARNING | Line exceeds 80 characters; contains 88 characters
    --------------------------------------------------------------------------------------------------------

    FILE: /Users/studenttrainees/Project10/drupalorg10/docroot/modules/contrib/facets_pretty_paths/src/PrettyPathsActiveFilters.php
    -------------------------------------------------------------------------------------------------------------------------------
    FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
    -------------------------------------------------------------------------------------------------------------------------------
    100 | WARNING | Unused variable $url_alias.
    -------------------------------------------------------------------------------------------------------------------------------

    Time: 408ms; Memory: 10MB

  • Comment about 2 years ago →
    🇮🇳India mrinalini9 New Delhi

    Updated patch #2 by addressing #5 all the warnings mentioned in except the unused variable $url_alias warning, so keeping it to Needs Work.

    Thanks!

  • Comment almost 2 years ago →
    🇮🇳India zkhan.aamir

    zkhan.aamir → made their first commit to this issue’s fork.

  • Comment over 1 year ago →
    bindu r

    bindu r → made their first commit to this issue’s fork.

  • Comment over 1 year ago →
    🇮🇳India zkhan.aamir

    Issue summary updated.

  • Assigned to amit.rawat777
  • Comment 23 days ago →
    🇮🇳India amit.rawat777

    I am working on this

  • Comment 23 days ago →
    🇮🇳India amit.rawat777

    I have resolved the issues in this patch. Please review.

  • Comment 8 days ago →
    🇮🇳India Asheesh kumar Pal New Delhi
  • Comment 8 days ago →
    🇮🇳India Asheesh kumar Pal New Delhi

    Thanks, amit. I have reviewed your patch, and it has been successfully applied.

  • Comment 8 days ago →
    🇮🇳India Asheesh kumar Pal New Delhi
contrib.social Blog FAQ Discussions
Production build 0.71.5 2024