Contrib.social

Resolved Coding standard issues.

Open on Drupal.org →
Created on 6 November 2023, 8 months ago
Updated 7 June 2024, 19 days ago

Problem/Motivation


FIle /routes_list/routes_list.module
----------------------------------------------------------------------------------------------------------------------------
FOUND 5 ERRORS AFFECTING 5 LINES
----------------------------------------------------------------------------------------------------------------------------
  1 | ERROR | [x] Missing file doc comment
  7 | ERROR | [x] Whitespace found at end of line
 17 | ERROR | [x] Inline comments must end in full-stops, exclamation marks, question marks, colons, or closing parentheses
 34 | ERROR | [x] Inline comments must end in full-stops, exclamation marks, question marks, colons, or closing parentheses
 43 | ERROR | [x] Inline comments must end in full-stops, exclamation marks, question marks, colons, or closing parentheses
----------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 5 MARKED SNIFF VIOLATIONS AUTOMATICALLY
----------------------------------------------------------------------------------------------------------------------------


FILE: /routes_list/src/Controller/RoutesListController.php
-------------------------------------------------------------------------------------------------------------------------------------------------
FOUND 8 ERRORS AND 2 WARNINGS AFFECTING 9 LINES
-------------------------------------------------------------------------------------------------------------------------------------------------
   7 | ERROR   | [x] Use statements should be sorted alphabetically. The first wrong one is Drupal\Core\Routing\RouteProviderInterface.
  18 | ERROR   | [ ] Missing short description in doc comment
  47 | ERROR   | [x] Data types in @param tags need to be fully namespaced
  49 | ERROR   | [x] Data types in @param tags need to be fully namespaced
  51 | ERROR   | [x] Data types in @param tags need to be fully namespaced
  53 | ERROR   | [x] Data types in @param tags need to be fully namespaced
 207 | WARNING | [x] '@TODO: We need to improve this with manual collection and tracking of' should match the format '@todo Fix problem X here.'
 209 | ERROR   | [x] Comma not allowed after last value in single-line array declaration
 209 | ERROR   | [x] Expected one space after the comma, 0 found
 222 | WARNING | [x] '@TODO: Handle entity + view overrides.' should match the format '@todo Fix problem X here.'
-------------------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 9 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-------------------------------------------------------------------------------------------------------------------------------------------------

Time: 85ms; Memory: 12MB

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

🐛 Bug report
Status

Needs review

Version

6.0

Component

Code

Created by

🇮🇳India adwivedi008

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 @adwivedi008
  • Status changed to Needs review 8 months ago
  • Comment 8 months ago →
    🇮🇳India adwivedi008

    Resolved remaining coding standard issues.

    Please review

  • Status changed to Needs work 20 days ago
  • Comment 20 days ago →
    a.aaronjake

    Hi @adwivedi008,

    Applied your patch successfully, but it still threw one error.

    routes_list git:(8.x-6.x) curl https://www.drupal.org/files/issues/2023-11-06/3399445-2.patch | patch -p1
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  4078  100  4078    0     0   5063      0 --:--:-- --:--:-- --:--:--  5142
patching file routes_list.module
patching file src/Controller/RoutesListController.php
➜  routes_list git:(8.x-6.x) ✗ cd ..
➜  contrib git:(master) ✗ phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig routes_list

FILE: ...ns/Demo-site/drupal-orgissue-v9/web/modules/contrib/routes_list/css/routes_list.css
-----------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
-----------------------------------------------------------------------------------------
 15 | ERROR | [x] CSS colours must be defined in lowercase; expected #f00 but found #F00
-----------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-----------------------------------------------------------------------------------------

Time: 185ms; Memory: 10MB

    Kindly check.

    Thanks,
    Jake

  • Comment 20 days ago →
    🇮🇳India atul ghate

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

  • Merge request !5Issue #3399445: fixed the phpcs. → (Open) created by Unnamed author
  • Status changed to Needs review 19 days ago
  • Comment 19 days ago →
    🇮🇳India atul ghate

    I have fixed remaining phpcs issue. please review and merge the MR

contrib.social Blog FAQ Discussions
Production build 0.69.0 2024