Drupal Coding Standard Issue

Created on 25 July 2023, over 1 year ago
Updated 31 May 2024, 6 months ago

Problem/Motivation

FILE: /var/www/html/drupal_cms/drupal9/web/modules/contrib/entity_pager/tests/fixtures/update/8.x-1.0-rc2/dump.php
------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
------------------------------------------------------------------------------------------------------------------
 52 | ERROR | Comment indentation error, expected only 1 spaces
------------------------------------------------------------------------------------------------------------------


FILE: /var/www/html/drupal_cms/drupal9/web/modules/contrib/entity_pager/src/Plugin/views/style/EntityPager.php
--------------------------------------------------------------------------------------------------------------
FOUND 2 ERRORS AFFECTING 2 LINES
--------------------------------------------------------------------------------------------------------------
 116 | ERROR | Array indentation error, expected 10 spaces but found 8
 117 | ERROR | Array indentation error, expected 10 spaces but found 8
--------------------------------------------------------------------------------------------------------------

Time: 1.57 secs; Memory: 12MB

📌 Task
Status

RTBC

Version

2.0

Component

Code

Created by

🇮🇳India Rohit Rana

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

  • Issue created by @Rohit Rana
  • Issue was unassigned.
  • Status changed to Needs review over 1 year ago
  • Open in Jenkins → Open on Drupal.org →
    Core: 9.5.x + Environment: PHP 7.4 & MySQL 5.7
    last update over 1 year ago
    6 pass
  • Status changed to Needs work 6 months ago
  • 🇵🇭Philippines paraderojether

    Hi

    I reviewed patch #2, and there is 1 phpcs remaining.

    phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig entity_pager
    
    FILE: /Users/jetherparadero/Q2_DO/q2_issues/web/modules/contrib/entity_pager/src/EntityPager.php
    -----------------------------------------------------------------------------------------------------------------------------------
    FOUND 1 ERROR AFFECTING 1 LINE
    -----------------------------------------------------------------------------------------------------------------------------------
     6 | ERROR | [x] Use statements should be sorted alphabetically. The first wrong one is Drupal\Core\Entity\ContentEntityInterface.
    -----------------------------------------------------------------------------------------------------------------------------------
    PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
    -----------------------------------------------------------------------------------------------------------------------------------
    
    Time: 372ms; Memory: 12MB
  • Merge request !73376711: Fix phpcs issues → (Open) created by paraderojether
  • Open in Jenkins → Open on Drupal.org →
    Core: 9.5.x + Environment: PHP 7.4 & MySQL 5.7
    last update 6 months ago
    6 pass
  • Status changed to Needs review 6 months ago
  • 🇵🇭Philippines paraderojether

    I created MR to fix remaining issue, please review.

    Thank you.

  • Status changed to RTBC 6 months ago
  • 🇮🇳India dev16.addweb

    I have tested phpcs issue, Now all issue are resolved, moving to RTBC.

    addweb@addweb-HP-ZBook-15-G2:~$ .config/composer/vendor/squizlabs/php_codesniffer/bin/phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,info,txt,md,yml,css Drupal8-vagrant/web/web/contribution/d9_contribution2/web/modules/custom/entity_pager/
    addweb@addweb-HP-ZBook-15-G2:~$ 
    
Production build 0.71.5 2024